Skip to content

Update toolchain to 2024-08-05#3416

Merged
tautschnig merged 6 commits intomodel-checking:mainfrom
tautschnig:toolchain-2024-08-05-manual
Aug 5, 2024
Merged

Update toolchain to 2024-08-05#3416
tautschnig merged 6 commits intomodel-checking:mainfrom
tautschnig:toolchain-2024-08-05-manual

Conversation

@tautschnig
Copy link
Member

@tautschnig tautschnig commented Aug 5, 2024

The toolchain upgrade to 2024-08-04 includes several bugfixes to dead-code analysis in rustc, explaining why we the recent PRs as listed above weren't flagged before for introducing dead code.

Resolves: #3411

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

* Remove `gen_function_local_variable` and `initializer_fn_name`, the
  last uses of both of which were removed in model-checking#3305.
* Remove `arg_count`, which was introduced in model-checking#3363, but not actually
  used.
* Remove `insert_bb`, which was introduced in model-checking#3382, but not actually
  used.

The toolchain upgrade to 2024-08-04 includes several bugfixes to
dead-code analysis in rustc, explaining why we the recent PRs as listed
above weren't flagged before for introducing dead code.

Resolves: model-checking#3411
@tautschnig tautschnig requested a review from a team as a code owner August 5, 2024 08:33
@github-actions github-actions bot added the Z-EndToEndBenchCI Tag a PR to run benchmark CI label Aug 5, 2024
@tautschnig tautschnig enabled auto-merge August 5, 2024 10:01
@tautschnig tautschnig disabled auto-merge August 5, 2024 10:02
@artemagvanian
Copy link
Contributor

Is it possible to temporarily mark insert_bb and arg_count as allow_unused instead of removing them altogether? They were added in separate PRs to make it easier to review, and I plan to use them in the next PR I am authoring.

@tautschnig
Copy link
Member Author

Is it possible to temporarily mark insert_bb and arg_count as allow_unused instead of removing them altogether? They were added in separate PRs to make it easier to review, and I plan to use them in the next PR I am authoring.

Done.

@tautschnig tautschnig added this pull request to the merge queue Aug 5, 2024
Merged via the queue into model-checking:main with commit 520cb3e Aug 5, 2024
@tautschnig tautschnig deleted the toolchain-2024-08-05-manual branch August 5, 2024 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Z-EndToEndBenchCI Tag a PR to run benchmark CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Toolchain upgrade to nightly-2024-08-04 failed

3 participants