[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