[PATCH] D124485: [AMDGPU] Remove vdata from buffer to lds load

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/BUFInstructions.td:497
   : MUBUF_Pseudo<opName,
-                 (outs vdata_op:$vdata),
+                 !if(isLds, (outs ), (outs vdata_op:$vdata)),
                  !con(getMUBUFIns<addrKindCopy, [], isLds>.ret,
----------------
arsenm wrote:
> rampitec wrote:
> > arsenm wrote:
> > > Extra space after outs
> > Did we finally fixed the issue when this extra space was needed?
> Not sure
Apparently we did. Removed space.


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

https://reviews.llvm.org/D124485