[PATCH] D103911: [OpenCL] Add support of __opencl_c_images feature macro

[Anton Zabaznov via Phabricator via cfe-commits]

azabaznov added inline comments.


================
Comment at: clang/test/SemaOpenCL/unsupported-image.cl:1
+// RUN: %clang_cc1 -verify -cl-std=CL3.0 -cl-ext=-__opencl_c_images %s
+// RUN: %clang_cc1 -verify -cl-std=CL3.0 -cl-ext=+__opencl_c_images %s
----------------
Anastasia wrote:
> svenvh wrote:
> > Should we add `-triple spir`, as some of the image types (e.g. msaa) are part of an extension that may not be available for every default triple?
> yes, the test might fail in some bots otherwise...
> 
> 
> Btw it would be good to test that 3d images work correctly for OpenCL 3. Would it work if we add `RUN` line with `CL3.0` into `test/SemaOpenCL/access-qualifier.cl` or are we missing some more work to enable the test?
> Should we add -triple spir, as some of the image types (e.g. msaa) are part of an extension that may not be available for every default triple?

> yes, the test might fail in some bots otherwise...

Yes, that's correct. Thanks. Now it works locally because the default triple I have is x86 which supports all the extensions as well as SPIR.

> Btw it would be good to test that 3d images work correctly for OpenCL 3. Would it work if we add RUN line with CL3.0 into test/SemaOpenCL/access-qualifier.cl or are we missing some more work to enable the test?

Yeah, to properly test 3d images actually we need to add `__opencl_c_images` support first, because `__opencl_c_3d_image_writes` depends on it


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D103911/new/

https://reviews.llvm.org/D103911