This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] remove obsolete code for Wuninitialized
- From: Richard Guenther <richard dot guenther at gmail dot com>
- To: Manuel López-Ibáñez <lopezibanez at gmail dot com>
- Cc: Gcc Patch List <gcc-patches at gcc dot gnu dot org>, "Joseph S. Myers" <joseph at codesourcery dot com>
- Date: Sun, 3 May 2009 19:22:55 +0200
- Subject: Re: [PATCH] remove obsolete code for Wuninitialized
- References: <6c33472e0905031006u51017f9fhee8c57264bc7f210@mail.gmail.com>
On Sun, May 3, 2009 at 7:06 PM, Manuel López-Ibáñez
<lopezibanez@gmail.com> wrote:
> Now that Wuninitialized is enabled at -O0, set_Wextra is not needed.
> Also, Wextra can be handled automatically.
>
> Not sure if this qualifies as obvious, so bootstrapped and regression
> tested on x86-64-unknown-linux-gnu.
>
> OK for trunk?
It's not obvious. The middle-end parts are ok but I cannot approve
the C frontend
changes.
Richard.
> 2009-05-03 ?Manuel López-Ibáñez ?<manu@gcc.gnu.org>
>
> ? ? ? ?* flags.h (extra_warnings): Delete.
> ? ? ? ?* toplev.c (process_options): Handle Wuninitialized here.
> ? ? ? ?* opts.c (extra_warnings): Delete.
> ? ? ? ?(set_Wextra): Delete.
> ? ? ? ?(common_handle_option): -Wextra can be handled automatically.
> ? ? ? ?* c-opts.c (c_common_handle_option): Delete obsolete code.
> ? ? ? ?(c_common_post_options): Simplify comment.
> ? ? ? ?* common.opt (W): Add Var.
> ? ? ? ?(Wextra): Add Var.
> ? ? ? ?(Wuninitialized): Initialize to -1.
>