mpl: clang-tidy and format

Signed-off-by: Arthur Koucher <[email protected]>

Author: AcKoucher

src/mpl/src/SimulatedAnnealingCore.h

@@ -135,8 +135,7 @@ class SimulatedAnnealingCore
   virtual void calPenalty() = 0;
   void calOutlinePenalty();
   void calWirelength();
-  void addClosestAvailableRegionDistToWL(const T& macro,
-                                         float net_weight);
+  void addClosestAvailableRegionDistToWL(const T& macro, float net_weight);
   bool isOutsideTheOutline(const T& macro) const;
   float computeDistance(const Point& a, const Point& b) const;
   void calGuidancePenalty();

src/mpl/src/clusterEngine.cpp

@@ -405,8 +405,7 @@ void ClusteringEngine::createIOClusters()
 
 Cluster* ClusteringEngine::findIOClusterWithSameConstraint(odb::dbBTerm* bterm)
 {
-  for (const auto& [cluster, cluster_constraint] :
-       io_cluster_and_region_list_) {
+  for (const auto& [cluster, cluster_constraint] : unplaced_ios_to_region_) {
     if (bterm->getConstraintRegion() == cluster_constraint) {
       return cluster;
     }
@@ -438,7 +437,7 @@ void ClusteringEngine::createClusterOfUnplacedIOs(odb::dbBTerm* bterm)
       block_->dbuToMicrons(constraint_shape.dx()),
       block_->dbuToMicrons(constraint_shape.dy()));
 
-  io_cluster_and_region_list_.push_back({cluster.get(), constraint_shape});
+  unplaced_ios_to_region_[cluster.get()] = constraint_shape;
 
   tree_->maps.bterm_to_cluster_id[bterm] = id_;
   tree_->maps.id_to_cluster[id_++] = cluster.get();

src/mpl/src/clusterEngine.h

@@ -185,7 +185,6 @@ class ClusteringEngine
 
  private:
   using UniqueClusterQueue = std::queue<std::unique_ptr<Cluster>>;
-  using IOClusterAndRegion = std::pair<Cluster*, odb::Rect>;
 
   void init();
   Metrics* computeModuleMetrics(odb::dbModule* module);
@@ -288,7 +287,7 @@ class ClusteringEngine
   PhysicalHierarchy* tree_{nullptr};
 
   // Only for clusters of unplaced IOs.
-  std::vector<IOClusterAndRegion> io_cluster_and_region_list_;
+  std::map<Cluster*, odb::Rect> unplaced_ios_to_region_;
 
   int level_{0};  // Current level
   int id_{0};     // Current "highest" id

src/mpl/src/hier_rtlmp.cpp

@@ -3236,10 +3236,11 @@ Pusher::Pusher(utl::Logger* logger,
 void Pusher::setIOBlockages(const std::vector<Rect>& io_blockages)
 {
   for (const Rect& blockage : io_blockages) {
-    io_blockages_.push_back(odb::Rect(block_->micronsToDbu(blockage.xMin()),
-                                      block_->micronsToDbu(blockage.yMin()),
-                                      block_->micronsToDbu(blockage.xMax()),
-                                      block_->micronsToDbu(blockage.yMax())));
+    io_blockages_.emplace_back(
+        odb::Rect(block_->micronsToDbu(blockage.xMin()),
+                  block_->micronsToDbu(blockage.yMin()),
+                  block_->micronsToDbu(blockage.xMax()),
+                  block_->micronsToDbu(blockage.yMax())));
   }
 }
 

src/mpl/src/hier_rtlmp.h

@@ -181,8 +181,8 @@ class HierRTLMP
   void createPinAccessBlockages();
   bool treeHasOnlyUnconstrainedIOs();
   std::vector<Cluster*> getClustersOfUnplacedIOPins();
-  void createPinAccessBlockage(const Rect& micron_region, const float depth);
-  float computePinAccessBaseDepth(const float io_span);
+  void createPinAccessBlockage(const Rect& micron_region, float depth);
+  float computePinAccessBaseDepth(float io_span);
   BoundaryToRegionsMap getBoundaryToBlockedRegionsMap(
       const std::vector<odb::Rect>& blocked_regions_for_pins);
   std::vector<odb::Rect> computeAvailableRegions(