forked from YosysHQ/yosys
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Upstream sync #73
Closed
Closed
Upstream sync #73
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
kernel: share a single CellTypes within a pass
The only user of this API right now is the puzzle3d benchmark and it sees a slight reduction in CNF size from this, but the performance difference is within the noise of measurement on my system. Signed-off-by: Claire Wolf <claire@symbioticeda.com>
Signed-off-by: Claire Wolf <claire@symbioticeda.com>
Using LFSR counter for ezSAT::manyhot()
Add info-file and cover features to write_btor
Improve write_btor symbol handling
Signed-off-by: Claire Wolf <claire@symbioticeda.com>
kernel: speedup by using more pass-by-const-ref
Co-Authored-By: Alberto Gonzalez <61295559+boqwxp@users.noreply.github.com>
This reverts commit 7b2a85a.
deminout: prevent any constant assignment from demoting to input
Adding error message for when size (width) of number literal is zero
This makes adding more FF types in the future much more manageable. Fixes YosysHQ#1824.
IdString: use more ID::*, make them easier to use, speed up IdString::in()
ast: cap dynamic range select to size of signal, suppresses warnings
Before this commit, memory_map (which is always a part of a synth script) would always pick up any $mem cell that was not processed by a preceding pass and lower it down to $dff/$mux cells. This is undesirable for two reasons: * If there is an explicit inference attribute set on a $mem cell, e.g. (* ram_block *), then it is arguably incorrect to map such a memory to $dff/$mux cells. * If memory_map tries to lower a memory that was intended to be mapped to a large BRAM, it often takes extraordinarily long time to finish, produces an extremely large log file, and outputs an unusable design. After this commit, properly invoked memory_map will not map any memory that has an explicit inference attribute specified, solving the first issue, and alleviating the second. The default behavior is not changed.
cxxrtl: minor documentation and usability improvements
And make {get,set}_src_attribute use those functions.
select: add select -unset option
Clean up pseudo-private member usage in `passes/cmds/rename.cc`.
Clean up `passes/cmds/scatter.cc`.
Clean up pseudo-private member usage and simplify `passes/techmap/abc.cc`
Verilog frontend: add location information to parsed constants
Signed-off-by: David Shah <dave@ds0.me>
rtlil: add AttrObject::{get,set}_string_attribute, AttrObject::has_attribute
qbfsat: Fix illegal use of 'stdout' identifier
[pull] master from YosysHQ:master
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
Signed-off-by: Keith Rothman <537074+litghost@users.noreply.github.com>
Signed-off-by: Keith Rothman <537074+litghost@users.noreply.github.com>
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
Signed-off-by: Keith Rothman <537074+litghost@users.noreply.github.com>
Signed-off-by: Keith Rothman <537074+litghost@users.noreply.github.com>
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
Signed-off-by: Maciej Kurc <mkurc@antmicro.com>
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
…isable_ramb18e_w2' into upstream_sync
This is an Octopus Merge commit of the following branches: wip/carry4-cout wip/disable-primitives wip/disable_ramb18e_w2 Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
Closing, superseded by #74 |
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.
Yosys upstream sync.
There is a fix for the ABC issue described here, that switches the w/b value in the abc9_xc7.box techlib