[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