This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
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