From 91a335f9fd982cfd3b9f9318a1086f715a048d8b Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Mon, 26 Dec 2022 22:27:31 -0500 Subject: [PATCH] sim: rl78: move libsim.a creation to top-level The objects are still compiled in the subdir, but the creation of the archive itself is in the top-level. This is a required step before we can move compilation itself up, and makes it easier to review. The downside is that each object compile is a recursive make instead of a single one. On my 4 core system, it adds ~100msec to the build per port, so it's not great, but it shouldn't be a big deal. This will go away of course once the top-level compiles objects. --- sim/Makefile.in | 83 ++++++++++++++++++++++++++++++-------------- sim/rl78/Makefile.in | 12 ++----- sim/rl78/local.mk | 24 ++++++++++--- 3 files changed, 78 insertions(+), 41 deletions(-) diff --git a/sim/Makefile.in b/sim/Makefile.in index 58ffbe50752..d9fc1d82462 100644 --- a/sim/Makefile.in +++ b/sim/Makefile.in @@ -314,18 +314,19 @@ TESTS = testsuite/common/bits32m0$(EXEEXT) \ @SIM_ENABLE_ARCH_pru_TRUE@am__append_118 = pru/run @SIM_ENABLE_ARCH_riscv_TRUE@am__append_119 = riscv/libsim.a @SIM_ENABLE_ARCH_riscv_TRUE@am__append_120 = riscv/run -@SIM_ENABLE_ARCH_rl78_TRUE@am__append_121 = rl78/run -@SIM_ENABLE_ARCH_rx_TRUE@am__append_122 = rx/run -@SIM_ENABLE_ARCH_sh_TRUE@am__append_123 = sh/run -@SIM_ENABLE_ARCH_sh_TRUE@am__append_124 = \ +@SIM_ENABLE_ARCH_rl78_TRUE@am__append_121 = rl78/libsim.a +@SIM_ENABLE_ARCH_rl78_TRUE@am__append_122 = rl78/run +@SIM_ENABLE_ARCH_rx_TRUE@am__append_123 = rx/run +@SIM_ENABLE_ARCH_sh_TRUE@am__append_124 = sh/run +@SIM_ENABLE_ARCH_sh_TRUE@am__append_125 = \ @SIM_ENABLE_ARCH_sh_TRUE@ sh/code.c \ @SIM_ENABLE_ARCH_sh_TRUE@ sh/ppi.c -@SIM_ENABLE_ARCH_sh_TRUE@am__append_125 = $(sh_BUILD_OUTPUTS) -@SIM_ENABLE_ARCH_sh_TRUE@am__append_126 = sh/gencode -@SIM_ENABLE_ARCH_sh_TRUE@am__append_127 = $(sh_BUILD_OUTPUTS) -@SIM_ENABLE_ARCH_v850_TRUE@am__append_128 = v850/run -@SIM_ENABLE_ARCH_v850_TRUE@am__append_129 = \ +@SIM_ENABLE_ARCH_sh_TRUE@am__append_126 = $(sh_BUILD_OUTPUTS) +@SIM_ENABLE_ARCH_sh_TRUE@am__append_127 = sh/gencode +@SIM_ENABLE_ARCH_sh_TRUE@am__append_128 = $(sh_BUILD_OUTPUTS) +@SIM_ENABLE_ARCH_v850_TRUE@am__append_129 = v850/run +@SIM_ENABLE_ARCH_v850_TRUE@am__append_130 = \ @SIM_ENABLE_ARCH_v850_TRUE@ v850/icache.h \ @SIM_ENABLE_ARCH_v850_TRUE@ v850/idecode.h \ @SIM_ENABLE_ARCH_v850_TRUE@ v850/semantics.h \ @@ -334,8 +335,8 @@ TESTS = testsuite/common/bits32m0$(EXEEXT) \ @SIM_ENABLE_ARCH_v850_TRUE@ v850/itable.h \ @SIM_ENABLE_ARCH_v850_TRUE@ v850/engine.h -@SIM_ENABLE_ARCH_v850_TRUE@am__append_130 = $(v850_BUILD_OUTPUTS) @SIM_ENABLE_ARCH_v850_TRUE@am__append_131 = $(v850_BUILD_OUTPUTS) +@SIM_ENABLE_ARCH_v850_TRUE@am__append_132 = $(v850_BUILD_OUTPUTS) subdir = . ACLOCAL_M4 = $(top_srcdir)/aclocal.m4 am__aclocal_m4_deps = $(top_srcdir)/../config/acx.m4 \ @@ -824,6 +825,14 @@ riscv_libsim_a_AR = $(AR) $(ARFLAGS) @SIM_ENABLE_ARCH_riscv_TRUE@ riscv/sim-resume.o am_riscv_libsim_a_OBJECTS = riscv_libsim_a_OBJECTS = $(am_riscv_libsim_a_OBJECTS) +rl78_libsim_a_AR = $(AR) $(ARFLAGS) +@SIM_ENABLE_ARCH_rl78_TRUE@rl78_libsim_a_DEPENDENCIES = \ +@SIM_ENABLE_ARCH_rl78_TRUE@ $(common_libcommon_a_OBJECTS) \ +@SIM_ENABLE_ARCH_rl78_TRUE@ rl78/load.o rl78/mem.o rl78/cpu.o \ +@SIM_ENABLE_ARCH_rl78_TRUE@ rl78/rl78.o rl78/gdb-if.o \ +@SIM_ENABLE_ARCH_rl78_TRUE@ rl78/modules.o rl78/trace.o +am_rl78_libsim_a_OBJECTS = +rl78_libsim_a_OBJECTS = $(am_rl78_libsim_a_OBJECTS) @SIM_ENABLE_IGEN_TRUE@am__EXEEXT_1 = $(IGEN) igen/filter$(EXEEXT) \ @SIM_ENABLE_IGEN_TRUE@ igen/gen$(EXEEXT) igen/ld-cache$(EXEEXT) \ @SIM_ENABLE_IGEN_TRUE@ igen/ld-decode$(EXEEXT) \ @@ -1160,12 +1169,12 @@ SOURCES = $(aarch64_libsim_a_SOURCES) $(arm_libsim_a_SOURCES) \ $(mips_libsim_a_SOURCES) $(mn10300_libsim_a_SOURCES) \ $(moxie_libsim_a_SOURCES) $(msp430_libsim_a_SOURCES) \ $(or1k_libsim_a_SOURCES) $(pru_libsim_a_SOURCES) \ - $(riscv_libsim_a_SOURCES) $(aarch64_run_SOURCES) \ - $(arm_run_SOURCES) $(avr_run_SOURCES) $(bfin_run_SOURCES) \ - $(bpf_run_SOURCES) $(cr16_gencode_SOURCES) $(cr16_run_SOURCES) \ - $(cris_run_SOURCES) $(cris_rvdummy_SOURCES) \ - $(d10v_gencode_SOURCES) $(d10v_run_SOURCES) \ - $(erc32_run_SOURCES) erc32/sis.c \ + $(riscv_libsim_a_SOURCES) $(rl78_libsim_a_SOURCES) \ + $(aarch64_run_SOURCES) $(arm_run_SOURCES) $(avr_run_SOURCES) \ + $(bfin_run_SOURCES) $(bpf_run_SOURCES) $(cr16_gencode_SOURCES) \ + $(cr16_run_SOURCES) $(cris_run_SOURCES) \ + $(cris_rvdummy_SOURCES) $(d10v_gencode_SOURCES) \ + $(d10v_run_SOURCES) $(erc32_run_SOURCES) erc32/sis.c \ $(example_synacor_run_SOURCES) $(frv_run_SOURCES) \ $(ft32_run_SOURCES) $(h8300_run_SOURCES) \ $(igen_filter_SOURCES) $(igen_gen_SOURCES) \ @@ -1729,12 +1738,12 @@ noinst_LIBRARIES = common/libcommon.a $(am__append_5) $(am__append_8) \ $(am__append_72) $(am__append_78) $(am__append_84) \ $(am__append_86) $(am__append_91) $(am__append_101) \ $(am__append_107) $(am__append_109) $(am__append_111) \ - $(am__append_117) $(am__append_119) + $(am__append_117) $(am__append_119) $(am__append_121) BUILT_SOURCES = $(am__append_19) $(am__append_24) $(am__append_32) \ $(am__append_37) $(am__append_49) $(am__append_58) \ $(am__append_64) $(am__append_75) $(am__append_94) \ - $(am__append_104) $(am__append_113) $(am__append_124) \ - $(am__append_129) + $(am__append_104) $(am__append_113) $(am__append_125) \ + $(am__append_130) CLEANFILES = common/version.c common/version.c-stamp \ testsuite/common/bits-gen testsuite/common/bits32m0.c \ testsuite/common/bits32m31.c testsuite/common/bits64m0.c \ @@ -1749,7 +1758,7 @@ MOSTLYCLEANFILES = core $(common_HW_CONFIG_H_TARGETS) $(patsubst \ $(am__append_51) $(am__append_60) $(am__append_66) \ $(am__append_71) $(am__append_77) $(am__append_83) \ $(am__append_99) $(am__append_106) $(am__append_115) \ - $(am__append_127) $(am__append_131) + $(am__append_128) $(am__append_132) AM_CFLAGS = $(WERROR_CFLAGS) $(WARN_CFLAGS) AM_CPPFLAGS = $(INCGNU) -I$(srcroot)/include -I../bfd -I.. \ $(SIM_HW_CFLAGS) $(SIM_INLINE) -I$(srcdir)/common \ @@ -1764,8 +1773,8 @@ SIM_ALL_RECURSIVE_DEPS = common/libcommon.a \ $(am__append_33) $(am__append_38) $(am__append_50) \ $(am__append_59) $(am__append_65) $(am__append_69) \ $(am__append_76) $(am__append_81) $(am__append_98) \ - $(am__append_105) $(am__append_114) $(am__append_125) \ - $(am__append_130) + $(am__append_105) $(am__append_114) $(am__append_126) \ + $(am__append_131) SIM_INSTALL_DATA_LOCAL_DEPS = SIM_INSTALL_EXEC_LOCAL_DEPS = $(am__append_43) SIM_UNINSTALL_LOCAL_DEPS = $(am__append_44) @@ -2746,6 +2755,17 @@ testsuite_common_CPPFLAGS = \ @SIM_ENABLE_ARCH_riscv_TRUE@ riscv/libsim.a \ @SIM_ENABLE_ARCH_riscv_TRUE@ $(SIM_COMMON_LIBS) +@SIM_ENABLE_ARCH_rl78_TRUE@rl78_libsim_a_SOURCES = +@SIM_ENABLE_ARCH_rl78_TRUE@rl78_libsim_a_LIBADD = \ +@SIM_ENABLE_ARCH_rl78_TRUE@ $(common_libcommon_a_OBJECTS) \ +@SIM_ENABLE_ARCH_rl78_TRUE@ rl78/load.o \ +@SIM_ENABLE_ARCH_rl78_TRUE@ rl78/mem.o \ +@SIM_ENABLE_ARCH_rl78_TRUE@ rl78/cpu.o \ +@SIM_ENABLE_ARCH_rl78_TRUE@ rl78/rl78.o \ +@SIM_ENABLE_ARCH_rl78_TRUE@ rl78/gdb-if.o \ +@SIM_ENABLE_ARCH_rl78_TRUE@ rl78/modules.o \ +@SIM_ENABLE_ARCH_rl78_TRUE@ rl78/trace.o + @SIM_ENABLE_ARCH_rl78_TRUE@rl78_run_SOURCES = @SIM_ENABLE_ARCH_rl78_TRUE@rl78_run_LDADD = \ @SIM_ENABLE_ARCH_rl78_TRUE@ rl78/main.o \ @@ -3277,6 +3297,14 @@ riscv/libsim.a: $(riscv_libsim_a_OBJECTS) $(riscv_libsim_a_DEPENDENCIES) $(EXTRA $(AM_V_at)-rm -f riscv/libsim.a $(AM_V_AR)$(riscv_libsim_a_AR) riscv/libsim.a $(riscv_libsim_a_OBJECTS) $(riscv_libsim_a_LIBADD) $(AM_V_at)$(RANLIB) riscv/libsim.a +rl78/$(am__dirstamp): + @$(MKDIR_P) rl78 + @: > rl78/$(am__dirstamp) + +rl78/libsim.a: $(rl78_libsim_a_OBJECTS) $(rl78_libsim_a_DEPENDENCIES) $(EXTRA_rl78_libsim_a_DEPENDENCIES) rl78/$(am__dirstamp) + $(AM_V_at)-rm -f rl78/libsim.a + $(AM_V_AR)$(rl78_libsim_a_AR) rl78/libsim.a $(rl78_libsim_a_OBJECTS) $(rl78_libsim_a_LIBADD) + $(AM_V_at)$(RANLIB) rl78/libsim.a clean-checkPROGRAMS: @list='$(check_PROGRAMS)'; test -n "$$list" || exit 0; \ @@ -3502,9 +3530,6 @@ pru/run$(EXEEXT): $(pru_run_OBJECTS) $(pru_run_DEPENDENCIES) $(EXTRA_pru_run_DEP riscv/run$(EXEEXT): $(riscv_run_OBJECTS) $(riscv_run_DEPENDENCIES) $(EXTRA_riscv_run_DEPENDENCIES) riscv/$(am__dirstamp) @rm -f riscv/run$(EXEEXT) $(AM_V_CCLD)$(LINK) $(riscv_run_OBJECTS) $(riscv_run_LDADD) $(LIBS) -rl78/$(am__dirstamp): - @$(MKDIR_P) rl78 - @: > rl78/$(am__dirstamp) rl78/run$(EXEEXT): $(rl78_run_OBJECTS) $(rl78_run_DEPENDENCIES) $(EXTRA_rl78_run_DEPENDENCIES) rl78/$(am__dirstamp) @rm -f rl78/run$(EXEEXT) @@ -5290,9 +5315,13 @@ testsuite/common/bits64m63.c: testsuite/common/bits-gen$(EXEEXT) testsuite/commo @SIM_ENABLE_ARCH_riscv_TRUE@riscv/%.o: common/%.c @SIM_ENABLE_ARCH_riscv_TRUE@ $(AM_V_at)$(MAKE) $(AM_MAKEFLAGS) -C $(@D) $(@F) +@SIM_ENABLE_ARCH_rl78_TRUE@$(rl78_libsim_a_OBJECTS) $(rl78_libsim_a_LIBADD): rl78/hw-config.h + +@SIM_ENABLE_ARCH_rl78_TRUE@rl78/%.o: rl78/%.c +@SIM_ENABLE_ARCH_rl78_TRUE@ $(AM_V_at)$(MAKE) $(AM_MAKEFLAGS) -C $(@D) $(@F) -@SIM_ENABLE_ARCH_rl78_TRUE@rl78/%.o: rl78/%.c | rl78/libsim.a $(SIM_ALL_RECURSIVE_DEPS) -@SIM_ENABLE_ARCH_rl78_TRUE@ $(MAKE) $(AM_MAKEFLAGS) -C $(@D) $(@F) +@SIM_ENABLE_ARCH_rl78_TRUE@rl78/%.o: common/%.c +@SIM_ENABLE_ARCH_rl78_TRUE@ $(AM_V_at)$(MAKE) $(AM_MAKEFLAGS) -C $(@D) $(@F) @SIM_ENABLE_ARCH_rx_TRUE@rx/%.o: rx/%.c | rx/libsim.a $(SIM_ALL_RECURSIVE_DEPS) @SIM_ENABLE_ARCH_rx_TRUE@ $(MAKE) $(AM_MAKEFLAGS) -C $(@D) $(@F) diff --git a/sim/rl78/Makefile.in b/sim/rl78/Makefile.in index 9ca400efe28..0cc55baf3b5 100644 --- a/sim/rl78/Makefile.in +++ b/sim/rl78/Makefile.in @@ -20,15 +20,7 @@ ## COMMON_PRE_CONFIG_FRAG -SIM_RUN_OBJS = \ - main.o - -SIM_OBJS = \ - load.o \ - mem.o \ - cpu.o \ - rl78.o \ - gdb-if.o \ - trace.o +SIM_LIBSIM = +SIM_RUN_OBJS = ## COMMON_POST_CONFIG_FRAG diff --git a/sim/rl78/local.mk b/sim/rl78/local.mk index 07ff86711c1..935fbf1eea0 100644 --- a/sim/rl78/local.mk +++ b/sim/rl78/local.mk @@ -16,6 +16,26 @@ ## You should have received a copy of the GNU General Public License ## along with this program. If not, see . +%C%_libsim_a_SOURCES = +%C%_libsim_a_LIBADD = \ + $(common_libcommon_a_OBJECTS) \ + %D%/load.o \ + %D%/mem.o \ + %D%/cpu.o \ + %D%/rl78.o \ + %D%/gdb-if.o \ + %D%/modules.o \ + %D%/trace.o +$(%C%_libsim_a_OBJECTS) $(%C%_libsim_a_LIBADD): %D%/hw-config.h + +noinst_LIBRARIES += %D%/libsim.a + +%D%/%.o: %D%/%.c + $(AM_V_at)$(MAKE) $(AM_MAKEFLAGS) -C $(@D) $(@F) + +%D%/%.o: common/%.c + $(AM_V_at)$(MAKE) $(AM_MAKEFLAGS) -C $(@D) $(@F) + %C%_run_SOURCES = %C%_run_LDADD = \ %D%/main.o \ @@ -23,7 +43,3 @@ $(SIM_COMMON_LIBS) noinst_PROGRAMS += %D%/run - -## Helper targets for running make from the top-level due to run's main.o. -%D%/%.o: %D%/%.c | %D%/libsim.a $(SIM_ALL_RECURSIVE_DEPS) - $(MAKE) $(AM_MAKEFLAGS) -C $(@D) $(@F) -- 2.30.2