Skip to content

bump OpenROAD: rsz repair_design hot path optimization#4255

Open
minjukim55 wants to merge 2 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:secure-rsz-issue1676-v2
Open

bump OpenROAD: rsz repair_design hot path optimization#4255
minjukim55 wants to merge 2 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:secure-rsz-issue1676-v2

Conversation

@minjukim55
Copy link
Copy Markdown
Contributor

Summary

  • Bumps `tools/OpenROAD` to bring in OR PR #3434: skip `ensureWireParasitic` in `repair_design` hot path via local `load_cap` hint, gated on `kGlobalRouting` parasitics.
  • Pure performance optimization for the GRT-stage `repair_design` buffer-insertion loop.

Type

  • Bump (OR submodule pointer update only)

Impact

Measured on a large customer design (497k inst, same cts.odb across runs):

Baseline Fix Δ
`repair_design` wall 1,570s 312s −79%
Total GRT step (5_1_grt) 3,996s 2,842s -29%
Final Hold viol count 32 23 -9 (better)

`asap7 jpeg_lvt` local reproduction with the gated fix: `finish__timing__setup__tns = -112` (threshold ≥ -120, passes).

Verification

  • All 18 `rsz.repair_design*` and `rsz.repair_wire*` regression tests pass on the bumped OR.

Related Issues

The-OpenROAD-Project-private/OpenROAD-flow-scripts#1676

(Supersedes #1681 — head branch was renamed to resolve a Jenkins multibranch indexing issue.)

@minjukim55 minjukim55 self-assigned this May 26, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the OpenROAD subproject commit to a newer version. There are no review comments, and I have no additional feedback to provide.

Bring in OR PR: skip ensureWireParasitic in repair_design hot path
via load_cap hint, gated on kGlobalRouting parasitics.

OR PR: The-OpenROAD-Project-private/OpenROAD#3434
Issue: The-OpenROAD-Project-private/OpenROAD-flow-scripts#1676

Signed-off-by: Minju Kim <mkim@precisioninno.com>
ariane133 nangate45 and microwatt sky130hd see small setup TNS drift
from the cell-selection change in OR PR #10507; bump thresholds with
~5% margin around the new measured values.

Related: The-OpenROAD-Project-private/OpenROAD-flow-scripts#1676
Signed-off-by: Minju Kim <mkim@precisioninno.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant