scripts: Add REPAIR_TIMING_MAX_PASSES_PER_ITER parameters

Signed-off-by: Eryk Szpotanski <[email protected]>
antmicro · Nov 25, 2024 · f69b905 · f69b905
1 parent 3cf48df
commit f69b905
Show file tree

Hide file tree

Showing 5 changed files with 32 additions and 6 deletions.
diff --git a/flow/scripts/cts.tcl b/flow/scripts/cts.tcl
@@ -69,7 +69,7 @@ if {![env_var_equals SKIP_CTS_REPAIR_TIMING 1]} {
       write_eqy_verilog 4_before_rsz.v
   }
 
-  repair_timing_helper
+  repair_timing_helper "cts"
 
   if {$::env(EQUIVALENCE_CHECK)} {
       run_equivalence_test

diff --git a/flow/scripts/floorplan.tcl b/flow/scripts/floorplan.tcl
@@ -108,13 +108,13 @@ if { [env_var_equals REMOVE_ABC_BUFFERS 1] } {
   # remove buffers inserted by yosys/abc
   remove_buffers
 } else {
-  repair_timing_helper 0
+  repair_timing_helper "floorplan" 0
 }
 
 ##### Restructure for timing #########
 if { [env_var_equals RESYNTH_TIMING_RECOVER 1] } {
   repair_design_helper
-  repair_timing_helper
+  repair_timing_helper "floorplan"
   # pre restructure area/timing report (ideal clocks)
   puts "Post synth-opt area"
   report_design_area
@@ -134,7 +134,7 @@ if { [env_var_equals RESYNTH_TIMING_RECOVER 1] } {
   # post restructure area/timing report (ideal clocks)
   remove_buffers
   repair_design_helper
-  repair_timing_helper
+  repair_timing_helper "floorplan"
 
   puts "Post restructure-opt wns"
   report_worst_slack -max -digits 3

diff --git a/flow/scripts/global_route.tcl b/flow/scripts/global_route.tcl
@@ -67,7 +67,7 @@ proc global_route_helper {} {
     puts "Repair setup and hold violations..."
     estimate_parasitics -global_routing
 
-    repair_timing_helper
+    repair_timing_helper "grt"
 
     if { $::env(DETAILED_METRICS) } {
       report_metrics 5 "global route post repair timing"

diff --git a/flow/scripts/util.tcl b/flow/scripts/util.tcl
@@ -15,7 +15,7 @@ proc fast_route {} {
   }
 }
 
-proc repair_timing_helper { {hold_margin 1} } {
+proc repair_timing_helper { stage {hold_margin 1} } {
   set additional_args "-verbose"
   append_env_var additional_args SETUP_SLACK_MARGIN -setup_margin 1
   if {$hold_margin || $::env(HOLD_SLACK_MARGIN) < 0} {
@@ -27,6 +27,14 @@ proc repair_timing_helper { {hold_margin 1} } {
   append_env_var additional_args SKIP_BUFFER_REMOVAL -skip_buffer_removal 0
   append_env_var additional_args SKIP_LAST_GASP -skip_last_gasp 0
   append_env_var additional_args MATCH_CELL_FOOTPRINT -match_cell_footprint 0
+  if { $stage == "floorplan"} {
+    set max_passes_prefix "FP_"
+  } elseif { $stage == "cts" } {
+    set max_passes_prefix "CTS_"
+  } elseif { $stage == "grt"} {
+    set max_passes_prefix "GRT_"
+  }
+  append_env_var additional_args "${max_passes_prefix}REPAIR_TIMING_MAX_PASSES_PER_ITER" -max_passes_per_iter 1
   puts "repair_timing [join $additional_args " "]"
   repair_timing {*}$additional_args
 }

diff --git a/flow/scripts/variables.yaml b/flow/scripts/variables.yaml
@@ -266,6 +266,12 @@ IO_PLACER_V:
   stages:
     - floorplan
     - place
+FP_REPAIR_TIMING_MAX_PASSES_PER_ITER:
+  description: >
+    Maximum number of repairs per iteration during repair_timing, called from floorplan.
+  default: 20
+  stages:
+    - floorplan
 GUI_TIMING:
   description: >
     Load timing information when opening GUI. For large designs, this can
@@ -609,6 +615,12 @@ CTS_CLUSTER_SIZE:
   default: 50
   stages:
     - cts
+CTS_REPAIR_TIMING_MAX_PASSES_PER_ITER:
+  description: >
+    Maximum number of repairs per iteration during repair_timing, called from CTS.
+  default: 20
+  stages:
+    - cts
 CTS_SNAPSHOT:
   description: >
     Creates ODB/SDC files prior to clock net and setup/hold repair.
@@ -640,6 +652,12 @@ GLOBAL_ROUTE_ARGS:
   stages:
     - grt
   default: -congestion_iterations 30 -congestion_report_iter_step 5 -verbose
+GRT_REPAIR_TIMING_MAX_PASSES_PER_ITER:
+  description: >
+    Maximum number of repairs per iteration during repair_timing, called from GRT.
+  default: 20
+  stages:
+    - grt
 MATCH_CELL_FOOTPRINT:
   description: >
     Enforce sizing operations to only swap cells that have the same layout boundary.