[PATCH] D144902: [AMDGPU][AsmParser] Distinguish literal and modifier SMEM offsets.

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 28 04:19:07 PST 2023


foad accepted this revision.
foad added a comment.
This revision is now accepted and ready to land.

>> I don't quite understand the bug here - can you explain how "s_load_dword s1, s[2:3], s0 0x1" would have been mishandled before your patch?
>
> The `0x1` bit is currently indistinguishable from `offset:0x1` and therefore accepted as a valid operand.

Thanks, LGTM!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144902/new/

https://reviews.llvm.org/D144902



More information about the llvm-commits mailing list