[cfe-commits] r141053 - in /cfe/trunk: include/clang/Basic/DiagnosticDriverKinds.td lib/Driver/Tools.cpp
Bob Wilson
bob.wilson at apple.com
Tue Oct 4 09:03:01 PDT 2011
I'm not sure this is a good idea. I understand the concern about having -W flags for all warnings, but we certainly don't anyone to start using -Wno-unsupported-debug-with-integrated-as. As soon as the integrated assembler can support debug info, we're going to rip out this warning, and we don't want to be stuck keeping the -W flags around for backward compatibility. Is there some existing warning group that we could use for this?
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
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
More information about the cfe-commits
mailing list