[PATCH] gcc parallel make check

Tom de Vries Tom_deVries@mentor.com
Tue Nov 25 16:18:00 GMT 2014


On 15-09-14 18:05, Jakub Jelinek wrote:
> libstdc++-v3/
> 	* testsuite/Makefile.am (check_p_numbers0, check_p_numbers1,
> 	check_p_numbers2, check_p_numbers3, check_p_numbers4,
> 	check_p_numbers5, check_p_numbers6, check_p_numbers,
> 	check_p_subdirs): New variables.
> 	(check_DEJAGNU_normal_targets): Use check_p_subdirs.
> 	(check-DEJAGNU): Rewritten so that for parallelized
> 	testing each job runs all the *.exp files, with
> 	GCC_RUNTEST_PARALLELIZE_DIR set in environment.
> 	* testsuite/Makefile.in: Regenerated.
> 	* testsuite/lib/libstdc++.exp (gcc_parallel_test_run_p,
> 	gcc_parallel_test_enable): New procedures.  If
> 	GCC_RUNTEST_PARALLELIZE_DIR is set in environment, override
> 	runtest_file_p to invoke also gcc_parallel_test_run_p.
> 	* testsuite/libstdc++-abi/abi.exp: Run all the tests serially
> 	by the first parallel runtest encountering it.  Fix up path
> 	of the extract_symvers script.
> 	* testsuite/libstdc++-xmethods/xmethods.exp: Run all the tests
> 	serially by the first parallel runtest encountering it.  Run
> 	dg-finish even in case of error.

When comparing test results of patch builds with test results of reference 
builds, the only differences I'm seeing are random differences in amount of 
'UNSUPPORTED: prettyprinter.exp'.

This patch fixes that by ensuring that we print that unsupported message only once.

The resulting test result comparison diff is:
...
--- without/FAIL  2014-11-24 17:46:32.202673282 +0100
+++ with/FAIL     2014-11-25 13:45:15.636131571 +0100
  libstdc++-v3/testsuite/libstdc++.sum:UNSUPPORTED: prettyprinters.exp
-libstdc++-v3/testsuite/libstdc++.sum:UNSUPPORTED: prettyprinters.exp
-libstdc++-v3/testsuite/libstdc++.sum:UNSUPPORTED: prettyprinters.exp
-libstdc++-v3/testsuite/libstdc++.sum:UNSUPPORTED: prettyprinters.exp
-libstdc++-v3/testsuite/libstdc++.sum:UNSUPPORTED: prettyprinters.exp
  libstdc++-v3/testsuite/libstdc++.sum:UNSUPPORTED: xmethods.exp
...

Furthermore, the patch adds a dg-finish in case the prettyprinters.exp file is 
unsupported, which AFAIU is also required in that case.

Bootstrapped and reg-tested on x86_64.

OK for trunk/stage3?

Thanks,
- Tom


-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Ensure-single-UNSUPPORTED-prettyprinters.exp-message.patch
Type: text/x-patch
Size: 979 bytes
Desc: not available
URL: <https://gcc.gnu.org/pipermail/gcc/attachments/20141125/f4e1afed/attachment.bin>


More information about the Gcc mailing list