[PATCH] D16501: [CUDA] Don't generate aliases for static extern "C" functions.

Justin Lebar via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 25 11:56:47 PST 2016


jlebar added a comment.

In http://reviews.llvm.org/D16501#335225, @tra wrote:

> Failing silently is not a good idea. At the very least there should produce an error.
>  The right thing to do here, IMO, would be to generate a stub with alias name that just jumps to or calls aliasee.


Discussed IRL.  We're not breaking anyone who's not already broken -- if you currently have "__device__ __attribute__((used)) static int foo()", that's just not going to compile for you without this patch.  With this patch, at least this will work for some people; we can always add this stub function later if we want (I have concerns about potentially surprising performance implications, but we can postpone worrying about that).


http://reviews.llvm.org/D16501





More information about the cfe-commits mailing list