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

Assoc item cleanup #90648

Merged
merged 1 commit into from
Nov 11, 2021
Merged

Assoc item cleanup #90648

merged 1 commit into from
Nov 11, 2021

Conversation

matthewjasper
Copy link
Contributor

@matthewjasper matthewjasper commented Nov 6, 2021

This removes some fields from ObligationCauseCode

Split out of #90639

@rust-highfive
Copy link
Collaborator

r? @jackh726

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 6, 2021
@cjgillot
Copy link
Contributor

cjgillot commented Nov 6, 2021

@bors try @rust-timer queue

@rust-timer
Copy link
Collaborator

Awaiting bors try build completion.

@rustbot label: +S-waiting-on-perf

@rustbot rustbot added the S-waiting-on-perf Status: Waiting on a perf run to be completed. label Nov 6, 2021
@cjgillot cjgillot requested a review from oli-obk November 6, 2021 13:50
@bors
Copy link
Contributor

bors commented Nov 6, 2021

⌛ Trying commit 37f8b7dfccd88f38abd91013092fbc8b1c7dea3e with merge 5378c9f4bd027bfc59e71c557bc0e3eeb8db49a7...

@cjgillot cjgillot self-assigned this Nov 6, 2021
@cjgillot
Copy link
Contributor

cjgillot commented Nov 6, 2021

Sorry for the ping oli-obk, I mis-clicked.

@bors
Copy link
Contributor

bors commented Nov 6, 2021

☀️ Try build successful - checks-actions
Build commit: 5378c9f4bd027bfc59e71c557bc0e3eeb8db49a7 (5378c9f4bd027bfc59e71c557bc0e3eeb8db49a7)

@rust-timer
Copy link
Collaborator

Queued 5378c9f4bd027bfc59e71c557bc0e3eeb8db49a7 with parent 3cd3bbe, future comparison URL.

@rust-timer
Copy link
Collaborator

Finished benchmarking commit (5378c9f4bd027bfc59e71c557bc0e3eeb8db49a7): comparison url.

Summary: This change led to large relevant mixed results 🤷 in compiler performance.

  • Small improvement in instruction counts (up to -0.8% on incr-full builds of ctfe-stress-4)
  • Large regression in instruction counts (up to 2.3% on full builds of keccak)

If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf.

Benchmarking this pull request likely means that it is perf-sensitive, so we're automatically marking it as not fit for rolling up. While you can manually mark this PR as fit for rollup, we strongly recommend not doing so since this PR led to changes in compiler perf.

Next Steps: If you can justify the regressions found in this try perf run, please indicate this with @rustbot label: +perf-regression-triaged along with sufficient written justification. If you cannot justify the regressions please fix the regressions and do another perf run. If the next run shows neutral or positive results, the label will be automatically removed.

@bors rollup=never
@rustbot label: +S-waiting-on-review -S-waiting-on-perf +perf-regression

@rustbot rustbot added perf-regression Performance regression. and removed S-waiting-on-perf Status: Waiting on a perf run to be completed. labels Nov 6, 2021
@matthewjasper
Copy link
Contributor Author

@bors try @rust-timer queue

@rust-timer
Copy link
Collaborator

Awaiting bors try build completion.

@rustbot label: +S-waiting-on-perf

@rustbot rustbot added the S-waiting-on-perf Status: Waiting on a perf run to be completed. label Nov 6, 2021
@bors
Copy link
Contributor

bors commented Nov 6, 2021

⌛ Trying commit f7989bda6cb1efb9705f429990c740d245c162f0 with merge 15c0e0e6a71c795b8ee72fdd01b9fcd2ad894be3...

@bors
Copy link
Contributor

bors commented Nov 6, 2021

☀️ Try build successful - checks-actions
Build commit: 15c0e0e6a71c795b8ee72fdd01b9fcd2ad894be3 (15c0e0e6a71c795b8ee72fdd01b9fcd2ad894be3)

@rust-timer
Copy link
Collaborator

Queued 15c0e0e6a71c795b8ee72fdd01b9fcd2ad894be3 with parent 3326f19, future comparison URL.

@rust-timer
Copy link
Collaborator

Finished benchmarking commit (15c0e0e6a71c795b8ee72fdd01b9fcd2ad894be3): comparison url.

Summary: This change led to large relevant mixed results 🤷 in compiler performance.

  • Small improvement in instruction counts (up to -0.8% on full builds of ctfe-stress-4)
  • Large regression in instruction counts (up to 2.3% on full builds of keccak)

If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf.

Benchmarking this pull request likely means that it is perf-sensitive, so we're automatically marking it as not fit for rolling up. While you can manually mark this PR as fit for rollup, we strongly recommend not doing so since this PR led to changes in compiler perf.

Next Steps: If you can justify the regressions found in this try perf run, please indicate this with @rustbot label: +perf-regression-triaged along with sufficient written justification. If you cannot justify the regressions please fix the regressions and do another perf run. If the next run shows neutral or positive results, the label will be automatically removed.

@bors rollup=never
@rustbot label: +S-waiting-on-review -S-waiting-on-perf +perf-regression

@rustbot rustbot removed the S-waiting-on-perf Status: Waiting on a perf run to be completed. label Nov 6, 2021
@matthewjasper
Copy link
Contributor Author

@bors try @rust-timer queue

@rust-timer
Copy link
Collaborator

Awaiting bors try build completion.

@rustbot label: +S-waiting-on-perf

@rustbot rustbot added the S-waiting-on-perf Status: Waiting on a perf run to be completed. label Nov 7, 2021
@bors
Copy link
Contributor

bors commented Nov 7, 2021

☀️ Try build successful - checks-actions
Build commit: c12bd3a2dfbe2dbf507f94d78372a7bf87be2930 (c12bd3a2dfbe2dbf507f94d78372a7bf87be2930)

@rust-timer
Copy link
Collaborator

Queued c12bd3a2dfbe2dbf507f94d78372a7bf87be2930 with parent 0727994, future comparison URL.

@rust-timer
Copy link
Collaborator

Finished benchmarking commit (c12bd3a2dfbe2dbf507f94d78372a7bf87be2930): comparison url.

Summary: This change led to large relevant regressions 😿 in compiler performance.

  • Large regression in instruction counts (up to 2.3% on full builds of keccak)

If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf.

Benchmarking this pull request likely means that it is perf-sensitive, so we're automatically marking it as not fit for rolling up. While you can manually mark this PR as fit for rollup, we strongly recommend not doing so since this PR led to changes in compiler perf.

Next Steps: If you can justify the regressions found in this try perf run, please indicate this with @rustbot label: +perf-regression-triaged along with sufficient written justification. If you cannot justify the regressions please fix the regressions and do another perf run. If the next run shows neutral or positive results, the label will be automatically removed.

@bors rollup=never
@rustbot label: +S-waiting-on-review -S-waiting-on-perf +perf-regression

@rustbot rustbot removed the S-waiting-on-perf Status: Waiting on a perf run to be completed. label Nov 7, 2021
@matthewjasper
Copy link
Contributor Author

@bors try @rust-timer queue

@rust-timer
Copy link
Collaborator

Awaiting bors try build completion.

@rustbot label: +S-waiting-on-perf

@rustbot rustbot added the S-waiting-on-perf Status: Waiting on a perf run to be completed. label Nov 7, 2021
@bors
Copy link
Contributor

bors commented Nov 7, 2021

⌛ Trying commit 9734c03 with merge 73a04f9e6b6dda76aca299429e4784b13e6685d3...

@bors
Copy link
Contributor

bors commented Nov 7, 2021

☀️ Try build successful - checks-actions
Build commit: 73a04f9e6b6dda76aca299429e4784b13e6685d3 (73a04f9e6b6dda76aca299429e4784b13e6685d3)

@rust-timer
Copy link
Collaborator

Queued 73a04f9e6b6dda76aca299429e4784b13e6685d3 with parent 90a273b, future comparison URL.

@rust-timer
Copy link
Collaborator

Finished benchmarking commit (73a04f9e6b6dda76aca299429e4784b13e6685d3): comparison url.

Summary: This benchmark run did not return any relevant changes.

If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf.

Benchmarking this pull request likely means that it is perf-sensitive, so we're automatically marking it as not fit for rolling up. While you can manually mark this PR as fit for rollup, we strongly recommend not doing so since this PR led to changes in compiler perf.

@bors rollup=never
@rustbot label: +S-waiting-on-review -S-waiting-on-perf -perf-regression

@rustbot rustbot removed S-waiting-on-perf Status: Waiting on a perf run to be completed. perf-regression Performance regression. labels Nov 7, 2021
@cjgillot
Copy link
Contributor

r=me with the PR message updated.

@apiraino apiraino added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Nov 11, 2021
@matthewjasper
Copy link
Contributor Author

I was hoping to readd some changes back into this PR, but I guess this can be merged as is for now.
@bors r=cjgillot

@bors
Copy link
Contributor

bors commented Nov 11, 2021

📌 Commit 9734c03 has been approved by cjgillot

@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 Nov 11, 2021
@bors
Copy link
Contributor

bors commented Nov 11, 2021

⌛ Testing commit 9734c03 with merge 3d29b68...

@bors
Copy link
Contributor

bors commented Nov 11, 2021

☀️ Test successful - checks-actions
Approved by: cjgillot
Pushing 3d29b68 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Nov 11, 2021
@bors bors merged commit 3d29b68 into rust-lang:master Nov 11, 2021
@rustbot rustbot added this to the 1.58.0 milestone Nov 11, 2021
@matthewjasper matthewjasper deleted the assoc-item-cleanup branch November 11, 2021 18:35
@rust-timer
Copy link
Collaborator

Finished benchmarking commit (3d29b68): comparison url.

Summary: This benchmark run did not return any relevant changes.

If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf.

@rustbot label: -perf-regression

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants