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 #57592

Merged
merged 14 commits into from
Jan 14, 2019
Merged

Rollup of 6 pull requests #57592

merged 14 commits into from
Jan 14, 2019

Conversation

Centril
Copy link
Contributor

@Centril Centril commented Jan 14, 2019

Successful merges:

Failed merges:

r? @ghost

lqd and others added 14 commits January 7, 2019 19:48
Use the `collect_and_partition_mono_items ` query to avoid calling the collector
This function has no side effects, and its result is ignored.
Include the kind of the binding that we're suggesting, and use a
structured suggestion.
It requires an unreachable IP address, but there is no such thing, and
this has caused it to fail for multiple people.

Fixes rust-lang#44698, fixes rust-lang#50065.
…oerister

Parallelize and optimize parts of HIR map creation

r? @michaelwoerister
…ster

MetadataOnlyCodegenBackend: run the collector only once

Use the `collect_and_partition_mono_items` query to avoid calling the collector directly twice.

Fixes rust-lang#57406.
…dor, r=joshtriplett,Centril

Stabilize cfg_target_vendor

This stabilizes the use of `cfg(target_vendor = "...")` and removes the corresponding `cfg_target_vendor` feature. Other unstable cfgs remain behind their existing feature gates.

This functionality was added back in 2015 in rust-lang#28612 to complete the coverage of target tuples (`<arch><sub>-<vendor>-<os>-<env>`). [RFC 131](https://github.com/rust-lang/rfcs/blob/master/text/0131-target-specification.md) governs the target specification, not including `target_vendor` seems to have just been an oversight. `target_os`, `target_family`, and `target_arch` are stable as of 1.0.0. `target_env` was also not mentioned in RFC 131, was added in rust-lang#24777, never behind a feature_gate, and insta-stable at 1.1.0.

The functionality is tested in [test/run-pass/cfg/cfg-target-vendor.rs](https://github.com/rust-lang/rust/blob/master/src/test/run-pass/cfg/cfg-target-vendor.rs).

Closes rust-lang#29718
…ckmdavis

clarify resolve typo suggestion

Include the kind of the binding that we're suggesting, and use a
structured suggestion.

Fixes rust-lang#53445.
privacy: Fix private-in-public check for existential types

Fixes rust-lang#53546 (regression from rust-lang#56878)

r? @arielb1
…utable, r=sfackler

Remove the `connect_timeout_unroutable` test.

It requires an unreachable IP address, but there is no such thing, and
this has caused it to fail for multiple people.

Fixes rust-lang#44698, fixes rust-lang#50065.

r? @sfackler
@Centril
Copy link
Contributor Author

Centril commented Jan 14, 2019

@bors r+ p=6

@bors
Copy link
Contributor

bors commented Jan 14, 2019

📌 Commit 47ccf2a has been approved by Centril

@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 Jan 14, 2019
@bors
Copy link
Contributor

bors commented Jan 14, 2019

⌛ Testing commit 47ccf2a with merge d106808...

bors added a commit that referenced this pull request Jan 14, 2019
Rollup of 6 pull requests

Successful merges:

 - #57232 (Parallelize and optimize parts of HIR map creation)
 - #57418 (MetadataOnlyCodegenBackend: run the collector only once)
 - #57465 (Stabilize cfg_target_vendor)
 - #57477 (clarify resolve typo suggestion)
 - #57556 (privacy: Fix private-in-public check for existential types)
 - #57584 (Remove the `connect_timeout_unroutable` test.)

Failed merges:

r? @ghost
@bors
Copy link
Contributor

bors commented Jan 14, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: Centril
Pushing d106808 to master...

@bors bors merged commit 47ccf2a into rust-lang:master Jan 14, 2019
@Centril Centril deleted the rollup branch January 14, 2019 15:46
@Centril Centril added the rollup A PR which is a rollup label Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

None yet

7 participants