[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.


Repository:
  rL LLVM

http://reviews.llvm.org/D20297





More information about the llvm-commits mailing list