Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Prepare Rust 1.80.1 point release #128635

Merged
merged 7 commits into from
Aug 6, 2024
Merged

Conversation

pietroalbini
Copy link
Member

@pietroalbini pietroalbini commented Aug 4, 2024

Noratrieb and others added 4 commits August 4, 2024 12:31
Jump threading stores values as `u128` (`ScalarInt`) and does its
comparisons for equality as integer comparisons.
This works great for integers. Sadly, not everything is an integer.

Floats famously have wonky equality semantcs, with `NaN!=NaN` and
`0.0 == -0.0`. This does not match our beautiful integer bitpattern
equality and therefore causes things to go horribly wrong.

While jump threading could be extended to support floats by remembering
that they're floats in the value state and handling them properly,
it's signficantly easier to just disable it for now.
…9, r=petrochenkov"

This reverts commit 977c5fd, reversing
changes made to 24c94f0.
…nce, r=pnkfelix"

This reverts commit 13314df, reversing
changes made to 6e534c7.
@rustbot

This comment was marked as resolved.

@rustbot

This comment was marked as resolved.

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-release Relevant to the release subteam, which will review and decide on the PR/issue. labels Aug 4, 2024
@pietroalbini

This comment was marked as resolved.

@rustbot

This comment was marked as resolved.

@pietroalbini pietroalbini marked this pull request as ready for review August 6, 2024 08:00
@rustbot
Copy link
Collaborator

rustbot commented Aug 6, 2024

Some changes occurred to MIR optimizations

cc @rust-lang/wg-mir-opt

@pietroalbini
Copy link
Member Author

@bors r+ p=1000 rollup=never

@bors
Copy link
Contributor

bors commented Aug 6, 2024

📌 Commit 403ca80 has been approved by pietroalbini

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 6, 2024
@bors
Copy link
Contributor

bors commented Aug 6, 2024

⌛ Testing commit 403ca80 with merge 04de459...

bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 6, 2024
Prepare Rust 1.80.1 point release

The point release is scheduled to include:

* rust-lang#128271
* rust-lang#128618
@rust-log-analyzer

This comment has been minimized.

@bors
Copy link
Contributor

bors commented Aug 6, 2024

💔 Test failed - checks-actions

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 6, 2024
@pietroalbini
Copy link
Member Author

@bors retry

Spurious failure.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 6, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 6, 2024
Prepare Rust 1.80.1 point release

The point release is scheduled to include:

* rust-lang#128271
* rust-lang#128618
@bors
Copy link
Contributor

bors commented Aug 6, 2024

⌛ Testing commit 403ca80 with merge 644b2cf...

@bors
Copy link
Contributor

bors commented Aug 6, 2024

💥 Test timed out

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 6, 2024
@matthiaskrgr
Copy link
Member

@bors ping

@bors
Copy link
Contributor

bors commented Aug 6, 2024

😪 I'm awake I'm awake

@matthiaskrgr
Copy link
Member

@bors retry

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 6, 2024
@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels Aug 6, 2024
@pietroalbini
Copy link
Member Author

@bors r+ p=1000 rollup=never

@bors
Copy link
Contributor

bors commented Aug 6, 2024

📌 Commit 1cc8da5 has been approved by pietroalbini

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Aug 6, 2024

⌛ Testing commit 1cc8da5 with merge 3f5fd8d...

@bors
Copy link
Contributor

bors commented Aug 6, 2024

☀️ Test successful - checks-actions
Approved by: pietroalbini
Pushing 3f5fd8d to stable...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Aug 6, 2024
@bors bors merged commit 3f5fd8d into rust-lang:stable Aug 6, 2024
7 checks passed
@rustbot rustbot added this to the 1.80.1 milestone Aug 6, 2024
@pietroalbini pietroalbini deleted the pa-1.80.1 branch August 6, 2024 21:35
@ysndr
Copy link

ysndr commented Aug 15, 2024

https://github.com/rust-lang/rust/releases/latest is still pointing to 1.80.0. Do you plan to make a Github Release for 1.80.1 as well?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. T-release Relevant to the release subteam, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants