This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: A few warning patches ...
- To: "Kaveh R. Ghazi" <ghazi at caip dot rutgers dot edu>
- Subject: Re: A few warning patches ...
- From: Jeffrey A Law <law at cygnus dot com>
- Date: Sun, 25 Oct 1998 13:29:52 -0700
- cc: egcs-patches at cygnus dot com
- Reply-To: law at cygnus dot com
In message <199810251757.MAA03907@caip.rutgers.edu>you write:
> Sun Oct 25 09:24:27 1998 Kaveh R. Ghazi <ghazi@caip.rutgers.edu>
>
> * Makefile.in (recog.o): Depend on toplev.h.
> (insn-emit.o): Depend on recog.h.
> (insn-peep.o): Depend on recog.h and insn-config.h.
Fine.
> * combine.c (simplify_set): Remove unused variable `scratches'.
Fine.
> * final.c (final_scan_insn): Wrap declaration of variables `vlen'
> and `idx' in macro conditional controlling their use.
Fine.
> * genemit.c (main): Make the generated output file include
> recog.h. Don't have it declare `insn_operand_constraint', since
> we get it from recog.h.
Fine.
> * genpeep.c (main): Make the generated output file include
> insn-config.h and recog.h.
Fine.
> * recog.h: Include toplev.h.
> (extract_insn): Remove unused variable `p'.
That should be "recog.c", not "recog.h" in the ChangeLog entry.
> * regclass.c (fix_register): Add missing braces around initializer
> for `what_option'.
> (allocate_reg_info): Move variable `i' into the scope where it is
> used. Change its type to `size_t'.
Fine.
> * reload1.c (reg_eliminate): Add missing initializers.
(delete_output_reload): Add explicit braces to avoid ambiguous `else'.
Can we delay this just a little while longer. I'd like to avoid mostly
cosmetic changes to reload until after we install Bernd's patch to avoid
suprious conflicts.
jeff