This is the mail archive of the
mailing list for the GCC project.
Re: egcs, warning patches
- To: "Kaveh R. Ghazi" <ghazi at caip dot rutgers dot edu>
- Subject: Re: egcs, warning patches
- From: Jeffrey A Law <law at cygnus dot com>
- Date: Mon, 17 Aug 1998 10:08:00 -0600
- cc: egcs-patches at cygnus dot com
- Reply-To: law at cygnus dot com
In message <199807111802.OAA21699@caip.rutgers.edu>you write:
> Here's a minor set of warning patches.
> The one patch appearing to need explanation might be varray.h.
> In that case gcc was issuing a warning about a value computed not being
> used. I looked at the VARRAY_FREE definition and apart from not seeing
> a case where you'd ever want to evaluate it, the expression looks like
> it'll always evaluate to zero anyway. So I replaced it with a
> do-while(0) to kill the warning.
> Okay to install?
> Sat Jul 11 10:57:02 1998 Kaveh R. Ghazi <email@example.com>
> * c-decl.c (init_decl_processing): Remove unneeded &.
> * alpha.h (alpha_initialize_trampoline): Provide prototype.
> * cse.c (cse_basic_block): Add parentheses around assignment used
> as truth value.
> * except.c (set_exception_lang_code, set_exception_version_code):
> Change parameter from `short' to `int' to avoid using a gcc
> * except.h (set_exception_lang_code, set_exception_version_code):
> Likewise for prototypes.
> * flow.c (count_reg_references): Remove unused variables `regno'
> and `i'.
> * gcse.c (hash_scan_insn): Declare parameter `in_libcall_block'.
> * prefix.c (translate_name): Cast the result of `alloca'.
> * regclass.c (allocate_reg_info): Remove unneeded &.
> * varray.h (VARRAY_FREE): Reimplement as a `do-while(0)' statement.
The cse.c patch was already in the mainline tree.
And there's still some questions about the regclass.c patch.
I've installed the rest of these changes into the mainline tree.