Skip to content

Commit eca5efe

Browse files
oharboeoharboe
committed
checkpoint: cleanup, remove vestiges of checkpoint
this variable is never set anywhere, so this conditional code is a vestige of something or other. Signed-off-by: Øyvind Harboe <[email protected]>
1 parent fb053e3 commit eca5efe

16 files changed

+29
-68
lines changed

‎flow/scripts/cts.tcl‎

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -125,10 +125,8 @@ if { [info exists ::env(POST_CTS_TCL)] } {
125125
source $::env(POST_CTS_TCL)
126126
}
127127

128-
if {![info exists save_checkpoint] || $save_checkpoint} {
129-
if {[info exists ::env(GALLERY_REPORT)] && $::env(GALLERY_REPORT) != 0} {
130-
write_def $::env(RESULTS_DIR)/4_1_cts.def
131-
}
132-
write_db $::env(RESULTS_DIR)/4_1_cts.odb
133-
write_sdc $::env(RESULTS_DIR)/4_cts.sdc
128+
if {[info exists ::env(GALLERY_REPORT)] && $::env(GALLERY_REPORT) != 0} {
129+
write_def $::env(RESULTS_DIR)/4_1_cts.def
134130
}
131+
write_db $::env(RESULTS_DIR)/4_1_cts.odb
132+
write_sdc $::env(RESULTS_DIR)/4_cts.sdc

‎flow/scripts/detail_place.tcl‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,4 @@ estimate_parasitics -placement
3434
source $::env(SCRIPTS_DIR)/report_metrics.tcl
3535
report_metrics 3 "detailed place" true false
3636

37-
if {![info exists save_checkpoint] || $save_checkpoint} {
38-
write_db $::env(RESULTS_DIR)/3_5_place_dp.odb
39-
}
37+
write_db $::env(RESULTS_DIR)/3_5_place_dp.odb

‎flow/scripts/detail_route.tcl‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,4 @@ if { [info exists ::env(POST_DETAIL_ROUTE_TCL)] } {
7272
source $::env(POST_DETAIL_ROUTE_TCL)
7373
}
7474

75-
if {![info exists save_checkpoint] || $save_checkpoint} {
76-
write_db $::env(RESULTS_DIR)/5_3_route.odb
77-
}
75+
write_db $::env(RESULTS_DIR)/5_3_route.odb

‎flow/scripts/fillcell.tcl‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,4 @@ set_propagated_clock [all_clocks]
77
filler_placement $::env(FILL_CELLS)
88
check_placement
99

10-
if {![info exists save_checkpoint] || $save_checkpoint} {
11-
write_db $::env(RESULTS_DIR)/5_2_fillcell.odb
12-
}
10+
write_db $::env(RESULTS_DIR)/5_2_fillcell.odb

‎flow/scripts/floorplan.tcl‎

Lines changed: 8 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -81,16 +81,12 @@ if { [info exist ::env(RESYNTH_TIMING_RECOVER)] && $::env(RESYNTH_TIMING_RECOVER
8181
puts "Post synth-opt tns"
8282
report_tns -digits 3
8383

84-
if {![info exists save_checkpoint] || $save_checkpoint} {
85-
write_verilog $::env(RESULTS_DIR)/2_pre_abc_timing.v
86-
}
84+
write_verilog $::env(RESULTS_DIR)/2_pre_abc_timing.v
8785

8886
restructure -target timing -liberty_file $::env(DONT_USE_SC_LIB) \
8987
-work_dir $::env(RESULTS_DIR)
9088

91-
if {![info exists save_checkpoint] || $save_checkpoint} {
92-
write_verilog $::env(RESULTS_DIR)/2_post_abc_timing.v
93-
}
89+
write_verilog $::env(RESULTS_DIR)/2_post_abc_timing.v
9490

9591
# post restructure area/timing report (ideal clocks)
9692
remove_buffers
@@ -123,9 +119,7 @@ if { [info exist ::env(RESYNTH_AREA_RECOVER)] && $::env(RESYNTH_AREA_RECOVER) ==
123119
report_design_area_metrics
124120
utl::pop_metrics_stage
125121

126-
if {![info exists save_checkpoint] || $save_checkpoint} {
127-
write_verilog $::env(RESULTS_DIR)/2_pre_abc.v
128-
}
122+
write_verilog $::env(RESULTS_DIR)/2_pre_abc.v
129123

130124
set tielo_cell_name [lindex $env(TIELO_CELL_AND_PORT) 0]
131125
set tielo_lib_name [get_name [get_property [lindex [get_lib_cell $tielo_cell_name] 0] library]]
@@ -143,9 +137,7 @@ if { [info exist ::env(RESYNTH_AREA_RECOVER)] && $::env(RESYNTH_AREA_RECOVER) ==
143137
# remove buffers inserted by abc
144138
remove_buffers
145139

146-
if {![info exists save_checkpoint] || $save_checkpoint} {
147-
write_verilog $::env(RESULTS_DIR)/2_post_abc.v
148-
}
140+
write_verilog $::env(RESULTS_DIR)/2_post_abc.v
149141
utl::push_metrics_stage "floorplan__{}__post_restruct"
150142
set num_instances [llength [get_cells -hier *]]
151143
puts "number instances after restructure is $num_instances"
@@ -159,10 +151,8 @@ if { [info exists ::env(POST_FLOORPLAN_TCL)] } {
159151
source $::env(POST_FLOORPLAN_TCL)
160152
}
161153

162-
if {![info exists save_checkpoint] || $save_checkpoint} {
163-
if {[info exists ::env(GALLERY_REPORT)] && $::env(GALLERY_REPORT) != 0} {
164-
write_def $::env(RESULTS_DIR)/2_1_floorplan.def
165-
}
166-
write_db $::env(RESULTS_DIR)/2_1_floorplan.odb
167-
write_sdc $::env(RESULTS_DIR)/2_floorplan.sdc
154+
if {[info exists ::env(GALLERY_REPORT)] && $::env(GALLERY_REPORT) != 0} {
155+
write_def $::env(RESULTS_DIR)/2_1_floorplan.def
168156
}
157+
write_db $::env(RESULTS_DIR)/2_1_floorplan.odb
158+
write_sdc $::env(RESULTS_DIR)/2_floorplan.sdc

‎flow/scripts/floorplan_debug_macros.tcl‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,4 @@ load_design 2_1_floorplan.odb 1_synth.sdc
33

44
source $::env(SCRIPTS_DIR)/macro_place_util.tcl
55

6-
if {![info exists save_checkpoint] || $save_checkpoint} {
7-
write_db $::env(RESULTS_DIR)/2_floorplan_debug_macros.odb
8-
}
6+
write_db $::env(RESULTS_DIR)/2_floorplan_debug_macros.odb

‎flow/scripts/global_place.tcl‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,4 @@ if {[info exist ::env(CLUSTER_FLOPS)]} {
6767
source $::env(SCRIPTS_DIR)/report_metrics.tcl
6868
report_metrics 5 "global place" false false
6969

70-
if {![info exists save_checkpoint] || $save_checkpoint} {
71-
write_db $::env(RESULTS_DIR)/3_3_place_gp.odb
72-
}
70+
write_db $::env(RESULTS_DIR)/3_3_place_gp.odb

‎flow/scripts/global_place_skip_io.tcl‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,4 @@ if {[info exists ::env(FLOORPLAN_DEF)] || ([info exists ::env(HAS_IO_CONSTRAINTS
2929
}
3030
}
3131

32-
if {![info exists save_checkpoint] || $save_checkpoint} {
33-
write_db $::env(RESULTS_DIR)/3_1_place_gp_skip_io.odb
34-
}
32+
write_db $::env(RESULTS_DIR)/3_1_place_gp_skip_io.odb

‎flow/scripts/global_route.tcl‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -106,6 +106,5 @@ report_metrics 5 "global route"
106106
# Write SDC to results with updated clock periods that are just failing.
107107
# Use make target update_sdc_clock to install the updated sdc.
108108
source [file join $env(SCRIPTS_DIR) "write_ref_sdc.tcl"]
109-
if {![info exists save_checkpoint] || $save_checkpoint} {
110-
write_db $env(RESULTS_DIR)/5_1_grt.odb
111-
}
109+
110+
write_db $env(RESULTS_DIR)/5_1_grt.odb

‎flow/scripts/io_placement.tcl‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,4 @@ if {[info exists ::env(FLOORPLAN_DEF)]} {
1212
{*}$::env(PLACE_PINS_ARGS)
1313
}
1414

15-
if {![info exists save_checkpoint] || $save_checkpoint} {
16-
write_db $::env(RESULTS_DIR)/3_2_place_iop.odb
17-
}
15+
write_db $::env(RESULTS_DIR)/3_2_place_iop.odb

0 commit comments

Comments
 (0)