Merge pull request #9072 from AcKoucher/mpl-dbu-checks

mpl: fix some remaining floating point inconsistencies

Author: eder-matheus

src/mpl/src/SACoreSoftMacro.cpp

@@ -855,8 +855,8 @@ void SACoreSoftMacro::fillDeadSpace()
     y_point.insert(macros_[macro_id].getY());
     y_point.insert(macros_[macro_id].getY() + macros_[macro_id].getHeight());
   }
-  x_point.insert(0.0);
-  y_point.insert(0.0);
+  x_point.insert(0);
+  y_point.insert(0);
   x_point.insert(outline_.dx());
   y_point.insert(outline_.dy());
   // create grid
@@ -872,7 +872,7 @@ void SACoreSoftMacro::fillDeadSpace()
   }
 
   for (int macro_id = 0; macro_id < pos_seq_.size(); macro_id++) {
-    if (macros_[macro_id].getArea() <= 0.0) {
+    if (macros_[macro_id].getArea() == 0) {
       continue;
     }
 
@@ -892,7 +892,7 @@ void SACoreSoftMacro::fillDeadSpace()
   // propagate from the MixedCluster and then StdCellCluster
   for (int order = 0; order <= 1; order++) {
     for (int macro_id = 0; macro_id < pos_seq_.size(); macro_id++) {
-      if (macros_[macro_id].getArea() <= 0.0) {
+      if (macros_[macro_id].getArea() == 0) {
         continue;
       }
       const bool forward_flag = (order == 0)

src/mpl/src/clusterEngine.cpp

@@ -1195,7 +1195,7 @@ void ClusteringEngine::setClusterMetrics(Cluster* cluster)
     std_cell_area += computeArea(std_cell);
   }
 
-  int64_t macro_area = 0.0f;
+  int64_t macro_area = 0;
   for (odb::dbInst* macro : cluster->getLeafMacros()) {
     macro_area += computeArea(macro);
   }
@@ -2525,13 +2525,13 @@ void ClusteringEngine::printPhysicalHierarchyTree(Cluster* parent, int level)
     line += fmt::format(" {}", parent->getIsLeafString());
 
     // Using 'or' on purpose to certify that there is no discrepancy going on.
-    if (parent->getNumStdCell() != 0 || parent->getStdCellArea() != 0.0f) {
+    if (parent->getNumStdCell() != 0 || parent->getStdCellArea() != 0) {
       line += fmt::format(", StdCells: {} ({} μ²)",
                           parent->getNumStdCell(),
                           parent->getStdCellArea());
     }
 
-    if (parent->getNumMacro() != 0 || parent->getMacroArea() != 0.0f) {
+    if (parent->getNumMacro() != 0 || parent->getMacroArea() != 0) {
       line += fmt::format(", Macros: {} ({} μ²),",
                           parent->getNumMacro(),
                           parent->getMacroArea());

src/mpl/src/hier_rtlmp.cpp

@@ -912,7 +912,7 @@ void HierRTLMP::createPinAccessBlockages()
       = tree_->maps.module_to_metrics.at(block_->getTopModule()).get();
 
   // Avoid creating blockages with zero depth.
-  if (top_module_metrics->getStdCellArea() == 0.0) {
+  if (top_module_metrics->getStdCellArea() == 0) {
     return;
   }
 
@@ -1805,8 +1805,8 @@ void HierRTLMP::createFixedTerminal(Cluster* cluster,
   // A conventional fixed terminal is just a point without
   // the cluster data.
   odb::Point location = cluster->getCenter();
-  float width = 0.0f;
-  float height = 0.0f;
+  int width = 0;
+  int height = 0;
   Cluster* terminal_cluster = nullptr;
 
   if (cluster->isClusterOfUnplacedIOPins()) {