[llvm-commits] [llvm] r158638 - in /llvm/trunk: include/llvm/ADT/DenseMap.h unittests/ADT/DenseMapTest.cpp

Chandler Carruth chandlerc at google.com
Tue Jun 19 23:42:43 PDT 2012


On Tue, Jun 19, 2012 at 11:40 PM, Duncan Sands <baldrick at free.fr> wrote:

> Hi Nick,
>
>
>  How would you feel about passing -Wno-uninitialized when you know you're
>> building with a version of GCC that warns about it too aggressively? We
>> would
>> still want it on for newer GCC's (that split it out into -Wuninitialized
>> and
>> -Wmaybe-initialized) and with Clang.
>>
>
> it sounds like a plan!
>
>
>  I could probably even write the configure script to detect a broken
>> -Wuninitialized implementation if that helps.
>>
>
> This is the Right Thing to Do (tm), but probably no-one will complain if
> you
> just check the GCC version.


For what its worth, this would help me too.

Knowing that when I get GCC's uninitialized warning it's for real and
likely a bug would help me, ahem, attend my code a bit better and fix the
issues that actually show up.

This is the toxic aspect of warnings with false positives -- they breed a
lack of caring about the warning. I now have a nice bitter taste in my
mouth. ;]
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120619/6ce6c6c7/attachment.html>


More information about the llvm-commits mailing list