This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Patch: Enforce -Werror during bootstrap


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

> Zack - Here's what I meant.  This patch survives bootstrap on
> sparc-sun-solaris2.7.  As you can see, the list of bypass files is
> rather small and I can zap a few more pretty easily.  A benefit of
> this patch (as opposed to the previous one) is that as new files get
> added to GCC we don't have to remember to add -Werror, it happens
> automatically.
>
> I can also address your point about needing to conditionalize on
> target triples by adding bypass elements to individual target makefile
> fragments if/when necessary.

I think this rather reemphasizes my point - the set of warnings I see
on i686-linux is larger and only partially overlapping the set you're
getting on solaris.  This is a build from this morning, so before the
patch you posted to deal with a few warnings; some of the difference
is addressed by that patch.

# Number of warnings per file:
#       5 objc-parse.c
#       5 gtype-desc.c
#       5 c-parse.c
#       4 insn-conditions.c
#       4 gtype-cp.h
#       3 gcc/tlink.c
#       2 gcc/gcc.c
#       2 gcc/f/sta.c
#       2 gcc/crtstuff.c
#       2 gcc/cp/decl2.c
#       1 insn-attrtab.c
#       1 gt-varasm.h
#       1 gt-c-decl.h
#       1 gengtype-yacc.c
#       1 gcc/timevar.c
#       1 gcc/ra-build.c
#       1 gcc/loop.c
#       1 gcc/local-alloc.c
#       1 gcc/java/jvspec.c
#       1 gcc/java/jcf-write.c
#       1 gcc/java/builtins.c
#       1 gcc/function.c
#       1 gcc/emit-rtl.c
#       1 gcc/cp/pt.c
#       1 gcc/collect2.c
#       1 gcc/ada/tracebak.c
#       1 ada/b_gnatb.c

I continue to believe that -Werror is premature.

zw


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]