Update yosys submodule#2
Draft
github-actions[bot] wants to merge 25 commits intoupdate_sky130hs_jpeg_metricsfrom
Draft
Update yosys submodule#2github-actions[bot] wants to merge 25 commits intoupdate_sky130hs_jpeg_metricsfrom
github-actions[bot] wants to merge 25 commits intoupdate_sky130hs_jpeg_metricsfrom
Conversation
github-actions
bot
force-pushed
the
update-yosys
branch
from
b856148 to
031d8a8
Compare
Update OR to get the new command Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
| Metric | Old | New | Type | | ------ | --- | --- | ---- | | placeopt__design__instance__area | 1591358 | 1579306 | Tighten | | placeopt__design__instance__count__stdcell | 1457522 | 1442363 | Tighten | | cts__design__instance__count__setup_buffer | 126741 | 125423 | Tighten | | cts__design__instance__count__hold_buffer | 126741 | 125423 | Tighten | | detailedroute__route__wirelength | 29312243 | 25903866 | Tighten | | detailedroute__route__drc_errors | 0 | 3 | Failing | | finish__timing__setup__ws | -114.98 | -239.22 | Failing | | finish__design__instance__area | 1622581 | 1610080 | Tighten | | finish__timing__drv__setup_violation_count | 63370 | 62711 | Tighten | | finish__timing__drv__hold_violation_count | 341 | 170 | Tighten | Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
…t-staging/bp-quad-metrics designs/gf12/bp_quad/rules-base.json updates:
…ky130hs_jpeg_metrics Update sky130hs/jpeg metrics
Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
`make gui_synth` is an important use-case for fast
turnaround, less accurate, RTL rinse and repeat cycles
when looking at timing reports, endpoint slack histograms
in the GUI.
The only reason why an .odb file and canonicalized .sdc
file is not always written out by the ORFS by default,
is that this takes a bit of extra time and space and it was
not deemed desirable. It is a bit of a toss-up, accept less
consistency in the flow, or pay a moderate performance
penalty. Switching to always writing .odb and canonicalized
.sdc is simple enough to do if we change our mind and want
simpler, more consistency and fewer suprises across stages.
- gui_synth: improve user experience, same code
as for gui_floorplan/grt, etc. GUI_TIMING=0 now works.
- sta-synth.tcl was trying to do the same as open.tcl. DRY,
now we only have open.tcl
- .odb file output from synth + canonicalization of 1_synth.sdc,
i.e. it is written out by OpenSTA instead of copied from user.
This means that any dependencies that the user's SDC_FILE has
is not dragged into floorplan.
bazel-orfs will be updated to use this, no change to normal
ORFS flow.
To use gui_synth, there is an additional step:
make synth
# New step that writes out 1_synth.odb and canonicalizes
# 1_synth.sdc
make do-1_3_synth
make gui_synth
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
…t-staging/report_fmax_metric Add report_fmax_metric
Signed-off-by: luis201420 <luisemv@precisioninno.com>
Signed-off-by: luis201420 <luisemv@precisioninno.com>
github-actions
bot
force-pushed
the
update-yosys
branch
from
031d8a8 to
b1f626d
Compare
Signed-off-by: luis201420 <luisemv@precisioninno.com>
github-actions
bot
force-pushed
the
update-yosys
branch
from
b1f626d to
2376fcf
Compare
Signed-off-by: luis201420 <luisemv@precisioninno.com>
github-actions
bot
force-pushed
the
update-yosys
branch
from
2376fcf to
15dafd6
Compare
…ui-synth Gui synth
Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
Signed-off-by: Martin Povišer <povik@cutebit.org>
…ate/OpenROAD-flow-scripts into add_variables_repair_antennas_iter
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
github-actions
bot
force-pushed
the
update-yosys
branch
from
15dafd6 to
bf611ec
Compare
…t-staging/rpt-report_clock_min_period Add report_clock_min_period to report_metrics.tcl
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
github-actions
bot
force-pushed
the
update-yosys
branch
from
bf611ec to
1086ffc
Compare
…t-staging/synth-retime Test retiming
…ate/OpenROAD-flow-scripts into add_variables_repair_antennas_iter
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions
bot
force-pushed
the
update-yosys
branch
from
1086ffc to
992acef
Compare
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
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.
Automated changes by create-pull-request GitHub action