[PATCH] D121624: [VPlan] Model pre-header explicitly.

Ayal Zaks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 8 14:11:52 PDT 2022


Ayal added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.cpp:1481
+
+  BasicBlock *VectorPH = State.CFG.getPreheaderBBFor(this);
+  Value *Res =
----------------
Ayal wrote:
> Do we need to getPreheaderBBFor(this), given that the recipe should reside in the preheader VPBB?
> 
> 
We do, at this time, until D122095 moves the recipe (from the header) to the preheader VPBB.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121624



More information about the llvm-commits mailing list