[PATCH] D19501: Add LoadStoreVectorizer pass

[Matt Arsenault via llvm-commits]

arsenm added a comment.

In http://reviews.llvm.org/D19501#426396, @arsenm wrote:

> In http://reviews.llvm.org/D19501#414459, @mzolotukhin wrote:
>
> > Hi Matt,
> >
> > I think it's ok to have it in a separate pass, but we really need to try to factor out common parts. For instance, we definitely have `propagateMetadata` and `isConsecutiveAccess` in SLP.
> >
> > Thanks,
> > Michael
>
>
> isConsecutiveAccess is already factored into a separate utility function, but it seems to have diverged from the one here. Some work will be needed to merge them. I've added a FIXME for it


http://reviews.llvm.org/D20639 splits SLPVectorizer's propagateMetadata into a utility function


http://reviews.llvm.org/D19501