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

Either SMT so we can gracefully handle SMT generation issues such as CopySlice #591

Merged
merged 4 commits into from
Oct 22, 2024

Conversation

msooseth
Copy link
Collaborator

@msooseth msooseth commented Oct 9, 2024

Description

This allows us to give results even for cases where some of the SMT cannot be generated due to symbolic copyslice. Then, we collect the errors and we continue running. This allows us to be more user-friendly and deal with more cases.

Please simply update (i.e. push into) this PR any changes you'd like to see about it!

Checklist

  • tested locally
  • added automated tests
  • updated the docs
  • updated the changelog

@msooseth msooseth force-pushed the maybe-smt branch 3 times, most recently from cf16288 to 3d7c720 Compare October 9, 2024 14:42
@msooseth msooseth force-pushed the maybe-smt branch 2 times, most recently from ae10a03 to cc26d4d Compare October 10, 2024 09:09
@msooseth msooseth changed the title [DRAFT] Maybe SMT so we can gracefully handle SMT generation issues such as CopySlice [DRAFT] Either SMT so we can gracefully handle SMT generation issues such as CopySlice Oct 10, 2024
@msooseth msooseth force-pushed the maybe-smt branch 2 times, most recently from 40d14e8 to 61249ae Compare October 10, 2024 15:54
@msooseth msooseth changed the title [DRAFT] Either SMT so we can gracefully handle SMT generation issues such as CopySlice Either SMT so we can gracefully handle SMT generation issues such as CopySlice Oct 10, 2024
@msooseth msooseth requested review from d-xo and arcz October 10, 2024 15:57
@msooseth msooseth requested a review from blishko October 21, 2024 09:27
Copy link
Collaborator

@blishko blishko left a comment

Choose a reason for hiding this comment

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

Looks good overall, as far as I can say.
I just have a few questions to clarify some things.

Copy link
Collaborator

@blishko blishko left a comment

Choose a reason for hiding this comment

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

LGTM!

@msooseth msooseth merged commit 749c7be into main Oct 22, 2024
8 checks passed
@msooseth msooseth deleted the maybe-smt branch October 24, 2024 12:23
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