testsuite: move architecture-specific logic from tests/asmgen to tools

ocaml · Apr 26, 2018 · 65090a8 · 65090a8
1 parent 3c98b3c
commit 65090a8
Show file tree

Hide file tree

Showing 9 changed files with 20 additions and 14 deletions.
diff --git a/testsuite/tests/asmgen/Makefile b/testsuite/tests/asmgen/Makefile
@@ -22,9 +22,7 @@ default:
 	  $(MAKE) all; \
 	fi
 
-all:
-	@$(MAKE) arch
-	@$(MAKE) tests
+all: tests
 
 CASES=fib tak quicksort quicksort2 soli \
       arith checkbound tagged-fib tagged-integr tagged-quicksort tagged-tak \
@@ -52,16 +50,17 @@ skips:
 	  echo " ... testing '$$c': => skipped"; \
 	done
 
+include $(BASEDIR)/makefiles/Makefile.common
+archmod = $(OTOPDIR)/testsuite/tools/asmgen_$(ARCH).$(O)
+
 one:
-	@$(call CCOMP,$(NAME).out $(ARGS_$(NAME)) $(NAME).$(O) $(ARCH).$(O)) \
+	@$(call CCOMP,$(NAME).out $(ARGS_$(NAME)) $(NAME).$(O) $(archmod)) \
 	&& echo " => passed" || echo " => failed"
 
 clean: defaultclean
 	@rm -f *.out *.out.manifest *.$(O) *.exe
 	@rm -f $(CASES:=.s)
 
-include $(BASEDIR)/makefiles/Makefile.common
-
 ifeq "$(CCOMPTYPE)-$(ARCH)" "msvc-amd64"
 # these tests are not ported to MSVC64 yet
 SKIP=true
@@ -86,9 +85,3 @@ tests: $(CASES:=.$(O))
 	done
 
 promote:
-
-arch: $(ARCH).$(O)
-
-i386.obj: i386nt.asm
-	@set -o pipefail ; \
-	$(ASM) $@ $^ | tail -n +2
diff --git a/testsuite/tools/Makefile b/testsuite/tools/Makefile
@@ -13,6 +13,7 @@
 #**************************************************************************
 
 BASEDIR=..
+include $(BASEDIR)/../config/Makefile
 expect_MAIN=expect_test
 expect_PROG=$(expect_MAIN)$(EXE)
 expect_COMPFLAGS=-I $(OTOPDIR)/parsing -I $(OTOPDIR)/utils \
@@ -37,10 +38,18 @@ codegen_OBJECTS=parsecmmaux.cmo parsecmm.cmo lexcmm.cmo codegen_main.cmo
 
 codegen_ADD_COMPFLAGS=$(codegen_INCLUDES) -w -40 -g
 
-all: $(expect_PROG) codegen
+ifeq "$(CCOMPTYPE)-$(ARCH)" "msvc-amd64"
+# The asmgen tests are not ported to MSVC64 yet
+# so do not compile any arch-specific module
+arch_target :=
+else
+arch_target := asmgen_$(ARCH).$(O)
+endif
+
+all: $(expect_PROG) codegen $(arch_target)
 
 $(expect_PROG): $(expect_LIBRARIES:=.cma) $(expect_MAIN).cmo
-	$(OCAMLC) -linkall -o $@ $^
+	@$(OCAMLC) -linkall -o $@ $^
 
 include $(BASEDIR)/makefiles/Makefile.common
 
@@ -64,3 +73,7 @@ parsecmm.mli parsecmm.ml: parsecmm.mly
 lexcmm.ml: lexcmm.mll
 	@$(OCAMLLEX) -q lexcmm.mll
 
+asmgen_i386.obj: asmgen_i386nt.asm
+	@set -o pipefail ; \
+	$(ASM) $@ $^ | tail -n +2
+
diff --git a/testsuite/tests/asmgen/amd64.S → testsuite/tools/asmgen_amd64.S b/testsuite/tests/asmgen/amd64.S → testsuite/tools/asmgen_amd64.S
diff --git a/testsuite/tests/asmgen/arm.S → testsuite/tools/asmgen_arm.S b/testsuite/tests/asmgen/arm.S → testsuite/tools/asmgen_arm.S
diff --git a/testsuite/tests/asmgen/arm64.S → testsuite/tools/asmgen_arm64.S b/testsuite/tests/asmgen/arm64.S → testsuite/tools/asmgen_arm64.S
diff --git a/testsuite/tests/asmgen/i386.S → testsuite/tools/asmgen_i386.S b/testsuite/tests/asmgen/i386.S → testsuite/tools/asmgen_i386.S
diff --git a/testsuite/tests/asmgen/i386nt.asm → testsuite/tools/asmgen_i386nt.asm b/testsuite/tests/asmgen/i386nt.asm → testsuite/tools/asmgen_i386nt.asm
diff --git a/testsuite/tests/asmgen/power.S → testsuite/tools/asmgen_power.S b/testsuite/tests/asmgen/power.S → testsuite/tools/asmgen_power.S
diff --git a/testsuite/tests/asmgen/s390x.S → testsuite/tools/asmgen_s390x.S b/testsuite/tests/asmgen/s390x.S → testsuite/tools/asmgen_s390x.S