Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Upgrade rust toolchain to 2023-04-30 #2456

Merged
merged 2 commits into from
Jun 9, 2023

Conversation

zhassan-aws
Copy link
Contributor

Description of changes:

Push rust toolchain by one day to 2023-04-30.

Resolved issues:

Resolves #2451

Related RFC:

Optional #ISSUE-NUMBER.

Call-outs:

Testing:

  • How is this change tested? Current regressions

  • Is this a refactor change? No

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.

@zhassan-aws
Copy link
Contributor Author

This is blocked by #2457.

tautschnig added a commit to tautschnig/cbmc that referenced this pull request May 30, 2023
With model-checking/kani#2456 we see examples
where a pointer of a different type is byte-extracted from a union. This
is caused by Rust's niche placement. `get_subexpression_at_offset`
already catered for that, but we didn't use it in field expansion.
@tautschnig
Copy link
Member

See #2457 (comment) for the link to the solution on the CBMC side. We might have to hold back on this upgrade until after the next CBMC release (due 2023-06-08).

tautschnig added a commit to tautschnig/cbmc that referenced this pull request May 31, 2023
With model-checking/kani#2456 we see examples
where a pointer of a different type is byte-extracted from a union. This
is caused by Rust's niche placement. `get_subexpression_at_offset`
already catered for that, but we didn't use it in field expansion.
tautschnig added a commit to tautschnig/cbmc that referenced this pull request May 31, 2023
With model-checking/kani#2456 we see examples
where a pointer of a different type is byte-extracted from a union. This
is caused by Rust's niche placement. `get_subexpression_at_offset`
already catered for that, but we didn't use it in field expansion.
@tautschnig
Copy link
Member

CBMC 5.85.0 has been released. A CBMC version bump PR should make this upgrade feasible.

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.

We really need to refactor this test... sorry

@zhassan-aws zhassan-aws marked this pull request as ready for review June 9, 2023 18:48
@zhassan-aws zhassan-aws requested a review from a team as a code owner June 9, 2023 18:48
@zhassan-aws zhassan-aws enabled auto-merge (squash) June 9, 2023 18:52
@zhassan-aws zhassan-aws merged commit 59b4759 into model-checking:main Jun 9, 2023
@zhassan-aws zhassan-aws deleted the toolchain-2023-04-30 branch June 9, 2023 21:23
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update rust toolchain version to nightly-2023-04-30
4 participants