arsenm added inline comments. ================ Comment at: include/llvm/IR/IntrinsicsAMDGPU.td:481 + [llvm_i32_ty, llvm_i32_ty, llvm_i32_ty, llvm_i32_ty], + [IntrNoMem]>; + ---------------- Shouldn't this be IntrReadMem? The doc says it reads parameters in LDS https://reviews.llvm.org/D26725