[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 mailing list