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

Rollup of 6 pull requests #69440

Merged
merged 13 commits into from
Feb 25, 2020
Merged

Rollup of 6 pull requests #69440

merged 13 commits into from
Feb 25, 2020

Conversation

Dylan-DPC-zz
Copy link

Successful merges:

Failed merges:

r? @ghost

wesleywiser and others added 13 commits February 17, 2020 06:41
…le_book, r=nikomatsakis

Add documentation for the `-Zself-profile` flag
…crum

Add rustdoc aliases to `ptr::copy` and `ptr::copy_nonoverlapping`

This PR adds a [rustdoc alias](https://doc.rust-lang.org/nightly/rustdoc/unstable-features.html#add-aliases-for-an-item-in-documentation-search) to `ptr::copy` and `ptr::copy_nonoverlapping` using the commonly used terms `memcpy` and `memmove`. The motivation for this PR is to improve discoverability for these functions, since I've noticed users overlook these functions on multiple occasions (and they have thus reached for [libc](https://crates.io/crates/libc) without need). Currently std docs state:

https://doc.rust-lang.org/nightly/std/ptr/fn.copy_nonoverlapping.html
> `copy_nonoverlapping` is semantically equivalent to C's `memcpy`, but with the argument order swapped.

https://doc.rust-lang.org/nightly/std/ptr/fn.copy.html
> `copy` is semantically equivalent to C's `memmove`, but with the argument order swapped.

#### search results before adding a rustdoc alias:
![screenshot 6517](https://user-images.githubusercontent.com/6868531/75102985-78fbb680-55c2-11ea-8e41-04979e6fa6f6.png)
![screenshot 6518](https://user-images.githubusercontent.com/6868531/75102984-78632000-55c2-11ea-9673-8822aae636d1.png)

#### after adding `#[doc(alias = "memcpy")]` and `#[doc(alias = "memmove")]`:
![screenshot 6516](https://user-images.githubusercontent.com/6868531/75102986-78fbb680-55c2-11ea-93b9-1929be940043.png)
![screenshot 6515](https://user-images.githubusercontent.com/6868531/75102987-78fbb680-55c2-11ea-9861-ce8a77a0c3b9.png)
don't explicitly compare against true or false
Replace uses of Cell::get + Cell::set with Cell::replace.
…c-morse

no more codegen for miri_start_panic

With rust-lang/miri#1136 landed, we don't generate code any more for crates that will be run by Miri. So the LLVM backend does not have to implement the `miri_start_panic` intrinsic any more.

Cc @Aaron1011
@Dylan-DPC-zz
Copy link
Author

@bors r+ rollup=never p=6

@bors
Copy link
Contributor

bors commented Feb 24, 2020

📌 Commit e238eb6 has been approved by Dylan-DPC

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Feb 24, 2020
@Dylan-DPC-zz Dylan-DPC-zz added the rollup A PR which is a rollup label Feb 24, 2020
@bors
Copy link
Contributor

bors commented Feb 24, 2020

⌛ Testing commit e238eb6 with merge 541650abffc588bebe6496d2fefbf11143b8115d...

@Centril
Copy link
Contributor

Centril commented Feb 24, 2020

@bors retry yielding to stable PR

@Centril
Copy link
Contributor

Centril commented Feb 25, 2020

@bors treeclosed-

@bors
Copy link
Contributor

bors commented Feb 25, 2020

⌛ Testing commit e238eb6 with merge b7544f3b1e35d7b51eabadb97dc959ab7d02e581...

@bors
Copy link
Contributor

bors commented Feb 25, 2020

💥 Test timed out

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Feb 25, 2020
@JohnTitor
Copy link
Member

@bors retry

@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 Feb 25, 2020
@bors
Copy link
Contributor

bors commented Feb 25, 2020

⌛ Testing commit e238eb6 with merge e3a2779...

@bors
Copy link
Contributor

bors commented Feb 25, 2020

☀️ Test successful - checks-azure
Approved by: Dylan-DPC
Pushing e3a2779 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Feb 25, 2020
@bors bors merged commit e3a2779 into rust-lang:master Feb 25, 2020
@rust-highfive
Copy link
Collaborator

📣 Toolstate changed by #69440!

Tested on commit e3a2779.
Direct link to PR: #69440

💔 miri on windows: test-pass → test-fail (cc @oli-obk @eddyb @RalfJung).
💔 miri on linux: test-pass → test-fail (cc @oli-obk @eddyb @RalfJung).

rust-highfive added a commit to rust-lang-nursery/rust-toolstate that referenced this pull request Feb 25, 2020
Tested on commit rust-lang/rust@e3a2779.
Direct link to PR: <rust-lang/rust#69440>

💔 miri on windows: test-pass → test-fail (cc @oli-obk @eddyb @RalfJung).
💔 miri on linux: test-pass → test-fail (cc @oli-obk @eddyb @RalfJung).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. rollup A PR which is a rollup 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.