[PATCH] D38190: Partial Inlining with multi-region outlining based on PGO information

[Graham Yiu via Phabricator via llvm-commits]

gyiu marked 2 inline comments as done.
gyiu added inline comments.


================
Comment at: lib/Transforms/IPO/PartialInlining.cpp:83
+                             cl::desc("Minimum block executions to consider "
+                                      "its BranchProbabilityInfo valid"));
+// Use for tuning outlining heuristics.
----------------
davidxl wrote:
> Why is option needed?
Just for added tune-ability.  I wasn't sure if '100' is a reasonable minimum execution count to have.


Repository:
  rL LLVM

https://reviews.llvm.org/D38190