Changeset 2755

Timestamp:

Jan 25, 2015, 1:42:34 AM (10 years ago)

Author:

bird

Message:

subheader.kmk,subfooter.kmk: Don't evaluate the def_subheader/footer define for old targets, only for targets which are new. Saves some times (saw 88000+ invokations of each of the two defines when loading the vbox makefiles).

Location:

trunk/kBuild

Files:

• subfooter.kmk (modified) (2 diffs)
• subheader.kmk (modified) (2 diffs)

trunk/kBuild/subfooter.kmk

@@ -60 +60 @@
 endef
 
+# TODO: Wish there was an easy way of only enumerating only new targets...
 $(foreach target,\
         $(ALL_TARGETS) \
@@ -73 +74 @@
         $(INSTALLS)    $(INSTALLS.$(KBUILD_TARGET))     $(INSTALLS.$(KBUILD_TARGET).$(KBUILD_TARGET_ARCH))    $(INSTALLS.$(KBUILD_TARGET_ARCH))    $(INSTALLS.$(KBUILD_TARGET_CPU))    $(INSTALLS.$(KBUILD_TYPE))    \
         $(OTHERS)      $(OTHERS.$(KBUILD_TARGET))       $(OTHERS.$(KBUILD_TARGET).$(KBUILD_TARGET_ARCH))      $(OTHERS.$(KBUILD_TARGET_ARCH))      $(OTHERS.$(KBUILD_TARGET_CPU))      $(OTHERS.$(KBUILD_TYPE))      \
-,$(evalval def_subheader))
+,$(if-expr defined($(target)_0_OUTDIR),,$(evalval def_subheader)))
 
 

trunk/kBuild/subheader.kmk

@@ -71 +71 @@
  endef
 
+ # TODO: Wish there was an easy way of only enumerating only new targets...
  $(foreach target,\
         $(ALL_TARGETS) \
@@ -84 +85 @@
         $(INSTALLS)    $(INSTALLS.$(KBUILD_TARGET))     $(INSTALLS.$(KBUILD_TARGET).$(KBUILD_TARGET_ARCH))    $(INSTALLS.$(KBUILD_TARGET_ARCH))    $(INSTALLS.$(KBUILD_TARGET_CPU))    $(INSTALLS.$(KBUILD_TYPE))    \
         $(OTHERS)      $(OTHERS.$(KBUILD_TARGET))       $(OTHERS.$(KBUILD_TARGET).$(KBUILD_TARGET_ARCH))      $(OTHERS.$(KBUILD_TARGET_ARCH))      $(OTHERS.$(KBUILD_TARGET_CPU))      $(OTHERS.$(KBUILD_TYPE))      \
- ,$(evalval def_subfooter))
+ ,$(if-expr defined($(target)_0_OUTDIR),,$(evalval def_subfooter)))