Re: bsd.sys.mk [-Wno-uninitialized]

[-stable to bcc; keeping -current]

On Tue, 15 Jul 2014, Hans Petter Selasky wrote:

> On 07/05/14 15:10, David Chisnall wrote:
>> On 5 Jul 2014, at 14:07, Dimitry Andric <dim_at_FreeBSD.org> wrote:
>> 
>>> Interestingly, -Wno-uninitialized has been in bsd.sys.mk since r76861,
>>> and the accompanying comment ("XXX Delete -Wuninitialized by default for
>>> now -- the compiler doesn't always get it right") has never been
>>> changed. :-)
>>> 
>>> It is probably time to re-enable that warning after 13 years, at least.
>> 
>> It probably only wants enabling for clang.  GCC (at least, GCC 4.2.1) 
>> performs this analysis based on analyses run by the optimisers and so the 
>> warnings are dependent on optimisation level.
>> 
>> David
>
> Hi,
>
> Is someone working on this?

I was going to chime in and claim that I had seen false positives from 
-Wuninitialized even from recent clang, but upon consulting my build logs, 
it seems that the false positives are actually from 
-Wconditional-uninitialized.  Is that known to be less reliable?

-Ben