[PATCH] D60745: Remove RunSLPAfterLoopVectorization option...
Hal Finkel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 16 18:45:15 PDT 2019
hfinkel added a comment.
In D60745#1469622 <https://reviews.llvm.org/D60745#1469622>, @echristo wrote:
> In D60745#1468829 <https://reviews.llvm.org/D60745#1468829>, @hfinkel wrote:
>
> > Do we need to adjust the new pass manager too?
>
>
> Nope. There's only a single instantiation of the SLP vectorizer in the new pass manager - it's in a fairly gratuitously different location and some loop passes have run before and after. I think that should be handled separately.
Sounds good.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60745/new/
https://reviews.llvm.org/D60745
More information about the llvm-commits
mailing list