[PATCH] D125700: [AMDGPU][GFX9] Support base+soffset+offset SMEM loads.

Mon May 16 14:03:06 PDT 2022

rampitec added inline comments.

================
Comment at: llvm/test/MC/Disassembler/AMDGPU/gfx9_dasm_all.txt:10077

+# CHECK: s_load_dword s5, s[2:3], s0 offset:0x12345 ; encoding: [0x41,0x41,0x02,0xc0,0x45,0x23,0x01,0x00]
+0x41,0x41,0x02,0xc0,0x45,0x23,0x01,0x00
----------------
arsenm wrote:
> rampitec wrote:
> > Is there a decoding conflict which requires to use brackets here? Something from the TODO list?
> I'm not sure what this question means. The encoding is always printed in brackets?
Printed yes. But disasm tests do not use brackets. These are needed in a stream of bytes when dis cannot find out a boundaries of an instruction itself, and that is usually an indication of disam conflict between subtargets.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125700