Merge pull request #2147 from The-OpenROAD-Project-staging/secure-tns-percent

improve QoR by more aggressive timing repair

Author: maliberty

flow/Makefile

@@ -104,8 +104,9 @@
 # Default design
 DESIGN_CONFIG ?= ./designs/nangate45/gcd/config.mk
 
-# Default TNS_END_PERCENT value
-export TNS_END_PERCENT ?=5
+# Default TNS_END_PERCENT value for post CTS timing repair
+# Try fixing all violating endpoints by default (reduce to 5% for runtime)
+export TNS_END_PERCENT ?=100
 
 # If we are running headless use offscreen rendering for save_image
 ifndef DISPLAY

flow/designs/gf12/bp_single/config.mk

@@ -3,6 +3,7 @@ export DESIGN_NAME = bsg_chip
 export PLATFORM    = gf12
 
 export SKIP_GATE_CLONING = 1
+export TNS_END_PERCENT = 5
 
 # export VERILOG_FILES = $(PLATFORM_DIR)/bp/bsg_ac_black_parrot_single_core_v0/bsg_chip.sv2v.v
 export VERILOG_FILES =  $(PLATFORM_DIR)/bp/bsg_ac_black_parrot_single_core_v0/yosys/synth_don_abc_script.v \

flow/designs/gf12/bp_single/config_mpl2.mk

@@ -2,6 +2,9 @@ export DESIGN_NICKNAME = bp_single
 export DESIGN_NAME = bsg_chip
 export PLATFORM    = gf12
 
+export SKIP_GATE_CLONING = 1
+export TNS_END_PERCENT = 5
+
 export SYNTH_HIERARCHICAL = 1
 export RTLMP_FLOW = True
 #

flow/designs/gf12/ca53/config.mk

@@ -65,5 +65,6 @@ else
 export DESIGN_TYPE = CELL_NODEN
 endif
 
-export SKIP_PIN_SWAP          = 1
+#export SKIP_PIN_SWAP          = 1
 export SKIP_INCREMENTAL_REPAIR = 1
+export TNS_END_PERCENT = 5

flow/designs/gf12/jpeg/config.mk

@@ -20,5 +20,6 @@ else
 export DESIGN_TYPE = CELL_NODEN
 endif
 
-export SKIP_PIN_SWAP     = 1
+#export SKIP_PIN_SWAP     = 1
 export SKIP_GATE_CLONING = 1
+

flow/designs/intel16/aes/config.mk

@@ -15,4 +15,5 @@ export CORE_AREA   = 1.26 1.89 248 248
 
 
 export PLACE_DENSITY = uniform
-export SKIP_PIN_SWAP = 1
+#export SKIP_PIN_SWAP = 1
+export SKIP_GATE_CLONING = 1

flow/designs/intel16/aes/metadata-base-ok.json

@@ -325,4 +325,4 @@
   "synth__mem__peak": 134532.0,
   "synth__runtime__total": "0:48.61",
   "total_time": "0:08:54.570000"
-}
+}

flow/designs/intel16/aes/rules-base.json

@@ -28,7 +28,7 @@
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
-        "value": 283519,
+        "value": 265022,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
@@ -59,4 +59,4 @@
         "value": -10.0,
         "compare": ">="
     }
-}
+}

flow/scripts/global_route.tcl

@@ -70,7 +70,14 @@ if { ![info exists ::env(SKIP_INCREMENTAL_REPAIR)] } {
   # Repair timing using global route parasitics
   puts "Repair setup and hold violations..."
   estimate_parasitics -global_routing
-  repair_timing -verbose
+
+  # process user settings
+  set additional_args "-verbose"
+  append_env_var additional_args SKIP_PIN_SWAP -skip_pin_swap 0
+  append_env_var additional_args SKIP_GATE_CLONING -skip_gate_cloning 0
+  puts "repair_timing [join $additional_args " "]"
+  repair_timing {*}$additional_args
+
   if {[info exist ::env(DETAILED_METRICS)]} {
     report_metrics 5 "global route post repair timing"
   }