[PATCH] D41468: AMDGPU: Implement getTgtMemIntrinsic for images

[Marek Olšák via Phabricator via llvm-commits]

mareko added inline comments.


================
Comment at: lib/Target/AMDGPU/SIISelLowering.cpp:680
+      *MF.getSubtarget<SISubtarget>().getInstrInfo(),
+      CI.getArgOperand(1));
+    Info.flags = MachineMemOperand::MOStore |
----------------
rsrc is ArgOperand(2) if I'm reading IntrinsicsAMDGPU.td correctly.


================
Comment at: lib/Target/AMDGPU/SIISelLowering.cpp:703
+      *MF.getSubtarget<SISubtarget>().getInstrInfo(),
+      CI.getArgOperand(1));
+
----------------
Same here.


https://reviews.llvm.org/D41468