azabaznov added a comment.

> I didn't want to end up with a big refactoring from this patch but now I 
> start to think that considering the situation it might be unavoidable. :(

Well, it was expected. However, I didn't want to mess this refactoring with 
OpenCL C 3.0 support at first.

I already do have some progress in this direction: 
https://reviews.llvm.org/D92277. We can differentiate between core and optional 
features by the defining them appropriately in `OpenCLExtensions.def`. Note 
that this also removes duplication of `OpenCLOptions` so we can query all 
features/extensions from `TargetInfo` directly. This will also allow 
simplifying `OpenCLOptions` in future


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

https://reviews.llvm.org/D89869

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

Reply via email to