[libcxx-commits] [clang] [libcxx] [C++20] [Modules] Convert '-fexperimental-modules-reduced-bmi' to '-fmodules-reduced-bmi' (PR #114382)

Chuanqi Xu via libcxx-commits libcxx-commits at lists.llvm.org
Mon Dec 9 22:56:26 PST 2024


ChuanqiXu9 wrote:

> I still find the current approach wrt the warning to be odd, we would basically force every modules user to change their command line, either by adding the new `-fmodules-reduced-bmi`, or add `-Wno-wathever` to suppress the warning, or just learn to live with the warning I guess.
> 
> I can't recall prior art on this.
> 
> If we think this has such a high chance of causing breakages, perhaps we shouldn't warn at all. Otherwise, we should just change the default.

Yeah, my intention to get more users get involved during the cycle to make the final output more stable. But it is not traditional indeed. Given it makes some problems already, let's follow the traditional step in the beginning.

https://github.com/llvm/llvm-project/pull/114382


More information about the libcxx-commits mailing list