forked from The-OpenROAD-Project/OpenROAD-flow-scripts
-
Notifications
You must be signed in to change notification settings - Fork 0
Update yosys submodule #91
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.
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
Draft
github-actions
wants to merge
49
commits into
asap7-cva6-reduce-clk-period
Choose a base branch
from
update-yosys
base: asap7-cva6-reduce-clk-period
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
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: LucasYuki <[email protected]>
…ecksum update klayout checksum for ubuntu 20.04
use-case: in bazel-orfs when generating issues for floorplan, yosys isn't a dependency and hence isn't in e.g. the `bazelisk build MockArray_floorplan --sandbox_debug` environment. Signed-off-by: Øyvind Harboe <[email protected]>
designs/gf12/bp_single/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | placeopt__design__instance__area | 509289 | 491681 | Tighten | | placeopt__design__instance__count__stdcell | 546190 | 535708 | Tighten | | cts__design__instance__count__setup_buffer | 47495 | 46583 | Tighten | | cts__design__instance__count__hold_buffer | 47495 | 46583 | Tighten | | detailedroute__route__wirelength | 7863419 | 6200511 | Tighten | | detailedroute__route__drc_errors | 1 | 0 | Tighten | | finish__timing__setup__ws | -183.6 | -144.83 | Tighten | | finish__design__instance__area | 519153 | 500408 | Tighten | | finish__timing__drv__setup_violation_count | 23747 | 23292 | Tighten | | finish__timing__drv__hold_violation_count | 230 | 479 | Failing | Signed-off-by: Matt Liberty <[email protected]>
…t-staging/gf12-bp_single-metrics Update gf12/bp_single for drv__hold_violation_count change
Signed-off-by: Øyvind Harboe <[email protected]>
…ersions-txt-fix versions.txt: non-zero exit code when commands don't exist
Signed-off-by: Mateusz Gancarz <[email protected]>
…m-hier-separator Allow user to specify a custom hierarchy separator for the synthesis `flatten` pass
010117c
to
b0c55ec
Compare
if YOSYS_EXE is empty, log it with iffy code. This avoids -V being passed to bash and strange error messages appearing. If YOSYS_EXE is non-empty, log error silently into versions.txt as versions.txt goes into the make issue report and the actual output from this command will not be part of the make issue Signed-off-by: Øyvind Harboe <[email protected]>
…6-reduce-clk-period reduced clock period and die size for asap7 cva6
b0c55ec
to
fdc0f95
Compare
…ersions-txt-less-confusing versions.txt: nicer output
Signed-off-by: Mateusz Gancarz <[email protected]>
…lt-flatten-flag Add `-flatten` to synth pass explicitly
27c2c8b
to
064479f
Compare
missed error handling Signed-off-by: Jeff Ng <[email protected]>
064479f
to
3e02f8d
Compare
…op-stage-support added support for AutoTuner to stop after a specified stage
Signed-off-by: Øyvind Harboe <[email protected]>
Signed-off-by: Øyvind Harboe <[email protected]>
Signed-off-by: Øyvind Harboe <[email protected]>
…ariables-cleanup variables: cleanup
3e02f8d
to
037c844
Compare
Signed-off-by: Øyvind Harboe <[email protected]>
Signed-off-by: Jeff Ng <[email protected]>
037c844
to
6f3a4ea
Compare
…eadme-bazel-blurb README: add bazel-orfs as an installation option
…cva6-autotuner initial AutoTuner config for asap7 cva6
Too early for hold and it creates new buffers with meaningless locations. Signed-off-by: Matt Liberty <[email protected]>
Remove extra column used by deprecated variables only(1 currently...) Signed-off-by: Øyvind Harboe <[email protected]>
simplifies .tcl and improves documentation Signed-off-by: Øyvind Harboe <[email protected]>
DRY, AutoTuner type use-cases can use this information, also it is documentation. Signed-off-by: Øyvind Harboe <[email protected]>
…ariables-better-docs-simpler-tcl variables: move default value into variables.yaml
DRY, AutoTuner type use-cases can use this information, also it is documentation. Signed-off-by: Øyvind Harboe <[email protected]>
Signed-off-by: Øyvind Harboe <[email protected]>
…ocs-variables-readability docs: more readable deprecated annotation
Update sg13g2.map file with latest one from IHP's Open PDK. Signed-off-by: Daniel Schultz <[email protected]>
Call the variable ADDITIONAL_TYP_LIBS instead of ADDITIONAL_LIBS and assign correct values to it. Also provide TYP_LIB_FILES next to LIB_FILES with the same content. Signed-off-by: Daniel Schultz <[email protected]>
Move the I2cDeviceCtrl into a macro to have one chip design with an embedded macro generated by OR. Signed-off-by: Daniel Schultz <[email protected]>
6f3a4ea
to
8aeb790
Compare
| Metric | Old | New | Type | | ------ | --- | --- | ---- | | detailedroute__route__wirelength | 5770855 | 5678019 | Tighten | | detailedroute__route__drc_errors | 0 | 1 | Failing | | finish__timing__drv__hold_violation_count | 100 | 266 | Failing | Signed-off-by: Matt Liberty <[email protected]>
…t-staging/secure-floorplan-setup-only Only do setup repair in floorplan.tcl
…ariables-tunable-type variables: add type info for tunable variables
- tensorboard < 2.16.2 used some old protobuf functions that were dropped in current protobuf, so a bump in version is needed. Signed-off-by: Jack Luar <[email protected]>
…sorboard-regression [Autotuner] Fix tensorboard hparam regression
…-block-logic ihp-sg13g2: Minor fixes to enabled BLOCKS
Signed-off-by: Øyvind Harboe <[email protected]>
Signed-off-by: Jack Luar <[email protected]>
Signed-off-by: Jack Luar <[email protected]>
…lint Add tclint CI
…ython-dependency-injection-missed-a-spot dependencies: missed a spot when adding PYTHON_EXE dependency injection
8aeb790
to
2abd8df
Compare
Signed-off-by: Jeff Ng <[email protected]>
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
2abd8df
to
8cfebf7
Compare
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.
Automated changes by create-pull-request GitHub action