[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