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

[Tom Stellard via Phabricator via llvm-commits]

tstellarAMD added a comment.

In https://reviews.llvm.org/D27586#662991, @nhaehnle wrote:

> I haven't looked in too much detail yet. I assume getelementptr doesn't work with these pointers, so it would be good to have a negative test which ensures that GEP use fails.


That's correct.  In order to support GEP, we would need to make i128 legal in the backend, which would be a pretty significant change.  GlobalISel will make this much much easier, so I'm not sure it's even worth trying to support with SelectionDAG.


https://reviews.llvm.org/D27586