[clang] Downgrade error to warning (PR #117840)
via cfe-commits
cfe-commits at lists.llvm.org
Tue Nov 26 21:53:30 PST 2024
https://github.com/cor3ntin requested changes to this pull request.
I don't think the discussion here has run its course
https://github.com/llvm/llvm-project/pull/115416
In particular, I agree with @boris-kolpackov that this is likely to lead to hard-to understand ODR violations issues, which is the exact things modules were meant to solve.
_If_ there was evidence of deployment challenges, we could consider a warning-defaulting-to-an-error, but no less, imo.
(But then again, deployment issues in a projects that mixes compilation flags is expected and it's very unclear to be there is value in deploying modules without first ensuring a consistent build environment)
@Bigcheese @AaronBallman
https://github.com/llvm/llvm-project/pull/117840
More information about the cfe-commits
mailing list