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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to