From e7409f11888da63443389e6cb08fb4d6ba5623e8 Mon Sep 17 00:00:00 2001 From: mrg Date: Sun, 19 Jul 2026 13:43:25 -0700 Subject: [PATCH 1/3] DO NOT MERGE: neighbor-check three-arm CI eval -- arm 3 (subgraph STA + WNS) Bump tools/OpenROAD to VLSIDA neighbor-check-sta (df28de9e10) and force repair_timing -neighbor_check on for the QoR comparison. The feature is off by default upstream; this branch exists only to A/B the three arms and will be bumped through them sequentially. Signed-off-by: mrg --- flow/scripts/util.tcl | 3 +++ tools/OpenROAD | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/flow/scripts/util.tcl b/flow/scripts/util.tcl index 5b7d98c8d1..53747a7d9a 100644 --- a/flow/scripts/util.tcl +++ b/flow/scripts/util.tcl @@ -40,6 +40,9 @@ proc repair_timing_helper { args } { append_env_var additional_args SKIP_VT_SWAP -skip_vt_swap 0 append_env_var additional_args SKIP_CRIT_VT_SWAP -skip_crit_vt_swap 0 append_env_var additional_args MATCH_CELL_FOOTPRINT -match_cell_footprint 0 + # DO NOT MERGE: neighbor-check CI eval -- force the (off-by-default) + # neighbor feasibility check on for every repair_timing call. + lappend additional_args -neighbor_check lappend additional_args {*}$args -verbose log_cmd repair_timing {*}$additional_args diff --git a/tools/OpenROAD b/tools/OpenROAD index 20d2d5c16e..df28de9e10 160000 --- a/tools/OpenROAD +++ b/tools/OpenROAD @@ -1 +1 @@ -Subproject commit 20d2d5c16e3077c5c5f66e4e977e24e3f479c0f9 +Subproject commit df28de9e10236e69c4bf42d363cce012c1e0f147 From 8797c7faae75a5fcde3b51c544f50dc5d9683862 Mon Sep 17 00:00:00 2001 From: mrg Date: Mon, 20 Jul 2026 05:01:07 -0700 Subject: [PATCH 2/3] DO NOT MERGE: bump to arm 2 (lumped model + WNS rule) for CI A/B tools/OpenROAD -> VLSIDA neighbor-check-wns (9d3b441bef). Same WNS decision rule as arm 3, lumped timing estimates instead of the subgraph STA: the diff against the arm-3 run isolates the timing model. Signed-off-by: mrg --- tools/OpenROAD | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/OpenROAD b/tools/OpenROAD index df28de9e10..9d3b441bef 160000 --- a/tools/OpenROAD +++ b/tools/OpenROAD @@ -1 +1 @@ -Subproject commit df28de9e10236e69c4bf42d363cce012c1e0f147 +Subproject commit 9d3b441beffefbcb004c668faee4e70636a9e187 From d8957231098c7df02f076ca12507ab94dd22cca4 Mon Sep 17 00:00:00 2001 From: mrg Date: Tue, 21 Jul 2026 06:18:23 -0700 Subject: [PATCH 3/3] DO NOT MERGE: bump to arm 1 (lumped model + lambda rule) for CI A/B tools/OpenROAD -> VLSIDA neighbor-check (2f0d25c289). The original neighbor check: same lumped estimates as arm 2 with the lambda harm/gain trade instead of the WNS rule; the diff against the arm-2 run isolates the decision rule. Signed-off-by: mrg --- tools/OpenROAD | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/OpenROAD b/tools/OpenROAD index 9d3b441bef..2f0d25c289 160000 --- a/tools/OpenROAD +++ b/tools/OpenROAD @@ -1 +1 @@ -Subproject commit 9d3b441beffefbcb004c668faee4e70636a9e187 +Subproject commit 2f0d25c289f1b7ae1eeecd1580d2d837c8ef3fb2