This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PR c/53063 Handle Wformat with LangEnabledBy
- From: "Joseph S. Myers" <joseph at codesourcery 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>
- Date: Sat, 3 Nov 2012 17:18:56 +0000
- Subject: Re: PR c/53063 Handle Wformat with LangEnabledBy
- References: <CAESRpQDPpB8y_JLL2hScsi+GiUGtdud1pnj+akDkxnktTthvVw@mail.gmail.com> <Pine.LNX.4.64.1210191615570.4249@digraph.polyomino.org.uk> <CAESRpQA75mqEBVM46Oz1pEGEx5T8NSPwZa4Qa3RsUxD6ZdBv+w@mail.gmail.com>
On Sat, 27 Oct 2012, Manuel López-Ibáñez wrote:
> On 19 October 2012 18:17, Joseph S. Myers <joseph@codesourcery.com> wrote:
> > On Wed, 17 Oct 2012, Manuel López-Ibáñez wrote:
> >
> >> documentation but I can also implement -Wformat=0 being an alias for
> >> -Wno-format and -Wformat=1 an alias for -Wformat and simply reject
> >> -Wno-format=.
> >
> > I think that's what's wanted; -Wno-format= should be rejected, -Wformat=
> > should take an arbitrary integer level (of which at present all those
> > above 2 are equivalent to 2, just as -O<n> for n > 3 is equivalent to
> > -O3).
>
> This patch implements exactly that. Bootstrapped and regression tested
> on x86_64-linux-gnu. OK?
OK, but unless there is a dependency between the opt*.awk refactoring and
the c-family changes, please make those two separate commits.
--
Joseph S. Myers
joseph@codesourcery.com