svenvh added a comment.

In https://reviews.llvm.org/D28860#653662, @yaxunl wrote:

> The tests should be added to SemaOpenCL/extension-version.cl


SemaOpenCL/extension-version.cl already has a test for the 
cl_khr_3d_image_writes extension.



================
Comment at: lib/Sema/SemaType.cpp:6683
+  // access qualifier unless the cl_khr_3d_image_writes extension is enabled.
+  if (CurType->isOCLImage3dWOType() &&
+      !S.getOpenCLOptions().isEnabled("cl_khr_3d_image_writes")) {
----------------
yaxunl wrote:
> should modify OpenCLImageTypes.def for image types associated with an 
> extension. The diagnostics will be emitted automatically.

That's neat, thanks for pointing it out!


https://reviews.llvm.org/D28860



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

Reply via email to