Merge pull request The-OpenROAD-Project#3462 from The-OpenROAD-Project-staging/secure-est_via_res

eder-matheus · web-flow · commit faa7fe5f7e7c · 2025-09-02T05:12:10.000-03:00
update metrics for est via resistance update
diff --git a/docs/user/FlowVariables.md b/docs/user/FlowVariables.md
@@ -307,11 +307,14 @@ configuration file.
 - [MACRO_WRAPPERS](#MACRO_WRAPPERS)
 - [MAKE_TRACKS](#MAKE_TRACKS)
 - [MATCH_CELL_FOOTPRINT](#MATCH_CELL_FOOTPRINT)
+- [MAX_ROUTING_LAYER](#MAX_ROUTING_LAYER)
+- [MIN_ROUTING_LAYER](#MIN_ROUTING_LAYER)
 - [PDN_TCL](#PDN_TCL)
 - [PLACE_DENSITY](#PLACE_DENSITY)
 - [PLACE_DENSITY_LB_ADDON](#PLACE_DENSITY_LB_ADDON)
 - [PLACE_SITE](#PLACE_SITE)
 - [REMOVE_ABC_BUFFERS](#REMOVE_ABC_BUFFERS)
+- [ROUTING_LAYER_ADJUSTMENT](#ROUTING_LAYER_ADJUSTMENT)
 - [RTLMP_AREA_WT](#RTLMP_AREA_WT)
 - [RTLMP_ARGS](#RTLMP_ARGS)
 - [RTLMP_BOUNDARY_WT](#RTLMP_BOUNDARY_WT)
diff --git a/flow/designs/asap7/ethmac/rules-base.json b/flow/designs/asap7/ethmac/rules-base.json
@@ -32,7 +32,7 @@
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
-        "value": 210959,
+        "value": 210949,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
@@ -48,7 +48,7 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -139.61,
+        "value": -156.06,
         "compare": ">="
     },
     "finish__design__instance__area": {
diff --git a/flow/designs/gf180/aes-hybrid/rules-base.json b/flow/designs/gf180/aes-hybrid/rules-base.json
@@ -28,11 +28,11 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 3,
+        "value": 9,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
-        "value": 1655593,
+        "value": 1650663,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
@@ -48,7 +48,7 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -1.22,
+        "value": -1.16,
         "compare": ">="
     },
     "finish__design__instance__area": {
@@ -64,7 +64,7 @@
         "compare": "<="
     },
     "finish__timing__wns_percent_delay": {
-        "value": -36.19,
+        "value": -36.02,
         "compare": ">="
     }
 }
diff --git a/flow/designs/ihp-sg13g2/jpeg/rules-base.json b/flow/designs/ihp-sg13g2/jpeg/rules-base.json
@@ -44,7 +44,7 @@
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {
-        "value": 134,
+        "value": 222,
         "compare": "<="
     },
     "finish__timing__setup__ws": {
diff --git a/flow/designs/nangate45/swerv/rules-base.json b/flow/designs/nangate45/swerv/rules-base.json
@@ -52,15 +52,15 @@
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 186100,
+        "value": 185087,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {
         "value": 4319,
         "compare": "<="
     },
     "finish__timing__drv__hold_violation_count": {
-        "value": 100,
+        "value": 279,
         "compare": "<="
     },
     "finish__timing__wns_percent_delay": {
diff --git a/flow/designs/rapidus2hp/hercules_idecode/rules-base.json b/flow/designs/rapidus2hp/hercules_idecode/rules-base.json
@@ -8,7 +8,7 @@
         "compare": "=="
     },
     "placeopt__design__instance__area": {
-        "value": 13567,
+        "value": 13530,
         "compare": "<="
     },
     "placeopt__design__instance__count__stdcell": {
@@ -32,11 +32,11 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -473.42,
+        "value": -504.31,
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 13843,
+        "value": 13812,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {
@@ -48,7 +48,7 @@
         "compare": "<="
     },
     "finish__timing__wns_percent_delay": {
-        "value": -76.58,
+        "value": -75.04,
         "compare": ">="
     }
 }
diff --git a/flow/designs/rapidus2hp/jpeg/rules-base.json b/flow/designs/rapidus2hp/jpeg/rules-base.json
@@ -8,7 +8,7 @@
         "compare": "=="
     },
     "placeopt__design__instance__area": {
-        "value": 3820,
+        "value": 3429,
         "compare": "<="
     },
     "placeopt__design__instance__count__stdcell": {
@@ -32,11 +32,11 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -166.18,
+        "value": -195.43,
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 3603,
+        "value": 3547,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {
diff --git a/flow/designs/sky130hd/chameleon/rules-base.json b/flow/designs/sky130hd/chameleon/rules-base.json
@@ -28,11 +28,11 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 219,
+        "value": 428,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
-        "value": 798366,
+        "value": 795476,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
diff --git a/flow/designs/sky130hd/microwatt/rules-base.json b/flow/designs/sky130hd/microwatt/rules-base.json
@@ -28,7 +28,7 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 3708,
+        "value": 3096,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
@@ -60,7 +60,7 @@
         "compare": "<="
     },
     "finish__timing__drv__hold_violation_count": {
-        "value": 106,
+        "value": 289,
         "compare": "<="
     },
     "finish__timing__wns_percent_delay": {
diff --git a/flow/designs/sky130hd/riscv32i/rules-base.json b/flow/designs/sky130hd/riscv32i/rules-base.json
@@ -8,7 +8,7 @@
         "compare": "=="
     },
     "placeopt__design__instance__area": {
-        "value": 77111,
+        "value": 77028,
         "compare": "<="
     },
     "placeopt__design__instance__count__stdcell": {
@@ -28,7 +28,7 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 2,
+        "value": 0,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
@@ -44,7 +44,7 @@
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {
-        "value": 6,
+        "value": 12,
         "compare": "<="
     },
     "finish__timing__setup__ws": {
diff --git a/flow/designs/sky130hs/aes/rules-base.json b/flow/designs/sky130hs/aes/rules-base.json
@@ -28,7 +28,7 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 26,
+        "value": 20,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
@@ -44,11 +44,11 @@
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {
-        "value": 18,
+        "value": 15,
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -0.18,
+        "value": -0.45,
         "compare": ">="
     },
     "finish__design__instance__area": {
diff --git a/flow/designs/sky130hs/riscv32i/rules-base.json b/flow/designs/sky130hs/riscv32i/rules-base.json
@@ -44,15 +44,15 @@
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {
-        "value": 16,
+        "value": 9,
         "compare": "<="
     },
     "finish__timing__setup__ws": {
         "value": -0.48,
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 126840,
+        "value": 126635,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {
@@ -64,7 +64,7 @@
         "compare": "<="
     },
     "finish__timing__wns_percent_delay": {
-        "value": -10.0,
+        "value": -22.09,
         "compare": ">="
     }
 }
diff --git a/flow/scripts/floorplan.tcl b/flow/scripts/floorplan.tcl
@@ -93,6 +93,15 @@ if { [env_var_exists_and_non_empty MAKE_TRACKS] } {
   make_tracks
 }
 
+if { [env_var_exists_and_non_empty FASTROUTE_TCL] } {
+  log_cmd source $::env(FASTROUTE_TCL)
+} else {
+  log_cmd \
+    set_global_routing_layer_adjustment \
+    $::env(MIN_ROUTING_LAYER)-$::env(MAX_ROUTING_LAYER) $::env(ROUTING_LAYER_ADJUSTMENT)
+  log_cmd set_routing_layers -signal $::env(MIN_ROUTING_LAYER)-$::env(MAX_ROUTING_LAYER)
+}
+
 source_env_var_if_exists FOOTPRINT_TCL
 
 if { ![env_var_equal SKIP_REPAIR_TIE_FANOUT 1] } {
diff --git a/flow/scripts/global_place.tcl b/flow/scripts/global_place.tcl
@@ -18,15 +18,6 @@ if { ![env_var_exists_and_non_empty FOOTPRINT] } {
   }
 }
 
-if { [env_var_exists_and_non_empty FASTROUTE_TCL] } {
-  log_cmd source $::env(FASTROUTE_TCL)
-} else {
-  log_cmd \
-    set_global_routing_layer_adjustment \
-    $::env(MIN_ROUTING_LAYER)-$::env(MAX_ROUTING_LAYER) $::env(ROUTING_LAYER_ADJUSTMENT)
-  log_cmd set_routing_layers -signal $::env(MIN_ROUTING_LAYER)-$::env(MAX_ROUTING_LAYER)
-}
-
 set global_placement_args {}
 
 # Parameters for routability mode in global placement
diff --git a/flow/scripts/variables.yaml b/flow/scripts/variables.yaml
@@ -59,6 +59,7 @@ ROUTING_LAYER_ADJUSTMENT:
     deltaDebug.py should weed out the possible routes and leave
     a minimal failing case that pinpoints the problem.
   stages:
+    - floorplan
     - place
     - grt
     - route
@@ -613,6 +614,7 @@ MIN_ROUTING_LAYER:
   description: |
     The lowest metal layer name to be used in routing.
   stages:
+    - floorplan
     - place
     - grt
     - route
@@ -621,6 +623,7 @@ MAX_ROUTING_LAYER:
   description: |
     The highest metal layer name to be used in routing.
   stages:
+    - floorplan
     - place
     - grt
     - route
diff --git a/tools/OpenROAD b/tools/OpenROAD
@@ -1 +1 @@
-Subproject commit d97df4813dba18edcd0e1bf17d7d0430fefb56c0
+Subproject commit ab81447620e91829e61bd0328897fbc2b86f85f9
