[PATCH] D111301: [VPlan] Set builder insert point instead of moving recipes. (NFC)

[Florian Hahn via Phabricator via llvm-commits]

fhahn marked an inline comment as done.
fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:9447
+      auto &VPBB = *PrevRecipe->getParent();
+      if (PrevRecipe->isPhi())
+        RecurSplice->moveBefore(VPBB, VPBB.getFirstNonPhi());
----------------
fhahn wrote:
> Ayal wrote:
> > Hmm, PrevRecipe cannot be a header phi because that would result in 2nd order recurrence (or greater); but it may be a phi of a replicated region? Should blends be excluded here? Curious to see a test...
> I thought there was a test case, but it looks like I remembered incorrectly. Removed the check.
> I thought there was a test case, but it looks like I remembered incorrectly. Removed the check.

Ok I now what phi-like case I originally hit: if the previous value is an induction value! So I think we still need the special handling?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111301