[llvm] [AMDGPU][NewPM] Port "AMDGPUResourceUsageAnalysis" to NPM (PR #130959)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 26 05:23:15 PDT 2025


================
@@ -44,23 +45,43 @@ struct AMDGPUResourceUsageAnalysis : public MachineFunctionPass {
     SmallVector<const Function *, 16> Callees;
   };
 
-  AMDGPUResourceUsageAnalysis() : MachineFunctionPass(ID) {}
+  SIFunctionResourceInfo
+  analyzeResourceUsage(const MachineFunction &MF,
+                       uint32_t AssumedStackSizeForDynamicSizeObjects,
+                       uint32_t AssumedStackSizeForExternalCall) const;
+};
+
+struct AMDGPUResourceUsageAnalysisWrapperPass : public MachineFunctionPass {
+  using ResultT = AMDGPUResourceUsageAnalysisImpl::SIFunctionResourceInfo;
----------------
arsenm wrote:

I would get rid of the ResultT. The only precedent for ResultT naming appear to be in template class parameters 

https://github.com/llvm/llvm-project/pull/130959


More information about the llvm-commits mailing list