[PATCH] D20648: Analysis pass to access profile summary info

Vedant Kumar via llvm-commits llvm-commits at lists.llvm.org
Fri May 27 15:21:34 PDT 2016

vsk added a comment.

Sorry for the delay. I have some preliminary comments, I'll take a closer look in a second.

Comment at: lib/Analysis/ProfileSummaryInfo.cpp:34
@@ +33,3 @@
+static cl::opt<int> ColdCountPercentile(
+    "cold-count-percentile", cl::Hidden, cl::init(999999), cl::ZeroOrMore,
+    cl::desc("A count is cold if it is below the minimum count"
I'm a bit confused as to why this is higher than the percentile for hot functions. Sorry if this is a basic question! Could you explain this a bit?

Comment at: lib/Analysis/ProfileSummaryInfo.cpp:44
@@ +43,3 @@
+  };
+  auto It = std::find_if(DS.begin(), DS.end(), FindPercentile);
+  // The required percentile has to match one of the percentiles in the
Why not a lower_bound? That should be faster, and it adds a bit of flexibility, I think.

Comment at: lib/Analysis/ProfileSummaryInfo.cpp:47
@@ +46,3 @@
+  // detailed summary.
+  assert(It != DS.end());
+  return It->MinCount;
Could you add an error string to this assert?


More information about the llvm-commits mailing list