This is the mail archive of the
mailing list for the GCC project.
Re: werror fallout for cross-builds (was: Re: [BUILDROBOT][PATCH] Fix mmix (unused variable))
- From: "Maciej W. Rozycki" <macro at linux-mips dot org>
- To: Mike Stump <mikestump at comcast dot net>
- Cc: Hans-Peter Nilsson <hp at bitrange dot com>, Richard Biener <rguenther at suse dot de>, Jan-Benedict Glaw <jbglaw at lug-owl dot de>, gcc-patches at gcc dot gnu dot org
- Date: Thu, 24 Jul 2014 12:08:11 +0100 (BST)
- Subject: Re: werror fallout for cross-builds (was: Re: [BUILDROBOT][PATCH] Fix mmix (unused variable))
- Authentication-results: sourceware.org; auth=none
- References: <20140718020136 dot GT21544 at lug-owl dot de> <alpine dot BSF dot 2 dot 02 dot 1407180256200 dot 66146 at arjuna dot pair dot com> <20140718124444 dot GU21544 at lug-owl dot de> <alpine dot BSF dot 2 dot 02 dot 1407182023490 dot 6913 at arjuna dot pair dot com> <alpine dot LSU dot 2 dot 11 dot 1407221456290 dot 3531 at zhemvz dot fhfr dot qr> <alpine dot BSF dot 2 dot 02 dot 1407221017320 dot 68621 at arjuna dot pair dot com> <107903C5-B7A0-46F9-A57A-99EF1C823F65 at comcast dot net> <alpine dot BSF dot 2 dot 02 dot 1407222028520 dot 85545 at arjuna dot pair dot com> <9E250913-2B11-49BD-BE89-F3C618426A3D at comcast dot net>
On Tue, 22 Jul 2014, Mike Stump wrote:
> Then Iʼm shadow boxing. I assumed that people wanted to turn it on by
> default. Iʼm all for that, I think it is a good idea and a fine
> direction. :-) The only limitation is whitelisting exactly when it
> pops on and preflighting those at least once to ensure they are clean.
I think at the very least the thing should be on whenever building with
itself, that is the build compiler's version is the same as the version of
the compiler being built. That can be probably reasonably broadened to
any compiler bearing the same major.minor version (or just major if we
switch to the two-part versioning scheme recently proposed).
The thing is to bring the code base to always compile without warnings
and then not to let it regress. Warnings are too easy to miss and
sometimes are a symptom of actual breakage rather than just harmless
noise, i.e. code builds and runs, but does something silly. I have wasted
hours of debugging time already on chasing such problems.