(DO NOT MERGE) ORFS QoR check for latch D-input path optimization#4339
Open
jhkim-pii wants to merge 5 commits into
Open
(DO NOT MERGE) ORFS QoR check for latch D-input path optimization#4339jhkim-pii wants to merge 5 commits into
jhkim-pii wants to merge 5 commits into
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Member
|
Is this a no-merge PR? |
jhkim-pii
changed the title
Contributor
Author
Yes. This PR is to check the ORFS design QoR impact. |
jhkim-pii
changed the title
Update the OpenROAD submodule to the RSZ latch time-borrow repair branch without the OpenSTA submodule update. This keeps the ORFS PR focused on the RSZ fix and avoids carrying the unrelated STA update commit. Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
openroad-ci
force-pushed
the
secure-rsz-time-borrow-opt
branch
from
b8837d3 to
2965a47
Compare
Update the OpenROAD submodule to the RSZ latch time-borrow repair branch after merging latest master. This restores the OpenSTA scene/mode object update required by ORFS save_images.tcl while keeping the RSZ time-borrow fixes on the branch. Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
Update only the metadata rules that failed in public PR-4339-merge build 4 after the OpenROAD submodule bump. The new thresholds match the archived Jenkins metadata for nangate45/tinyRocket and sky130hd/microwatt so local metadata checks pass without changing unrelated rules. Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
Update the cts setup TNS rule that failed in public PR-4339-merge build 5 after the previous QoR rule update allowed the flow to progress further. The updated threshold matches the archived Jenkins metadata for nangate45/mempool_group and leaves unrelated rules unchanged. Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
Merge the latest master branch into secure-rsz-time-borrow-opt to resolve QoR rules conflicts before public sync. Keep the OpenROAD submodule on the RSZ time-borrow optimization commit while accepting the relaxed master QoR rule thresholds that cover the Jenkins metadata failures. 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
2 participants
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
secure-rsz-time-borrow-optcommit.Problem
Solution
tools/OpenROADfrom8eade99b1to9896dd27ff.RSZ-0008logging cleanup.Impact
Testing
bazelisk test //src/rsz/test:all --cache_test_results=no --test_output=errors.Reviewer Notes
Related