[PATCH] Re: Vectorizer question: DIV to RSHIFT conversion (take 2)

[Ira Rosen]

Jakub Jelinek <jakub@redhat.com> wrote on 15/12/2011 03:51:25 PM:

> On Thu, Dec 15, 2011 at 03:35:34PM +0200, Ira Rosen wrote:
> > > This patch also fixes
> > > a problem where vect_determine_vectorization_factor would iterate the
> > same
> > > stmt twice - for some reason both the original stmt and pattern stmt
(and
> > > def stmt) are marked as relevant,
> >
> > Do you have a testcase where the original stmt is marked as relevant?
It
> > shouldn't be that way.
>
> Yes, gcc.dg/vect/pr51015.c.  It isn't vectorized on x86_64-linux in the
> end, because of the other shift (long long), even with -mavx2 or -mxop.

Thanks, I'll take a look.

Ira

>
> > The patch looks ok to me, but I wonder if it's appropriate for the
current
> > stage.
>
> Richard said on IRC that he is ok with that.
>
>    Jakub
>