Update rust toolchain to 2023-04-29#2452
Conversation
|
Keeping this as a draft because there are still a few failures in the |
|
All regressions now pass except for two perf regressions that are running out of memory 😞 |
|
On one of the BTreeSet performance tests ( The size of the program expression also increased > 5X: @tautschnig is there a way to pinpoint what is causing the difference? |
|
I narrowed it down to a change that happened between the 2023-04-29 and 2023-04-30 nightly toolchains: just changing the toolchain to 2023-04-29 in this PR restores performance on those btreeset tests. |
|
For the sake of making progress on the toolchain updates, I've backtracked the toolchain date by 1 day to 2023-04-29 to avoid the performance regression that occurs with the 2023-04-30 toolchain. The PR is now ready for review. |
celinval
left a comment
There was a problem hiding this comment.
Thanks Zyad! I just have questions about the changes to offset implementation.
Description of changes:
Update toolchain to 2023-04-29. Relevant PRs that required changes:
Removal of WithOptConstParam: rust-lang/rust#96840
Moving of IndexVec into its own module: rust-lang/rust#110539
Hashes use their own type instead of
u64: rust-lang/rust#110083New
OffsetOfvariant inrustc_middle::mir::NullOp: rust-lang/rust#106934Lowering of
offsetintrinsic toBinOp::Offset: rust-lang/rust#110822Also had to add an
isizesuffix to the second argument to theoffsetintrinsic due to rust-lang/rust#111699.Resolved issues:
Towards #2451
Related RFC:
Optional #ISSUE-NUMBER.
Call-outs:
Testing:
How is this change tested? Current regressions
Is this a refactor change? No
Checklist
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.