From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 20111 invoked by alias); 5 May 2014 16:08:18 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 20095 invoked by uid 89); 5 May 2014 16:08:17 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.6 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 05 May 2014 16:08:15 +0000 Received: from int-mx12.intmail.prod.int.phx2.redhat.com (int-mx12.intmail.prod.int.phx2.redhat.com [10.5.11.25]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s45G8EEr024808 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 5 May 2014 12:08:14 -0400 Received: from stumpy.slc.redhat.com (ovpn-113-142.phx2.redhat.com [10.3.113.142]) by int-mx12.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id s45G8E17027284; Mon, 5 May 2014 12:08:14 -0400 Message-ID: <5367B76D.4060200@redhat.com> Date: Mon, 05 May 2014 16:08:00 -0000 From: Jeff Law User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0 MIME-Version: 1.0 To: gcc-patches@gcc.gnu.org, rdsandiford@googlemail.com Subject: Re: RFA: speeding up dg-extract-results.sh References: <878utfe5g0.fsf@talisman.default> In-Reply-To: <878utfe5g0.fsf@talisman.default> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-IsSubscribed: yes X-SW-Source: 2014-05/txt/msg00218.txt.bz2 On 02/13/14 02:18, Richard Sandiford wrote: > > http://gcc.gnu.org/ml/gcc-testresults/2014-02/msg00025.html > > the script takes just over 5 hours to produce the gcc.log file. Good grief... > > 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. 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. 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? > > Thanks, > Richard > > > contrib/ > * dg-extract-results.py: New file. > * dg-extract-results.sh: Use it if the environment seems suitable. OK by me. jeff