Merge pull request #3906 from The-OpenROAD-Project-staging/secure-fix-swap-master-0220

maliberty · web-flow · commit 69009ae42dfb · 2026-02-28T18:34:59.000Z
Enabled `replace_design_check_sanity` for `replace_arith_module` post sanity check
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.516,
+        "value": -0.482,
         "compare": ">="
     },
     "globalroute__timing__setup__tns": {
@@ -64,7 +64,7 @@
         "compare": ">="
     },
     "detailedroute__route__wirelength": {
-        "value": 8313088,
+        "value": 8312391,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
@@ -80,11 +80,11 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -0.531,
+        "value": -0.516,
         "compare": ">="
     },
     "finish__timing__setup__tns": {
-        "value": -622.0,
+        "value": -649.0,
         "compare": ">="
     },
     "finish__timing__hold__ws": {
diff --git a/flow/scripts/floorplan.tcl b/flow/scripts/floorplan.tcl
@@ -130,6 +130,8 @@ if { !$::env(SKIP_REPAIR_TIE_FANOUT) } {
 }
 
 if { [env_var_exists_and_non_empty SWAP_ARITH_OPERATORS] } {
+  # Enable sanity checker until replace_arith_modules becomes stable
+  set_debug_level ODB replace_design_check_sanity 1
   replace_arith_modules
 }
 
diff --git a/flow/scripts/resize.tcl b/flow/scripts/resize.tcl
@@ -15,6 +15,8 @@ if { [env_var_exists_and_non_empty EARLY_SIZING_CAP_RATIO] } {
 }
 
 if { [env_var_exists_and_non_empty SWAP_ARITH_OPERATORS] } {
+  # Enable sanity checker until replace_arith_modules becomes stable
+  set_debug_level ODB replace_design_check_sanity 1
   replace_arith_modules
   global_placement -incremental
 }
diff --git a/tools/OpenROAD b/tools/OpenROAD
@@ -1 +1 @@
-Subproject commit 93cb34c5d4faafea180de896c4d631cddb5c56bf
+Subproject commit b984a8d7e869745a923bf28037b373a3a6488a17

Original file line number	Diff line number	Diff line change
`@@ -130,6 +130,8 @@ if { !$::env(SKIP_REPAIR_TIE_FANOUT) } {`
`130`	`130`	`}`
`131`	`131`
`132`	`132`	`if { [env_var_exists_and_non_empty SWAP_ARITH_OPERATORS] } {`
	`133`	`+ # Enable sanity checker until replace_arith_modules becomes stable`
	`134`	`+ set_debug_level ODB replace_design_check_sanity 1`
`133`	`135`	`replace_arith_modules`
`134`	`136`	`}`
`135`	`137`
Original file line number	Diff line number	Diff line change
`@@ -15,6 +15,8 @@ if { [env_var_exists_and_non_empty EARLY_SIZING_CAP_RATIO] } {`
`15`	`15`	`}`
`16`	`16`
`17`	`17`	`if { [env_var_exists_and_non_empty SWAP_ARITH_OPERATORS] } {`
	`18`	`+ # Enable sanity checker until replace_arith_modules becomes stable`
	`19`	`+ set_debug_level ODB replace_design_check_sanity 1`
`18`	`20`	`replace_arith_modules`
`19`	`21`	`global_placement -incremental`
`20`	`22`	`}`