mpl: cleanup leftover subtree code

Signed-off-by: João Mai <[email protected]>

Author: joaomai

src/mpl/src/clusterEngine.cpp

@@ -1981,12 +1981,6 @@ void ClusteringEngine::breakMixedLeaf(Cluster* mixed_leaf)
   Cluster* parent = mixed_leaf->getParent();
   const float macro_dominated_cluster_ratio = 0.01;
 
-  // Split by replacement if macro dominated.
-  //if (mixed_leaf->getNumStdCell() * macro_dominated_cluster_ratio
-  //    < mixed_leaf->getNumMacro()) {
-  //  parent = mixed_leaf->getParent();
-  //}
-
   mapMacroInCluster2HardMacro(mixed_leaf);
 
   std::vector<HardMacro*> hard_macros = mixed_leaf->getHardMacros();
@@ -2027,12 +2021,7 @@ void ClusteringEngine::breakMixedLeaf(Cluster* mixed_leaf)
   // Never use SetInstProperty in the following lines for the reason above!
   std::vector<int> virtual_conn_clusters;
 
-  // Deal with the std cells
-  if (parent == mixed_leaf) {
-    addStdCellClusterToSubTree(parent, mixed_leaf, virtual_conn_clusters);
-  } else {
-    replaceByStdCellCluster(mixed_leaf, virtual_conn_clusters);
-  }
+  replaceByStdCellCluster(mixed_leaf, virtual_conn_clusters);
 
   // Deal with the macros
   for (int i = 0; i < macro_class.size(); i++) {
@@ -2256,29 +2245,6 @@ void ClusteringEngine::groupSingleMacroClusters(
   }
 }
 
-void ClusteringEngine::addStdCellClusterToSubTree(
-    Cluster* parent,
-    Cluster* mixed_leaf,
-    std::vector<int>& virtual_conn_clusters)
-{
-  std::string std_cell_cluster_name = mixed_leaf->getName();
-  auto std_cell_cluster
-      = std::make_unique<Cluster>(id_, std_cell_cluster_name, logger_);
-
-  std_cell_cluster->copyInstances(*mixed_leaf);
-  std_cell_cluster->clearLeafMacros();
-  std_cell_cluster->setClusterType(StdCellCluster);
-
-  setClusterMetrics(std_cell_cluster.get());
-
-  virtual_conn_clusters.push_back(std_cell_cluster->getId());
-
-  tree_->maps.id_to_cluster[id_++] = std_cell_cluster.get();
-  std_cell_cluster->setParent(parent);
-  parent->addChild(std::move(std_cell_cluster));
-}
-
-// We don't modify the physical hierarchy when spliting by replacement
 void ClusteringEngine::replaceByStdCellCluster(
     Cluster* mixed_leaf,
     std::vector<int>& virtual_conn_clusters)

src/mpl/src/clusterEngine.h

@@ -230,9 +230,6 @@ class ClusteringEngine
                                 const std::vector<int>& signature_class,
                                 std::vector<int>& interconn_class,
                                 std::vector<int>& macro_class);
-  void addStdCellClusterToSubTree(Cluster* parent,
-                                  Cluster* mixed_leaf,
-                                  std::vector<int>& virtual_conn_clusters);
   void replaceByStdCellCluster(Cluster* mixed_leaf,
                                std::vector<int>& virtual_conn_clusters);