[clang] [C++20] [Modules] Convert '-fexperimental-modules-reduced-bmi' to '-fmodules-reduced-bmi' (PR #114382)
Chuanqi Xu via cfe-commits
cfe-commits at lists.llvm.org
Sun Nov 10 18:16:40 PST 2024
ChuanqiXu9 wrote:
> 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?
I originally mentioned it in https://github.com/llvm/llvm-project/pull/85050 and then in the document. And it looks like no one had a question about it.
>
> 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.
We can disable it with the same way with other warnings, e.g., `-Wno-missing-reduced-bmi`
https://github.com/llvm/llvm-project/pull/114382
More information about the cfe-commits
mailing list