Skip to content

Conversation

ljedrz
Copy link
Contributor

@ljedrz ljedrz commented Nov 30, 2018

  • speed up TypedArena::clear: improves its performance by up to 33% (in case of a single entry)
  • simplify DroplessArena::in_arena

@rust-highfive
Copy link
Contributor

r? @sfackler

(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 30, 2018
@ljedrz
Copy link
Contributor Author

ljedrz commented Dec 2, 2018

Comments addressed.

@nagisa
Copy link
Member

nagisa commented Dec 2, 2018

@bors r+

@bors
Copy link
Collaborator

bors commented Dec 2, 2018

📌 Commit 95f32f1 has been approved by nagisa

@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 Dec 2, 2018
@bors
Copy link
Collaborator

bors commented Dec 2, 2018

⌛ Testing commit 95f32f1 with merge 9abc231...

bors added a commit that referenced this pull request Dec 2, 2018
arena: speed up TypedArena::clear and improve common patterns

- speed up `TypedArena::clear`: improves its performance by up to **33%** (in case of a single entry)
- simplify `DroplessArena::in_arena`
@bors
Copy link
Collaborator

bors commented Dec 2, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: nagisa
Pushing 9abc231 to master...

@bors bors merged commit 95f32f1 into rust-lang:master Dec 2, 2018
@ljedrz ljedrz deleted the arena_tweaks branch December 2, 2018 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants