This is the mail archive of the
mailing list for the GCC project.
Re: RFA: speeding up dg-extract-results.sh
- From: Jeff Law <law at redhat dot com>
- To: gcc-patches at gcc dot gnu dot org, rdsandiford at googlemail dot com
- Date: Mon, 05 May 2014 10:08:13 -0600
- Subject: Re: RFA: speeding up dg-extract-results.sh
- Authentication-results: sourceware.org; auth=none
- References: <878utfe5g0 dot fsf at talisman dot default>
On 02/13/14 02:18, Richard Sandiford wrote:
the script takes just over 5 hours to produce the gcc.log file.
Can't argue with that reasoning. There's been talk of rewriting the
testsuite drivers to use python rather than tcl under the hood. I'd be
surprised if anyone in our community really prefers tcl over python.
This patch tries to reduce that by providing an alternative single-script
version. I was torn between Python and Tcl, but given how most people
tend to react to Tcl, I thought I'd better go for Python. I wouldn't
mind rewriting it in Tcl if that seems better though, not least because
expect is already a prerequisite.
It would mean a bit of pain for initial system bootstraps by the
distributors, but those are relatively rare occurrences and they can
always seed that process by building without testing to get the initial
tools, when are then used to build python, then rebuild gcc to get the
testing done. I don't see this as being a major issue.
Python isn't yet required and I'm pretty sure this script needs 2.6
or later. I'm also worried that the seek/tell stuff might not work on
Windows. The patch therefore gets dg-extract-results.sh to check the
environment first and call into the python version if possible,
otherwise it falls back on the current approach. This also means
that the patch is contained entirely within contrib/. If this does
indeed not work on Windows then we should either fix the python code
(obviously preferred) or get dg-extract-results.sh to skip it on
Windows for now.
Kai can probably test an answer questions about the Windows space.
I checked that the outputs were otherwise identical for a set of
mips64-linux-gnu, mipsisa64-sde-elf and x86_64-linux-gnu runs. I also
reran the acats tests with some nobbled testcases in order to test the
failure paths there.
Also bootstrapped & regression-tested on x86_64-linux-gnu. OK to install?
* dg-extract-results.py: New file.
* dg-extract-results.sh: Use it if the environment seems suitable.
OK by me.