echristo accepted this revision.
echristo added a reviewer: echristo.
echristo added a comment.
This revision is now accepted and ready to land.

OK. One question inline, but feel free to commit after answering one way or the 
other :)

-eric


================
Comment at: test/SemaCUDA/alias.cu:1-2
@@ +1,3 @@
+// REQUIRES: x86-registered-target
+// REQUIRES: nvptx-registered-target
+// RUN: %clang_cc1 -triple nvptx-unknown-cuda -fsyntax-only -fcuda-is-device 
-verify -DEXPECT_ERR %s
----------------
Why does this require a registered target? Pretty sure TargetInfo should be 
able to cope with all of these.


http://reviews.llvm.org/D16502



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

Reply via email to