New VT swap for critical cells near end of setup fixing #8378
Conversation
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
Signed-off-by: Cho Moon <[email protected]>
Signed-off-by: Cho Moon <[email protected]>
Signed-off-by: Cho Moon <[email protected]>
Signed-off-by: Cho Moon <[email protected]>
Signed-off-by: Cho Moon <[email protected]>
Signed-off-by: Cho Moon <[email protected]>
Signed-off-by: Cho Moon <[email protected]>
Signed-off-by: Cho Moon <[email protected]>
|
clang-tidy review says "All clean, LGTM! 👍" |
QuantamHD
reviewed
Sep 20, 2025
|
PR https://github.com/The-OpenROAD-Project-private/OpenROAD-flow-scripts/pull/1256 covers CI tests for this enhancement. |
|
ORFS PR The-OpenROAD-Project/OpenROAD-flow-scripts#3512 covers the CI tests. The previous PR had other synthesis changes that polluted the results. All the results are green in jenkins. For asap7 cva6, TNS is reduced from -2124.6 to 0. 2 nm is_int WNS is reduced from -476.3 to -417.2 and # setup viols from 32486 to 29981. |
maliberty
reviewed
Sep 24, 2025
povik
reviewed
Sep 24, 2025
povik
reviewed
Sep 25, 2025
…ivate/OpenROAD into secure-setup-qor
Signed-off-by: Cho Moon <[email protected]>
|
clang-tidy review says "All clean, LGTM! 👍" |
povik
reviewed
Sep 29, 2025
Signed-off-by: Cho Moon <[email protected]>
|
clang-tidy review says "All clean, LGTM! 👍" |
1) removed new doMove from BaseMove class 2) removed extra slack check from traverseFaninCone Signed-off-by: Cho Moon <[email protected]>
|
clang-tidy review says "All clean, LGTM! 👍" |
povik
approved these changes
Oct 1, 2025
maliberty
approved these changes
Oct 1, 2025
Signed-off-by: Matt Liberty <[email protected]>
maliberty
approved these changes
Oct 1, 2025
|
clang-tidy review says "All clean, LGTM! 👍" |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 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.
No description provided.