[PATCH] D120508: introducing profi flags
Sergey Pupyrev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 8 12:16:05 PST 2022
spupyrev added inline comments.
================
Comment at: llvm/lib/Transforms/Utils/SampleProfileLoaderBaseUtil.cpp:46
+cl::opt<bool> SampleProfileInferEntryCount(
+ "sample-profile-infer-entry-count", cl::init(true), cl::Hidden,
+ cl::ZeroOrMore, cl::desc("Use profi to infer function entry count."));
----------------
hoy wrote:
> Will there be a separate change to flip the default value?
I suggest we keep it as is and test on a couple of prod binaries. If confirmed, we may change the default value
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D120508/new/
https://reviews.llvm.org/D120508
More information about the llvm-commits
mailing list