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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to