Skip to content

Commit 9e42db5

Browse files
committed
reports: report_metrics now reports into the reports folder
much less spam in the normal log output from report_metrics and easier to navigate instead of scrolling in endless logs. There are a few things that can't be redirected to a file, a future improvement. Signed-off-by: Øyvind Harboe <[email protected]>
1 parent d723079 commit 9e42db5

File tree

10 files changed

+147
-132
lines changed

10 files changed

+147
-132
lines changed

flow/scripts/cts.tcl

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,14 +51,14 @@ utl::push_metrics_stage "cts__{}__pre_repair"
5151
source $::env(SCRIPTS_DIR)/report_metrics.tcl
5252

5353
estimate_parasitics -placement
54-
report_metrics "cts pre-repair"
54+
report_metrics 4 "cts pre-repair"
5555
utl::pop_metrics_stage
5656

5757
repair_clock_nets
5858

5959
utl::push_metrics_stage "cts__{}__post_repair"
6060
estimate_parasitics -placement
61-
report_metrics "cts post-repair"
61+
report_metrics 4 "cts post-repair"
6262
utl::pop_metrics_stage
6363

6464
set_placement_padding -global \
@@ -118,7 +118,7 @@ if {$result != 0} {
118118

119119
check_placement -verbose
120120

121-
report_metrics "cts final"
121+
report_metrics 4 "cts final"
122122

123123
if { [info exists ::env(POST_CTS_TCL)] } {
124124
source $::env(POST_CTS_TCL)

flow/scripts/detail_place.tcl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ utl::info FLW 12 "Placement violations [check_placement -verbose]."
2727
estimate_parasitics -placement
2828

2929
source $::env(SCRIPTS_DIR)/report_metrics.tcl
30-
report_metrics "detailed place" true false
30+
report_metrics 3 "detailed place" true false
3131

3232
if {![info exists save_checkpoint] || $save_checkpoint} {
3333
write_db $::env(RESULTS_DIR)/3_5_place_dp.odb

flow/scripts/final_report.tcl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ if {[info exist ::env(RCX_RULES)]} {
6161
}
6262

6363
source $::env(SCRIPTS_DIR)/report_metrics.tcl
64-
report_metrics "finish"
64+
report_metrics 6 "finish"
6565

6666
# Save a final image if openroad is compiled with the gui
6767
if {[expr [llength [info procs save_image]] > 0]} {

flow/scripts/floorplan.tcl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ puts "Default units for flow"
111111
report_units
112112
report_units_metric
113113
source $::env(SCRIPTS_DIR)/report_metrics.tcl
114-
report_metrics "floorplan final" false false
114+
report_metrics 2 "floorplan final" false false
115115

116116
if { [info exist ::env(RESYNTH_AREA_RECOVER)] && $::env(RESYNTH_AREA_RECOVER) == 1 } {
117117

flow/scripts/global_place.tcl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ if {[info exist ::env(CLUSTER_FLOPS)]} {
6565
}
6666

6767
source $::env(SCRIPTS_DIR)/report_metrics.tcl
68-
report_metrics "global place" false false
68+
report_metrics 5 "global place" false false
6969

7070
if {![info exists save_checkpoint] || $save_checkpoint} {
7171
write_db $::env(RESULTS_DIR)/3_3_place_gp.odb

flow/scripts/global_route.tcl

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -48,12 +48,12 @@ if {[info exist env(DONT_USE_CELLS)]} {
4848
source $env(SCRIPTS_DIR)/report_metrics.tcl
4949

5050
if { ![info exists ::env(SKIP_INCREMENTAL_REPAIR)] } {
51-
report_metrics "global route pre repair design"
51+
report_metrics 5 "global route pre repair design"
5252

5353
# Repair design using global route parasitics
5454
puts "Perform buffer insertion..."
5555
repair_design
56-
report_metrics "global route post repair design"
56+
report_metrics 5 "global route post repair design"
5757

5858
# Running DPL to fix overlapped instances
5959
# Run to get modified net by DPL
@@ -66,7 +66,7 @@ if { ![info exists ::env(SKIP_INCREMENTAL_REPAIR)] } {
6666
puts "Repair setup and hold violations..."
6767
estimate_parasitics -global_routing
6868
repair_timing
69-
report_metrics "global route post repair timing"
69+
report_metrics 5 "global route post repair timing"
7070

7171
# Running DPL to fix overlapped instances
7272
# Run to get modified net by DPL
@@ -97,7 +97,7 @@ check_placement -verbose
9797
check_antennas -report_file $env(REPORTS_DIR)/antenna.log -report_violating_nets
9898

9999
estimate_parasitics -global_routing
100-
report_metrics "global route"
100+
report_metrics 5 "global route"
101101

102102
# Write SDC to results with updated clock periods that are just failing.
103103
# Use make target update_sdc_clock to install the updated sdc.

flow/scripts/report_metrics.tcl

Lines changed: 132 additions & 117 deletions
Large diffs are not rendered by default.

flow/scripts/resize.tcl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ estimate_parasitics -placement
1313

1414
utl::push_metrics_stage "placeopt__{}__pre_opt"
1515
source $::env(SCRIPTS_DIR)/report_metrics.tcl
16-
report_metrics "resizer pre" false false
16+
report_metrics 3 "resizer pre" false false
1717
utl::pop_metrics_stage
1818

1919
print_banner "instance_count"
@@ -78,7 +78,7 @@ print_banner "report_floating_nets"
7878
report_floating_nets
7979

8080
source $::env(SCRIPTS_DIR)/report_metrics.tcl
81-
report_metrics "resizer" true false
81+
report_metrics 3 "resizer" true false
8282

8383
print_banner "instance_count"
8484
puts [sta::network_leaf_instance_count]

flow/scripts/synth_metrics.tcl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,4 @@ source $::env(SCRIPTS_DIR)/load.tcl
33
load_design 1_1_yosys.v 1_synth.sdc "Post synthesis"
44

55
source $::env(SCRIPTS_DIR)/report_metrics.tcl
6-
report_metrics "Post synthesis" false false
6+
report_metrics 1 "Post synthesis" false false

flow/util/genMetrics.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,7 @@ def extract_metrics(cwd, platform, design, flow_variant, output, hier_json):
295295
metrics_dict,
296296
baseRegEx.format('finish slack div critical path delay',
297297
'(\S+)'),
298-
logPath + '/6_report.log')
298+
rptPath + '/6_finish.rpt')
299299

300300
extractGnuTime('finish', metrics_dict, logPath + '/6_report.log')
301301

0 commit comments

Comments
 (0)