[all-commits] [llvm/llvm-project] d09315: [opt][NewPM] Add isRequired to passes named as *Pr...

[Björn Pettersson via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d09315d986fea239e462da454334482abc4ad461
      https://github.com/llvm/llvm-project/commit/d09315d986fea239e462da454334482abc4ad461
  Author: Björn Pettersson <bjorn.a.pettersson at ericsson.com>
  Date:   2024-01-03 (Wed, 03 Jan 2024)

  Changed paths:
    M llvm/include/llvm/Analysis/AliasSetTracker.h
    M llvm/include/llvm/Analysis/AssumptionCache.h
    M llvm/include/llvm/Analysis/BlockFrequencyInfo.h
    M llvm/include/llvm/Analysis/BranchProbabilityInfo.h
    M llvm/include/llvm/Analysis/CFGSCCPrinter.h
    M llvm/include/llvm/Analysis/CallGraph.h
    M llvm/include/llvm/Analysis/CallPrinter.h
    M llvm/include/llvm/Analysis/CostModel.h
    M llvm/include/llvm/Analysis/CycleAnalysis.h
    M llvm/include/llvm/Analysis/DDG.h
    M llvm/include/llvm/Analysis/DDGPrinter.h
    M llvm/include/llvm/Analysis/Delinearization.h
    M llvm/include/llvm/Analysis/DemandedBits.h
    M llvm/include/llvm/Analysis/DependenceAnalysis.h
    M llvm/include/llvm/Analysis/DominanceFrontier.h
    M llvm/include/llvm/Analysis/FunctionPropertiesAnalysis.h
    M llvm/include/llvm/Analysis/IRSimilarityIdentifier.h
    M llvm/include/llvm/Analysis/InlineAdvisor.h
    M llvm/include/llvm/Analysis/InlineCost.h
    M llvm/include/llvm/Analysis/InlineSizeEstimatorAnalysis.h
    M llvm/include/llvm/Analysis/LazyCallGraph.h
    M llvm/include/llvm/Analysis/LazyValueInfo.h
    M llvm/include/llvm/Analysis/LoopCacheAnalysis.h
    M llvm/include/llvm/Analysis/LoopInfo.h
    M llvm/include/llvm/Analysis/LoopNestAnalysis.h
    M llvm/include/llvm/Analysis/MemDerefPrinter.h
    M llvm/include/llvm/Analysis/MemorySSA.h
    M llvm/include/llvm/Analysis/ModuleDebugInfoPrinter.h
    M llvm/include/llvm/Analysis/MustExecute.h
    M llvm/include/llvm/Analysis/PhiValues.h
    M llvm/include/llvm/Analysis/PostDominators.h
    M llvm/include/llvm/Analysis/ProfileSummaryInfo.h
    M llvm/include/llvm/Analysis/RegionInfo.h
    M llvm/include/llvm/Analysis/ScalarEvolution.h
    M llvm/include/llvm/Analysis/StackLifetime.h
    M llvm/include/llvm/Analysis/StackSafetyAnalysis.h
    M llvm/include/llvm/Analysis/StructuralHash.h
    M llvm/include/llvm/Analysis/UniformityAnalysis.h
    M llvm/include/llvm/IR/Dominators.h
    M llvm/include/llvm/Transforms/Scalar/IVUsersPrinter.h
    M llvm/include/llvm/Transforms/Scalar/LoopAccessAnalysisPrinter.h
    M llvm/include/llvm/Transforms/Utils/PredicateInfo.h
    M llvm/lib/Target/DirectX/DXILResourceAnalysis.h

  Log Message:
  -----------
  [opt][NewPM] Add isRequired to passes named as *PrinterPass (#76516)

Passes that print the result of analysis passes should be of interest, and
are expected to run even if a function for example is marked as optnone.
So when adding such passes explicitly to a pipeline it makes sense to
run the pass regardless of standard instrumentation gates such as
OptNoneInstrumentation.

In this patch all passes named as *PrinterPass are marked as required.
That should make sure that those passes are executed
without being skipped due to standard instrumentations.

The polly passes are not touched in this patch.

Partial fix for: https://github.com/llvm/llvm-project/issues/76762