[llvm] 75270e3 - [VPlan] Don't print VPlan DT after VPlan construction. (NFC)

[Florian Hahn via llvm-commits]

Author: Florian Hahn
Date: 2025-03-01T21:15:56Z
New Revision: 75270e3750db13e20ddbf42df6b7094c6266ed57

URL: https://github.com/llvm/llvm-project/commit/75270e3750db13e20ddbf42df6b7094c6266ed57
DIFF: https://github.com/llvm/llvm-project/commit/75270e3750db13e20ddbf42df6b7094c6266ed57.diff

LOG: [VPlan] Don't print VPlan DT after VPlan construction. (NFC)

Remove unnecessary code to just print VPlan dominator tree.

Added: 
    

Modified: 
    llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
    llvm/lib/Transforms/Vectorize/VPlanHCFGBuilder.cpp
    llvm/lib/Transforms/Vectorize/VPlanHCFGBuilder.h
    llvm/unittests/Transforms/Vectorize/VPlanVerifierTest.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
index e2612698b6b0f..c447fa4843591 100644
--- a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
+++ b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
@@ -58,6 +58,7 @@
 #include "VPRecipeBuilder.h"
 #include "VPlan.h"
 #include "VPlanAnalysis.h"
+#include "VPlanCFG.h"
 #include "VPlanHCFGBuilder.h"
 #include "VPlanHelpers.h"
 #include "VPlanPatternMatch.h"

diff  --git a/llvm/lib/Transforms/Vectorize/VPlanHCFGBuilder.cpp b/llvm/lib/Transforms/Vectorize/VPlanHCFGBuilder.cpp
index dcf1057b991ee..19af0225c128f 100644
--- a/llvm/lib/Transforms/Vectorize/VPlanHCFGBuilder.cpp
+++ b/llvm/lib/Transforms/Vectorize/VPlanHCFGBuilder.cpp
@@ -496,9 +496,4 @@ void VPlanHCFGBuilder::buildHierarchicalCFG() {
   // Build Top Region enclosing the plain CFG.
   buildPlainCFG();
   LLVM_DEBUG(Plan.setName("HCFGBuilder: Plain CFG\n"); dbgs() << Plan);
-
-  // Compute plain CFG dom tree for VPLInfo.
-  VPDomTree.recalculate(Plan);
-  LLVM_DEBUG(dbgs() << "Dominator Tree after building the plain CFG.\n";
-             VPDomTree.print(dbgs()));
 }

diff  --git a/llvm/lib/Transforms/Vectorize/VPlanHCFGBuilder.h b/llvm/lib/Transforms/Vectorize/VPlanHCFGBuilder.h
index bc853bf7a1395..f7f98ed7b1755 100644
--- a/llvm/lib/Transforms/Vectorize/VPlanHCFGBuilder.h
+++ b/llvm/lib/Transforms/Vectorize/VPlanHCFGBuilder.h
@@ -24,7 +24,7 @@
 #ifndef LLVM_TRANSFORMS_VECTORIZE_VPLAN_VPLANHCFGBUILDER_H
 #define LLVM_TRANSFORMS_VECTORIZE_VPLAN_VPLANHCFGBUILDER_H
 
-#include "VPlanDominatorTree.h"
+#include "llvm/ADT/DenseMap.h"
 
 namespace llvm {
 
@@ -33,6 +33,8 @@ class LoopInfo;
 class VPRegionBlock;
 class VPlan;
 class VPlanTestIRBase;
+class VPBlockBase;
+class BasicBlock;
 
 /// Main class to build the VPlan H-CFG for an incoming IR.
 class VPlanHCFGBuilder {
@@ -48,11 +50,6 @@ class VPlanHCFGBuilder {
   // The VPlan that will contain the H-CFG we are building.
   VPlan &Plan;
 
-  // Dominator analysis for VPlan plain CFG to be used in the
-  // construction of the H-CFG. This analysis is no longer valid once regions
-  // are introduced.
-  VPDominatorTree VPDomTree;
-
   /// Map of create VP blocks to their input IR basic blocks, if they have been
   /// created for a input IR basic block.
   DenseMap<VPBlockBase *, BasicBlock *> VPB2IRBB;

diff  --git a/llvm/unittests/Transforms/Vectorize/VPlanVerifierTest.cpp b/llvm/unittests/Transforms/Vectorize/VPlanVerifierTest.cpp
index f818b49fdbe7f..182c39cadecfa 100644
--- a/llvm/unittests/Transforms/Vectorize/VPlanVerifierTest.cpp
+++ b/llvm/unittests/Transforms/Vectorize/VPlanVerifierTest.cpp
@@ -8,6 +8,7 @@
 
 #include "../lib/Transforms/Vectorize/VPlanVerifier.h"
 #include "../lib/Transforms/Vectorize/VPlan.h"
+#include "../lib/Transforms/Vectorize/VPlanCFG.h"
 #include "VPlanTestBase.h"
 #include "llvm/IR/Instruction.h"
 #include "llvm/IR/Instructions.h"