[PATCH] D20297: AMDGPU/SI: Add implicitarg.segment.ptr intrinsic.

Jan Vesely via llvm-commits llvm-commits at lists.llvm.org
Thu May 26 10:58:16 PDT 2016

jvesely added a comment.

In http://reviews.llvm.org/D20297#441156, @arsenm wrote:

> Why not the offset from the base pointer intrinsic rather than an intrinsic to the offset?

Does not really matter, but pointer appeared more flexible. if you decide to move implicit args elsewhere (that is not offset to kernarg segment) you only need to reimplement the intrinsic, without updating library.



More information about the llvm-commits mailing list