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 BB_VISITED is used by

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 > > * bb-reorder.c (reo

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 @@ reorder_basic_blocks_simple (void) >> To start w

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 assigned yet. */ > >