[cfe-commits] r141053 - in /cfe/trunk: include/clang/Basic/DiagnosticDriverKinds.td lib/Driver/Tools.cpp

[Ted Kremenek]

Hi Chad,

I went ahead and reverted r141060, and added the warning under a -Wflag (r141078).  I know it seems weird to put driver warnings under -W flags, but it is consistent with out goal of putting every warning under a -Wflag.  This allows users to suppress any warning (group) they choose, or promote any warning group to an error.

Cheers,
Ted

On Oct 3, 2011, at 9:46 PM, Chad Rosier wrote:

> Should be fixed with r141060.  Sorry for the breakage.
> 
> Chad
> 
> On Oct 3, 2011, at 8:21 PM, NAKAMURA Takumi wrote:
> 
>> FYI, diagtool list-warnings shows me;
>> 
>> Warnings with flags (355):
>> (snip)
>> Warnings without flags (305):
>> (snip)
>> 
>> STATISTICS:
>> 
>> Percentage of warnings with flags: 53.79%
>> Number of unique flags: 168
>> Average number of diagnostics per flag: 2.113
>> 
>> ******************** TEST 'Clang :: Misc/warning-flags.c' FAILED
>> ********************
>> Script:
>> --
>> diagtool list-warnings 2>&1 | FileCheck
>> /home/chapuni/bb/cmake-clang-x86_64-linux/llvm-project/clang/test/Misc/warning-flags.c
>> --
>> Exit Code: 1
>> Command Output (stderr):
>> --
>> /home/chapuni/bb/cmake-clang-x86_64-linux/llvm-project/clang/test/Misc/warning-flags.c:20:8:
>> error: expected string not found in input
>> CHECK: Warnings without flags (304):
>>      ^
>> <stdin>:1:1: note: scanning from here
>> Warnings with flags (355):
>> ^
>> --
>> 
>> ********************
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits