diff options
author | Nicholas Piggin <npiggin@gmail.com> | 2018-02-10 18:42:49 +1000 |
---|---|---|
committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2018-02-28 20:28:39 -0600 |
commit | f6159cff5d91e5c0810d0c9285a1d2370a38e2b7 (patch) | |
tree | c6148aef1cc7f967baaf9b7a65486b7b4c95678f /Makefile.rules | |
parent | f30286c494312caa81333a077da58f501894c2bb (diff) | |
download | skiboot-f6159cff5d91e5c0810d0c9285a1d2370a38e2b7.zip skiboot-f6159cff5d91e5c0810d0c9285a1d2370a38e2b7.tar.gz skiboot-f6159cff5d91e5c0810d0c9285a1d2370a38e2b7.tar.bz2 |
build: use thin archives rather than incremental linking
This changes to build system to use thin archives rather than
incremental linking for built-in.o, similar to recent change to Linux.
built-in.o is renamed to built-in.a, and is created as a thin archive
with no index, for speed and size. All built-in.a are aggregated into
a skiboot.tmp.a which is a thin archive built with an index, making it
suitable or linking. This is input into the final link.
The advantags of build size and linker code placement flexibility are
not as great with skiboot as a bigger project like Linux, but it's a
conceptually better way to build, and is more compatible with link
time optimisation in toolchains which might be interesting for skiboot
particularly for size reductions.
Size of build tree before this patch is 34.4MB, afterwards 23.1MB.
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'Makefile.rules')
-rw-r--r-- | Makefile.rules | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/Makefile.rules b/Makefile.rules index 80121c3..e5f6dfc 100644 --- a/Makefile.rules +++ b/Makefile.rules @@ -67,8 +67,9 @@ endif %.i : %.c $(call Q,CC, $(CC) $(call cook_cflags,$@) -E -c $< -o $@, $@) -%built-in.o : - $(call Q,LD, $(LD) $(LDRFLAGS) -r $^ -o $@, $@) +%built-in.a : + @rm -f $@ + $(call Q,AR, $(AR) rcSTPD $@ $^, $@) %.lds : %.lds.S $(call Q,CC, $(CC) $(CPPFLAGS) -P -E $< -o $@, $@) |