[PATCH] D127966: [LV] Move LoopVersioning creation to LVP::execute.
Ayal Zaks via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Jun 26 14:09:56 PDT 2022
Ayal added a comment.
Curious if current lack of proper noalias metadata of an epilogue vector loop may result in missed optimization or potentially wrong code?
================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:7587
+ // Only use noalias metadata when using memory checks guaranteeing no overlap
+ // across all iterations.
+ const LoopAccessInfo *LAI = ILV.Legal->getLAI();
----------------
Above comment still holds, given that here it's not past an `if (!MemCheckBlock)` early-exit?
C/Should it move to appear before BestVPlan.prepareToExecute(), but presumably must appear after ILV.createVectorizedLoopSkeleton()?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D127966/new/
https://reviews.llvm.org/D127966
More information about the llvm-commits
mailing list