Skip to content

[do not merge] Use Arc instead of Rc #59641

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
Closed

Conversation

Zoxc
Copy link
Contributor

@Zoxc Zoxc commented Apr 2, 2019

Checking performance overhead of doing so.

@rust-highfive
Copy link
Contributor

r? @pnkfelix

(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 Apr 2, 2019
@Zoxc
Copy link
Contributor Author

Zoxc commented Apr 2, 2019

@bors try

@bors
Copy link
Collaborator

bors commented Apr 2, 2019

⌛ Trying commit a0638d9 with merge 7cf1f9951d16ffa79d81073c73dc130a3f78f3c2...

@Zoxc
Copy link
Contributor Author

Zoxc commented Apr 2, 2019

@rust-timer build 7cf1f9951d16ffa79d81073c73dc130a3f78f3c2

@rust-timer
Copy link
Collaborator

Success: Queued 7cf1f9951d16ffa79d81073c73dc130a3f78f3c2 with parent 428943c, comparison URL.

@rust-timer
Copy link
Collaborator

Finished benchmarking try commit 7cf1f9951d16ffa79d81073c73dc130a3f78f3c2

# 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