Rollup of 7 pull requests #86788
Closed
Rollup of 7 pull requests #86788
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
For example, drop glue generated for struct below, doesn't have any
statements, only terminators. Previously it received an estimate of 0,
the new estimate is 13 (6+5 drop terminators, +1 resume, +1 return).
struct S {
a: String,
b: String,
c: String,
d: String,
e: String,
f: String,
}
Originally reported in rust-lang#69382 (comment)
This commit adds a test case for generics, re-adds generics data to the search index, and tweaks function indexing to use less space in JSON. This reverts commit 14ca894.
…c, r=GuillaumeGomez fix(rustdoc): generics search This commit adds a test case for generics, re-adds generics data to the search index, and tweaks function indexing to use less space in JSON. This partially reverts commit 14ca894.
Add myself to mailmap
…schievink Test for const trait impls behind feature gates - Make the previous cross-crate tests use revisions instead of being separate files - Added test for gating const trait impls. cc `@oli-obk` `@jonas-schievink`
…ebank
Include terminators in instance size estimate
For example, drop glue generated for struct below, doesn't have any
statements, only terminators. Previously it received an estimate of 0,
the new estimate is 13 (6+5 drop terminators, +1 resume, +1 return).
```rust
struct S {
a: String,
b: String,
c: String,
d: String,
e: String,
f: String,
}
```
Originally reported in rust-lang#69382 (comment)
…acrum Allow anyone to add or remove any label starting with perf- In order to fully realize planned changes to the performance tracking process, we'd like to allow anyone to change the perf related labels. r? `@Mark-Simulacrum`
Move Mutex::unlock to T: ?Sized r? `@mbrubeck` cc rust-lang#81872
proc_macro/bridge: Remove dead code Slice type See rust-lang#85390 (comment)
|
@bors r+ p=7 rollup=never |
|
📌 Commit 5d7a793 has been approved by |
bors
added
the
S-waiting-on-bors
|
#86777 should not be rolled up since it could affect perf. Can we redo this rollup without that PR? |
|
Sure! |
bors
added
S-waiting-on-author
|
Thanks @JohnTitor! 🙂 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Successful merges:
Failed merges:
r? @ghost
@rustbot modify labels: rollup
Create a similar rollup