Merge pull request #8440 from arthurjolo/cts_remove_unused_functions

Cts remove unused functions

Author: maliberty

src/cts/BUILD

@@ -19,7 +19,6 @@ cc_library(
         "src/CtsObserver.h",
         "src/HTreeBuilder.h",
         "src/LatencyBalancer.h",
-        "src/LevelBalancer.h",
         "src/SinkClustering.h",
         "src/TreeBuilder.h",
         "src/Util.h",
@@ -41,7 +40,6 @@ cc_library(
         "src/CtsOptions.cpp",
         "src/HTreeBuilder.cpp",
         "src/LatencyBalancer.cpp",
-        "src/LevelBalancer.cpp",
         "src/SinkClustering.cpp",
         "src/TechChar.cpp",
         "src/TreeBuilder.cpp",

src/cts/include/cts/TritonCTS.h

@@ -208,22 +208,6 @@ class TritonCTS
   void setAllClocksPropagated();
   void repairClockNets();
   void balanceMacroRegisterLatencies();
-  float getVertexClkArrival(sta::Vertex* sinkVertex,
-                            odb::dbNet* topNet,
-                            odb::dbITerm* iterm);
-  void computeAveSinkArrivals(TreeBuilder* builder, sta::Graph* graph);
-  void computeSinkArrivalRecur(odb::dbNet* topClokcNet,
-                               odb::dbITerm* iterm,
-                               float& sumArrivals,
-                               unsigned& numSinks,
-                               sta::Graph* graph);
-  bool propagateClock(odb::dbITerm* input);
-  void adjustLatencies(TreeBuilder* macroBuilder, TreeBuilder* registerBuilder);
-  void computeTopBufferDelay(TreeBuilder* builder);
-  odb::dbInst* insertDelayBuffer(odb::dbInst* driver,
-                                 const std::string& clockName,
-                                 int locX,
-                                 int locY);
 
   sta::dbSta* openSta_ = nullptr;
   sta::dbNetwork* network_ = nullptr;

src/cts/src/CMakeLists.txt

@@ -14,7 +14,6 @@ add_library(cts_lib
     TechChar.cpp
     TritonCTS.cpp
     Clustering.cpp
-    LevelBalancer.cpp
     LatencyBalancer.cpp
     CtsOptions.cpp
 )

src/cts/src/CtsOptions.h

@@ -277,8 +277,6 @@ class CtsOptions : public odb::dbBlockCallBackObj
     return macroSinkClustersSizeSet_;
   }
   unsigned getNumStaticLayers() const { return numStaticLayers_; }
-  void setBalanceLevels(bool balance) { balanceLevels_ = balance; }
-  bool getBalanceLevels() const { return balanceLevels_; }
   void setNumStaticLayers(unsigned num) { numStaticLayers_ = num; }
   void resetNumStaticLayers() { numStaticLayers_ = 0; }
   void setSinkBuffer(const std::string& buffer) { sinkBuffer_ = buffer; }
@@ -400,7 +398,6 @@ class CtsOptions : public odb::dbBlockCallBackObj
   bool macroMaxDiameterSet_ = false;
   unsigned macroSinkClustersSize_ = 4;
   bool macroSinkClustersSizeSet_ = true;
-  bool balanceLevels_ = false;
   unsigned sinkClusteringLevels_ = 0;
   unsigned numStaticLayers_ = 0;
   std::vector<std::string> bufferList_;

src/cts/src/LatencyBalancer.cpp

@@ -45,8 +45,6 @@ int LatencyBalancer::run()
                 33,
                 "Balancing latency for clock {}",
                 root_->getClock().getSdcName());
-  worseDelay_ = std::numeric_limits<float>::min();
-  delayBufIndex_ = 0;
   initSta();
   findLeafBuilders(root_);
   buildGraph(root_->getTopInputNet());
@@ -136,8 +134,8 @@ void LatencyBalancer::buildGraph(odb::dbNet* clkInputNet)
   }
   int builderSrcId = graph_.size();
   GraphNode builderSrcNode
-      = GraphNode(builderSrcId, rootSrcName, rootOutputITerm);
-  graph_.push_back(builderSrcNode);
+      = GraphNode(builderSrcId, std::move(rootSrcName), rootOutputITerm);
+  graph_.push_back(std::move(builderSrcNode));
 
   std::stack<int> visitNode;
   visitNode.push(builderSrcId);
@@ -164,7 +162,7 @@ void LatencyBalancer::buildGraph(odb::dbNet* clkInputNet)
         odb::dbInst* sinkInst = sinkIterm->getInst();
         std::string sinkName = sinkInst->getName();
         GraphNode sinkNode = GraphNode(sinkId, sinkName, sinkIterm);
-        graph_.push_back(sinkNode);
+        graph_.push_back(std::move(sinkNode));
         graph_[driverId].childrenIds.push_back(sinkId);
 
         if (inst2builder_.find(sinkName) != inst2builder_.end()) {
@@ -440,7 +438,7 @@ void LatencyBalancer::balanceLatencies(int nodeId)
     if (!previouBufToInsert) {
       previouBufToInsert = bufToInsert;
       sinksInput.clear();
-      sinksInput = children;
+      sinksInput = std::move(children);
       continue;
     }
 
@@ -449,7 +447,7 @@ void LatencyBalancer::balanceLatencies(int nodeId)
         = insertDelayBuffers(numBuffers, srcX, srcY, sinksInput);
 
     sinksInput.clear();
-    sinksInput = children;
+    sinksInput = std::move(children);
     sinksInput.push_back(delauBuffInput);
 
     previouBufToInsert = bufToInsert;

src/cts/src/LatencyBalancer.h

@@ -66,7 +66,9 @@ class LatencyBalancer
         network_(network),
         openSta_(sta),
         wireSegmentUnit_(scalingUnit),
-        capPerDBU_(capPerDBU)
+        capPerDBU_(capPerDBU),
+        worseDelay_(std::numeric_limits<float>::min()),
+        delayBufIndex_(0)
   {
   }
 
@@ -115,9 +117,9 @@ class LatencyBalancer
   sta::dbSta* openSta_ = nullptr;
   sta::Graph* timingGraph_ = nullptr;
   double wireSegmentUnit_;
-  float worseDelay_;
   float bufferDelay_;
   double capPerDBU_;
+  float worseDelay_;
   int delayBufIndex_;
   std::vector<GraphNode> graph_;
   std::map<std::string, TreeBuilder*> inst2builder_;