[llvm] [VPlan] Introduce child regions as VPlan transform. (PR #129402)
Florian Hahn via llvm-commits
llvm-commits at lists.llvm.org
Sun Apr 13 04:47:29 PDT 2025
================
@@ -130,43 +98,23 @@ void PlainCFGBuilder::fixHeaderPhis() {
auto *VPPhi = cast<VPWidenPHIRecipe>(VPVal);
assert(VPPhi->getNumOperands() == 0 &&
"Expected VPInstruction with no operands.");
-
- Loop *L = LI->getLoopFor(Phi->getParent());
- assert(isHeaderBB(Phi->getParent(), L));
+ assert(isHeaderBB(Phi->getParent(), LI->getLoopFor(Phi->getParent())));
// For header phis, make sure the incoming value from the loop
// predecessor is the first operand of the recipe.
----------------
fhahn wrote:
I dropped the comment, I think at this point the header phis are not really special (the fixed predecessor order comes when we introduce the corresponding regions)
https://github.com/llvm/llvm-project/pull/129402
More information about the llvm-commits
mailing list