Anastasia added a comment.

I am generally not clear about the scope of this patch. Is it intended for 
OpenCL or C++?

As for OpenCL, boolean vector types are not permitted (Table 6.2). Also 
conversions between vector types are disallowed in general (Section 6.2.2) but 
unfortunately Clang doesn't diagnose it properly at the moment. :(


================
Comment at: test/CodeGenCXX/bool-vector-conversion.cpp:8
@@ +7,3 @@
+
+// Nothing but OpenCL allows vectors of booleans.
+// CHECK-LABEL: @_Z4testv
----------------
Can you explain why? Table 6.2 doesn't list it among allowed valid types and 
Table 6.4 only says it's a reserved data type, but doesn't allow to use in 
application code anyways (Section 6.1.4).


http://reviews.llvm.org/D15721



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

Reply via email to