[llvm] 20d491b - [VPlan] Remove re-using vector PH in VPBasicBlock::execute (NFC).

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 3 11:57:56 PST 2025


Author: Florian Hahn
Date: 2025-01-03T19:56:44Z
New Revision: 20d491bb993218eae6a13e4901da943ebd804f7a

URL: https://github.com/llvm/llvm-project/commit/20d491bb993218eae6a13e4901da943ebd804f7a
DIFF: https://github.com/llvm/llvm-project/commit/20d491bb993218eae6a13e4901da943ebd804f7a.diff

LOG: [VPlan] Remove re-using vector PH in VPBasicBlock::execute (NFC).

Remove logic to re-use the previous basic block for the vector pre
header from VPBasicBlock::execute. The preheader is now modeled as
VPIRBasicBlock, so the code is no longer needed.

Split off from https://github.com/llvm/llvm-project/pull/108378.

Added: 
    

Modified: 
    llvm/lib/Transforms/Vectorize/VPlan.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Vectorize/VPlan.cpp b/llvm/lib/Transforms/Vectorize/VPlan.cpp
index 6d02efc05614a3..06c36396a17f38 100644
--- a/llvm/lib/Transforms/Vectorize/VPlan.cpp
+++ b/llvm/lib/Transforms/Vectorize/VPlan.cpp
@@ -487,11 +487,9 @@ void VPBasicBlock::execute(VPTransformState *State) {
   };
 
   // 1. Create an IR basic block.
-  if (this == getPlan()->getVectorPreheader() ||
-      (Replica && this == getParent()->getEntry()) ||
+  if ((Replica && this == getParent()->getEntry()) ||
       IsReplicateRegion(getSingleHierarchicalPredecessor())) {
     // Reuse the previous basic block if the current VPBB is either
-    //  * the vector preheader,
     //  * the entry to a replicate region, or
     //  * the exit of a replicate region.
     State->CFG.VPBB2IRBB[this] = NewBB;


        


More information about the llvm-commits mailing list