Patch: Enforce -Werror during bootstrap

Zack Weinberg zack@codesourcery.com
Tue Jan 14 07:36:00 GMT 2003


"Kaveh R. Ghazi" <ghazi@caip.rutgers.edu> writes:

> Here's an attempt to make use of -Werror during bootstrap to stop
> warning regressions from creeping back in.
>
> Following Zack's suggestion from the last time I tried this:
> http://gcc.gnu.org/ml/gcc-patches/2001-09/msg00280.html
...

I hate to rain on your parade, but my attitude toward this is still
pretty much what it was back then:

| I think this is backward.  First we should get to zero warnings
| throughout the compiler, _then_ we talk about -Werror.  If only
| because if we do it your way, we'll be adding explicit compile rules
| to module after module.  I don't think there's enough marginal benefit
| from -Werror applied to a few modules, to eat that kind of complexity
| burden in the Makefile which is already too complicated.
[http://gcc.gnu.org/ml/gcc-patches/2001-09/msg00129.html]

You've got the complexity burden down as far as it's going to go, but
think about the consequence of this: there's eventually going to be a
long long list of

foobar.o-warn = $(WERROR_FLAGS)

which will, I venture to guess, never go away.  Now think about what
will happen when (not if) you start needing to conditionalize this on
target triples.

zw



More information about the Gcc-patches mailing list