Skip to content

Commit 58a54ec

Browse files
committed
scripts: update SKIP_DETAILED_ROUTE logic and typo
Use correct name SKIP_DETAILED_ROUTE instead of SKIP_DRT. Since SKIP_DETAILED_ROUTE is defined at variables.yaml it will always exist in the route stage, so the check env_var_exists_and_non_empty will always be true. The check should be done with env_var_equals. Since the variable is also used in the final stage to decide if OpenRCX should run we need "final" included in the variables.yaml file. Signed-off-by: Vitor Bandeira <[email protected]>
1 parent 00722fb commit 58a54ec

File tree

3 files changed

+3
-2
lines changed

3 files changed

+3
-2
lines changed

flow/scripts/detail_route.tcl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ if { ![grt::have_routes] } {
66
in DRC viewer to view congestion"
77
}
88

9-
if { [env_var_exists_and_non_empty SKIP_DRT] } {
9+
if { [env_var_equals SKIP_DETAILED_ROUTE 1] } {
1010
write_db $::env(RESULTS_DIR)/5_2_route.odb
1111
exit
1212
}

flow/scripts/final_report.tcl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ write_verilog $::env(RESULTS_DIR)/6_final.v
2020
# Run extraction and STA
2121
if {
2222
[env_var_exists_and_non_empty RCX_RULES]
23-
&& ![env_var_exists_and_non_empty SKIP_DRT]
23+
&& [env_var_equals SKIP_DETAILED_ROUTE 0]
2424
} {
2525
# RCX section
2626
define_process_corner -ext_model_index 0 X

flow/scripts/variables.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1004,3 +1004,4 @@ SKIP_DETAILED_ROUTE:
10041004
Skips detailed route.
10051005
stages:
10061006
- route
1007+
- final

0 commit comments

Comments
 (0)