[llvm] r176341 - GCC thinks that this variable might be used uninitialized (it isn't).

[Michael Liao]

On Fri, 2013-03-01 at 02:50 -0800, Chandler Carruth wrote:
> 
> On Fri, Mar 1, 2013 at 1:46 AM, Duncan Sands <baldrick at free.fr> wrote:
>         GCC thinks that this variable might be used uninitialized (it
>         isn't).
> 
> 
> Sorry to keep kicking the horse here, but I really oppose adding dead
> stores to silence a broken warning.
> 
> 
> We'll no longer be able to use Valgrind to catch a bug where the
> control flow that intends to initialize this variable suddenly changes
> and fails to do so in one case. =[ That seems really bad.
> 
> 
> Clang's warnings don't have this false positive, and there is even a
> way to silence GCCs in modern GCCs (-Wno-maybe-uninitialized) because
> of this problem. Can we just turn off old GCC -Wuninitialized and rely
> on the modern compilers' warnings instead?

shall we add that option by default in our makefile*?

- michael

> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits