[PATCH] D50482: Added another optimization pass to make vectorizing possible

Matt Morehouse via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 8 17:09:50 PDT 2018


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.


================
Comment at: clang/tools/clang-fuzzer/handle-llvm/handle_llvm.cpp:126
+
+  auto &LTM = static_cast<LLVMTargetMachine &>(*TM);
+  Passes.add(LTM.createPassConfig(Passes));
----------------
Is TM guaranteed to be an LLVMTargetMachine?


Repository:
  rC Clang

https://reviews.llvm.org/D50482





More information about the cfe-commits mailing list