Skip to content

internal: Cleanup code for old Rust versions - #22914

Merged
ChayimFriedman2 merged 6 commits into
rust-lang:masterfrom
ChayimFriedman2:remove-old-versions
Jul 26, 2026
Merged

internal: Cleanup code for old Rust versions#22914
ChayimFriedman2 merged 6 commits into
rust-lang:masterfrom
ChayimFriedman2:remove-old-versions

Conversation

@ChayimFriedman2

@ChayimFriedman2 ChayimFriedman2 commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

That we no longer support after #22876.

Reviewing commit-by-commit is recommended.

It was replaced with an intrinsic in rust-lang/rust#135046, released in 1.86.0.

And add MIR eval support for the intrinsic.
Required since 1.88.0.
Renamed to `align_of[_val]()` in 1.89.0.
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 25, 2026
@Veykril

Veykril commented Jul 26, 2026

Copy link
Copy Markdown
Member

That bump might've been a bit too aggressive by us as 1.94 is only from march.

@ChayimFriedman2

Copy link
Copy Markdown
Contributor Author

Maybe, but I'm actually quite happy about it. It lets us cleanup a lot of code. Since Rust 1.94.0 was released more than 4 months have passed. If you have a legitimate reason to use such old version (and remember we only officially support the last version), you can very well downgrade your r-a. I believe most people using this toolchain just didn't see a reason to upgrade, so we gave them it 😸

@ChayimFriedman2
ChayimFriedman2 added this pull request to the merge queue Jul 26, 2026
@Veykril

Veykril commented Jul 26, 2026

Copy link
Copy Markdown
Member

Yea, in this case the code we get rid of is nice. I am fine with merging either way (since otherwise we'd need to revert some PRs anyways), just something to note down.

Merged via the queue into rust-lang:master with commit 0eb393b Jul 26, 2026
18 checks passed
@ChayimFriedman2
ChayimFriedman2 deleted the remove-old-versions branch July 26, 2026 15:23
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 26, 2026
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.

3 participants