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

Tom Stellard via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 1 03:56:39 PST 2017

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.


More information about the llvm-commits mailing list