[PATCH] D109002: [OpenCL] Supports optional image types in C++ for OpenCL 2021

Wed Sep 1 02:45:57 PDT 2021

Topotuna added inline comments.

================
Comment at: clang/lib/Sema/SemaType.cpp:1729
     bool IsOpenCLC30 = (S.getLangOpts().OpenCLVersion == 300);
+    bool IsOpenCLC30Comp = S.getLangOpts().getOpenCLCompatibleVersion() == 300;
     // OpenCL C v3.0 s6.3.3 - OpenCL image types require __opencl_c_images
----------------
Anastasia wrote:
> I think we should just replace `IsOpenCLC30` as it is when only used in the diagnostic that we should report the same as for OpenCL 3.0.
> 
> 
> Also I would suggest changing name to `IsOpenCLC30Compatible` to make it clearer. 
That diagnostic is responsible for `__opencl_c_3d_image_writes` feature while current commit addresses `__opencl_c_images`. I wanted to keep commits separate and was planning to finish transitioning from `IsOpenCLC30` to `IsOpenCLC30Compatible` in a future commit.

I agree that `IsOpenCLC30Compatible` would be clearer. Thank you.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109002