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 4 pull requests #64466

Merged
merged 11 commits into from
Sep 15, 2019
Merged

Rollup of 4 pull requests #64466

merged 11 commits into from
Sep 15, 2019

Conversation

Centril
Copy link
Contributor

@Centril Centril commented Sep 14, 2019

Successful merges:

Failed merges:

r? @ghost

Thomasdezeeuw and others added 11 commits August 25, 2019 13:06
Unfortunately, the diagnotic machinery does not cope well with an empty
span which can happen if the crate is empty, in which case we merely set
a spanless note.
Intrinsics can only be defined by the compiler.
Same checks are performed during name resolution, and all attributes go through name resolution now
Use `PostExpansionVisitor` directly instead
…ebank

Provide a span if main function is not present in crate

Unfortunately, the diagnostic machinery does not cope well with an empty
span which can happen if the crate is empty, in which case we merely set
a spanless note.

Tests are already updated for this change, so a dedicated test is not added.

Resolves rust-lang#36561.
…nsic, r=Centril

Ban non-extern rust intrinsics

Intrinsics can only be defined by the compiler.

Fixes rust-lang#36979
feature_gate: Remove dead code from attribute checking

rust-lang#63468 is merged, so all attributes go through name resolution now, so we can remove code that previously performed some checks for attributes not going through resolution.
@Centril
Copy link
Contributor Author

Centril commented Sep 14, 2019

@bors r+ p=4 rollup=never

@bors
Copy link
Contributor

bors commented Sep 14, 2019

📌 Commit 45e50e2 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 Sep 14, 2019
@bors
Copy link
Contributor

bors commented Sep 14, 2019

⌛ Testing commit 45e50e2 with merge b35ebac...

bors added a commit that referenced this pull request Sep 14, 2019
Rollup of 4 pull requests

Successful merges:

 - #61797 (Stabilise weak_ptr_eq)
 - #64290 (Provide a span if main function is not present in crate)
 - #64406 (Ban non-extern rust intrinsics)
 - #64462 (feature_gate: Remove dead code from attribute checking)

Failed merges:

r? @ghost
@bors
Copy link
Contributor

bors commented Sep 15, 2019

☀️ Test successful - checks-azure
Approved by: Centril
Pushing b35ebac to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Sep 15, 2019
@bors bors merged commit 45e50e2 into rust-lang:master Sep 15, 2019
@rust-highfive
Copy link
Collaborator

📣 Toolstate changed by #64466!

Tested on commit b35ebac.
Direct link to PR: #64466

💔 rls on windows: test-pass → test-fail (cc @Xanewok, @rust-lang/infra).
💔 rls on linux: test-pass → test-fail (cc @Xanewok, @rust-lang/infra).

rust-highfive added a commit to rust-lang-nursery/rust-toolstate that referenced this pull request Sep 15, 2019
Tested on commit rust-lang/rust@b35ebac.
Direct link to PR: <rust-lang/rust#64466>

💔 rls on windows: test-pass → test-fail (cc @Xanewok, @rust-lang/infra).
💔 rls on linux: test-pass → test-fail (cc @Xanewok, @rust-lang/infra).
@Centril Centril deleted the rollup-s3nlb9e branch September 15, 2019 00:12
@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
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.

6 participants