[llvm] [VPlan] Connect Entry to scalar preheader during initial construction. (PR #140132)

via llvm-commits llvm-commits at lists.llvm.org
Tue May 27 02:18:35 PDT 2025


================
@@ -1846,7 +1846,7 @@ static void removeBranchOnCondTrue(VPlan &Plan) {
   using namespace llvm::VPlanPatternMatch;
   for (VPBasicBlock *VPBB : VPBlockUtils::blocksOnly<VPBasicBlock>(
            vp_depth_first_shallow(Plan.getEntry()))) {
-    if (VPBB->getNumSuccessors() != 2 ||
+    if (VPBB->getNumSuccessors() != 2 || isa<VPIRBasicBlock>(VPBB) ||
----------------
ayalz wrote:

Excluding VPIRBB's only because of the temporary terminator-less entry block with two successors? If so, suffice to exclude `VPBB == Plan.getEntry()`? But from an earlier comment: "we still need to allow terminator-less VPIRBBs for various parts of the skeleton."

https://github.com/llvm/llvm-project/pull/140132


More information about the llvm-commits mailing list