[PATCH] D113785: [llvm-profgen] Trim cold function profiles for non-CS AutoFDO

[Wenlei He via Phabricator via llvm-commits]

wenlei added inline comments.


================
Comment at: llvm/tools/llvm-profgen/ProfileGenerator.cpp:63
 
 static cl::opt<bool> CSProfTrimColdContext(
     "csprof-trim-cold-context", cl::init(false), cl::ZeroOrMore,
----------------
hoy wrote:
> How about merge the new switch with this one?
Yeah, let's merge them. And this may also mean we need to put it on afdo-specific path, instead having it in `ProfileGenerator::postProcessProfiles`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113785