-
Notifications
You must be signed in to change notification settings - Fork 273
CONTRACTS: Add inductive checks for assigns clauses [depends-on: #7127] #7300
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
Codecov ReportBase: 78.50% // Head: 78.28% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## develop #7300 +/- ##
===========================================
- Coverage 78.50% 78.28% -0.23%
===========================================
Files 1663 1642 -21
Lines 191297 189988 -1309
===========================================
- Hits 150174 148728 -1446
- Misses 41123 41260 +137
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
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.
This would benefit from a rebase to make CI pass all checks.
if(expr.id() == ID_index || expr.id() == ID_dereference) | ||
{ | ||
address_of_exprt address_of_expr(expr); | ||
if(!is_constant(address_of_expr)) | ||
{ | ||
append_object_havoc_code_for_expr(location, address_of_expr, dest); | ||
return; | ||
} | ||
} |
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 would appreciate if the commit message could explain why this change is needed here/is the right thing to do.
da1e346
to
a7cc073
Compare
a7cc073
to
4f9b5cc
Compare
Fixes #7208