This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] PR28901 Add two levels for -Wunused-const-variable.
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Jeff Law <law at redhat dot com>
- Cc: Mark Wielaard <mjw at redhat dot com>, gcc-patches at gcc dot gnu dot org, Bernd Schmidt <bschmidt at redhat dot com>, Martin Sebor <msebor at gmail dot com>, sellcey at imgtec dot com, Manuel López-Ibáñez <lopezibanez at gmail dot com>
- Date: Mon, 22 Feb 2016 20:00:20 +0100
- Subject: Re: [PATCH] PR28901 Add two levels for -Wunused-const-variable.
- Authentication-results: sourceware.org; auth=none
- References: <5603E4D3 dot 4050806 at redhat dot com> <1456018952-25270-1-git-send-email-mjw at redhat dot com> <56CB5A34 dot 8030301 at redhat dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Mon, Feb 22, 2016 at 11:57:56AM -0700, Jeff Law wrote:
> >diff --git a/gcc/cgraphunit.c b/gcc/cgraphunit.c
> >index 0a745f0..27a073a 100644
> >--- a/gcc/cgraphunit.c
> >+++ b/gcc/cgraphunit.c
> >@@ -971,7 +974,7 @@ check_global_declaration (symtab_node *snode)
> > (TREE_CODE (decl) == FUNCTION_DECL)
> > ? OPT_Wunused_function
> > : (TREE_READONLY (decl)
> >- ? OPT_Wunused_const_variable
> >+ ? OPT_Wunused_const_variable_
> Typo here?
The _ stands for = from the option name, as -Wunused-const-variable
is an alias to -Wunused-const-variable=2, I think we want the trailing
underscore.
Jakub