-
Notifications
You must be signed in to change notification settings - Fork 263
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
Add documentation to loop contracts, __CPROVER_loop_entry #8377
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.
Thank you for adding documentation!
src/goto-instrument/contracts/doc/user/contracts-history-variables.md
Outdated
Show resolved
Hide resolved
src/goto-instrument/contracts/doc/user/contracts-history-variables.md
Outdated
Show resolved
Hide resolved
src/goto-instrument/contracts/doc/user/contracts-history-variables.md
Outdated
Show resolved
Hide resolved
src/goto-instrument/contracts/doc/user/contracts-memory-predicates.md
Outdated
Show resolved
Hide resolved
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #8377 +/- ##
===========================================
- Coverage 78.35% 78.32% -0.03%
===========================================
Files 1726 1726
Lines 188424 188506 +82
Branches 18248 18249 +1
===========================================
+ Hits 147631 147646 +15
- Misses 40793 40860 +67 ☔ View full report in Codecov by Sentry. |
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.
It looks pretty good. Thank you!
@qinheping I added a nested loop example to the loop contract documentation to explain the havoc nature of the assigns clause. Could you please take a look to make sure everything is accurate? |
All suggestions have been resolved. @tautschnig @qinheping Please approve this PR if everything looks good! ^_^ |
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
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.
Please combine your commits into a single one.
…try_old add loop contract example emphasize size in bytes document assign clause for nested loop Apply suggestions from code review Co-authored-by: Michael Tautschnig <[email protected]> Apply suggestions from code review Co-authored-by: Qinheping Hu <[email protected]> add nested loop example Apply suggestions from code review Co-authored-by: Qinheping Hu <[email protected]> modify comments in nested loop example
Commits squashed. |
This PR is documentation only.