================ @@ -40,12 +41,20 @@ class AMDGPUAsmPrinter final : public AsmPrinter { AMDGPUResourceUsageAnalysis *ResourceUsage; + std::unique_ptr<MCResourceInfo> RI; + SIProgramInfo CurrentProgramInfo; std::unique_ptr<AMDGPU::HSAMD::MetadataStreamer> HSAMetadataStream; MCCodeEmitter *DumpCodeInstEmitter = nullptr; + // ValidateMCResourceInfo cannot recompute parts of the occupancy as it does + // for other metadata to validate (e.g., NumSGPRs) so a map is necessary if we + // really want to track and validate the occupancy. + std::unique_ptr<DenseMap<const Function *, const MCExpr *>> ---------------- arsenm wrote:
Same, why not just a plain class member? https://github.com/llvm/llvm-project/pull/102913 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits