re PR testsuite/49741 (make -k check-c++0x not multilib aware)

	PR testsuite/49741
gcc/
	* Makefile.in ($(lang_checks_parallelized)): Allow --extra_opts
	rather than --tool_opts.
gcc/cp/
	* Make-lang.in (check-c++0x): Use --extra_opts instead of--tool_opts.
gcc/testsuite/
	* lib/g++.exp (${tool}_option_help, ${tool}_option_proc): Restore.
	Use --extra_opts instead of --additional_options.

	* Makefile.in (check-c++): Move check-gcc-c++0x after
	check-target-libstdc++-v3.

From-SVN: r176332

diff --git a/ChangeLog b/ChangeLog
index 7cf79b05fd0b353ebda7b244afa33ac27059fc4b..5794a5d1a1037c86e2a1c2caeaa598281597413b 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2011-07-15  Jason Merrill  <jason@redhat.com>
+
+       * Makefile.in (check-c++): Move check-gcc-c++0x after
+       check-target-libstdc++-v3.
+
 2011-07-15  Bernd Schmidt  <bernds@codesourcery.com>
 
        * MAINTAINERS (c6x port): New entry.

diff --git a/Makefile.in b/Makefile.in
index 506d26e544c97fb519e73d41122713b6457c4595..0d40358d209db2b763f1907005eb9a3411cf9c04 100644 (file)
--- a/Makefile.in
+++ b/Makefile.in
@@ -40157,7 +40157,7 @@ check-gcc-c++0x:
        s=`cd $(srcdir); ${PWD_COMMAND}`; export s; \
        $(HOST_EXPORTS) \
        (cd gcc && $(MAKE) $(GCC_FLAGS_TO_PASS) check-c++0x);
-check-c++: check-gcc-c++ check-gcc-c++0x check-target-libstdc++-v3
+check-c++: check-gcc-c++ check-target-libstdc++-v3 check-gcc-c++0x
 
 .PHONY: check-gcc-fortran check-fortran
 check-gcc-fortran:

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index a49896a16ad13bad9353e4132e2f5c302cfec424..7d52dd3ecbb5256906b8a423314dc5def3db799a 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2011-07-15  Jason Merrill  <jason@redhat.com>
+
+       PR testsuite/49741
+       * Makefile.in ($(lang_checks_parallelized)): Allow --extra_opts
+       rather than --tool_opts.
+
 2011-07-15  Basile Starynkevitch  <basile@starynkevitch.net>
 
        * doc/plugins.texi (Building GCC plugins): gengtype needs its

diff --git a/gcc/Makefile.in b/gcc/Makefile.in
index 0fded4e3988773858daf0e16ad955cf8c0b1ee23..47e14fa969757327116979a82f3c31cc991b5ea1 100644 (file)
--- a/gcc/Makefile.in
+++ b/gcc/Makefile.in
@@ -5019,7 +5019,7 @@ check_p_subdirs=$(wordlist 1,$(words $(check_$*_parallelize)),$(check_p_numbers)
 
 # For parallelized check-% targets, this decides whether parallelization
 # is desirable (if -jN is used and RUNTESTFLAGS doesn't contain anything
-# but optional --target_board or --tool_opts arguments).  If it is desirable,
+# but optional --target_board or --extra_opts arguments).  If desirable,
 # recursive make is run with check-parallel-$lang{,1,2,3,4,5} etc. goals,
 # which can be executed in parallel, as they are run in separate directories.
 # check-parallel-$lang{1,2,3,4,5} etc. goals invoke runtest with the longest
@@ -5036,7 +5036,7 @@ check_p_subdirs=$(wordlist 1,$(words $(check_$*_parallelize)),$(check_p_numbers)
 # to lang_checks_parallelized variable and define check_$lang_parallelize
 # variable (see above check_gcc_parallelize description).
 $(lang_checks_parallelized): check-% : site.exp
-       @if [ -z "$(filter-out --target_board=%,$(filter-out --tool_opts%,$(RUNTESTFLAGS)))" ] \
+       @if [ -z "$(filter-out --target_board=%,$(filter-out --extra_opts%,$(RUNTESTFLAGS)))" ] \
            && [ "$(filter -j, $(MFLAGS))" = "-j" ]; then \
          $(MAKE) TESTSUITEDIR="$(TESTSUITEDIR)" RUNTESTFLAGS="$(RUNTESTFLAGS)" \
            check-parallel-$* \

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 8e45b2ee196ddb2ad69ec8b211ae5b535202569d..a0ce80dadbd5d769d381913bde27dd5757b00bb9 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,8 @@
+2011-07-15  Jason Merrill  <jason@redhat.com>
+
+       PR testsuite/49741
+       * Make-lang.in (check-c++0x): Use --extra_opts instead of--tool_opts.
+
 2011-07-13  Jason Merrill  <jason@redhat.com>
 
        * Make-lang.in (check-c++0x): New.

diff --git a/gcc/cp/Make-lang.in b/gcc/cp/Make-lang.in
index b9251a44099385d43930a7e78f80e9d90c5e6e83..ad466b2dc163107a8a8d65aaa42cdcf5467defab 100644 (file)
--- a/gcc/cp/Make-lang.in
+++ b/gcc/cp/Make-lang.in
@@ -151,7 +151,7 @@ c++.srcman: doc/g++.1
 check-c++ : check-g++
 # Run the testsute in C++0x mode.
 check-c++0x:
-       $(MAKE) RUNTESTFLAGS="$(RUNTESTFLAGS) --tool_opts=-std=gnu++0x" \
+       $(MAKE) RUNTESTFLAGS="$(RUNTESTFLAGS) --extra_opts,-std=gnu++0x" \
          TESTSUITEDIR="$(TESTSUITEDIR).c++0x" check-g++
 check-c++-subtargets : check-g++-subtargets
 # List of targets that can use the generic check- rule and its // variant.

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 1d82350648b9a336b126bcc3007120ffa68cfbb2..c49833478f34e6db4997697cca7ee1cc1e1e6086 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,9 @@
+2011-07-15  Jason Merrill  <jason@redhat.com>
+
+       PR testsuite/49741
+       * lib/g++.exp (${tool}_option_help, ${tool}_option_proc): Restore.
+       Use --extra_opts instead of --additional_options.
+
 2011-07-15  Jakub Jelinek  <jakub@redhat.com>
 
        PR testsuite/49753

diff --git a/gcc/testsuite/lib/g++.exp b/gcc/testsuite/lib/g++.exp
index 81c4398a4e1a558cb04d95cc9917017d7110b498..9e269b12d66c4428bc58dc650b3b7c2791f45bed 100644 (file)
--- a/gcc/testsuite/lib/g++.exp
+++ b/gcc/testsuite/lib/g++.exp
@@ -307,3 +307,34 @@ proc g++_target_compile { source dest type options } {
 
     return $result
 }
+
+#
+# ${tool}_option_help
+#
+# Changed "additional" to "extra" because runtest.exp treats --a* as --all.
+#
+# This shouldn't be necessary at all; it should be entirely redundant with
+# --tool_opts, except that --tool_opts currently breaks multilib: see
+# http://lists.gnu.org/archive/html/dejagnu/2002-10/msg00007.html
+
+proc ${tool}_option_help { } {
+    send_user " --extra_opts,OPTIONS\t\tUse OPTIONS to compile the testcase files. OPTIONS should be comma-separated.\n"
+}
+
+#
+# ${tool}_option_proc
+#
+
+proc ${tool}_option_proc { option } {
+    if [regexp "^--extra_opts," $option] {
+       global gpp_compile_options
+       regsub "^--extra_opts," $option "" option
+       foreach x [split $option ","] {
+           lappend gpp_compile_options "additional_flags=$x"
+       }
+       verbose -log "gpp_compile_options set to $gpp_compile_options"
+       return 1
+    } else {
+       return 0
+    }
+}