> From: Mike Stump <mikest...@comcast.net> > Date: Sat, 11 Jan 2014 01:55:09 +0100
> On Jan 10, 2014, at 11:26 AM, Hans-Peter Nilsson > <hans-peter.nils...@axis.com> wrote: > > This patch "fixes" a regression on trunk and the 4.8 branch: > > > Index: gcc/testsuite/gcc.dg/pr46309.c > > =================================================================== > > --- gcc/testsuite/gcc.dg/pr46309.c (revision 206534) > > +++ gcc/testsuite/gcc.dg/pr46309.c (working copy) > > @@ -1,5 +1,5 @@ > > /* PR tree-optimization/46309 */ > > -/* { dg-do compile } */ > > +/* { dg-do compile { target { ! { cris*-*-* } } } } */ > > If you could add /* branch cost==1 */ or some such to it, it > would help others that have ports and see it fail, why this > might be appropriate to just skip on their target, thanks. > It's already there in the two lines you cut out, isn't it? Index: gcc/testsuite/gcc.dg/pr46309.c =================================================================== --- gcc/testsuite/gcc.dg/pr46309.c (revision 206534) +++ gcc/testsuite/gcc.dg/pr46309.c (working copy) @@ -1,5 +1,5 @@ /* PR tree-optimization/46309 */ -/* { dg-do compile } */ +/* { dg-do compile { target { ! { cris*-*-* } } } } */ /* { dg-options "-O2 -fdump-tree-reassoc-details" } */ /* The transformation depends on BRANCH_COST being greater than 1 (see the notes in the PR), so try to force that. */ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ brgds, H-P