yaxunl added a comment.

With this change, clang basically will have no knowledge about the removed 
extensions, i.e., it will not know which extension is supported in which 
version of OpenCL and have no way to enable/disable those extensions. There 
will be no way to define corresponding macros in clang.

Basically the responsibility of defining those macros will be shifted to OpenCL 
runtime for JIT and shifted to users for offline compilation. They need to have 
knowledge about which extensions are supported in which version of OpenCL and 
which cpu/platform supports them. I am not sure whether this is the direction 
we want to move to.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89372

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

Reply via email to