Skip to content

Commit 536254f

Browse files
committed
mpl: improvements to placeChildren
Signed-off-by: João Mai <[email protected]>
1 parent 5623e3c commit 536254f

File tree

1 file changed

+1
-24
lines changed

1 file changed

+1
-24
lines changed

src/mpl/src/hier_rtlmp.cpp

Lines changed: 1 addition & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1312,10 +1312,6 @@ void HierRTLMP::adjustMacroBlockageWeight()
13121312
}
13131313
}
13141314

1315-
// This function is used in cases with very high density, in which it may
1316-
// be very hard to generate a valid tiling for the clusters.
1317-
// Here, we may want to try setting the area of all standard-cell clusters to 0.
1318-
// This should be only be used in mixed clusters.
13191315
void HierRTLMP::placeChildren(Cluster* parent, bool minimum_target_util)
13201316
{
13211317
if (!minimum_target_util) {
@@ -1533,7 +1529,7 @@ void HierRTLMP::placeChildren(Cluster* parent, bool minimum_target_util)
15331529
}
15341530
}
15351531
} else {
1536-
target_utils.push_back(1.6);
1532+
target_utils.push_back(1e6);
15371533
target_dead_spaces.push_back(0.99999);
15381534
}
15391535

@@ -1683,25 +1679,6 @@ void HierRTLMP::placeChildren(Cluster* parent, bool minimum_target_util)
16831679
if (!minimum_target_util) {
16841680
placeChildren(parent, true);
16851681
} else {
1686-
debugPrint(logger_,
1687-
MPL,
1688-
"hierarchical_macro_placement",
1689-
1,
1690-
"SA Summary for cluster {}",
1691-
parent->getName());
1692-
1693-
for (auto i = 0; i < sa_containers.size(); i++) {
1694-
debugPrint(logger_,
1695-
MPL,
1696-
"hierarchical_macro_placement",
1697-
1,
1698-
"sa_id: {}, target_util: {}, target_dead_space: {}",
1699-
i,
1700-
target_util_list[i],
1701-
target_dead_space_list[i]);
1702-
1703-
sa_containers[i]->printResults();
1704-
}
17051682
logger_->error(MPL, 40, "Failed on cluster {}", parent->getName());
17061683
}
17071684
} else {

0 commit comments

Comments
 (0)