yaxunl added a comment.

In D78979#2007643 <https://reviews.llvm.org/D78979#2007643>, @Anastasia wrote:

> ah I guess if we leave it under `-cc1 ` we will have the command line 
> interface as follows:
>
> - Driver (without `-cc1`) adds OpenCL header by default that can be 
> overridden by the flag added in this patch.
> - Frontend (with `-cc1`) doesn't add the header by default but there are two 
> flags `-fdeclare-opencl-builtins` or `-finclude-default-header` that allow to 
> include the header.
>
>   Is my understanding correct? We should reflect this is the User Manual.


I think -finclude-default-header -fno-include-default-header better becomes 
both driver and -cc1 option since users are likely to use it with driver. It 
may also be used with other languages e.g. HIP.


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

https://reviews.llvm.org/D78979



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

Reply via email to