[llvm] [AMDGPU] Handle MachineOperandType global address in SIFoldOperands. (PR #135424)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 11 12:20:17 PDT 2025


================
@@ -1161,8 +1161,10 @@ void SIFoldOperandsImpl::foldOperand(
 
         if (OpToFold.isImm())
           UseMI->getOperand(1).ChangeToImmediate(OpToFold.getImm());
-        else
+        else if(OpToFold.isFI())
           UseMI->getOperand(1).ChangeToFrameIndex(OpToFold.getIndex());
+        else if(OpToFold.isGlobal())
+          return;
----------------
arsenm wrote:

Missing space after if, but this should probably be an else return, this will just break in the same way on any other operand type 

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


More information about the llvm-commits mailing list