Skip to content

Refactor Contracts and Harnesses for <*mut T>::add, sub and offset #203

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

Merged
merged 39 commits into from
Dec 4, 2024

Conversation

xsxszab
Copy link

@xsxszab xsxszab commented Dec 3, 2024

Summary

This PR synchronizes updates from PR #166 and applies them to mut function contracts and proof for contracts. Since most reviews are applicable to both const and mut versions, please review PR #166 first and provide feedbacks there. Any common updates to PR #166 will be incorporated into this PR afterward.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

xsxszab and others added 30 commits October 7, 2024 12:44
implemented integer type proof for contract for fn add, sub and offset
Combines macros for different types.
…allocation api, modified their proof for harness accordingly
Copy link
Member

@tautschnig tautschnig left a comment

Choose a reason for hiding this comment

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

Thank you!

Copy link

@feliperodri feliperodri left a comment

Choose a reason for hiding this comment

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

Thank you!

@feliperodri feliperodri enabled auto-merge (squash) December 3, 2024 18:48
@tautschnig tautschnig disabled auto-merge December 4, 2024 17:37
@tautschnig tautschnig enabled auto-merge (squash) December 4, 2024 17:37
@tautschnig tautschnig merged commit 2338dad into model-checking:main Dec 4, 2024
9 checks passed
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.

5 participants