Repository navigation
flow: enable GPL_PLACE_IOS for more designs - #4733
Open
minjukim55 wants to merge 1 commit into
Open
minjukim55 wants to merge 1 commit into
minjukim55 wants to merge 1 commit into
Conversation
Follow-up to The-OpenROAD-Project#4706, from the same seed 1-10 sweep for the designs that had not been measured yet: turn on global_placement -place_ios for 11 more public designs where setup WS improves or stays within the flag-off seed spread and placement runtime drops. asap7: coralnpu nangate45: ariane136, black_parrot, bp_be_top, bp_multi_top, swerv, swerv_wrapper ihp-sg13g2: gcd, ibex, riscv32i, spi Signed-off-by: Minju Kim <mkim@precisioninno.com>
Contributor
There was a problem hiding this comment.
Code Review
This pull request enables the GPL_PLACE_IOS option by setting export GPL_PLACE_IOS = 1 across multiple design configuration files (config.mk) for various platforms, including asap7, ihp-sg13g2, and nangate45. There are no review comments, and I have no feedback to provide.
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
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.
Follow-up to #4706: enable
GPL_PLACE_IOSin 11 more public designs, from the designs #4706 had not measured yet.How it was measured (same as #4706):
GPL_PLACE_IOSoff vs on, same OpenROAD binary, shared synth/floorplan-random_seed), run through GRTIO_CONSTRAINTSexclude regions still hold (black_parrot, bp_be_top, bp_multi_top, swerv_wrapper, ariane136)Means over seeds 1-10, off → on, all at GRT (5_1). fmax = ORFS
timing__fmax(reg2reg, see #4583). GPL wirelength = 3_3 final HPWL, GRT wirelength = 5_1 routed, placement runtime = 3_1..3_5.* ariane136's
timing__fmaxis the virtual IO clockvclk;core_clockfmax goes 0.177 → 0.180 GHz.Disabled (13):
Not measured (1): nangate45/bp_quad, floorplan fails with PDN-0179 on both arms.