-
Notifications
You must be signed in to change notification settings - Fork 44
Distinguish \bottom from stuck states #2451
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
Conversation
…amework/kore into distinguish-bottom-from-stuck-states
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wrote a summary in the pull request description, but please do that in the future.
kore/src/Kore/Step.hs
Outdated
configs <- lift $ Pattern.simplifyTopConfiguration config | ||
filteredConfigs <- SMT.Evaluator.filterMultiOr configs | ||
asum (pure <$> toList filteredConfigs) | ||
if filteredConfigs == mempty |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if filteredConfigs == mempty | |
if null filteredConfigs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, disregard this comment. I added it before you changed this line, but for some reason the review didn't go through.
Another attempt at addressing runtimeverification/evm-semantics#1944. Addresses #3630 (needs a follow-up in booster) This change mimicks what kore-exe in prover mode returns(as introduced by #2451), namely the configuration goes to bottom
When the last configuration rewrites to \bottom, the backend will report that instead of reporting the last configuration as "stuck".
Fixes #2445.
Review checklist
The author performs the actions on the checklist. The reviewer evaluates the work and checks the boxes as they are completed.