[llvm] 9c0ba62 - [ctx_prof] Relax the "profile use" case around `PGOOpt` (#108265)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 11 13:39:53 PDT 2024
Author: Mircea Trofin
Date: 2024-09-11T13:39:50-07:00
New Revision: 9c0ba62010b5850adf6b4c3979128aa6e9189aca
URL: https://github.com/llvm/llvm-project/commit/9c0ba62010b5850adf6b4c3979128aa6e9189aca
DIFF: https://github.com/llvm/llvm-project/commit/9c0ba62010b5850adf6b4c3979128aa6e9189aca.diff
LOG: [ctx_prof] Relax the "profile use" case around `PGOOpt` (#108265)
`PGOOpt` could have a value if, for instance, debug info for profiling
is requested. Relaxing the requirement, for now, following that
eventually we would factor `PGOOpt` to better capture the supported
interplay between the various profiling options.
Added:
Modified:
llvm/lib/Passes/PassBuilderPipelines.cpp
llvm/test/Transforms/PGOProfile/ctx-prof-use-prelink.ll
Removed:
################################################################################
diff --git a/llvm/lib/Passes/PassBuilderPipelines.cpp b/llvm/lib/Passes/PassBuilderPipelines.cpp
index 6ebf262379c2fb..8f151a99b11709 100644
--- a/llvm/lib/Passes/PassBuilderPipelines.cpp
+++ b/llvm/lib/Passes/PassBuilderPipelines.cpp
@@ -1181,8 +1181,8 @@ PassBuilder::buildModuleSimplificationPipeline(OptimizationLevel Level,
// Enable contextual profiling instrumentation.
const bool IsCtxProfGen = !IsPGOInstrGen && IsPreLink &&
PGOCtxProfLoweringPass::isCtxIRPGOInstrEnabled();
- const bool IsCtxProfUse = !UseCtxProfile.empty() && !PGOOpt &&
- Phase == ThinOrFullLTOPhase::ThinLTOPreLink;
+ const bool IsCtxProfUse =
+ !UseCtxProfile.empty() && Phase == ThinOrFullLTOPhase::ThinLTOPreLink;
if (IsPGOInstrGen || IsPGOInstrUse || IsMemprofUse || IsCtxProfGen ||
IsCtxProfUse)
@@ -1673,7 +1673,7 @@ PassBuilder::buildThinLTOPreLinkDefaultPipeline(OptimizationLevel Level) {
// In pre-link, for ctx prof use, we stop here with an instrumented IR. We let
// thinlto use the contextual info to perform imports; then use the contextual
// profile in the post-thinlink phase.
- if (!UseCtxProfile.empty() && !PGOOpt) {
+ if (!UseCtxProfile.empty()) {
addRequiredLTOPreLinkPasses(MPM);
return MPM;
}
diff --git a/llvm/test/Transforms/PGOProfile/ctx-prof-use-prelink.ll b/llvm/test/Transforms/PGOProfile/ctx-prof-use-prelink.ll
index 7959e4d0760edb..56ed92ea1b7ffb 100644
--- a/llvm/test/Transforms/PGOProfile/ctx-prof-use-prelink.ll
+++ b/llvm/test/Transforms/PGOProfile/ctx-prof-use-prelink.ll
@@ -4,6 +4,7 @@
; There is no scenario currently of doing ctx profile use without thinlto.
;
; RUN: opt -passes='thinlto-pre-link<O2>' -use-ctx-profile=something_that_does_not_exist %s -S | FileCheck %s
+; RUN: opt -debug-info-for-profiling -passes='thinlto-pre-link<O2>' -use-ctx-profile=something_that_does_not_exist %s -S | FileCheck %s
declare void @bar()
More information about the llvm-commits
mailing list