Skip to content

SMV: set type #1175

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

SMV: set type #1175

wants to merge 1 commit into from

Conversation

kroening
Copy link
Member

This fixes typechecking for set literals, assignments from set literals to variables of the element type, and typechecking for the in set membership operator.

@kroening kroening added the SMV label Jun 29, 2025
@kroening kroening force-pushed the smv-set branch 7 times, most recently from f8f63ac to 43eb3fd Compare June 30, 2025 17:55
@kroening kroening marked this pull request as ready for review June 30, 2025 18:02
Comment on lines 659 to 660
else if(src.id() == ID_smv_set)
return convert_nondet_choice(src);
Copy link
Collaborator

Choose a reason for hiding this comment

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

This looks strange. Is this just a re-use of a function that wasn't meant to be used that way?

Copy link
Member Author

Choose a reason for hiding this comment

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

Renamed the method

Comment on lines -794 to -795
else if(src.id() == ID_constraint_select_one)
return convert_nondet_choice(src);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why is this now gone?

This fixes typechecking for set literals, assignments from set literals to
variables of the element type, and typechecking for the "in" set membership
operator.
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