[PATCH] D41604: Add a pass to generate synthetic function entry counts.

Easwaran Raman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 8 11:01:52 PST 2018


eraman added inline comments.


================
Comment at: lib/Transforms/IPO/SyntheticCountsPropagation.cpp:60
+static cl::opt<int> ColdSyntheticCount(
+    "cold-synthetic-count", cl::Hidden, cl::init(5), cl::ZeroOrMore,
+    cl::desc("Initial synthetic entry count for cold functions."));
----------------
davidxl wrote:
> make it zero perhaps?
I have no strong opinion, but I expect the default values to change after we make the inliner use these synthetic counts and tune the inliner. So I'll leave it as 5 for now. 


https://reviews.llvm.org/D41604





More information about the llvm-commits mailing list