https://github.com/mizvekov commented:

I think the overall plan is fine, thanks for working on it!

I think the idea of issuing a warning in order to get users to opt-in to a flag 
is somewhat novel. Have I missed discussion about this approach, versus simply 
changing the default?

Regardless, for any new warning we add here, we will probably need to add a 
targeted way to disable that warning, as otherwise users of Werror are likely 
to complain.

https://github.com/llvm/llvm-project/pull/114382
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to