Skip to content

Conversation

@danielsn
Copy link
Contributor

@danielsn danielsn commented May 2, 2022

Description of changes:

Currently, there is no description of how UB interacts with kani verification in the docs.
Add one

Resolved issues:

Resolves #936

Call-outs:

Testing:

  • How is this change tested? Documentation change

  • Is this a refactor change? Documentation change

Checklist

  • Each commit message has a non-empty body, explaining why the change was made
  • Methods or procedures are documented
  • Regression or unit tests are included, or existing tests cover the modified code
  • My PR is restricted to a single feature or bugfix

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

@danielsn danielsn requested a review from a team as a code owner May 2, 2022 18:59
@danielsn danielsn force-pushed the UB branch 2 times, most recently from b93e268 to fea8c2f Compare May 2, 2022 19:07
Copy link
Contributor

@celinval celinval left a comment

Choose a reason for hiding this comment

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

It looks good to me. Can you please get a second review before pushing please? Thanks

Copy link
Contributor

@adpaco-aws adpaco-aws left a comment

Choose a reason for hiding this comment

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

Please edit SUMMARY.md to replace the "Guarantees" section (which is empty) with this one. Otherwise this will not be accessible from the index.

@danielsn danielsn merged commit 11ff580 into model-checking:main May 3, 2022
@danielsn danielsn deleted the UB branch May 3, 2022 15:17
tautschnig added a commit to tautschnig/kani that referenced this pull request Jul 29, 2025
Cleanup of documentation previously introduced in model-checking#1141:

1. No need to repeatedly discuss compiler intrinsics.
2. I am not aware of invalid dereferences that we would _not_ catch,
   even place expressions.
github-merge-queue bot pushed a commit that referenced this pull request Jul 29, 2025
Cleanup of documentation previously introduced in #1141:

1. No need to repeatedly discuss compiler intrinsics.
2. I am not aware of invalid dereferences that we would _not_ catch,
even place expressions.

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 and MIT licenses.
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.

Clear documentation on what checks are in/out of scope

5 participants