Skip to content

Update yosys submodule#29

Closed
github-actions[bot] wants to merge 1270 commits intosky130hd-gcd-grt-dpl-failurefrom
update-yosys
Closed

Update yosys submodule#29
github-actions[bot] wants to merge 1270 commits intosky130hd-gcd-grt-dpl-failurefrom
update-yosys

Conversation

@github-actions
Copy link
Copy Markdown

Automated changes by create-pull-request GitHub action

@github-actions github-actions Bot force-pushed the update-yosys branch 11 times, most recently from 1b60c6c to 026bfde Compare February 20, 2025 17:29
@github-actions github-actions Bot force-pushed the update-yosys branch 13 times, most recently from 838986a to 407fb24 Compare February 28, 2025 16:12
@github-actions github-actions Bot force-pushed the update-yosys branch 6 times, most recently from eeea265 to 62bc5c4 Compare March 4, 2025 22:12
oharboe and others added 30 commits August 24, 2025 14:55
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
…ariables-openroad-hierarchical

variables: add OPENROAD_HIERARCHICAL
designs/asap7/aes-block/rules-base.json updates:
| Metric                                        | Old      | New      | Type     |
| ------                                        | ---      | ---      | ----     |
| placeopt__design__instance__area              |     7258 |     7235 | Tighten  |
| cts__design__instance__count__hold_buffer     |     1593 |     1121 | Tighten  |
| detailedroute__route__wirelength              |    62330 |    57659 | Tighten  |
| finish__timing__setup__ws                     |   -89.15 |  -122.15 | Failing  |

designs/nangate45/swerv_wrapper/rules-base.json updates:
| Metric                                        | Old      | New      | Type     |
| ------                                        | ---      | ---      | ----     |
| placeopt__design__instance__area              |   755158 |   733024 | Tighten  |
| detailedroute__route__wirelength              |  5365759 |  5031735 | Tighten  |
| finish__timing__setup__ws                     |    -0.32 |    -0.45 | Failing  |
| finish__design__instance__area                |   762884 |   739736 | Tighten  |
| finish__timing__drv__hold_violation_count     |      656 |      578 | Tighten  |

Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
The current logic in variables.mk results in OPENROAD_EXE and YOSYS_EXE
to always be set to something, either the absolute path to tools/install
or the value from the parent env. With this behavior, when calling make
versions.txt the if statements are always true and the "not available"
is never printed to the versions.txt file. If the bins are not valid it
just prints an empty string.

Before this change, calling make versions.txt would fail silently ($? == 0)
and other make commands would fail when trying to run openroad or yosys.

After this change, calling any make command without having a valid
openroad or yosys binary will fail with an error message, which is the
same behaviour already implemented for klayout.

Signed-off-by: Vitor Bandeira <vvbandeira@precisioninno.com>
…t-staging/metrics-mpl-sa-fixes

Update metrics for The-OpenROAD-Project/OpenROAD#8106 (mpl SA fixes)
1) SYNTH_OPT_HIER to enable optimization across hier boundaries
2) SKIP_VT_SWAP to disable new VT swap opto
3) SKIP_REPAIR_TIE_FANOUT to skip tie fanout repair

Signed-off-by: Cho Moon <cmoon@precisioninno.com>
Signed-off-by: Cho Moon <cmoon@precisioninno.com>
Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
Before:
$ docker run ...
$ openroad -version
OpenROAD HEAD-HASH-NOTFOUND

After:
$ docker run ...
$ openroad -version
v2.0-23724-ge561a140c1

Signed-off-by: Vitor Bandeira <vvbandeira@precisioninno.com>
Signed-off-by: Cho Moon <cmoon@precisioninno.com>
…t-staging/rm-coyote_tc.v

rm stale designs/src/coyote_tc/coyote_tc.v
Signed-off-by: Cho Moon <cmoon@precisioninno.com>
Signed-off-by: Cho Moon <cmoon@precisioninno.com>
…t-staging/versions-check

make: check if binaries are valid before continuing
Signed-off-by: Cho Moon <cmoon@precisioninno.com>
…t-staging/secure-opto-flow

Added flow support for new optimization transformns
…t-staging/docker-or-version

docker: include openroad version in docker images
Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
| Metric                                        | Old      | New      | Type     |
| ------                                        | ---      | ---      | ----     |
| detailedroute__route__wirelength              |    13998 |    16435 | Failing  |
| finish__timing__setup__ws                     |    -1.76 |    -1.96 | Failing  |

Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
…t-staging/string-break

Fix bad string continuation in synth.tcl
…t-staging/sky130hd-gcd-metrics

designs/sky130hd/gcd/rules-base.json updates:
Grabbed sky130hs/ibex by mistake but just included it.

designs/asap7/aes-block/rules-base.json updates:
| Metric                                        | Old      | New      | Type     |
| ------                                        | ---      | ---      | ----     |
| placeopt__design__instance__area              |     7235 |     7233 | Tighten  |
| cts__design__instance__count__hold_buffer     |     1121 |     1449 | Failing  |
| finish__timing__setup__ws                     |  -122.15 |   -99.09 | Tighten  |

designs/sky130hs/ibex/rules-base.json updates:
| Metric                                        | Old      | New      | Type     |
| ------                                        | ---      | ---      | ----     |
| placeopt__design__instance__area              |   261133 |   230231 | Tighten  |
| globalroute__antenna_diodes_count             |      130 |       32 | Tighten  |
| detailedroute__route__wirelength              |   908310 |   845495 | Tighten  |
| finish__design__instance__area                |   280694 |   268183 | Tighten  |

designs/sky130hs/jpeg/rules-base.json updates:
| Metric                                        | Old      | New      | Type     |
| ------                                        | ---      | ---      | ----     |
| placeopt__design__instance__area              |   723127 |   685257 | Tighten  |
| globalroute__antenna_diodes_count             |      336 |      220 | Tighten  |
| detailedroute__antenna__violating__nets       |        1 |        3 | Failing  |
| detailedroute__antenna_diodes_count           |      142 |      136 | Tighten  |
| finish__design__instance__area                |   760037 |   724792 | Tighten  |

Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
…t-staging/metrics-update

Update metrics recent changes
Signed-off-by: João Mai <jmai@precisioninno.com>
…t-staging/mpl-notch-penalty

flow: update sky130hd/uW mpl params
Signed-off-by: Austin Rovinski <rovinski@users.noreply.github.com>
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.