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

Tue May 17 00:15:04 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
----------------
kosarev wrote:
> rampitec wrote:
> > 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.
> I too see no bracketed input bytes here, only printed ones. May this be the case that you just see them on a separate line because of the larger length of the instruction?
Oh, yes. This is formatting, not really brackets on input.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125700