From mboxrd@z Thu Jan 1 00:00:00 1970 X-Msuck: nntp://news.gmane.org/gmane.linux.lib.musl.general/4826 Path: news.gmane.org!not-for-mail From: Rich Felker Newsgroups: gmane.linux.lib.musl.general Subject: Re: build with clang-3.4 warnings report Date: Mon, 7 Apr 2014 13:23:34 -0400 Message-ID: <20140407172334.GC26358@brightrain.aerifal.cx> References: <533F0F1A.6040707@embtoolkit.org> <20140404204701.GO26358@brightrain.aerifal.cx> <533F56FB.8060509@embtoolkit.org> <20140405015443.GU26358@brightrain.aerifal.cx> <53428963.9030609@f-prot.com> Reply-To: musl@lists.openwall.com NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: ger.gmane.org 1396891436 7074 80.91.229.3 (7 Apr 2014 17:23:56 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Mon, 7 Apr 2014 17:23:56 +0000 (UTC) To: musl@lists.openwall.com Original-X-From: musl-return-4830-gllmg-musl=m.gmane.org@lists.openwall.com Mon Apr 07 19:23:49 2014 Return-path: Envelope-to: gllmg-musl@plane.gmane.org Original-Received: from mother.openwall.net ([195.42.179.200]) by plane.gmane.org with smtp (Exim 4.69) (envelope-from ) id 1WXDGu-00055N-08 for gllmg-musl@plane.gmane.org; Mon, 07 Apr 2014 19:23:48 +0200 Original-Received: (qmail 20336 invoked by uid 550); 7 Apr 2014 17:23:46 -0000 Mailing-List: contact musl-help@lists.openwall.com; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: Original-Received: (qmail 20325 invoked from network); 7 Apr 2014 17:23:46 -0000 Content-Disposition: inline In-Reply-To: <53428963.9030609@f-prot.com> User-Agent: Mutt/1.5.21 (2010-09-15) Xref: news.gmane.org gmane.linux.lib.musl.general:4826 Archived-At: 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