[PATCH] D21405: [PGO] IRPGO pre-cleanup pass changes

[Vedant Kumar via llvm-commits]

vsk added a comment.

This looks great! Thanks for putting together the detailed performance reports.

I noticed that there is one small slowdown with 401.bzip. Do you know what aspect of the pipeline is responsible for the change? What were your criteria for picking the pre-inline pipeline?


================
Comment at: lib/Transforms/IPO/PassManagerBuilder.cpp:122
@@ +121,3 @@
+static cl::opt<bool>
+    RunPreInliner("preinline", cl::init(true), cl::Hidden, cl::ZeroOrMore,
+                  cl::desc("Enable pre-instrumentation inliner"));
----------------
Is zero or more needed?


http://reviews.llvm.org/D21405