[PATCH] D11663: [NVPTX] Added an option to run NVVMReflect pass.
Artem Belevich via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 8 13:04:02 PDT 2015
tra added a comment.
In http://reviews.llvm.org/D11663#241611, @echristo wrote:
> Seems this could be unified with with the nvvm-reflect-enable option in the pass itself? Actually, is there any reason this shouldn't be on by default for the backend? (i.e. am I missing something here?)
Your point makes sense as linking with libdevice will be the common case for compiling real CUDA apps.
Adding NVVMReflect pass unconditionally would break whoever may want to compile their own libdevice variant.
It can be easily fixed with -nvvm-reflect-enable=0, so it should not be too big of a deal.
More information about the llvm-commits