[PATCH] D20691: AMDGPU: Remove gcc builtin names from workitem intrinsics

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


jvesely added a comment.

In http://reviews.llvm.org/D20691#441248, @arsenm wrote:

> Why do you need this? I know I had a patch to do the same thing at some point, but don't remember why. Since these don't need mangling I would expect GCCBuiltin to work correctly


clang in CodeGenFunction::EmitBuiltinExpr() considers embedded intrinsic names before calling EmitTargetBuiltinExpr(). Since we want these intrinsics to be emitted with range metadata we need to do it in EmitAMDGPUBuiltinExpr(called by EmitTargetBUiltinExpr). The embedded names need to be removed, otherwise the Emit* functions won't run.


Repository:
  rL LLVM

http://reviews.llvm.org/D20691





More information about the llvm-commits mailing list