tra requested changes to this revision.
tra added a comment.
This revision now requires changes to proceed.

I'm not sure this is the right thing to do. What if user explicitly wants 
device-side dependencies and runs `clang --cuda-device-only -M` ? This patch 
makes it impossible.

I'd rather tell users that they have to explicitly provide *which* compilation 
they want to generate dependencies for. If they want them for the host, just 
pass along --cuda-host-only.


https://reviews.llvm.org/D47376



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to