[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