[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