arsenm added inline comments.
================ Comment at: lib/CodeGen/CGBuiltin.cpp:8391-8398 + case AMDGPU::BI__builtin_amdgcn_ds_permute: + return emitBinaryBuiltin(*this, E, Intrinsic::amdgcn_ds_permute); + case AMDGPU::BI__builtin_amdgcn_ds_bpermute: + return emitBinaryBuiltin(*this, E, Intrinsic::amdgcn_ds_bpermute); + case AMDGPU::BI__builtin_amdgcn_readfirstlane: + return emitUnaryBuiltin(*this, E, Intrinsic::amdgcn_readfirstlane); + case AMDGPU::BI__builtin_amdgcn_readlane: ---------------- yaxunl wrote: > arsenm wrote: > > Since these don't have mangling, you could add GCCBuiltin to the intrinsic > > definition and then you wouldn't need these to be handled here > If I remove these, there will be error: > > cannot compile this builtin function yet > > > Yes, the GCCBuiltin isn't on the intrinsic declaration in llvm. If you fix that you shouldn't see that https://reviews.llvm.org/D30551 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
