[PATCH] D19501: Add LoadStoreVectorizer pass

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed May 25 13:01:53 PDT 2016

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


More information about the llvm-commits mailing list