forked from YosysHQ/yosys
-
Notifications
You must be signed in to change notification settings - Fork 41
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
bump fork for version 0.42 #21
Merged
Merged
Conversation
This file contains 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
Separates `test-linux` into `build-linux` and `test-linux`, wherein `build-` builds out of tree, and uploading the build for the `test-` job. Tar compression is done to retain execution permissions when downloading build artifact. When calling `make test`, override `TARGETS` and `EXTRA_TARGETS` to prevent rebuild.
e.g. when calling `make clean` out-of-tree
Also reduce `${{}}` expansion in `run` blocks.
Switch build artifact to a default clang build. Testing with the build artifact locally, `make test` is failing with `/lib/x86_64-linux-gnu/libstdc++.so.6: version 'GLIBCXX_3.4.29' not found`. Using the gcc-11 build (might be?) installing GLIBCXX_3.4.29 but not linking it into the build. Rather than trying to get it to link, just use the pre-installed `clang` instead.
Also rename `build-artifact` to use `matrix.os` for compatibility with testing across OS.
`os_name` in include section needs to be explicit (putting it at the end doesn't apply to the extra jobs). Move macOS test to extra job instead of doing all gcc/clang (which isn't setup for mac anyway). Also adds name to build-yosys task.
Allow test suite to run if, for example, the C++20 builds are failing but C++11 are fine.
Remove compiler and cpp_std from `build-yosys` matrix. Using `config-$CC` will instead fall back to default values. Drop `Tool versions` step and introduce `yosys-config` output instead. Rename `test-builds` to `test-compile`.
`test-macos.yml` included c++17 which was missing in `test-build.yml`.
New Setup Cpp step uses fully qualified paths for $CC and $CXX so ${CC%%-*} no longer works. Remove os_name since it's not needed anymore.
intel_alm: drop quartus support
export define marco to qtcreator.config
Since 0.41, when building docs with ABCEXTERNAL set, the rule for yosys-abc docs usage would attempt to build in-tree ABC. This fix this regression by only including yosys-abc in DOCS_USAGE_STDERR when ABCEXTERNAL is not set. Signed-off-by: Mary Guillemard <[email protected]>
Skip duplicate actions on nix build.
Only build yosys-abc usage docs when ABCEXTERNAL is not set
ci: Skip duplicate actions on nix build
docs: add todo for $demux
box_derive: New command to derive modules for boxes
Update workflow so it creates trigger as user so GitHub Action is triggering actual build
These came up when using the experimental incremental interface and are also in code that was recently refactored to support that interface.
smtbmc: Fix two .yw handling related crashes
eder-matheus
merged commit Jun 7, 2024
ea9ff29
into
The-OpenROAD-Project:master
20 of 21 checks passed
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.
No description provided.