This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: parallel check output changes?
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Segher Boessenkool <segher at kernel dot crashing dot org>
- Cc: Andrew MacLeod <amacleod at redhat dot com>, Bernd Schmidt <bernds at codesourcery dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>, richard dot sandiford at arm dot com
- Date: Thu, 2 Oct 2014 19:05:03 +0200
- Subject: Re: parallel check output changes?
- Authentication-results: sourceware.org; auth=none
- References: <20140918130109 dot GH17454 at tucnak dot redhat dot com> <541AD880 dot 7080703 at redhat dot com> <541AF451 dot 3070406 at redhat dot com> <541B1710 dot 8060809 at codesourcery dot com> <20140918173609 dot GM17454 at tucnak dot redhat dot com> <20140918184455 dot GB28595 at gate dot crashing dot org> <20140919093723 dot GA26414 at gate dot crashing dot org> <87iokel5c0 dot fsf at e105548-lin dot cambridge dot arm dot com> <5422DB41 dot 1090800 at redhat dot com> <20141002164739 dot GA25260 at gate dot crashing dot org>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Thu, Oct 02, 2014 at 11:47:39AM -0500, Segher Boessenkool wrote:
> On Wed, Sep 24, 2014 at 10:54:57AM -0400, Andrew MacLeod wrote:
> > Is this suppose to be resolved now? I'm still seeing some issues with a
> > branch cut from mainline from yesterday.
>
> Confirmed. The following patch works for me, and Andrew has tested it
> as well. The comment it removes isn't valid before the patch either.
>
> Okay for mainline?
>
>
> Segher
>
>
> 2014-10-02 Segher Boessenkool <segher@kernel.crashing.org>
>
> contrib/
> * dg-extract-results.py (output_variation): Always sort if do_sum.
Ok, thanks.
> --- a/contrib/dg-extract-results.py
> +++ b/contrib/dg-extract-results.py
> @@ -495,15 +495,7 @@ class Prog:
> key = attrgetter ('name')):
> sys.stdout.write ('Running ' + harness.name + ' ...\n')
> if self.do_sum:
> - # Keep the original test result order if there was only
> - # one segment for this harness. This is needed for
> - # unsorted.exp, which has unusual test names. Otherwise
> - # sort the tests by test filename. If there are several
> - # subtests for the same test filename (such as 'compilation',
> - # 'test for excess errors', etc.) then keep the subtests
> - # in the original order.
> - if len (harness.segments) > 1:
> - harness.results.sort()
> + harness.results.sort()
> for (key, line) in harness.results:
> sys.stdout.write (line)
> else:
Jakub