[PATCH] D23086: [OpenCL] Generate concrete struct type for ndrange_t

[Anastasia Stulova via cfe-commits]

Anastasia added a comment.

In https://reviews.llvm.org/D23086#507055, @majnemer wrote:

> This approach seems wrong to me.
>
> Instead, why not just make `ndrange_t` a typedef of a real struct in `Sema::Initialize`?


I think we have an issue because in that case during the diagnostic of enqueue_kernel Builtin in SemaChecking.cpp we won't have a good way to identify this type apart from checking its name... That's why ideally we would like to leave it as a special Builtin type.

What kind of issues do you see with the current implementation?

Also do you suggest to create the struct in Sema::Initialize? Alternatively, we also have an OpenCL header file for such things.


Repository:
  rL LLVM

https://reviews.llvm.org/D23086