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.
> 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
