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

[FEATURE] Refactor SMT encoding option flag use #1159

Closed
rodrigo7491 opened this issue Dec 7, 2021 · 0 comments · Fixed by #1161
Closed

[FEATURE] Refactor SMT encoding option flag use #1159

rodrigo7491 opened this issue Dec 7, 2021 · 0 comments · Fixed by #1161
Assignees
Labels
Farrays Feature: New SMT encoding with arrays refactoring

Comments

@rodrigo7491
Copy link
Collaborator

The new SMT encoding option flag is currently hardcoded in many places, making it hard to maintain it. We should have it as a global constant, as suggested by @Kukovec in #1093, and as an enum type instead of plain string, as suggested by @shonfeder in a discussion. A check for the correct encoding use by different symbolic state rewriter implementations also should be added, as suggested by @konnov in #1121.

@rodrigo7491 rodrigo7491 added refactoring Farrays Feature: New SMT encoding with arrays labels Dec 7, 2021
@rodrigo7491 rodrigo7491 self-assigned this Dec 7, 2021
@shonfeder shonfeder added this to the Arrays Encoding milestone Dec 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Farrays Feature: New SMT encoding with arrays refactoring
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants