[PATCH] D106244: [LV] Move recurrence backedge fixup code to VPlan::execute (NFC).

[Ayal Zaks via Phabricator via llvm-commits]

Ayal accepted this revision.
Ayal added a comment.
This revision is now accepted and ready to land.

This looks good to me, with a couple of minor comments; thanks for following-up!

Would also be good to hook-up other header phis here as well.



================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.cpp:818
 
+  // Fix the latch value of the first-order recurrenxes in the vector loop.
+  VPBasicBlock *Header = Entry->getEntryBasicBlock();
----------------
recurrenxes

Mention that they have a single part, regardless of UF?

Use VectorLatchBB?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D106244/new/

https://reviews.llvm.org/D106244