Skip to content

Commit 546d2fb

Browse files
committed
gpl: do not print area change if routability or timing driven modes are off
Signed-off-by: Augusto Berndt <[email protected]>
1 parent 5d34609 commit 546d2fb

File tree

2 files changed

+32
-25
lines changed

2 files changed

+32
-25
lines changed

src/gpl/src/nesterovBase.cpp

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2844,17 +2844,19 @@ bool NesterovBase::checkConvergence(int gpl_iter_count,
28442844

28452845
if (is_power_domain) {
28462846
log_->info(GPL,
2847-
1002,
2847+
1016,
28482848
"Power domain '{}' placement finished at iteration {}",
28492849
group_name,
28502850
final_iter);
28512851
} else {
28522852
log_->info(
28532853
GPL, 1001, "Global placement finished at iteration {}", final_iter);
2854-
log_->info(GPL,
2855-
1003,
2856-
"Routability mode iteration count: {}",
2857-
routability_gpl_iter_count);
2854+
if (npVars_->routability_driven_mode) {
2855+
log_->info(GPL,
2856+
1003,
2857+
"Routability mode iteration count: {}",
2858+
routability_gpl_iter_count);
2859+
}
28582860
}
28592861

28602862
if (npVars_->routability_driven_mode) {

src/gpl/src/nesterovPlace.cpp

Lines changed: 25 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -880,31 +880,36 @@ void NesterovPlace::reportResults(const int nesterov_iter,
880880
npVars_.maxNesterovIter);
881881
}
882882

883-
log_->info(GPL,
884-
1011,
885-
"Original area (um^2): {:.2f}",
886-
block->dbuAreaToMicrons(original_area));
883+
if (npVars_.routability_driven_mode || npVars_.timingDrivenMode) {
884+
log_->info(GPL,
885+
1011,
886+
"Original area (um^2): {:.2f}",
887+
block->dbuAreaToMicrons(original_area));
888+
}
889+
890+
if (npVars_.routability_driven_mode) {
891+
const float routability_diff
892+
= 100.0 * (end_routability_area - original_area) / original_area;
893+
log_->info(GPL,
894+
1012,
895+
"Total routability artificial inflation: {:.2f} ({:+.2f}%)",
896+
block->dbuAreaToMicrons(end_routability_area - original_area),
897+
routability_diff);
898+
}
899+
900+
if (npVars_.timingDrivenMode) {
901+
const float td_diff = 100.0 * td_accumulated_delta_area / original_area;
902+
log_->info(GPL,
903+
1013,
904+
"Total timing-driven delta area: {:.2f} ({:+.2f}%)",
905+
block->dbuAreaToMicrons(td_accumulated_delta_area),
906+
td_diff);
907+
}
887908

888909
int64_t new_area = 0;
889910
for (auto& nb : nbVec_) {
890911
new_area += nb->nesterovInstsArea();
891912
}
892-
893-
const float routability_diff
894-
= 100.0 * (end_routability_area - original_area) / original_area;
895-
log_->info(GPL,
896-
1012,
897-
"Total routability artificial inflation: {:.2f} ({:+.2f}%)",
898-
block->dbuAreaToMicrons(end_routability_area - original_area),
899-
routability_diff);
900-
901-
const float td_diff = 100.0 * td_accumulated_delta_area / original_area;
902-
log_->info(GPL,
903-
1013,
904-
"Total timing-driven delta area: {:.2f} ({:+.2f}%)",
905-
block->dbuAreaToMicrons(td_accumulated_delta_area),
906-
td_diff);
907-
908913
const float placement_diff
909914
= 100.0 * (new_area - original_area) / original_area;
910915
log_->info(GPL,

0 commit comments

Comments
 (0)