https://github.com/optimisan updated https://github.com/llvm/llvm-project/pull/118779
>From 7ee86211673112f061749d3ce7e1f2671313159f Mon Sep 17 00:00:00 2001 From: Akshat Oke <akshat....@amd.com> Date: Thu, 5 Dec 2024 10:05:30 +0000 Subject: [PATCH] [CodeGen][PM] Initialize analyses with isAnalysis=true Analyses should be marked as analyses else they are prone to be ignored by the legacy analysis cache mechanism and get scheduled redundantly. --- llvm/lib/CodeGen/GCMetadata.cpp | 2 +- llvm/lib/CodeGen/LiveDebugVariables.cpp | 2 +- llvm/lib/CodeGen/LiveIntervals.cpp | 2 +- llvm/lib/CodeGen/LiveRegMatrix.cpp | 2 +- llvm/lib/CodeGen/LiveStacks.cpp | 2 +- llvm/lib/CodeGen/SlotIndexes.cpp | 2 +- llvm/lib/CodeGen/VirtRegMap.cpp | 2 +- llvm/lib/Transforms/Utils/LoopSimplify.cpp | 4 ++-- 8 files changed, 9 insertions(+), 9 deletions(-) diff --git a/llvm/lib/CodeGen/GCMetadata.cpp b/llvm/lib/CodeGen/GCMetadata.cpp index 6d1cc1a58e27df..f33008c9e0f232 100644 --- a/llvm/lib/CodeGen/GCMetadata.cpp +++ b/llvm/lib/CodeGen/GCMetadata.cpp @@ -66,7 +66,7 @@ GCFunctionAnalysis::run(Function &F, FunctionAnalysisManager &FAM) { } INITIALIZE_PASS(GCModuleInfo, "collector-metadata", - "Create Garbage Collector Module Metadata", false, false) + "Create Garbage Collector Module Metadata", false, true) // ----------------------------------------------------------------------------- diff --git a/llvm/lib/CodeGen/LiveDebugVariables.cpp b/llvm/lib/CodeGen/LiveDebugVariables.cpp index 317d3401f000a4..79085e587ebc45 100644 --- a/llvm/lib/CodeGen/LiveDebugVariables.cpp +++ b/llvm/lib/CodeGen/LiveDebugVariables.cpp @@ -82,7 +82,7 @@ INITIALIZE_PASS_BEGIN(LiveDebugVariablesWrapperLegacy, DEBUG_TYPE, INITIALIZE_PASS_DEPENDENCY(MachineDominatorTreeWrapperPass) INITIALIZE_PASS_DEPENDENCY(LiveIntervalsWrapperPass) INITIALIZE_PASS_END(LiveDebugVariablesWrapperLegacy, DEBUG_TYPE, - "Debug Variable Analysis", false, false) + "Debug Variable Analysis", false, true) void LiveDebugVariablesWrapperLegacy::getAnalysisUsage( AnalysisUsage &AU) const { diff --git a/llvm/lib/CodeGen/LiveIntervals.cpp b/llvm/lib/CodeGen/LiveIntervals.cpp index f9ee6e4563f8d6..f38527a3ce6a31 100644 --- a/llvm/lib/CodeGen/LiveIntervals.cpp +++ b/llvm/lib/CodeGen/LiveIntervals.cpp @@ -83,7 +83,7 @@ INITIALIZE_PASS_BEGIN(LiveIntervalsWrapperPass, "liveintervals", INITIALIZE_PASS_DEPENDENCY(MachineDominatorTreeWrapperPass) INITIALIZE_PASS_DEPENDENCY(SlotIndexesWrapperPass) INITIALIZE_PASS_END(LiveIntervalsWrapperPass, "liveintervals", - "Live Interval Analysis", false, false) + "Live Interval Analysis", false, true) bool LiveIntervalsWrapperPass::runOnMachineFunction(MachineFunction &MF) { LIS.Indexes = &getAnalysis<SlotIndexesWrapperPass>().getSI(); diff --git a/llvm/lib/CodeGen/LiveRegMatrix.cpp b/llvm/lib/CodeGen/LiveRegMatrix.cpp index bc8c59381a40e1..9744c47d5a8510 100644 --- a/llvm/lib/CodeGen/LiveRegMatrix.cpp +++ b/llvm/lib/CodeGen/LiveRegMatrix.cpp @@ -41,7 +41,7 @@ INITIALIZE_PASS_BEGIN(LiveRegMatrixWrapperLegacy, "liveregmatrix", INITIALIZE_PASS_DEPENDENCY(LiveIntervalsWrapperPass) INITIALIZE_PASS_DEPENDENCY(VirtRegMapWrapperLegacy) INITIALIZE_PASS_END(LiveRegMatrixWrapperLegacy, "liveregmatrix", - "Live Register Matrix", false, false) + "Live Register Matrix", false, true) void LiveRegMatrixWrapperLegacy::getAnalysisUsage(AnalysisUsage &AU) const { AU.setPreservesAll(); diff --git a/llvm/lib/CodeGen/LiveStacks.cpp b/llvm/lib/CodeGen/LiveStacks.cpp index 92cc6699f2d331..d615caf48c0ad3 100644 --- a/llvm/lib/CodeGen/LiveStacks.cpp +++ b/llvm/lib/CodeGen/LiveStacks.cpp @@ -25,7 +25,7 @@ INITIALIZE_PASS_BEGIN(LiveStacksWrapperLegacy, DEBUG_TYPE, "Live Stack Slot Analysis", false, false) INITIALIZE_PASS_DEPENDENCY(SlotIndexesWrapperPass) INITIALIZE_PASS_END(LiveStacksWrapperLegacy, DEBUG_TYPE, - "Live Stack Slot Analysis", false, false) + "Live Stack Slot Analysis", false, true) char &llvm::LiveStacksID = LiveStacksWrapperLegacy::ID; diff --git a/llvm/lib/CodeGen/SlotIndexes.cpp b/llvm/lib/CodeGen/SlotIndexes.cpp index 1b92a5aa59d18c..8d2832b3fdd56e 100644 --- a/llvm/lib/CodeGen/SlotIndexes.cpp +++ b/llvm/lib/CodeGen/SlotIndexes.cpp @@ -45,7 +45,7 @@ SlotIndexes::~SlotIndexes() { } INITIALIZE_PASS(SlotIndexesWrapperPass, DEBUG_TYPE, "Slot index numbering", - false, false) + false, true) STATISTIC(NumLocalRenum, "Number of local renumberings"); diff --git a/llvm/lib/CodeGen/VirtRegMap.cpp b/llvm/lib/CodeGen/VirtRegMap.cpp index 1352102a93d01b..b28c74600e7a29 100644 --- a/llvm/lib/CodeGen/VirtRegMap.cpp +++ b/llvm/lib/CodeGen/VirtRegMap.cpp @@ -60,7 +60,7 @@ STATISTIC(NumIdCopies, "Number of identity moves eliminated after rewriting"); char VirtRegMapWrapperLegacy::ID = 0; INITIALIZE_PASS(VirtRegMapWrapperLegacy, "virtregmap", "Virtual Register Map", - false, false) + false, true) void VirtRegMap::init(MachineFunction &mf) { MRI = &mf.getRegInfo(); diff --git a/llvm/lib/Transforms/Utils/LoopSimplify.cpp b/llvm/lib/Transforms/Utils/LoopSimplify.cpp index 44fdfe530178a9..d8298646e18d7e 100644 --- a/llvm/lib/Transforms/Utils/LoopSimplify.cpp +++ b/llvm/lib/Transforms/Utils/LoopSimplify.cpp @@ -777,8 +777,8 @@ INITIALIZE_PASS_BEGIN(LoopSimplify, "loop-simplify", INITIALIZE_PASS_DEPENDENCY(AssumptionCacheTracker) INITIALIZE_PASS_DEPENDENCY(DominatorTreeWrapperPass) INITIALIZE_PASS_DEPENDENCY(LoopInfoWrapperPass) -INITIALIZE_PASS_END(LoopSimplify, "loop-simplify", - "Canonicalize natural loops", false, false) +INITIALIZE_PASS_END(LoopSimplify, "loop-simplify", "Canonicalize natural loops", + false, true) // Publicly exposed interface to pass... char &llvm::LoopSimplifyID = LoopSimplify::ID; _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits