[PATCH] D35962: Separate the ICP total threshold and remaining threshold.

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 27 17:11:12 PDT 2017


tejohnson added inline comments.


================
Comment at: lib/Analysis/IndirectCallPromotionAnalysis.cpp:47
+    "icp-remaining-percent-threshold", cl::init(30), cl::Hidden, cl::ZeroOrMore,
+    cl::desc("The percentage threshold against remaining unpromoted indirect "
+             "call count for the promotion"));
----------------
In your 99.9% and  0.1% case, won't the 0.1% still get promoted since it is 100% of the remaining count after promoting the 99.9% target?


https://reviews.llvm.org/D35962





More information about the llvm-commits mailing list