[PATCH] D135017: [LV] Move exit cond simplification to separate transform.
Ayal Zaks via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Oct 2 07:40:05 PDT 2022
Ayal added inline comments.
================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:7623
+ if (!IsEpilogueVectorization)
+ VPlanTransforms::optimizeForConcreteVFAndUF(BestVPlan, BestVF, BestUF, PSE);
+
----------------
nit: can drop "Concrete".
Restrict the VF range of BestPlan to include only BestVF?
When printing BestPlan would be good to now indicate that it applies to BestUF only.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135017/new/
https://reviews.llvm.org/D135017
More information about the llvm-commits
mailing list