This is the mail archive of the
libc-hacker@sourceware.cygnus.com
mailing list for the glibc project.
this time for sure!
- To: libc-hacker@gnu.org
- Subject: this time for sure!
- From: Zack Weinberg <zack@rabi.phys.columbia.edu>
- Date: Fri, 01 May 1998 21:25:04 -0400
Awright, finally I have an ar speed patch that actually works. It's
tested to complete make, make check, make install successfully. Time
for a full rebuild on my machine is reduced by nearly an hour.
zw
Fri May 1 21:01:56 1998 Zack Weinberg <zack@rabi.phys.columbia.edu>
Build speed improvement: make ar less of a bottleneck.
* Makerules (stamp$o): Just record the object list for later
use.
(libc_foo.a): New rule, depends on subdir stamp files; builds
and ranlibs the library from the lists saved in those files.
(subdirs-stamps): Define also if subdir == elf (FIXME: kludge).
(subdirs-stamp-o): Likewise.
(ar-symtab-name): Delete variable.
(lib%.a): Delete target.
(libc_foo.a(bar.%)): Delete target.
(libc_foo.a($(ar-symtab-name))): Delete target.
(objects objs): Just depend on libobjs and extra-objs.
(libobjs): Depend on the stamp files, not the objects.
(extra-objs): New target, just depends on the extra-objs list.
(CREATE_ARFLAGS): Kill the `ar S' hack.
* config.make.in (have-ar-S): Deleted.
* configure.in: Delete test for `ar S'. Add test to determine
whether ranlib does anything useful.
Index: Makerules
===================================================================
RCS file: /egcs/carton/cvsfiles/libc/Makerules,v
retrieving revision 1.288
diff -u -r1.288 Makerules
--- Makerules 1998/04/23 22:43:34 1.288
+++ Makerules 1998/05/02 00:38:12
@@ -72,15 +72,6 @@
# `+includes' in Makeconfig references $(+sysdep-includes).
+sysdep-includes := $(addprefix -I,$(+sysdep_dirs))
-# The name of the symbol table archive member. The default is suitable for
-# BSD style archives. It can be overridden in sysdep Makefiles when SYSV
-# style archive are used.
-ifeq (no,$(elf))
-ar-symtab-name = __.SYMDEF
-else
-ar-symtab-name = # The null name is used in ELF archives.
-endif
-
# This variable is used in ``include $(o-iterator)'' after defining
# $(o-iterator-doit) to produce some desired rule using $o for the object
# suffix, and setting $(object-suffixes-left) to $(object-suffixes); a copy
@@ -510,29 +501,18 @@
endif # not -s
ARFLAGS := r$(verbose)
-ifeq ($(have-ar-S),yes)
-CREATE_ARFLAGS := Scru$(verbose)
-else
CREATE_ARFLAGS := cru$(verbose)
-endif
# This makes all the object files in the parent library archive.
-.PHONY: lib libobjs lib-noranlib
+.PHONY: lib lib-noranlib
lib: lib-noranlib $(foreach l,$(libtypes),$(patsubst %,$(common-objpfx)$l,c))
-# Pattern rule for libraries: depend on the __.SYMDEF member updated by ranlib.
-lib%.a: lib%.a($(ar-symtab-name)) ;
-
# For object-suffix $o, the list of objects with that suffix.
# Makefiles can define `elide-routines.so = foo' to leave foo.so out.
o-objects = $(patsubst %.o,%$o,$(filter-out $(patsubst %,$(objpfx)%.o,\
$(elide-routines$o)),\
$(objects)))
-
-libobjs: $(foreach o,$(object-suffixes-for-libc),\
- $(common-objpfx)$(patsubst %,$(libtype$o),c)(\
- $(notdir $(o-objects))))
lib-noranlib: libobjs
others: $(addprefix $(objpfx),$(install-lib))
@@ -544,43 +524,25 @@
$(make-target-directory)
rm -f $@; > $@
else
-
-# Define a pattern rule that will match many targets libc.a(foo.%), for
-# each foo.o in $(objects) (% will just happen always to match `o'). This is
-# the only way to define a rule that updates many targets at once with one
-# sequence of commands.
-define o-iterator-doit
-$(common-objpfx)$(patsubst %,$(libtype$o),c)(\
-$(addsuffix .%,$(filter-out $(elide-routines$o),$(notdir $(objects:.o=))))): \
- $(objpfx)stamp.% ;
-endef
-object-suffixes-left := $(object-suffixes-for-libc)
-include $(o-iterator)
-# The pattern rule tells Make to remake $(objpfx)stamp.% as
-# the way to update all the foo.% object files in $(objects). Now we
-# define explicit rules to update each $(objpfx)stamp.SUFFIX
+# Define explicit rules to update each $(objpfx)stamp.SUFFIX
# timestamp file; these rules (one explicit rule is generated for each
-# object suffix) will update the parent archive with ar. Use a static
-# pattern rule so $* is set to the object type during the commands.
+# object suffix) write a list of objects to update in the stamp file.
+# The parent will then actually add them all to the archive in the
+# archive rule, below.
define o-iterator-doit
-$(objpfx)stamp$o: $(objpfx)stamp%: $(o-objects); $$(do-ar)
+$(objpfx)stamp$o: $(o-objects); $$(do-stamp)
endef
+define do-stamp
+ echo '$(patsubst $(common-objpfx)%,%,$^)' > $@T
+ mv -f $@T $@
+endef
ifdef static-only-routines
object-suffixes-left := $(object-suffixes-for-libc)
else
object-suffixes-left := $(object-suffixes)
endif
include $(o-iterator)
-define do-ar
-topdir=`cd $(..). && pwd`; \
-$(patsubst %/,cd % &&,$(objpfx)) \
-$(SHELL) $$topdir/autolock.sh ${O%-lib}.lck $(AR) $(CREATE_ARFLAGS) ${O%-lib} \
- $(patsubst $(objpfx)%,%,$?)
-rm -f $@
-touch $@
-endef
-O%-lib = $(..)$(patsubst %,$(libtype$*),c)
ifndef static-only-routines
subdir_lib: $(objpfx)stamp.oS
@@ -591,29 +553,42 @@
endif
-# Rules to update the $(ar-symtab-name) member with ranlib,
-# one for each object flavor.
+# Now define explicit rules to build the library archives; these depend
+# on the stamp files built above. The rule always destroys and recreates
+# the library from scratch; it's faster (!) that way.
define o-iterator-doit
-$(common-objpfx)$(patsubst %,$(libtype$o),c)($(ar-symtab-name)): \
- $(common-objpfx)$(patsubst %,$(libtype$o),c)(\
- $(patsubst $(objpfx)%,%,$(o-objects))) $(subdirs-stamp-o); \
- $(SHELL) $$(..)./autolock.sh \
- $$(common-objpfx)$$(patsubst %,$$(libtype$o),c).lck \
- $$(RANLIB) $$(common-objpfx)$$(patsubst %,$$(libtype$o),c)
+$(common-objpfx)$(patsubst %,$(libtype$o),c): \
+ $(subdirs-stamp-o) $(common-objpfx)stamp$o; $$(do-makelib)
endef
+define do-makelib
+ (set -e; cd $(common-objdir); \
+ $(AR) $(CREATE_ARFLAGS) $(@F)T \
+ `cat $(patsubst $(common-objpfx)%,%,$^)`; \
+ $(RANLIB) $(@F)T; \
+ mv -f $(@F)T $(@F))
+endef
ifndef subdir
subdirs-stamps := $(foreach d,$(subdirs),$(common-objpfx)$d/stamp%)
subdirs-stamp-o = $(subst %,$o,$(subdirs-stamps))
$(subdirs-stamps): subdir_lib;
endif
+# The elf directory needs to know subdirs-stamp-o; otherwise libc_pic.a
+# is mangled. FIXME: It would be cleaner to move the final construction
+# of ld.so to top level.
+ifeq ($(subdir),elf)
+subdirs-stamps := $(foreach d,$(subdirs),$(common-objpfx)$d/stamp%)
+subdirs-stamp-o = $(subst %,$o,$(subdirs-stamps))
+endif
+
object-suffixes-left = $(object-suffixes-for-libc)
include $(o-iterator)
# This makes all the object files.
-.PHONY: objects objs
-objects objs: $(foreach o,$(object-suffixes-for-libc),$(o-objects)) \
- $(addprefix $(objpfx),$(extra-objs))
+.PHONY: objects objs libobjs extra-objs
+objects objs: libobjs extra-objs
+libobjs: $(foreach o,$(object-suffixes-for-libc),$(objpfx)stamp$o)
+extra-objs: $(addprefix $(objpfx),$(extra-objs))
# Canned sequence for building an extra library archive.
define build-extra-lib
Index: config.make.in
===================================================================
RCS file: /egcs/carton/cvsfiles/libc/config.make.in,v
retrieving revision 1.52
diff -u -r1.52 config.make.in
--- config.make.in 1998/04/21 17:58:51 1.52
+++ config.make.in 1998/05/02 00:38:12
@@ -35,7 +35,6 @@
versioning = @VERSIONING@
-have-ar-S=@libc_cv_ar_S@
have-no-whole-archive = @libc_cv_ld_no_whole_archive@
have-no-exceptions = @libc_cv_gcc_no_exceptions@
have-bash2 = @libc_cv_have_bash2@
Index: configure.in
===================================================================
RCS file: /egcs/carton/cvsfiles/libc/configure.in,v
retrieving revision 1.210
diff -u -r1.210 configure.in
--- configure.in 1998/04/21 17:59:02 1.210
+++ configure.in 1998/05/02 00:38:12
@@ -492,19 +492,26 @@
fi
AC_SUBST(SYSINCLUDES)
-# check if ar takes S
-AC_CACHE_CHECK(for ar S, libc_cv_ar_S, [dnl
-tmpo=$$.o
-tmplib=lib$$.a
-rm -f $tmpo $tmplib
-touch $tmpo
-if AC_TRY_COMMAND([${AR-ar} rcuS $tmplib $tmpo > /dev/null 2>&1]); then
- libc_cv_ar_S=yes
+# check if ranlib is necessary
+AC_CACHE_CHECK(whether ranlib is necessary, libc_cv_ranlib_necessary, [dnl
+cat > conftest.c <<EOF
+int a;
+char b;
+void c(void) {}
+EOF
+$CC $CFLAGS -c conftest.c
+$AR cr conftest.a conftest.c
+cp conftest.a conftest2.a
+$RANLIB conftest.a
+if cmp -s conftest.a conftest2.a; then
+ libc_cv_ranlib_necessary=no
else
- libc_cv_ar_S=no
+ libc_cv_ranlib_necessary=yes
fi
-rm -f $tmpo $tmplib])
-AC_SUBST(libc_cv_ar_S)dnl
+rm -rf conftest*])
+if test "$libc_cv_ranlib_necessary" = no; then
+ RANLIB=:
+fi
# Test if LD_LIBRARY_PATH contains the notation for the current directory
# since this would lead to problems installing/building glibc.