On Mon, 17 Oct 2016, Andrew Pinski wrote:
> On Mon, Oct 17, 2016 at 5:26 AM, Richard Biener <rguent...@suse.de> 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 the vectorizer - can you point
me to that?
Thanks,
Richard.
> Thanks,
> Andrew
>
> >
> > Richard.
> >
> > 2016-10-17 Richard Biener <rguent...@suse.de>
> >
> > * bb-reorder.c (reorder_basic_blocks_simple): Clear BB_VISITED
> > before using it.
> >
> > Index: gcc/bb-reorder.c
> > ===================================================================
> > --- 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.
> > */
> >
> > FOR_ALL_BB_FN (bb, cfun)
> > - bb->aux = bb;
> > + {
> > + bb->aux = bb;
> > + bb->flags &= ~BB_VISITED;
> > + }
> >
> > EXIT_BLOCK_PTR_FOR_FN (cfun)->aux = 0;
> >
