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

sby_design: Also track fairness assumptions #260

Merged
merged 1 commit into from
Feb 1, 2024
Merged

Conversation

jix
Copy link
Member

@jix jix commented Jan 24, 2024

If we track $assume we should also track $fair and certainly need to support $check cells where the flavor is fair.

@nakengelhardt
Copy link
Member

Fairness properties aren't supported at all yet in sby, are they?

@jix
Copy link
Member Author

jix commented Jan 29, 2024

$fair is roughly to $live as $assume is to $assert so $fair would be used for for liveness properties and in fact the one liveness example included in SBY does have a $fair cell which sby_design so far ignored. With a $check cell it wouldn't ignore it but instead error out making the test for that example fail.

@jix jix merged commit 5c649c8 into YosysHQ:master Feb 1, 2024
3 checks passed
@jix jix deleted the prepare-check branch February 1, 2024 14:34
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