Messages in this thread | ![/](/images/icornerl.gif) | | Date | Mon, 24 Feb 2003 15:49:09 -0500 (EST) | From | "Richard B. Johnson" <> | Subject | Re: [PATCH] s390 (7/13): gcc 3.3 adaptions. |
| |
On Mon, 24 Feb 2003, Arnd Bergmann wrote:
> Richard B. Johnson wrote: > > > I think you must keep these warnings in! There are many bugs > > that these uncover uncluding loops that don't terminate correctly > > but seem to work for "most all" cases. These are the hard-to-find > > bugs that hit you six months after release. > > This was my change. Obviously the warning is a good idea in general, > but I don't see the point of scrolling through hundreds of lines > with the same warning in someone else's code. I actually plan to fix > these warnings in arch/s390 and drivers/s390 as well as include/ > and make the s390 kernel compile with -Werror, but the rest looks > more like a task for the Janitors. Note that before gcc-3.3, > -Wsign-compare has not been part of -Wall. >
Yep. There are a lot of things, not part of -Wall. To find 'need coffee' errors, I use:
-Wall -Werror -Wstrict-prototypes -Wwrite-strings -Wshadow \ -Wsign-compare -Wno-trigraphs
plus... -fno-builtin -fno-common -fomit-frame-pointer.
I also use -march=1486 so it will boot. Some machines don't like `cmov` and other stuff the compiler may emit.
Cheers, Dick Johnson Penguin : Linux version 2.4.18 on an i686 machine (797.90 BogoMips). Why is the government concerned about the lunatic fringe? Think about it.
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| ![\](/images/icornerr.gif) |