Rollup of 9 pull requests #36635
Closed
Rollup of 9 pull requests #36635
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
Currently `TypedArena` allocates its first chunk, which is usually 4096 bytes, as soon as it is created. If no allocations are ever made from the arena then this allocation (and the corresponding deallocation) is wasted effort. This commit changes `TypedArena` so it doesn't allocate the first chunk until the first allocation is made. This change speeds up rustc by a non-trivial amount because rustc uses `TypedArena` heavily: compilation speed (producing debug builds) on several of the rustc-benchmarks increases by 1.02--1.06x. The change should never cause a slow-down because the hot `alloc` function is unchanged. It does increase the size of `TypedArena` by one `usize` field, however. The commit also fixes some out-of-date comments.
…rner Updated E0560 to new error format Added a small bit of code to add a label for E0560. Also renamed src/test/compile-fail/E560.rs file to E0560.rs (not sure why it was named E560). Updated all test cases which check this error. Closes rust-lang#36199
fix top level attr spans issue rust-lang#36530 ?r @jonathandturner
…cs, r=GuillaumeGomez Tweak std::rc docs Fixes rust-lang#29372. r? @GuillaumeGomez
…bnik Replace 'e.g.' by 'i.e.' Fixes rust-lang#36577. r? @steveklabnik
…, r=nrc Add the ability to merge spans to codemap This PR adds the ability to merge Spans. To do so, it builds on the Codemap's ability to verify the locations of spans, namely that following can be verified: * the expn_id of both spans much match * the lhs span needs to end on the same line the rhs span begins * the lhs span must start at or before the rhs span If all of these are met, a new span is returned that is min(lo), max(hi) of the two spans. This PR also removes an older Span merge, as this new functionality subsumes it. r? @nrc
…_fix, r=bluss fixed the safety header/wording in option.rs Fixes rust-lang#36581 screenshot of the rendered documentation: http://imgur.com/14kLVrA r? @steveklabnik
Lazily allocate TypedArena's first chunk Currently `TypedArena` allocates its first chunk, which is usually 4096 bytes, as soon as it is created. If no allocations are ever made from the arena then this allocation (and the corresponding deallocation) is wasted effort. This commit changes `TypedArena` so it doesn't allocate the first chunk until the first allocation is made. This change speeds up rustc by a non-trivial amount because rustc uses `TypedArena` heavily: compilation speed (producing debug builds) on several of the rustc-benchmarks increases by 1.02--1.06x. The change should never cause a slow-down because the hot `alloc` function is unchanged. It does increase the size of `TypedArena` by one `usize` field, however. The commit also fixes some out-of-date comments.
…pasel422 Minor correction in `sort_by_key` doc comment The parameter to `sort_by_key` is named `f`, not `key`.
Implement Debug for DirEntry. None
|
@bors r+ p=1 |
|
📌 Commit 9b25f10 has been approved by |
|
⌛ Testing commit 9b25f10 with merge 4703011... |
|
(rust_highfive has picked a reviewer for you, use r? to override) |
|
💔 Test failed - auto-win-gnu-32-opt-rustbuild |
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.
sort_by_keydoc comment #36600, Implement Debug for DirEntry. #36631