yaxunl added a comment.
In https://reviews.llvm.org/D45223#1056187, @rjmccall wrote:
> I think the appropriate place to do this is in IsStandardConversion,
> immediately after the call to ResolveAddressOfOverloadedFunction. You might
> want to add a general utility for getting the type-of-reference of a function
> decl.
We may need to resolve overloaded functions with dropped calling conventions,
e.g.
__global__ void EmptyKernel(float) {}
__global__ void EmptyKernel(double) {}
struct Dummy {
/// Type definition of the EmptyKernel kernel entry point
typedef void (*EmptyKernelPtr)(float);
EmptyKernelPtr Empty() { return EmptyKernel; }
};
In this case we have to drop the calling convention during the resolution.
Since the calling convention is invisible in the AST, why don't why just do not
represent it in AST?
Going back to the original implementation in CodeGen:
if ((getTriple().getArch() == llvm::Triple::amdgcn) &&
D->hasAttr<CUDAGlobalAttr>())
Fn->setCallingConv(llvm::CallingConv::AMDGPU_KERNEL);
It is much simpler and straightforward.
Can we just reconsider implement this in CodeGen instead of Sema?
https://reviews.llvm.org/D45223
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
