[PATCH] D124884: [AMDGPU] Add intrinsic llvm.amdgcn.raw.buffer.load.lds

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp:4289
+    .addImm(HasVIndex ? -1 : 0)     // idxen(imm)
+    .addImm(MemTy.getSizeInBytes()) // data byte size
+    .addMemOperand(MMO);
----------------
arsenm wrote:
> Why is this needed if it's in the MMO?
I believe to select based on the MMO I would need to write a complex pattern.


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

https://reviews.llvm.org/D124884