[Request for Help] Reducing gcc 4.9 compilation warnings
adrian at freebsd.org
Sun Apr 19 15:19:31 UTC 2015
On 19 April 2015 at 02:10, Eitan Adler <lists at eitanadler.com> wrote:
> On 18 April 2015 at 11:12, Craig Rodrigues <rodrigc at freebsd.org> wrote:
>> After the latest commits by members of freebsd-toolchain@ ,
>> I have managed to compile latest CURRENT world and GENERIC
>> kernel on amd64 with an gcc 4.9 external toolchain by doing:
>> pkg install devel/amd64-xtoolchain-gcc
>> cd /usr/src
>> sed -i "" -e 's/boot2//' sys/boot/i386/Makefile
>> make buildworld CROSS_TOOLCHAIN=amd64-gcc NO_WERROR=yes WERROR=
>> make buildkernel CROSS_TOOLCHAIN=amd64-gcc NO_WERROR=yes WERROR=
>> It was necessary to skip boot2 from building because of errors
>> reported here:
>> The boot2 compilation errors still need to be worked on.
>> However, most other things compile with warnings. If folks are
>> interested in looking at the warnings, you can see them here:
>> Please look at these warnings, and if you see places to
>> patch the code to eliminate the warnings, please submit patches
>> and commit them if you can.
> Some time ago I tried to do something similar, but with gcc4.6 instead
> of gcc4.9.
> The vast majority of warnings (1548) are "unused-but-set-variable" and
> it may be worthwhile to just silence this warning, at least at low
> WARNS levels. It has caught real errors but is quite noisy and can
> cause quite a bit of churn.
> "inline" (count 515) is generally pure noise, and I think we should
> disable it. It can be helpful to find certain types of performance
> optimizations, but it should be opt-in.
> "strict-aliasing" (154) are likely all real bugs of some form. Very
> few exist in contrib code, and it would be good to fix them.
> "Warning" (111) seem to be a mix of warnings, but mostly in contrib code.
> "maybe-uninitialized" (111) is often noisy, and in a spot check were
> all false positives.
> "deprecated-declarations" (22) is all in contrib code
> Many of these are useful though and I've already fixed a few of the
> bugs reported.
> Perhaps it would be useful to do a second run of this, but with a
> modified share/mk to silence the most useless of these warnings?
I just got a booting mips32 kernel using gcc-4.9.2, and boy are there
a lot of warnings. I'm going to start fixing the ones I find - cleaner
code is better code. Mostly.
(I'd be happy with -Wall -Werror.)
More information about the freebsd-testing