[llvm] [AMDGPU][NewPM] Port "AMDGPUResourceUsageAnalysis" to NPM (PR #130959)
Vikram Hegde via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 12 22:59:45 PDT 2025
================
@@ -1657,8 +1657,8 @@ bool AMDGPUAsmPrinter::PrintAsmOperand(const MachineInstr *MI, unsigned OpNo,
}
void AMDGPUAsmPrinter::getAnalysisUsage(AnalysisUsage &AU) const {
- AU.addRequired<AMDGPUResourceUsageAnalysis>();
- AU.addPreserved<AMDGPUResourceUsageAnalysis>();
+ AU.addRequired<AMDGPUResourceUsageAnalysisWrapperPass>();
+ AU.addPreserved<AMDGPUResourceUsageAnalysisWrapperPass>();
----------------
vikramRH wrote:
I don't see Analysis wrappers using legacy in pass name (wrapper itself should be indicative of that i guess). However i don't see the "analysis" being used explicitly in wrapper names too, I dont really know if this is the convention..
https://github.com/llvm/llvm-project/pull/130959
More information about the llvm-commits
mailing list