[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


================
@@ -4464,12 +4464,13 @@ bool AMDGPUAsmParser::validateOffset(const MCInst &Inst,
     return validateSMEMOffset(Inst, Operands);
 
   const auto &Op = Inst.getOperand(OpNum);
+  // GFX12+ buffer ops: InstOffset is signed 24, but must be positive
----------------
jayfoad wrote:

```suggestion
  // GFX12+ buffer ops: InstOffset is signed 24, but must not be negative.
```

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


More information about the llvm-commits mailing list