emmettneyman created this revision.
emmettneyman added reviewers: morehouse, kcc.
Herald added a subscriber: cfe-commits.
I noticed that my code wasn't going deep into the loop vectorizer code so added
another pass that makes it go further.
Repository:
rC Clang
https://reviews.llvm.org/D5048
emmettneyman updated this revision to Diff 159831.
emmettneyman added a comment.
- minor style fix
Repository:
rC Clang
https://reviews.llvm.org/D50482
Files:
clang/tools/clang-fuzzer/handle-llvm/handle_llvm.cpp
Index: clang/tools/clang-fuzzer/handle-llvm/handle_llvm.cpp
morehouse added inline comments.
Comment at: clang/tools/clang-fuzzer/handle-llvm/handle_llvm.cpp:90
+getCodeModel(), OLvl);
+}
+
If you have to pass that many parameters to a 3 line function, just inline
instead.
==
emmettneyman added inline comments.
Comment at: clang/tools/clang-fuzzer/handle-llvm/handle_llvm.cpp:126
+
+ auto Is TM guaranteed to be an LLVMTargetMachine?
Yes, since the t
emmettneyman updated this revision to Diff 159834.
emmettneyman added a comment.
- Inlined function
Repository:
rC Clang
https://reviews.llvm.org/D50482
Files:
clang/tools/clang-fuzzer/handle-llvm/handle_llvm.cpp
Index: clang/tools/clang-fuzzer/handle-llvm/handle_llvm.cpp
===
morehouse accepted this revision.
morehouse added inline comments.
This revision is now accepted and ready to land.
Comment at: clang/tools/clang-fuzzer/handle-llvm/handle_llvm.cpp:103
+ Triple ModuleTriple(M->getTargetTriple());
+ const TargetOptions Options = InitTargetOpti
emmettneyman added inline comments.
Comment at: clang/tools/clang-fuzzer/handle-llvm/handle_llvm.cpp:103
+ Triple ModuleTriple(M->getTargetTriple());
+ const TargetOptions Options = InitTargetOptionsFromCodeGenFlags();
morehouse wrote:
> I think you can avoid
emmettneyman updated this revision to Diff 159838.
emmettneyman added a comment.
Rebase
Repository:
rC Clang
https://reviews.llvm.org/D50482
Files:
clang/tools/clang-fuzzer/handle-llvm/handle_llvm.cpp
Index: clang/tools/clang-fuzzer/handle-llvm/handle_llvm.cpp
=
This revision was automatically updated to reflect the committed changes.
Closed by commit rL339305: Added another optimization pass to make vectorizing
possible (authored by emmettneyman, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D50482?vs=159838&id=159839#toc
Reposito