[llvm] a5d3cac - [llvm-profgen] Turn off cold context trimming by default

[Wenlei He via llvm-commits]

Author: Wenlei He
Date: 2021-09-02T12:29:06-07:00
New Revision: a5d3cac0336b7318a4b0c1fb4a0b76e991630be6

URL: https://github.com/llvm/llvm-project/commit/a5d3cac0336b7318a4b0c1fb4a0b76e991630be6
DIFF: https://github.com/llvm/llvm-project/commit/a5d3cac0336b7318a4b0c1fb4a0b76e991630be6.diff

LOG: [llvm-profgen] Turn off cold context trimming by default

We merge cold context by default to save profile size. However trimming cold context after merging doesn't save size much, so default to off to reflect how it's commonly used.

Differential Revision: https://reviews.llvm.org/D109166

Added: 
    

Modified: 
    llvm/tools/llvm-profgen/ProfileGenerator.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/tools/llvm-profgen/ProfileGenerator.cpp b/llvm/tools/llvm-profgen/ProfileGenerator.cpp
index 581e4dd6f0bf..5b9984cc8986 100644
--- a/llvm/tools/llvm-profgen/ProfileGenerator.cpp
+++ b/llvm/tools/llvm-profgen/ProfileGenerator.cpp
@@ -46,7 +46,7 @@ static cl::opt<bool> CSProfMergeColdContext(
              "profile."));
 
 static cl::opt<bool> CSProfTrimColdContext(
-    "csprof-trim-cold-context", cl::init(true), cl::ZeroOrMore,
+    "csprof-trim-cold-context", cl::init(false), cl::ZeroOrMore,
     cl::desc("If the total count of the profile after all merge is done "
              "is still smaller than threshold, it will be trimmed."));