Anastasia added a comment.

In D96515#2559424 <https://reviews.llvm.org/D96515#2559424>, @svenvh wrote:

> It probably makes sense to update `clang/docs/UsersManual.rst` as part of 
> this change.  In particular the following sentence is no longer true after 
> this patch: "By default the OpenCL headers are not loaded and therefore 
> certain builtin types and most of builtin functions are not declared."

Yes, that's right but I think there is a bigger change that needs to be made 
i.e. I would completely remove `-finclude-default-header` and let it live on 
OpenCLSupport page. I would prefer a separate review for docs though.


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

https://reviews.llvm.org/D96515

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

Reply via email to