[PATCH] Add specific warning flags for GNU ext in Sema

Peter N Lewis peter at stairways.com.au
Tue Sep 17 19:28:11 PDT 2013


On 18/09/2013, at 6:38 , Eli Friedman <eli.friedman at gmail.com> wrote:
> Sorry about the delay.
> 
> LGTM.

No problem.  Eli, if you're happy, unless it needs a seconder, please commit it for me.

Thanks,
   Peter.

> 
> On Mon, Sep 16, 2013 at 7:16 PM, Peter N Lewis <peter at stairways.com.au> wrote:
> Ping x 2.
> 
> On 05/09/2013, at 18:33 , Peter Lewis <peter at stairways.com.au> wrote:
> 
> > This patch adds the following specific warning flags to -Wgnu:
> >
> > gnu-anonymous-struct
> > gnu-compound-literal-initializer
> > gnu-empty-struct
> > gnu-flexible-array-initializer
> > gnu-flexible-array-union-member
> > gnu-folding-constant
> > redeclared-class-member
> > gnu-redeclared-enum
> > gnu-union-cast
> > gnu-variable-sized-type-not-at-end
> >
> > At Eli's suggestion, redeclared-class-member does not have the gnu prefix, and this patch subsumes ext_vla_folded_to_constant into gnu-folding-constant (a visible effect of this is that -Wgnu enables gnu-folding-constant which enabled ext_vla_folded_to_constant where previously it would not have).
> >
> > http://llvm-reviews.chandlerc.com/D1605
> >
> > Files:
> > test/Sema/gnu-flags.c
> > test/Misc/warning-flags-tree.c
> > test/Misc/warning-flags.c
> > test/SemaCXX/gnu-flags.cpp
> > include/clang/Basic/DiagnosticGroups.td
> > include/clang/Basic/DiagnosticSemaKinds.td
> > <D1605.1.patch>
> 
> --
> Keyboard Maestro 6.2 now out - control Mail, reveal a file, format AppleScripts and more.
> 
> Keyboard Maestro <http://www.keyboardmaestro.com/> Macros for your Mac
> <http://www.stairways.com/>           <http://download.keyboardmaestro.com/>
> 
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> 

-- 
Keyboard Maestro 6.2 now out - control Mail, reveal a file, format AppleScripts and more.

Keyboard Maestro <http://www.keyboardmaestro.com/> Macros for your Mac
<http://www.stairways.com/>           <http://download.keyboardmaestro.com/>





More information about the cfe-commits mailing list