[PATCH] D127966: [LV] Move LoopVersioning creation to LVP::execute.

[Ayal Zaks via Phabricator via llvm-commits]

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