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

[circt-test] Add support for contracts #8166

Merged
merged 1 commit into from
Feb 8, 2025

Conversation

fabianschuiki
Copy link
Contributor

Add the LowerContracts or StripContracts pass to the circt-test pipeline in order to leverage contracts during formal verification. To make this work, circ-test has to run a preparatory pass pipeline on the input MLIR, and pass the result as input to the MLIR runners instead of the original file.

Also add an --ir option to dump the IR after the preparatory passes.

@fabianschuiki fabianschuiki force-pushed the fschuiki/circt-test-contracts branch from bc19415 to de4ab82 Compare February 4, 2025 18:14
@fabianschuiki fabianschuiki force-pushed the fschuiki/lower-firrtl-contracts branch 2 times, most recently from 9d4e065 to e78f330 Compare February 4, 2025 21:14
@fabianschuiki fabianschuiki force-pushed the fschuiki/circt-test-contracts branch 2 times, most recently from ac03265 to 854cd9e Compare February 5, 2025 21:24
@fabianschuiki fabianschuiki force-pushed the fschuiki/lower-firrtl-contracts branch from 545ae59 to ee14267 Compare February 6, 2025 17:46
@fabianschuiki fabianschuiki force-pushed the fschuiki/circt-test-contracts branch from 854cd9e to 3595fdd Compare February 6, 2025 17:46
@fabianschuiki fabianschuiki force-pushed the fschuiki/lower-firrtl-contracts branch from ee14267 to 3bf3a99 Compare February 6, 2025 21:37
@fabianschuiki fabianschuiki force-pushed the fschuiki/circt-test-contracts branch 2 times, most recently from 161b8f4 to 83e5e8e Compare February 6, 2025 21:39
@fabianschuiki fabianschuiki force-pushed the fschuiki/lower-firrtl-contracts branch 2 times, most recently from 050e628 to 439231c Compare February 6, 2025 21:44
@fabianschuiki fabianschuiki force-pushed the fschuiki/circt-test-contracts branch from 83e5e8e to 327884d Compare February 6, 2025 21:44
@fabianschuiki fabianschuiki force-pushed the fschuiki/lower-firrtl-contracts branch from 439231c to c6dd04c Compare February 7, 2025 23:18
@fabianschuiki fabianschuiki force-pushed the fschuiki/circt-test-contracts branch from 327884d to 0a6b003 Compare February 7, 2025 23:18
Base automatically changed from fschuiki/lower-firrtl-contracts to main February 7, 2025 23:34
Add the `LowerContracts` or `StripContracts` pass to the circt-test
pipeline in order to leverage contracts during formal verification. To
make this work, circ-test has to run a preparatory pass pipeline on the
input MLIR, and pass the result as input to the MLIR runners instead of
the original file.

Also add an `--ir` option to dump the IR after the preparatory passes.
@fabianschuiki fabianschuiki force-pushed the fschuiki/circt-test-contracts branch from 0a6b003 to 96404fa Compare February 7, 2025 23:35
Copy link
Member

@dobios dobios left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@fabianschuiki fabianschuiki merged commit d667c17 into main Feb 8, 2025
5 checks passed
@fabianschuiki fabianschuiki deleted the fschuiki/circt-test-contracts branch February 8, 2025 00:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants