[PATCH] D20297: AMDGPU/SI: Add amdgcn versions of remaining builtins

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Mon May 16 13:53:07 PDT 2016


arsenm added a comment.

In http://reviews.llvm.org/D20297#431298, @jvesely wrote:

> In http://reviews.llvm.org/D20297#431268, @arsenm wrote:
>
> > We don't actually want these. We now have the kernarg.segment.ptr intrinsic, so the library should just directly read the offsets from there
>
>
> I assume it points to the beginning of the kernel args.
>  Is the idea to have another intrinsic to read from the end (workdim, global offset), or is it OK to have intrinsics for those?


We should fix clover to not read from the end of the arguments (I thought this is what it already did)?


Repository:
  rL LLVM

http://reviews.llvm.org/D20297





More information about the llvm-commits mailing list