jlebar added inline comments.

================
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
----------------
jlebar wrote:
> echristo wrote:
> > Why does this require a registered target? Pretty sure TargetInfo should be 
> > able to cope with all of these.
> Honestly I've been cargo-culting those lines from other SemaCUDA tests.  
> Would you mind elaborating on what they're for?
Removed them in the meantime.


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