Skip to content

Revert "Stabilize -Znext-solver=coherence" #130249

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

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

compiler-errors
Copy link
Member

@compiler-errors compiler-errors commented Sep 11, 2024

This is a clean revert of #121848, prepared by running:

$ git revert 17b322fa69eed7216dccc9f097eb68237cf62234 -m1

Which effectively reverts:

see: https://rust-lang.zulipchat.com/#narrow/stream/364551-t-types.2Ftrait-system-refactor/topic/nalgebra.20hang

Closes #130056

r? lcnr

@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. labels Sep 11, 2024
@rustbot
Copy link
Collaborator

rustbot commented Sep 11, 2024

Some changes occurred in engine.rs, potentially modifying the public API of ObligationCtxt.

cc @lcnr, @compiler-errors

@compiler-errors compiler-errors changed the title Revert 'Stabilize -Znext-solver=coherence' Revert "Stabilize -Znext-solver=coherence" Sep 11, 2024
@lcnr
Copy link
Contributor

lcnr commented Sep 11, 2024

r=me after CI is green

@bors rollup=never

@lcnr
Copy link
Contributor

lcnr commented Sep 11, 2024

Thanks for opening this PR!

@bors r+

@bors
Copy link
Collaborator

bors commented Sep 11, 2024

📌 Commit e866f8a has been approved by lcnr

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 Sep 11, 2024
@bors
Copy link
Collaborator

bors commented Sep 12, 2024

⌛ Testing commit e866f8a with merge f753bc7...

@bors
Copy link
Collaborator

bors commented Sep 12, 2024

☀️ Test successful - checks-actions
Approved by: lcnr
Pushing f753bc7 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Sep 12, 2024
@bors bors merged commit f753bc7 into rust-lang:master Sep 12, 2024
7 checks passed
@rustbot rustbot added this to the 1.83.0 milestone Sep 12, 2024
@rust-timer
Copy link
Collaborator

Finished benchmarking commit (f753bc7): comparison URL.

Overall result: ❌✅ regressions and improvements - ACTION NEEDED

Next Steps: If you can justify the regressions found in this perf run, please indicate this with @rustbot label: +perf-regression-triaged along with sufficient written justification. If you cannot justify the regressions please open an issue or create a new PR that fixes the regressions, add a comment linking to the newly created issue or PR, and then add the perf-regression-triaged label to this PR.

@rustbot label: +perf-regression
cc @rust-lang/wg-compiler-performance

Instruction count

This is a highly reliable metric that was used to determine the overall result at the top of this comment.

mean range count
Regressions ❌
(primary)
3.7% [2.9%, 4.5%] 7
Regressions ❌
(secondary)
- - 0
Improvements ✅
(primary)
-0.5% [-0.9%, -0.2%] 13
Improvements ✅
(secondary)
-0.5% [-0.7%, -0.1%] 9
All ❌✅ (primary) 1.0% [-0.9%, 4.5%] 20

Max RSS (memory usage)

Results (primary -0.5%)

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

mean range count
Regressions ❌
(primary)
- - 0
Regressions ❌
(secondary)
- - 0
Improvements ✅
(primary)
-0.5% [-0.5%, -0.5%] 1
Improvements ✅
(secondary)
- - 0
All ❌✅ (primary) -0.5% [-0.5%, -0.5%] 1

Cycles

Results (primary 3.7%, secondary 2.4%)

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

mean range count
Regressions ❌
(primary)
3.7% [2.9%, 4.7%] 6
Regressions ❌
(secondary)
2.4% [2.1%, 2.6%] 4
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
- - 0
All ❌✅ (primary) 3.7% [2.9%, 4.7%] 6

Binary size

This benchmark run did not return any relevant results for this metric.

Bootstrap: 758.873s -> 759.185s (0.04%)
Artifact size: 341.30 MiB -> 341.20 MiB (-0.03%)

@rustbot rustbot added the perf-regression Performance regression. label Sep 12, 2024
@Mark-Simulacrum Mark-Simulacrum added the perf-regression-triaged The performance regression has been triaged. label Sep 16, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
merged-by-bors This PR was explicitly merged by bors. perf-regression Performance regression. perf-regression-triaged The performance regression has been triaged. 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Regression in nightly-2024-09-07: hang affecting nalgebra
6 participants