Anastasia added inline comments.

================
Comment at: lib/Sema/SemaDeclAttr.cpp:5067
@@ +5066,3 @@
+      if (DeclTy->isPipeType() ||
+          (S.getLangOpts().OpenCLVersion < 200 && DeclTy->isImageType())) {
+        S.Diag(D->getLocation(), diag::err_opencl_invalid_read_write)
----------------
pxli168 wrote:
> I used arc to update the patch and it case some line error.
> I have check about OpenCL version here and will handle the read_write with 
> image before OpenCL 2.0. As spec mentioned that it is reversed before OpenCL 
> 2.0. 
Do you think we could just check:
  if (S.getLangOpts().OpenCLVersion < 200 || DeclTy->isPipeType())


http://reviews.llvm.org/D16040



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

Reply via email to