From: Rich Felker <dalias@libc.org>
To: musl@lists.openwall.com
Cc: Denys Vlasenko <vda.linux@googlemail.com>
Subject: Re: [PATCH] configure: add gcc flags for better link-time optimization
Date: Mon, 26 Oct 2015 21:21:31 -0400 [thread overview]
Message-ID: <20151027012131.GR8645@brightrain.aerifal.cx> (raw)
In-Reply-To: <20151023131202.GI10551@port70.net>
On Fri, Oct 23, 2015 at 03:12:02PM +0200, Szabolcs Nagy wrote:
> * Denys Vlasenko <vda.linux@googlemail.com> [2015-10-23 14:30:26 +0200]:
> > libc.so size reduction:
> >
> > text data bss dec hex filename
> > 564099 1944 11768 577811 8d113 libc.so.before
> > 562277 1924 11576 575777 8c921 libc.so
> >
>
> i assume this is x86_64, nice improvement.
I suspect all of this difference comes from optimizing out dummy weak
functions that are replaced by strong versions in other files. The
same savings could be achieved by eliminating them with #ifndef
SHARED, but as noted elsewhere I actually want to eliminate all such
#ifdefs and build only one set of .o files to use for libc.so and
libc.a. So I think this looks like a nice way to get the same benefit
without #ifdef.
BTW I noticed that __simple_malloc (in lite_malloc.c) is external
despite there being no need for it to be external. If we make it
static, I think gc-sections will be able to remove it too. (Denys, if
you want to try the numbers with it made static before I get around to
it, I'd be happy to hear results.)
> > +# When linker merges sections, a tiny section (such as one resulting
> > +# from "static char flag_var") with no alignment restrictions
> > +# can end up logded between two more strongly aligned ones (say,
> > +# "static int global_cnt1/2", both of which want 32-bit alignment).
> > +# Then this byte-sized "flag_var" gets 3 bytes of padding.
> > +#
> > +# With section sorting by alignment, one-byte flag variables have
> > +# higher chance of being grouped together and not require padding.
> > +# (It can be made even better. Linker is too dumb.
> > +# ld needs to grow -Wl,--pack-sections-optimally)
> > +#
> > +# For us, this affects the size of only one file: libc.so
> > +#
> > +tryldflag LDFLAGS_AUTO -Wl,--sort-section=alignment
> > +tryldflag LDFLAGS_AUTO -Wl,--sort-common
>
> i think this came up before
> https://sourceware.org/bugzilla/show_bug.cgi?id=14156
I don't think this bug affects linkin musl itself, since we don't have
init or fini sections, so we probably don't have to worry about it.
Does this sound correct? If there's a risk of these options making
breakage with some binutils versions then we probably have to detect
that, but hopefully there's not.
Rich
next prev parent reply other threads:[~2015-10-27 1:21 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-23 12:30 Denys Vlasenko
2015-10-23 13:12 ` Szabolcs Nagy
2015-10-23 14:41 ` Denys Vlasenko
2015-10-23 14:48 ` Rich Felker
2015-10-23 22:00 ` Denys Vlasenko
2015-10-24 12:43 ` Szabolcs Nagy
2015-10-24 19:20 ` Rich Felker
2015-10-27 1:21 ` Rich Felker [this message]
2015-10-27 19:09 ` Denys Vlasenko
2015-10-27 21:01 ` Rich Felker
2015-10-28 9:53 ` Denys Vlasenko
2015-10-28 10:05 ` Denys Vlasenko
2015-11-01 19:56 ` Rich Felker
2015-11-02 22:36 ` Rich Felker
2015-11-03 0:01 ` Rich Felker
2015-11-05 2:58 ` Rich Felker
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20151027012131.GR8645@brightrain.aerifal.cx \
--to=dalias@libc.org \
--cc=musl@lists.openwall.com \
--cc=vda.linux@googlemail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/musl/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).