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 simple runner interface for choosing modes and depth #7763

Merged
merged 2 commits into from
Nov 5, 2024

Conversation

leonardt
Copy link
Contributor

@leonardt leonardt commented Nov 1, 2024

  • Maintains default behavior of using cover, bmc, and induction tasks, but allows the user to supply a custom comma-separated mode string selecting specific tasks. NOTE: The runner script still expects mode to be some subset of the above three.
  • Adds a depth attribute/parameter for controlling bound or induction factor. For now, this parameter is shared across all tasks.

* Maintains default behavior of using `cover`, `bmc`, and `induction`
  tasks, but allows the user to supply a custom comma-separated `mode`
  string selecting specific tasks.  NOTE: The runner script still
  expects mode to be some subset of the above three.
* Adds a depth attribute/parameter for controlling bound or induction
  factor.  For now, this parameter is shared across all tasks.
@leonardt
Copy link
Contributor Author

leonardt commented Nov 1, 2024

Maybe we should rename mode to tasks for the attribute and argument. Also we could make it an array of strings rather than a comma-separated string.

Copy link
Contributor

@fabianschuiki fabianschuiki left a comment

Choose a reason for hiding this comment

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

Very cool! Thanks 🥳

@leonardt leonardt merged commit 93b08df into main Nov 5, 2024
4 checks passed
@leonardt leonardt deleted the lenny/verif-formal-sby-params branch November 5, 2024 18:15
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