-
Notifications
You must be signed in to change notification settings - Fork 102
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
[python-frontend] Add support for non-determinism #1439
Conversation
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.
Can you please add some regression tests with __VERIFIER_nondet_X()? See the definition here: https://sv-comp.sosy-lab.org/2023/rules.php
In particular, try to add some regression tests that fail too.
ebf90a8
to
1cab16e
Compare
I agree with @fbrausse. Please check these definitions: https://github.com/esbmc/esbmc/blob/master/src/clang-c-frontend/clang_c_language.cpp#L431 https://github.com/esbmc/esbmc/blob/master/src/clang-c-frontend/clang_c_language.cpp#L445 We should match them. |
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.
Excellent!
Many thanks for addressing our comments, @brcfarias.
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.
LGTM. One nitpick: The test whether the underscore was found could be removed (or become an assertion).
This PR: