[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Mon, 7 Apr 2014 13:23:34 -0400
From: Rich Felker <dalias@...ifal.cx>
To: musl@...ts.openwall.com
Subject: Re: build with clang-3.4 warnings report
On Mon, Apr 07, 2014 at 11:17:55AM +0000, Oliver Schneider wrote:
> On 2014-04-05 01:54, Rich Felker wrote:
> >> src/env/__init_security.c:8:13: warning: function 'dummy' is not needed and will not be emitted [-Wunneeded-internal-declaration]
> >> static void dummy(void *ent)
> >> ^
> >> 1 warning generated.
> >
> > This is a bug in the compiler; the function dummy has an externally
> > visible alias, so it will not be omitted (and if it is actually
> > omitted, that means the compiler is generating broken output).
> There is yet another possibility ;)
>
> The ccc-analyzer could simply be buggy in this case. Would be
> interesting to look at the optimized intermediate code whether this gets
> optimized out. I think it probably doesn't, but who knows.
At one point it was getting optimized out, which made it impossible to
build musl with clang, so I think this was fixed and it's just a
residual bug in the warning generation.
> The dangling else warnings could also be resolved by simply adding
> braces. It doesn't add to the code size, and the times when developers
> where paid by LoC are long over (I hope).
Our developers are paid by 1/LoC. ;-)
Rich
Powered by blists - more mailing lists
Confused about mailing lists and their use?
Read about mailing lists on Wikipedia
and check out these
guidelines on proper formatting of your messages.
