[PATCH] D19501: Add LoadStoreVectorizer pass

[Matt Arsenault via llvm-commits]

arsenm marked an inline comment as done.
arsenm added a comment.

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/D19501