[ctx_prof] Relax the "profile use" case around PGOOpt (llvm#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.

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;
  }

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()