[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