yaxunl 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`? ndrange_t is an OpenCL builtin type and the spec does not define its layout. There are sema checks which requires to check if a type is ndrange_t. How to do the check if it is defined as a typedef of a common struct type? How about implement it as a child class of Type then typedef it? Repository: rL LLVM https://reviews.llvm.org/D23086 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
