Anastasia added a comment.


> 
> 
>> I am looping in @svenvh to discuss this further. Sven, do you think we 
>> should rename this flag due to current proposed change? Should it be moved 
>> out of `-cc1` too?
> 
> I would leave `-fdeclare-opencl-builtins` as it is, until it is complete and 
> taken out of its "experimental" status.  Then we can discuss if we want to 
> use TableGen instead of the header as a default.

Would it not become confusing since the builtins are going to be included by 
default? Should we rename the flag at least? Also ideally it should be 
documented in https://clang.llvm.org/docs/UsersManual.html#opencl-header


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