Skip to content

Conversation

kroening
Copy link
Member

@kroening kroening commented Aug 4, 2024

This adds SVA abort properties (IEEE 1800-2017 16.12.14).

@kroening kroening force-pushed the sva-abort-properties branch 2 times, most recently from 4eae7fd to 00a1eda Compare August 4, 2024 19:56
@kroening kroening marked this pull request as ready for review August 4, 2024 20:03
@kroening kroening force-pushed the sva-abort-properties branch from 00a1eda to 924c621 Compare August 4, 2024 20:23
Copy link
Collaborator

@tautschnig tautschnig left a comment

Choose a reason for hiding this comment

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

A few more tests would be nice.

@@ -20,6 +20,10 @@ Author: Daniel Kroening, dkr@amazon.com
/// sva_s_nexttime φ --> Xφ
/// sva_if --> ? :
/// a sva_disable_iff b --> a ∨ b
/// a sva_accept_on b --> a ∨ b
/// a sva_reject_on b --> ¬a ∧ b
/// a sva_sync_accept_on b --> a ∨ b
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are there really three different ways that all just express disjunction?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes. They differ slightly if you are building a simulator.

This adds SVA abort properties (IEEE 1800-2017 16.12.14).
@kroening kroening force-pushed the sva-abort-properties branch from 924c621 to 59d332b Compare August 7, 2024 03:11
@kroening kroening merged commit b2b8a05 into main Aug 7, 2024
8 checks passed
@kroening kroening deleted the sva-abort-properties branch August 7, 2024 03:36
Romy15200 pushed a commit to Romy15200/nws that referenced this pull request Aug 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants