[PATCH] D147245: [AMDGPU] Make amdgcn.s.buffer.load a memory-reading intrinsic

[Jay Foad via Phabricator via llvm-commits]

foad added a comment.

I had a quick look at some codegen differences this caused in Vulkan shaders compiled by LLPC <https://github.com/GPUOpen-Drivers/llpc>. The biggest difference was that the DAG nodes for scalar loads now have a chain operand, which has a big effect on DAG scheduling.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147245