[PATCH] D20948: [OpenCL] Fix access qualifiers handling for typedefs

Anastasia Stulova via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 7 09:25:00 PDT 2016


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

LGTM! Could you please address these last minor comments though before committing.

Thanks!


================
Comment at: test/SemaOpenCL/access-qualifier.cl:10
@@ +9,3 @@
+#if __OPENCL_C_VERSION__ >= 200
+  typedef read_write image1d_t img1d_rw;
+#endif
----------------
don't indent here!

================
Comment at: test/SemaOpenCL/access-qualifier.cl:68
@@ +67,3 @@
+#else
+kernel void k13(__read_write image1d_t i){} // expected-error{{access qualifier '__read_write' can not be used for '__read_write image1d_t' earlier than OpenCL version 2.0}}
+#endif
----------------
It reads a bit strange. Would it be better:
"earlier than OpenCL version 2.0" -> "prior to OpenCL version 2.0"


http://reviews.llvm.org/D20948





More information about the cfe-commits mailing list