Skip to content

validate post GRT WNS opto#4320

Open
openroad-ci wants to merge 7 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:secure-opt-grt-wns
Open

validate post GRT WNS opto#4320
openroad-ci wants to merge 7 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:secure-opt-grt-wns

Conversation

@openroad-ci

@openroad-ci openroad-ci commented Jul 1, 2026

Copy link
Copy Markdown
Member

Timing benefits on private PDK designs seem quite significant.
20260630 private PDK results

Runtime impact seems small for hercules_is_int:
Base runtime (verific): 582 sec
Test runtime (verific): 575 sec

Signed-off-by: Cho Moon <cmoon@precisioninno.com>
Signed-off-by: Cho Moon <cmoon@precisioninno.com>
Signed-off-by: Cho Moon <cmoon@precisioninno.com>

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

Copy link
Copy Markdown
Contributor

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 introduces a new optimization step, controlled by the OPT_POST_GRT_WNS variable, to optimize worst negative slack (WNS) after global routing using VT swapping and wire rerouting. Feedback on these changes highlights a redundant detailed placement and global routing run that occurs when this optimization is enabled, and recommends propagating the MATCH_CELL_FOOTPRINT and SETUP_SLACK_MARGIN parameters to the repair_timing command to ensure user configurations are respected.

Comment thread flow/scripts/global_route.tcl
Signed-off-by: Cho Moon <cmoon@precisioninno.com>
Signed-off-by: Cho Moon <cmoon@precisioninno.com>
@precisionmoon precisionmoon requested a review from maliberty July 1, 2026 02:22
@precisionmoon

Copy link
Copy Markdown
Contributor

@jhkim-pii, @jfgava, FYI.

@precisionmoon precisionmoon added the UpdateRules Starts GHA to update rules label Jul 1, 2026
Signed-off-by: Cho Moon <cmoon@precisioninno.com>
@openroad-ci openroad-ci removed the UpdateRules Starts GHA to update rules label Jul 1, 2026
@jfgava

jfgava commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

If the focus is just on WNS, we can try different phases, such as the "WNS" policy (-phases "WNS"). The problem is that it will focus only on the most critical path. If it can't be optimized, the run ends. There is no optimization on the secondary paths (no TNS gains).
But maybe it is worth trying. I think the objective here is to go a bit further on QoR, using optimizations that don't disturb placement and routing.
What are your thoughts? @jhkim-pii

@jhkim-pii

Copy link
Copy Markdown
Contributor

I agree that focusing on WNS rather than TNS is required to minimize the PNR impact.
But @precisionmoon said that WNS policy does not work well unfortunately.

I also investigate how to fix the post-route timing violations effectively.
I am trying to insert buffers more precisely to avoid causing additional displacement impact or routing detour for nets w/ large delay.

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.

4 participants