]> git.ipfire.org Git - thirdparty/glibc.git/commitdiff
Fix math/Makefile dependency on libm-test.stmp for libmvec tests.
authorJoseph Myers <joseph@codesourcery.com>
Tue, 23 Jun 2015 16:27:25 +0000 (16:27 +0000)
committerJoseph Myers <joseph@codesourcery.com>
Tue, 23 Jun 2015 16:27:25 +0000 (16:27 +0000)
Since the libmvec tests went in I've noticed build failures from
parallel testing in math/, when those tests start building before
libm-test.c has been fully generated.  (This only applies if libm test
sources have been modified after the original glibc build, because
otherwise libm-test.stmp was generated during the original build and
doesn't get regenerated during testing.)

Those tests depend on libm-test.stmp, but the dependency uses
$(libmvec-tests), which is set in the sysdeps Makefile fragments, and
appears before the inclusion of ../Rules, which is what includes those
fragments; thus, the dependency does not work and parallel make can
start building the vector tests too soon.  This patch moves the
dependency further down so that the required variable is defined when
the dependency is.

Tested for x86_64.

* math/Makefile [$(PERL) != no]
($(addprefix $(objpfx), $(addsuffix .o, $(libm-vec-tests)))): Move
dependency on libm-test.stmp below the inclusion of Rules.

ChangeLog
math/Makefile

index b76a184e71d320eaf01c004f69cb837746dea9c5..d2132be61634ff9bfe31c80a5954d1a915ff75ce 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2015-06-23  Joseph Myers  <joseph@codesourcery.com>
+
+       * math/Makefile [$(PERL) != no]
+       ($(addprefix $(objpfx), $(addsuffix .o, $(libm-vec-tests)))): Move
+       dependency on libm-test.stmp below the inclusion of Rules.
+
 2015-06-23  Andrew Senkevich  <andrew.senkevich@intel.com>
 
        * sysdeps/x86_64/fpu/Makefile (libmvec-support): Fixed files list.
index 7f6b85ec0d4e8ae1ec0500af021681455311dbd5..2c78dd0bb9c3ebd883f64b46063c28c328b07f43 100644 (file)
@@ -144,7 +144,6 @@ $(objpfx)test-double.o: $(objpfx)libm-test.stmp
 $(objpfx)test-idouble.o: $(objpfx)libm-test.stmp
 $(objpfx)test-ldouble.o: $(objpfx)libm-test.stmp
 $(objpfx)test-ildoubl.o: $(objpfx)libm-test.stmp
-$(addprefix $(objpfx), $(addsuffix .o, $(libm-vec-tests))): $(objpfx)libm-test.stmp
 endif
 
 libm-test-fast-math-cflags = -fno-builtin -D__FAST_MATH__ -DTEST_FAST_MATH
@@ -196,6 +195,9 @@ extra-objs += libieee.a ieee-math.o
 include ../Rules
 
 ifneq (no,$(PERL))
+# This must come after the inclusion of sysdeps Makefiles via Rules.
+$(addprefix $(objpfx), $(addsuffix .o, $(libm-vec-tests))): $(objpfx)libm-test.stmp
+
 # Run the math programs to automatically generate ULPs files.
 .PHONY: regen-ulps