[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