Merge pull request #3877 from The-OpenROAD-Project-staging/secure-fix-repair-tie-name-collision

maliberty · web-flow · commit 1426c0532d3d · 2026-02-10T17:15:45.000Z
update metrics for `repair_tie_fanout` and `insert_buffer` name collision fix
diff --git a/flow/designs/asap7/aes-block/rules-base.json b/flow/designs/asap7/aes-block/rules-base.json
@@ -24,15 +24,15 @@
         "compare": "<="
     },
     "cts__design__instance__count__hold_buffer": {
-        "value": 1445,
+        "value": 923,
         "compare": "<="
     },
     "cts__timing__setup__ws": {
-        "value": -162.0,
+        "value": -124.0,
         "compare": ">="
     },
     "cts__timing__setup__tns": {
-        "value": -9000.0,
+        "value": -5920.0,
         "compare": ">="
     },
     "cts__timing__hold__ws": {
@@ -48,11 +48,11 @@
         "compare": "<="
     },
     "globalroute__timing__setup__ws": {
-        "value": -164.0,
+        "value": -150.0,
         "compare": ">="
     },
     "globalroute__timing__setup__tns": {
-        "value": -8520.0,
+        "value": -6750.0,
         "compare": ">="
     },
     "globalroute__timing__hold__ws": {
@@ -80,7 +80,7 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -129.0,
+        "value": -116.0,
         "compare": ">="
     },
     "finish__timing__setup__tns": {
diff --git a/flow/designs/asap7/ibex/rules-base.json b/flow/designs/asap7/ibex/rules-base.json
@@ -32,7 +32,7 @@
         "compare": ">="
     },
     "cts__timing__setup__tns": {
-        "value": -35900.0,
+        "value": -29100.0,
         "compare": ">="
     },
     "cts__timing__hold__ws": {
@@ -52,7 +52,7 @@
         "compare": ">="
     },
     "globalroute__timing__setup__tns": {
-        "value": -57400.0,
+        "value": -57800.0,
         "compare": ">="
     },
     "globalroute__timing__hold__ws": {
@@ -84,7 +84,7 @@
         "compare": ">="
     },
     "finish__timing__setup__tns": {
-        "value": -34700.0,
+        "value": -22600.0,
         "compare": ">="
     },
     "finish__timing__hold__ws": {
diff --git a/flow/designs/asap7/mock-alu/rules-base.json b/flow/designs/asap7/mock-alu/rules-base.json
@@ -84,7 +84,7 @@
         "compare": ">="
     },
     "finish__timing__setup__tns": {
-        "value": -17200.0,
+        "value": -17300.0,
         "compare": ">="
     },
     "finish__timing__hold__ws": {
diff --git a/flow/designs/asap7/riscv32i-mock-sram/rules-base.json b/flow/designs/asap7/riscv32i-mock-sram/rules-base.json
@@ -64,7 +64,7 @@
         "compare": ">="
     },
     "detailedroute__route__wirelength": {
-        "value": 73284,
+        "value": 72999,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
@@ -84,7 +84,7 @@
         "compare": ">="
     },
     "finish__timing__setup__tns": {
-        "value": -12600.0,
+        "value": -8360.0,
         "compare": ">="
     },
     "finish__timing__hold__ws": {
diff --git a/flow/designs/asap7/riscv32i/rules-base.json b/flow/designs/asap7/riscv32i/rules-base.json
@@ -52,7 +52,7 @@
         "compare": ">="
     },
     "globalroute__timing__setup__tns": {
-        "value": -200.0,
+        "value": -503.0,
         "compare": ">="
     },
     "globalroute__timing__hold__ws": {
@@ -64,7 +64,7 @@
         "compare": ">="
     },
     "detailedroute__route__wirelength": {
-        "value": 70525,
+        "value": 70494,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
diff --git a/flow/designs/asap7/swerv_wrapper/rules-base.json b/flow/designs/asap7/swerv_wrapper/rules-base.json
@@ -80,19 +80,19 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -412.0,
+        "value": -402.0,
         "compare": ">="
     },
     "finish__timing__setup__tns": {
-        "value": -78000.0,
+        "value": -75900.0,
         "compare": ">="
     },
     "finish__timing__hold__ws": {
-        "value": -215.0,
+        "value": -200.0,
         "compare": ">="
     },
     "finish__timing__hold__tns": {
-        "value": -45700.0,
+        "value": -45200.0,
         "compare": ">="
     },
     "finish__design__instance__area": {
diff --git a/flow/designs/gf12/swerv_wrapper/macros.v b/flow/designs/gf12/swerv_wrapper/macros.v
@@ -11,7 +11,7 @@ module ram_2048x39(CLK, ADR, D, Q, WE);
   wire n_21;
   gf12_1rf_lg11_w40_all mem(.CLK (CLK), .Q ({Q_int[39], Q}), .CEN
        (1'b1), .GWEN (n_21), .A (ADR), .D ({1'b0, D}), .EMA (3'b011),
-       .EMAW (2'b01), .RET1N (1'b1));
+       .EMAW (2'b01), .EMAS (1'b0), .STOV (1'b0), .RET1N (1'b1));
   assign n_21 = ~(WE);
 endmodule
 
@@ -28,7 +28,7 @@ module ram_64x21(CLK, ADR, D, Q, WE);
   wire n_16;
   gf12_1rf_lg6_w22_all mem(.CLK (CLK), .Q ({Q_int[21], Q}), .CEN
        (1'b1), .GWEN (n_16), .A (ADR), .D ({1'b0, D}), .EMA (3'b011),
-       .EMAW (2'b01), .RET1N (1'b1));
+       .EMAW (2'b01), .EMAS (1'b0), .STOV (1'b0), .RET1N (1'b1));
   assign n_16 = ~(WE);
 endmodule
 
@@ -43,7 +43,7 @@ module ram_256x34(CLK, ADR, D, Q, WE);
   wire [33:0] Q;
   wire n_51;
   gf12_1rf_lg8_w34_all mem(.CLK (CLK), .Q (Q), .CEN (1'b1), .GWEN
-       (n_51), .A (ADR), .D (D), .EMA (3'b011), .EMAW (2'b01), .RET1N
-       (1'b1));
+       (n_51), .A (ADR), .D (D), .EMA (3'b011), .EMAW (2'b01),
+       .EMAS (1'b0), .STOV (1'b0), .RET1N (1'b1));
   assign n_51 = ~(WE);
 endmodule
diff --git a/flow/designs/gf12/swerv_wrapper/rules-base.json b/flow/designs/gf12/swerv_wrapper/rules-base.json
@@ -28,15 +28,15 @@
         "compare": "<="
     },
     "cts__timing__setup__ws": {
-        "value": -172.0,
+        "value": -84.0,
         "compare": ">="
     },
     "cts__timing__setup__tns": {
-        "value": -899.0,
+        "value": -374.0,
         "compare": ">="
     },
     "cts__timing__hold__ws": {
-        "value": -108.0,
+        "value": -197.0,
         "compare": ">="
     },
     "cts__timing__hold__tns": {
diff --git a/flow/designs/gf12/tinyRocket/rules-base.json b/flow/designs/gf12/tinyRocket/rules-base.json
@@ -32,7 +32,7 @@
         "compare": ">="
     },
     "cts__timing__setup__tns": {
-        "value": -1720.0,
+        "value": -166.0,
         "compare": ">="
     },
     "cts__timing__hold__ws": {
diff --git a/flow/designs/gf180/ibex/rules-base.json b/flow/designs/gf180/ibex/rules-base.json
@@ -64,7 +64,7 @@
         "compare": ">="
     },
     "detailedroute__route__wirelength": {
-        "value": 1368568,
+        "value": 1368088,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
@@ -80,7 +80,7 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -0.679,
+        "value": -0.672,
         "compare": ">="
     },
     "finish__timing__setup__tns": {
diff --git a/flow/designs/gf180/riscv32i/rules-base.json b/flow/designs/gf180/riscv32i/rules-base.json
@@ -64,7 +64,7 @@
         "compare": ">="
     },
     "detailedroute__route__wirelength": {
-        "value": 657404,
+        "value": 653747,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
diff --git a/flow/designs/ihp-sg13g2/ibex/rules-base.json b/flow/designs/ihp-sg13g2/ibex/rules-base.json
@@ -64,7 +64,7 @@
         "compare": ">="
     },
     "detailedroute__route__wirelength": {
-        "value": 897596,
+        "value": 895142,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
diff --git a/flow/designs/nangate45/ariane133/rules-base.json b/flow/designs/nangate45/ariane133/rules-base.json
@@ -48,7 +48,7 @@
         "compare": "<="
     },
     "globalroute__timing__setup__ws": {
-        "value": -0.414,
+        "value": -0.391,
         "compare": ">="
     },
     "globalroute__timing__setup__tns": {
@@ -80,7 +80,7 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -0.421,
+        "value": -0.389,
         "compare": ">="
     },
     "finish__timing__setup__tns": {
diff --git a/flow/designs/nangate45/ariane136/rules-base.json b/flow/designs/nangate45/ariane136/rules-base.json
@@ -40,7 +40,7 @@
         "compare": ">="
     },
     "cts__timing__hold__tns": {
-        "value": -5.13,
+        "value": -2.54,
         "compare": ">="
     },
     "globalroute__antenna_diodes_count": {
diff --git a/flow/designs/nangate45/bp_fe_top/rules-base.json b/flow/designs/nangate45/bp_fe_top/rules-base.json
@@ -84,7 +84,7 @@
         "compare": ">="
     },
     "finish__timing__setup__tns": {
-        "value": -6.17,
+        "value": -1.42,
         "compare": ">="
     },
     "finish__timing__hold__ws": {
diff --git a/flow/designs/nangate45/bp_multi_top/rules-base.json b/flow/designs/nangate45/bp_multi_top/rules-base.json
@@ -64,7 +64,7 @@
         "compare": ">="
     },
     "detailedroute__route__wirelength": {
-        "value": 3751068,
+        "value": 3700293,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
diff --git a/flow/designs/nangate45/swerv_wrapper/rules-base.json b/flow/designs/nangate45/swerv_wrapper/rules-base.json
@@ -28,11 +28,11 @@
         "compare": "<="
     },
     "cts__timing__setup__ws": {
-        "value": -0.163,
+        "value": -0.153,
         "compare": ">="
     },
     "cts__timing__setup__tns": {
-        "value": -24.1,
+        "value": -16.4,
         "compare": ">="
     },
     "cts__timing__hold__ws": {
@@ -52,7 +52,7 @@
         "compare": ">="
     },
     "globalroute__timing__setup__tns": {
-        "value": -1.57,
+        "value": -1.27,
         "compare": ">="
     },
     "globalroute__timing__hold__ws": {
@@ -80,19 +80,19 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -0.2,
+        "value": -0.19,
         "compare": ">="
     },
     "finish__timing__setup__tns": {
-        "value": -7.3,
+        "value": -6.6,
         "compare": ">="
     },
     "finish__timing__hold__ws": {
         "value": -0.11,
         "compare": ">="
     },
     "finish__timing__hold__tns": {
-        "value": -2.0,
+        "value": -1.73,
         "compare": ">="
     },
     "finish__design__instance__area": {
diff --git a/flow/designs/nangate45/tinyRocket/rules-base.json b/flow/designs/nangate45/tinyRocket/rules-base.json
@@ -32,7 +32,7 @@
         "compare": ">="
     },
     "cts__timing__setup__tns": {
-        "value": -34.7,
+        "value": -14.8,
         "compare": ">="
     },
     "cts__timing__hold__ws": {
@@ -52,7 +52,7 @@
         "compare": ">="
     },
     "globalroute__timing__setup__tns": {
-        "value": -44.8,
+        "value": -27.0,
         "compare": ">="
     },
     "globalroute__timing__hold__ws": {
@@ -64,7 +64,7 @@
         "compare": ">="
     },
     "detailedroute__route__wirelength": {
-        "value": 545695,
+        "value": 539547,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
@@ -84,7 +84,7 @@
         "compare": ">="
     },
     "finish__timing__setup__tns": {
-        "value": -40.3,
+        "value": -19.1,
         "compare": ">="
     },
     "finish__timing__hold__ws": {
diff --git a/flow/designs/rapidus2hp/cva6/rules-base.json b/flow/designs/rapidus2hp/cva6/rules-base.json
diff --git a/flow/designs/rapidus2hp/cva6/rules-verific.json b/flow/designs/rapidus2hp/cva6/rules-verific.json
diff --git a/flow/designs/sky130hd/ibex/rules-base.json b/flow/designs/sky130hd/ibex/rules-base.json
diff --git a/flow/designs/sky130hd/microwatt/rules-base.json b/flow/designs/sky130hd/microwatt/rules-base.json
diff --git a/flow/designs/sky130hd/riscv32i/rules-base.json b/flow/designs/sky130hd/riscv32i/rules-base.json
diff --git a/flow/designs/sky130hs/ibex/rules-base.json b/flow/designs/sky130hs/ibex/rules-base.json
diff --git a/flow/designs/sky130hs/riscv32i/rules-base.json b/flow/designs/sky130hs/riscv32i/rules-base.json
diff --git a/tools/OpenROAD b/tools/OpenROAD
