[llvm] [AMDGPU] Ensure positive InstOffset for buffer operations (PR #145504)

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 14 07:06:23 PDT 2025


================
@@ -4552,7 +4553,9 @@ bool AMDGPUAsmParser::validateSMEMOffset(const MCInst &Inst,
     return true;
 
   Error(getSMEMOffsetLoc(Operands),
-        isGFX12Plus()          ? "expected a 24-bit signed offset"
+        isGFX12Plus() && IsBuffer
----------------
jayfoad wrote:

It is pretty horrible how we have to guess at the reason for `isLegalSMRDEncoded*Offset` rejecting our offset. That is not your fault. But maybe we should try to clean it up, as a a separate patch.

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


More information about the llvm-commits mailing list