[llvm] [ctx_prof] Fix the pre-thinlink "use" case (PR #102511)
Aiden Grossman via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 8 11:50:18 PDT 2024
================
@@ -1173,13 +1173,12 @@ PassBuilder::buildModuleSimplificationPipeline(OptimizationLevel Level,
const bool IsMemprofUse = IsPGOPreLink && !PGOOpt->MemoryProfile.empty();
// We don't want to mix pgo ctx gen and pgo gen; we also don't currently
// enable ctx profiling from the frontend.
- assert(
- !(IsPGOInstrGen && PGOCtxProfLoweringPass::isContextualIRPGOEnabled()) &&
- "Enabling both instrumented FDO and contextual instrumentation is not "
- "supported.");
+ assert(!(IsPGOInstrGen && PGOCtxProfLoweringPass::isCtxIRPGOInstrEnabled()) &&
+ "Enabling both instrumented FDO and contextual instrumentation is not "
----------------
boomanaiden154 wrote:
I know this is copy and paste, but `s/FDO/PGO` given everything in the context/variable naming is also PGO?
https://github.com/llvm/llvm-project/pull/102511
More information about the llvm-commits
mailing list