[llvm] [AMDGPU] Support SIProgramInfo MCExpr for comments and remarks (PR #94350)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 6 08:03:36 PDT 2024


================
@@ -554,13 +585,11 @@ bool AMDGPUAsmPrinter::runOnMachineFunction(MachineFunction &MF) {
 
     OutStreamer->emitRawComment(" Kernel info:", false);
     emitCommonFunctionComments(
-        getMCExprValue(CurrentProgramInfo.NumArchVGPR, Ctx),
-        STM.hasMAIInsts() ? getMCExprValue(CurrentProgramInfo.NumAccVGPR, Ctx)
-                          : std::optional<uint32_t>(),
-        getMCExprValue(CurrentProgramInfo.NumVGPR, Ctx),
-        getMCExprValue(CurrentProgramInfo.NumSGPR, Ctx),
-        getMCExprValue(CurrentProgramInfo.ScratchSize, Ctx),
-        getFunctionCodeSize(MF), MFI);
+        CurrentProgramInfo.NumArchVGPR,
+        STM.hasMAIInsts() ? CurrentProgramInfo.NumAccVGPR
+                          : std::optional<const MCExpr *>(),
----------------
arsenm wrote:

optional of pointer is kind of obnoxious 

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


More information about the llvm-commits mailing list