jlebar accepted this revision.
jlebar added a comment.
This revision is now accepted and ready to land.

Yay, this is great.


================
Comment at: test/SemaCUDA/target_attr_inheritance.cu:4
@@ +3,3 @@
+// REQUIRES: x86-registered-target
+// REQUIRES: nvptx-registered-target
+
----------------
Other reviewers have pointed out to me that we don't usually (ever?) need this. 
 I think these have to do with llvm's ability to generate code for our targets, 
but it's not relevant to clang here.


https://reviews.llvm.org/D24522



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

Reply via email to