From c18bc7f7feac94e9abf2e65345c33709a2451eb0 Mon Sep 17 00:00:00 2001 From: Florian Hahn Date: Wed, 5 Apr 2023 14:29:24 +0100 Subject: [PATCH] [VPlan] Replace check for replicate regions with assert (NFCI). After recent changes, replication regions only get introduced later, so there's no need to check for them. --- llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp b/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp index 99f6000..07abcdc 100644 --- a/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp +++ b/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp @@ -643,14 +643,10 @@ static bool properlyDominates(const VPRecipeBase *A, const VPRecipeBase *B, if (ParentA == ParentB) return LocalComesBefore(A, B); - const VPRegionBlock *RegionA = - GetReplicateRegion(const_cast(A)); - const VPRegionBlock *RegionB = - GetReplicateRegion(const_cast(B)); - if (RegionA) - ParentA = RegionA->getExiting(); - if (RegionB) - ParentB = RegionB->getExiting(); + assert(!GetReplicateRegion(const_cast(A)) && + "No replicate regions expected at this point"); + assert(!GetReplicateRegion(const_cast(B)) && + "No replicate regions expected at this point"); return VPDT.properlyDominates(ParentA, ParentB); } -- 2.7.4