[PATCH] D15828: [PGO] Passmanagerbuilder change that enable IR level PGO instrumentation

David Li via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 30 14:54:36 PST 2015


davidxl added inline comments.

================
Comment at: lib/Transforms/IPO/PassManagerBuilder.cpp:208
@@ +207,3 @@
+  // Preinstrumenation inline. Set the OptLevel to 1 to limit the inlining. 
+  MPM.add(createFunctionInliningPass(1, SizeLevel));
+  if (OptLevel != 0) {
----------------
Is it better to do max(SizeLevel, 1) instead of SizeLevel?


http://reviews.llvm.org/D15828





More information about the llvm-commits mailing list