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

Aleksandar Spasojevic via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 18 03:56:53 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
----------------
aleksandar-amd wrote:

I completely agree. Cleaning it up as a separate patch sounds like a great idea.

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


More information about the llvm-commits mailing list