Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

update Yosys to version 0.48 #2620

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

update Yosys to version 0.48 #2620

wants to merge 13 commits into from

Conversation

openroad-robot
Copy link
Contributor

@openroad-robot openroad-robot commented Dec 11, 2024 •
edited by eder-matheus
Loading

sky130hd/aes

Metric Old New Type
synth__design__instance__area__stdcell 100410.81 99830.94 Tighten
placeopt__design__instance__area 133285 129776 Tighten
placeopt__design__instance__count__stdcell 20411 20216 Tighten
cts__design__instance__count__setup_buffer 1775 1758 Tighten
cts__design__instance__count__hold_buffer 1775 1758 Tighten
globalroute__antenna_diodes_count 218 200 Tighten
detailedroute__antenna_diodes_count 15 26 Failing
finish__design__instance__area 140715 135766 Tighten
finish__timing__drv__setup_violation_count 887 879 Tighten

sky130hd/chameleon

Metric Old New Type
synth__design__instance__area__stdcell 27456.71 27373.26 Tighten
placeopt__design__instance__area 6534944 6533817 Tighten
placeopt__design__instance__count__stdcell 70314 69796 Tighten
cts__design__instance__count__setup_buffer 6114 6069 Tighten
cts__design__instance__count__hold_buffer 6114 6069 Tighten
globalroute__antenna_diodes_count 144 216 Failing
detailedroute__route__wirelength 843488 821835 Tighten
detailedroute__antenna_diodes_count 161 335 Failing
finish__design__instance__area 6538739 6538118 Tighten
finish__timing__drv__setup_violation_count 3057 3035 Tighten

sky130hd/ibex

Metric Old New Type
finish__timing__setup__ws -1.2 -1.96 Failing

sky130hs/ibex

Metric Old New Type
globalroute__antenna_diodes_count 36 0 Tighten
detailedroute__antenna_diodes_count 14 22 Failing

gf180/jpeg

Metric Old New Type
globalroute__antenna_diodes_count 0 7 Failing
finish__timing__setup__ws -0.53 -0.31 Tighten
finish__timing__wns_percent_delay -11.78 -10.0 Tighten

sky130hd/jpeg

Metric Old New Type
synth__design__instance__area__stdcell 554199.03 552475.25 Tighten
placeopt__design__instance__area 598574 593412 Tighten
placeopt__design__instance__count__stdcell 73131 65167 Tighten
cts__design__instance__count__setup_buffer 6359 5667 Tighten
cts__design__instance__count__hold_buffer 6359 5667 Tighten
globalroute__antenna_diodes_count 269 78 Tighten
detailedroute__route__wirelength 1536290 1429919 Tighten
detailedroute__antenna_diodes_count 56 103 Failing
finish__timing__setup__ws -0.11 -0.09 Tighten
finish__design__instance__area 621399 615298 Tighten
finish__timing__drv__setup_violation_count 3180 2833 Tighten

sky130hd/riscv32i

Metric Old New Type
synth__design__instance__area__stdcell 71922.42 70918.08 Tighten
placeopt__design__instance__count__stdcell 9731 7358 Tighten
cts__design__instance__count__setup_buffer 846 640 Tighten
cts__design__instance__count__hold_buffer 846 640 Tighten
globalroute__antenna_diodes_count 12 0 Tighten
detailedroute__route__wirelength 333349 314342 Tighten
detailedroute__antenna_diodes_count 13 9 Tighten
finish__timing__setup__ws -1.31 -2.28 Failing
finish__timing__drv__setup_violation_count 423 516 Failing

sky130hs/riscv32i

Metric Old New Type
placeopt__design__instance__count__stdcell 9921 7591 Tighten
cts__design__instance__count__setup_buffer 863 660 Tighten
cts__design__instance__count__hold_buffer 863 660 Tighten
globalroute__antenna_diodes_count 25 0 Tighten
detailedroute__route__wirelength 385662 352023 Tighten
detailedroute__antenna_diodes_count 3 11 Failing
finish__timing__drv__setup_violation_count 431 330 Tighten

ihp-sg13g2/spi

Metric Old New Type
synth__design__instance__area__stdcell 2259.36 2244.75 Tighten
detailedroute__route__wirelength 3437 4194 Failing

asap7/aes-block

Metric Old New Type
synth__design__instance__area__stdcell 2314.54 2310.54 Tighten
placeopt__design__instance__count__stdcell 12211 12203 Tighten
cts__design__instance__count__setup_buffer 1062 1061 Tighten
cts__design__instance__count__hold_buffer 1262 1178 Tighten
finish__timing__setup__ws -227.71 -213.94 Tighten
finish__timing__drv__hold_violation_count 248 166 Tighten
finish__timing__wns_percent_delay -44.5 -41.6 Tighten

nangate45/bp_fe

Metric Old New Type
synth__design__instance__area__stdcell 244565.53 244461.52 Tighten
detailedroute__route__wirelength 3080205 2733057 Tighten
finish__timing__setup__ws -0.68 -0.16 Tighten
finish__timing__drv__hold_violation_count 1154 678 Tighten
finish__timing__wns_percent_delay -35.7 -16.09 Tighten

intel16/gcd

Metric Old New Type
placeopt__design__instance__area 210 253 Failing
placeopt__design__instance__count__stdcell 512 500 Tighten
detailedroute__route__wirelength 3020 2807 Tighten
finish__design__instance__area 360 350 Tighten

gf12/swerv_wrapper

Metric Old New Type
finish__timing__setup__ws 0.0 -153.25 Failing
finish__timing__drv__hold_violation_count 660 638 Tighten

sky130hd/microwatt

Metric Old New Type
placeopt__design__instance__count__stdcell 191836 183347 Tighten
cts__design__instance__count__setup_buffer 16681 15943 Tighten
cts__design__instance__count__hold_buffer 16681 15943 Tighten
globalroute__antenna_diodes_count 3344 5019 Failing
detailedroute__antenna_diodes_count 2196 3042 Failing
finish__timing__setup__ws -4.68 -2.91 Tighten
finish__timing__drv__setup_violation_count 8341 7972 Tighten
  • sky130hs/aes
    • [ERROR DRT-1231] Pin 22197/B does not have access point

eder-matheus and others added 3 commits December 11, 2024 11:53
@eder-matheus
use latest openroad
659f54e 
Signed-off-by: Eder Monteiro <[email protected]>
@eder-matheus
use Yosys 0.48
bc82ba4 
Signed-off-by: Eder Monteiro <[email protected]>
@povik @eder-matheus
synth: Adjust blackbox loading
3046d7d 
Load whiteboxes with functional and timing description of cells where
available.

Signed-off-by: Martin Povišer <[email protected]>
@povik
Copy link

povik commented Dec 11, 2024

If there's larger-than-usual number of metric failures it may be because Yosys has started inferring flip-flops with clock enables (YosysHQ/yosys#4698)

@eder-matheus
Copy link
Collaborator

@povik We actually have a bunch of errors in this PR. The metric errors are not concerning me, but we have some congestion errors and a DRT error regarding a pin not having access points. I will see what needs to be done for each of them.

@eder-matheus
increase padding for diode count
b98068a 
Signed-off-by: Eder Monteiro <[email protected]>
@eder-matheus
update public designs metrics
320224b 
Signed-off-by: Eder Monteiro <[email protected]>
@eder-matheus
bump or
8433e67 
Signed-off-by: Eder Monteiro <[email protected]>
@eder-matheus
Copy link
Collaborator

I've updated the description of the PR with the metric updates and the remaining failures. We can see a bunch of improvements in the sky130hd and sky130hs designs, specially regarding buffer count. The four remaining violations are being investigated.

@eder-matheus
update nangate45/bp_fe config
2414a5a 
Signed-off-by: Eder Monteiro <[email protected]>
@eder-matheus
change asap7/aes-block config
b376a5a 
Signed-off-by: Eder Monteiro <[email protected]>
@eder-matheus
update private designs metrics
39afa59 
Signed-off-by: Eder Monteiro <[email protected]>
@eder-matheus
bump or again
d587dbd 
Signed-off-by: Eder Monteiro <[email protected]>
@eder-matheus
decrease sky130hd/microwatt density
4d20778 
Signed-off-by: Eder Monteiro <[email protected]>
@eder-matheus
update microwatt metrics
8ae3ae3 
Signed-off-by: Eder Monteiro <[email protected]>
@eder-matheus
Copy link
Collaborator

@povik FYI, we're almost ready to merge this. Only one design is failing in DRT. I'm investigating what's going on here.

@eder-matheus
bump or again
2b51394 
Signed-off-by: Eder Monteiro <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@openroad-robot @povik @eder-matheus