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

[Chad Rosier]

Thanks for taking care of this and I apologize for not doing so in the first place.

 Regards,
  Chad

On Oct 4, 2011, at 7:54 AM, Ted Kremenek wrote:

> 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
>