[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