[PATCH] D27586: AMDGPU/SI: Add llvm.amdgcn.s.buffer.load intrinsic

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

mareko added inline comments.


================
Comment at: include/llvm/IR/IntrinsicsAMDGPU.td:409
+   llvm_i1_ty], // glc
+  [IntrReadMem, IntrArgMemOnly, NoCapture<0>]>;
+
----------------
Do these flags prevent CSE? Mesa re-loads TGSI constants on each use. There is no reuse. It relies on CSE to do its the job.


https://reviews.llvm.org/D27586