Skip to content

[wip] Link to alloc_system by default #55202

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 1 commit into from

Conversation

alexcrichton
Copy link
Member

@alexcrichton alexcrichton commented Oct 19, 2018

I'm curious to see the perf impact we have from moving away from jemalloc, so using this as a placeholder to run a perf run!

@alexcrichton alexcrichton changed the title Link to alloc_system by default [wip] Link to alloc_system by default Oct 19, 2018
@rust-highfive
Copy link
Contributor

r? @oli-obk

(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 Oct 19, 2018
@alexcrichton
Copy link
Member Author

r? @alexcrichton

@bors: try

@bors
Copy link
Collaborator

bors commented Oct 19, 2018

⌛ Trying commit 635cb31 with merge 7e7928d...

bors added a commit that referenced this pull request Oct 19, 2018
[wip] Link to alloc_system by default

I'm curious to see the perf impact we have from moving away from jemalloc, so using this as a placeholder to run a perf run!
@bors
Copy link
Collaborator

bors commented Oct 19, 2018

☀️ Test successful - status-travis
State: approved= try=True

@alexcrichton

This comment has been minimized.

@alexcrichton
Copy link
Member Author

@rust-timer build 7e7928d

@rust-timer
Copy link
Collaborator

Success: Queued 7e7928d with parent 74ff7dc, comparison URL.

@rust-timer
Copy link
Collaborator

Finished benchmarking try commit 7e7928d

@alexcrichton
Copy link
Member Author

Mischief managed! (data collected)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants