Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Avoid some Place clones. #53551

Merged
merged 1 commit into from Aug 21, 2018

Conversation

Projects
None yet
5 participants
@nnethercote
Copy link
Contributor

nnethercote commented Aug 21, 2018

This is a 0.5% speedup on ripgrep.

r? @nikomatsakis

Avoid some Place clones.
This is a 0.5% speedup on ripgrep.
@varkor

This comment has been minimized.

Copy link
Member

varkor commented Aug 21, 2018

@bors r+ rollup

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Aug 21, 2018

📌 Commit c0636ab has been approved by varkor

kennytm added a commit to kennytm/rust that referenced this pull request Aug 21, 2018

Rollup merge of rust-lang#53551 - nnethercote:access_place_error_repo…
…rted, r=varkor

Avoid some Place clones.

This is a 0.5% speedup on ripgrep.

bors added a commit that referenced this pull request Aug 21, 2018

Auto merge of #53530 - kennytm:rollup, r=kennytm
Rollup of 17 pull requests

Successful merges:

 - #53030 (Updated RELEASES.md for 1.29.0)
 - #53104 (expand the documentation on the `Unpin` trait)
 - #53213 (Stabilize IP associated constants)
 - #53296 (When closure with no arguments was expected, suggest wrapping)
 - #53329 (Replace usages of ptr::offset with ptr::{add,sub}.)
 - #53363 (add individual docs to `core::num::NonZero*`)
 - #53370 (Stabilize macro_vis_matcher)
 - #53393 (Mark libserialize functions as inline)
 - #53405 (restore the page title after escaping out of a search)
 - #53452 (Change target triple used to check for lldb in build-manifest)
 - #53462 (Document Box::into_raw returns non-null ptr)
 - #53465 (Remove LinkMeta struct)
 - #53492 (update lld submodule to include RISCV patch)
 - #53496 (Fix typos found by codespell.)
 - #53521 (syntax: Optimize some literal parsing)
 - #53540 (Moved issue-53157.rs into src/test/ui/consts/const-eval/)
 - #53551 (Avoid some Place clones.)

Failed merges:

r? @ghost

@bors bors merged commit c0636ab into rust-lang:master Aug 21, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@nnethercote nnethercote deleted the nnethercote:access_place_error_reported branch Aug 21, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.