[PATCH] D110212: [llvm-profgen] Use hot threshold for context merging and trimming

[Wenlei He via Phabricator via llvm-commits]

wenlei added inline comments.


================
Comment at: llvm/tools/llvm-profgen/ProfileGenerator.cpp:431
-  if (!ProfileSummaryCutoffCold.getNumOccurrences())
-    ProfileSummaryCutoffCold = 999000;
-
----------------
hoy wrote:
> Does it still make sense to keep this? Would profile summary builder use this value?
Profile summary builder still use it, but the removed tuning was only needed for merging/trimming. Now that merging/trimming changed to use hot threshold, the tuning is no longer needed either. The tuning may not general enough for a good cold threshold. 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110212/new/

https://reviews.llvm.org/D110212