[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