Re: [PATCH] Clear BB_VISITED in bb-reorder

2016-10-18 Thread Richard Biener
On Mon, 17 Oct 2016, Andrew Pinski wrote: > On Mon, Oct 17, 2016 at 5:26 AM, Richard Biener wrote: > > > > $subject, applied as obvious. > > I think you should do the same for the vectorizer too. I noticed that > when testing the patch for loop splitting. Can't see where

Re: [PATCH] Clear BB_VISITED in bb-reorder

2016-10-17 Thread Andrew Pinski
On Mon, Oct 17, 2016 at 5:26 AM, Richard Biener wrote: > > $subject, applied as obvious. I think you should do the same for the vectorizer too. I noticed that when testing the patch for loop splitting. Thanks, Andrew > > Richard. > > 2016-10-17 Richard Biener

Re: Use FOR_ALL_BB_FN in a few more places (was: [PATCH] Clear BB_VISITED in bb-reorder)

2016-10-17 Thread Richard Biener
On October 17, 2016 6:09:02 PM GMT+02:00, Thomas Schwinge wrote: >Hi! > >On Mon, 17 Oct 2016 15:17:13 +0200, Richard Biener > wrote: >> On Mon, Oct 17, 2016 at 3:12 PM, Thomas Schwinge >> wrote: >> > On Mon, 17 Oct

Use FOR_ALL_BB_FN in a few more places (was: [PATCH] Clear BB_VISITED in bb-reorder)

2016-10-17 Thread Thomas Schwinge
Hi! On Mon, 17 Oct 2016 15:17:13 +0200, Richard Biener wrote: > On Mon, Oct 17, 2016 at 3:12 PM, Thomas Schwinge > wrote: > > On Mon, 17 Oct 2016 14:26:42 +0200 (CEST), Richard Biener > > wrote: > >> --- gcc/bb-reorder.c

Re: [PATCH] Clear BB_VISITED in bb-reorder

2016-10-17 Thread Richard Biener
On Mon, Oct 17, 2016 at 3:12 PM, Thomas Schwinge wrote: > Hi! > > On Mon, 17 Oct 2016 14:26:42 +0200 (CEST), Richard Biener > wrote: >> --- gcc/bb-reorder.c (revision 241228) >> +++ gcc/bb-reorder.c (working copy) >> @@ -2355,7 +2355,10 @@

Re: [PATCH] Clear BB_VISITED in bb-reorder

2016-10-17 Thread Thomas Schwinge
Hi! On Mon, 17 Oct 2016 14:26:42 +0200 (CEST), Richard Biener wrote: > --- gcc/bb-reorder.c (revision 241228) > +++ gcc/bb-reorder.c (working copy) > @@ -2355,7 +2355,10 @@ reorder_basic_blocks_simple (void) > To start with, everything points to itself, nothing is

[PATCH] Clear BB_VISITED in bb-reorder

2016-10-17 Thread Richard Biener
$subject, applied as obvious. Richard. 2016-10-17 Richard Biener * bb-reorder.c (reorder_basic_blocks_simple): Clear BB_VISITED before using it. Index: gcc/bb-reorder.c === ---