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

Chandler Carruth chandlerc at gmail.com
Mon Jun 18 03:49:43 PDT 2012


On Mon, Jun 18, 2012 at 3:39 AM, Tobias von Koch <
T.EdlerVonKoch at freescale.com> wrote:

> Hi Chandler,
>
> I'm getting lots of warnings like the following after this commit:
>
> /build/compiler/llvm/include/llvm/ADT/DenseMap.h:493:39: warning:
> ‘TheBucket’ may be used uninitialised in this function [-Wuninitialized]
> /build/compiler/llvm/include/llvm/ADT/DenseMap.h:107:14: note:
> ‘TheBucket’ was declared here
>
> This may be g++ being overly clever, but it's not pretty.
>

You should turn off that GCC warning. It is useless I'm afraid. GCC folks
themselves split it into -Wmaybe-uninitialized because of the incredibly
high false positive rate.


The codebase already has piles of these incorrect warnings in it, and
they're not likely to be fixed. My suggestion? Use clang. ;] It's
-Wuninitialized doesn't have this problem.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120618/bc86cb02/attachment.html>


More information about the llvm-commits mailing list