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

David Blaikie dblaikie at gmail.com
Tue Oct 4 09:27:46 PDT 2011


Could/should we support -Wno-* by default anyway? I mean if we don't
have the warning it is certainly not on.
(on the other hand, as a user I wouldn't mind knowing if I'm passing
useless flags. Except when I'm trying to use multiple compilers and
turn something off in one which isn't present in the other. Minor
convenience to have that silently(ish) accepted on the no supporting
compiler)
From: Bob Wilson
Sent: 10/4/2011 9:04 AM
To: kremenek at apple.com
Cc: cfe-commits at cs.uiuc.edu
Subject: Re: [cfe-commits] r141053 - in /cfe/trunk:
include/clang/Basic/DiagnosticDriverKinds.td lib/Driver/Tools.cpp
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

_______________________________________________
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