Skip to content

Rollup of 6 pull requests - #100174

Merged
bors merged 14 commits into
rust-lang:masterfrom
Dylan-DPC:rollup-wnskbk6
Aug 5, 2022
Merged

Rollup of 6 pull requests#100174
bors merged 14 commits into
rust-lang:masterfrom
Dylan-DPC:rollup-wnskbk6

Conversation

@Dylan-DPC

Copy link
Copy Markdown
Member

Successful merges:

Failed merges:

r? @ghost
@rustbot modify labels: rollup

Create a similar rollup

TaKO8Kiand others added 14 commits July 28, 2022 09:55
…ref-for-binding-pattern, r=estebank
Suggest adding/removing `ref` for binding patterns
This fixes what a fixme comment says.
r? `@estebank`
…ckh726
Use `node_type_opt` to skip over generics that were not expected
Fixesrust-lang#100154
…d-of-repeated-push, r=Dylan-DPC
rustdoc: use `collect()` instead of repeatedly pushing
…-98246, r=joshtriplett
kmc-solid: Add a stub implementation of rust-lang#98246 (`File::set_times`)
Fixes the build failure of the [`*-kmc-solid_*`](https://doc.rust-lang.org/nightly/rustc/platform-support/kmc-solid.html) Tier 3 targets after rust-lang#98246.
This target does not support setting a modification time and access time separately, hence stubbing out the implementation.
…ylan-DPC
Remove more Clean trait implementations
Follow-up of rust-lang#99638.
r? `@notriddle`
…or_missing_type_in_a_const_item, r=compiler-errors
Improve diagnostics for `const a: = expr;`
Adds a suggestion to write a type when there is a colon, but the type is not present.
I've also shrunk spans a little, so the suggestions are a little nicer.
Resolvesrust-lang#100146
r? `@compiler-errors`
@rustbotrustbot added T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. rollup A PR which is a rollup labels Aug 5, 2022
@Dylan-DPC

Copy link
Copy Markdown
MemberAuthor

@bors r+ rollup=never p=5

@bors

bors commented Aug 5, 2022

Copy link
Copy Markdown
Collaborator

📌 Commit e7ed844 has been approved by Dylan-DPC

It is now in the queue for this repository.

@borsbors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Aug 5, 2022
@bors

bors commented Aug 5, 2022

Copy link
Copy Markdown
Collaborator

⌛ Testing commit e7ed844 with merge affe0d3...

@bors

bors commented Aug 5, 2022

Copy link
Copy Markdown
Collaborator

☀️ Test successful - checks-actions
Approved by: Dylan-DPC
Pushing affe0d3 to master...

@borsbors added the merged-by-bors This PR was explicitly merged by bors. label Aug 5, 2022
@bors
bors merged commit affe0d3 into rust-lang:masterAug 5, 2022
@rustbotrustbot added this to the 1.64.0 milestone Aug 5, 2022
@rust-timer

Copy link
Copy Markdown
Collaborator

📌 Perf builds for each rolled up PR:

PR#Perf Build Sha
#1001686ce4eb0a29d66b833f4ae426b515ff87b5f02aca
#1001660fb7c3d620edae78145bb57bcf128cb04326221e
#100158558a66aaf6ca137a8c67911a534a866ea0e908ff
#10015722fc0c340e326cccfd1da756b86d5afde242ab98
#100155ab4fb18c98601f8a6ed7c3b08599b3baa0af42e5
#99835372680824d80928e543a528abbd43bcfc88362e1

In the case of a perf regression, run the following command for each PR you suspect might be the cause: @rust-timer build $SHA

This was referenced Aug 5, 2022
@rust-timer

Copy link
Copy Markdown
Collaborator

Finished benchmarking commit (affe0d3): comparison url.

Instruction count

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

Max RSS (memory usage)

Results
  • Primary benchmarks: no relevant changes found
  • Secondary benchmarks: 😿 relevant regression found
mean1maxcount2
Regressions 😿
(primary)
N/AN/A0
Regressions 😿
(secondary)
2.1%2.1%1
Improvements 🎉
(primary)
N/AN/A0
Improvements 🎉
(secondary)
N/AN/A0
All 😿🎉 (primary)N/AN/A0

Cycles

Results
  • Primary benchmarks: 🎉 relevant improvement found
  • Secondary benchmarks: 🎉 relevant improvement found
mean1maxcount2
Regressions 😿
(primary)
N/AN/A0
Regressions 😿
(secondary)
N/AN/A0
Improvements 🎉
(primary)
-2.4%-2.4%1
Improvements 🎉
(secondary)
-2.5%-2.5%1
All 😿🎉 (primary)-2.4%-2.4%1

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

@rustbot label: -perf-regression

Footnotes

  1. the arithmetic mean of the percent change2

  2. number of relevant changes2

@ehussehuss modified the milestones: 1.64.0, 1.65.0Aug 31, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged-by-borsThis PR was explicitly merged by bors.rollupA PR which is a rollupS-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.T-libsRelevant to the library team, which will review and decide on the PR/issue.T-rustdocRelevant to the rustdoc team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

11 participants

@Dylan-DPC@bors@rust-timer@ehuss@rustbot@TaKO8Ki@kawadakk@compiler-errors@notriddle@GuillaumeGomez@WaffleLapkin