serialize linking of large libraries only if USE_LD is not set
This problem with large memory usage during parallel linking of several large libraries is presumably only relevant for the poorly performing BFD linker. I can barely notice any memory usage increase even with something as "old" as gold. Change-Id: I20038c98543b1b920d75d8f645c6b90afb5fb6e4 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/133250 Reviewed-by: Michael Stahl <michael.stahl@allotropia.de> Reviewed-by: Luboš Luňák <l.lunak@collabora.com> Tested-by: Jenkins
This commit is contained in:
@@ -200,9 +200,16 @@ ifeq (,$(DISABLE_DYNLOADING))
|
||||
# libraries takes enormous amounts of RAM. To prevent annoying OOM situations
|
||||
# etc., try to prevent linking these in parallel by adding artificial build
|
||||
# order dependencies here.
|
||||
# Do this only if a linker is not explicitly set, as this should only apply
|
||||
# to the BFD linker and any decently modern linker presumably performs better.
|
||||
ifeq (,$(USE_LD))
|
||||
define repositorymodule_serialize1
|
||||
$(call gb_Library_get_linktarget_target,$(1)) :| $(foreach lib,$(2),$(call gb_Library_get_target,$(lib)))
|
||||
endef
|
||||
else
|
||||
define repositorymodule_serialize1
|
||||
endef
|
||||
endif
|
||||
|
||||
define repositorymodule_serialize
|
||||
$(if $(filter-out 0 1,$(words $(1))),\
|
||||
|
||||
Reference in New Issue
Block a user