Skip to content

gf12/ibex: tighten clock period so setup WS < 0 - #4703

Merged
maliberty merged 1 commit into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:tightening_designs
Oct 20, 2026
Merged

maliberty merged 1 commit into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:tightening_designs

Conversation

@eder-matheus

Copy link
Copy Markdown
Member
  • Remove SETUP_SLACK_MARGIN (180) and HOLD_SLACK_MARGIN (50). The flow
    only times the BC corner, so the margins over-repaired at BC without
    closing WC. Removing them cut area 18%, power 32% and flow runtime
    from 966 s to 307 s.
  • Clock period 1020 -> 840 ps.
    Finish: setup WS -34 ps, TNS -9.4 ns (493 endpoints), hold clean, 0 DRC.

Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
@eder-matheus eder-matheus self-assigned this Oct 19, 2026

@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 removes the setup and hold slack margins from the configuration file and decreases the clock period constraint from 1020 to 840. However, decreasing the clock period to 840 results in negative setup slack and timing violations. It is recommended to increase the clock period to at least 880 to ensure a timing-closed design.

Comment thread flow/designs/gf12/ibex/constraint.sdc
@openroad-ci

Copy link
Copy Markdown
Member

🔍 QoR check

Metrics reflect the PR merge build — i.e. what will land on the target branch.

Commit fc1342a · Jenkins build #1 · Baseline: build · View build on dashboard

62 design(s) checked — 0 with regression(s), 0 without a comparable baseline.

@maliberty
maliberty merged commit c8772e2 into The-OpenROAD-Project:master Oct 20, 2026
12 checks passed
@maliberty
maliberty deleted the tightening_designs branch October 20, 2026 02:23
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.

3 participants