flow: Global sizing exploration#4293
Draft
jhkim-pii wants to merge 7 commits into
Draft
Conversation
Point tools/OpenROAD to secure-explore-global-sizing (5e83a17805), which is OR master (f020ab6cfa) plus the 8 commits from upstream PR The-OpenROAD-Project/OpenROAD#10599 (rsz: Global sizing). TEST side of the global sizing BASE vs TEST QoR experiment. Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
Pass -phases "GLOBAL_SIZING LEGACY LAST_GASP CRIT_VT_SWAP" to the post-CTS repair_timing call so the LR global sizing policy from OpenROAD PR 10599 runs ahead of the default legacy phases. TEST-branch-only change to measure global sizing QoR impact in CI; all other repair_timing call sites keep the default phase pipeline. Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
Lower core utilization for selected GF12 and IHP-SG13G2 secure exploration designs to provide additional placement and repair headroom. Remove the temporary CTS comment now that the global sizing repair_timing phase sequence is the branch behavior. Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
This pull request updates the OpenROAD subproject commit, reduces core utilization targets across several design configurations, and modifies the timing repair script in cts.tcl to use a specific set of phases. Feedback suggests making these hardcoded phases configurable via an environment variable to avoid compatibility or regression issues on designs and platforms that do not support them.
Trigger ORFS public CI after publishing the pinned OpenROAD submodule commit through the private-to-public sync flow. No source files are changed by this commit. Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
jhkim-pii
changed the title
Update failing ORFS QoR guardrails after the secure exploration global sizing changes. Refresh only the public CI metric failures observed on the head and merge Jenkins runs, preserving unrelated warning-level netlist hash rules. The updated rules pass local checkMetadata validation against the downloaded public CI head and merge metadata artifacts. Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
…al-sizing # Conflicts: # flow/designs/nangate45/bp_be_top/rules-base.json Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
Update the sky130hd microwatt antenna violation guardrail observed in public head and merge CI after rebasing the branch on master. The refreshed rule passes local checkMetadata validation against the public Jenkins head and merge metadata artifacts. Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Problem
Solution
CORE_UTILIZATIONforgf12/ibex,gf12/jpeg, andihp-sg13g2/ibexto reduce density pressure.repair_timing_helperon theGLOBAL_SIZING LEGACY LAST_GASP CRIT_VT_SWAPphase sequence.Impact
Testing
Reviewer Notes
Related