Skip to content

Rollup of 7 pull requests - #100151

Merged
bors merged 20 commits into
rust-lang:masterfrom
matthiaskrgr:rollup-irqwvj2
Aug 4, 2022
Merged

Rollup of 7 pull requests#100151
bors merged 20 commits into
rust-lang:masterfrom
matthiaskrgr:rollup-irqwvj2

Conversation

@matthiaskrgr

Copy link
Copy Markdown
Member

Successful merges:

Failed merges:

r? @ghost
@rustbot modify labels: rollup

Create a similar rollup

compiler-errorsand others added 20 commits July 23, 2022 17:52
…=estebank
Do not exclusively suggest `;` when `,` is also a choice
Fixesrust-lang#96791
…=Mark-Simulacrum
Re-enable submodule archive downloads.
This is effectively a revert of rust-lang#98423 (though it keeps the `--depth 1` flag since that is still helpful).
GitHub has indicated that they have been working on the original issue, and my testing shows that the llvm-project archive download now succeeds 100% of the time.
This should save about a minute on every job.
…ing-argument-instead-of-format-args-capture, r=estebank
Suggest a positional formatting argument instead of a captured argument
This patch fixes a part of rust-lang#96999.
fixesrust-lang#98241fixesrust-lang#97311
r? `@estebank`
…tch-arms, r=petrochenkov
Enable unused_parens for match arms
Fixes: rust-lang#92751
Currently I can't get the `stderr` to work with `./x.py test`, but this should fix the issue. Help would be appreciated!
More EarlyBinder cleanups
Each commit is independent
r? types
…otriddle
Remove more Clean trait implementations
Follow-up of rust-lang#99638.
r? `@notriddle`
RustWrapper: update for TypedPointerType in LLVM
This is a result of https://reviews.llvm.org/D130592.
r? `@nikic`
@rustbotrustbot added 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-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. rollup A PR which is a rollup labels Aug 4, 2022
@matthiaskrgr

Copy link
Copy Markdown
MemberAuthor

@bors r+ rollup=never p=7

@bors

bors commented Aug 4, 2022

Copy link
Copy Markdown
Collaborator

📌 Commit c2d7321 has been approved by matthiaskrgr

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 4, 2022
@bors

bors commented Aug 4, 2022

Copy link
Copy Markdown
Collaborator

⌛ Testing commit c2d7321 with merge f6f9d5e...

@bors

bors commented Aug 4, 2022

Copy link
Copy Markdown
Collaborator

☀️ Test successful - checks-actions
Approved by: matthiaskrgr
Pushing f6f9d5e to master...

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

Copy link
Copy Markdown
Collaborator

@rust-timer

Copy link
Copy Markdown
Collaborator

Finished benchmarking commit (f6f9d5e): 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 improvement found
mean1maxcount2
Regressions 😿
(primary)
N/AN/A0
Regressions 😿
(secondary)
N/AN/A0
Improvements 🎉
(primary)
N/AN/A0
Improvements 🎉
(secondary)
-2.9%-2.9%1
All 😿🎉 (primary)N/AN/A0

Cycles

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

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

@matthiaskrgr
matthiaskrgr deleted the rollup-irqwvj2 branch October 9, 2022 00:31
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-infraRelevant to the infrastructure 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

@matthiaskrgr@bors@rust-timer@rustbot@compiler-errors@ehuss@TaKO8Ki@jackh726@GuillaumeGomez@wcampbell0x2a@durin42