Skip to content
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

Add support for Assert in simulation #427

Closed
jeanthom opened this issue Jul 9, 2020 · 4 comments
Closed

Add support for Assert in simulation #427

jeanthom opened this issue Jul 9, 2020 · 4 comments

Comments

@jeanthom
Copy link
Contributor

jeanthom commented Jul 9, 2020

Adding Assert support in pysim would be helpful for migrating LiteX code that relies on Display statements. While not equivalent to the Display statement, it would give enough information to figure out what's happening inside an elaboratable without exposing every single internal signal.

@whitequark whitequark changed the title Add support for Assert in pysim Add support for Assert in simulation Jul 13, 2020
@whitequark
Copy link
Member

(Tagging upstream because the Yosys cxxrtl backend needs to be taught about asserts, too.)

@jeanthom
Copy link
Contributor Author

This issue was discussed on August 16th as part of the weekly IRC meeting: https://freenode.irclog.whitequark.org/nmigen/2020-08-17#1597691839-1597695703;

In short:

@whitequark whitequark added this to the 0.5 milestone Feb 1, 2024
@whitequark
Copy link
Member

Support for capturing assertions in CXXRTL is about to land upstream as part of YosysHQ/yosys#4128.

Amaranth will require modification to emit $check cell from Assert statements once that PR lands. The format string within $check cell is the same as one within $print cell, which will be the basis of the implementation of #432.

@whitequark
Copy link
Member

Support for this feature has been accepted via RFC 50 and implementation is tracked in issue #1186.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

2 participants