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
Yosys sensitivity to input changes #1788
Comments
To temporary work-around the litex differences between different builds, a patch might be applied to litex, as this line in the bios source code produce the small changes in the mem init file: |
Could we propose a PR for LiteX to |
@litghost Sure, opened PR here: enjoy-digital/litex#702 |
@acomodi Can we split this bug into two seperate things?
It could turn out that due to (a) the assumption that in (b) the two eblifs are actually "almost identical" is incorrect. |
This issue is to keep track of the issue reported here: #1776.
Problem statement
The auto-generated designs, such as litex, present some small variations in the memory initialization of the BRAMs. This causes a small perturbation in the initial conditions of a test, which lead to major changes in the output results.
This problem might not be only related to the BRAM initialization, but to any small changes that are applied to a specific design, but for the sake of this issue description, we will keep the small BRAM initialization changes as the triggering factor for this issue.
This issue covers the sensitivity of the synthesis step which produces two very different
eblifs
given a small change in the memory initialization values.Synthesis
The mem.init changes make yosys produce two completely different output eblifs. There are some tests that need to be performed in order to asses whether the two different eblifs represent the same circuit, and it is just a matter of ordering.
The expected output of the synthesis step would be to have only a small number of lines affected, relative to the BRAM initializations.
Steps to reproduce
arty_soc
directory.eblif
with the ones saved from the previous run.The text was updated successfully, but these errors were encountered: