Skip to content

Rollup of 18 pull requests - #65152

Merged
bors merged 52 commits into
rust-lang:masterfrom
tmandry:rollup-btn4a01
Oct 6, 2019
Merged

Rollup of 18 pull requests#65152
bors merged 52 commits into
rust-lang:masterfrom
tmandry:rollup-btn4a01

Conversation

@tmandry

@tmandrytmandry commented Oct 6, 2019

Copy link
Copy Markdown
Member

This contains changes from all the successful runs that bors marked as timed out, plus a revert of #63649 which appears to be the immediate cause of the timeouts.

Successful merges:

Failed merges:

r? @ghost

SimonSapinand others added 30 commits September 23, 2019 17:14
The tracking issue rust-lang#50264
still has unresolved question for the corresponding `Result` methods.
This is a stylistic change to libstd to reduce the number of checks of
`feature = "backtrace"` now that we unconditionally depend on the
`backtrace` crate and rely on it having an empty implementation.
otherwise.
Co-Authored-By: Mazdak Farrokhzad <twingoow@gmail.com>
Still debating if this is the right approach
Co-Authored-By: Pietro Albini <pietro@pietroalbini.org>
visit_statement default definition does just this, there's no need to
redefine it.
It has been fully disabled by default since rust-lang#54639.
Do not ICE when dereferencing non-Copy raw pointer
CC rust-lang#52262. Confirmed to remove the unnecessary ICE, but without a repro case.
…by-anyone, r=nikomatsakis
permit asyncawait-ondeck to be added by anyone
Still debating if this is the right approach
but r? @pietroalbini to check the format
…olymorphic_promoted_mir, r=oli-obk
[const-prop] Fix ICE when trying to eval polymorphic promoted MIR
Fixesrust-lang#64908
r? @oli-obk
cc @nikomatsakis@pnkfelix
…ikomatsakis
Split out some passes from librustc
This is just moving them out to librustc_passes -- I've not measured compile time or run time. I don't expect any significant impact, but this seems prudent regardless.
…r=Centril
Allow unused attributes to avoid incremental bug
cc rust-lang#65023
This isn't labeled as fixing that issue because it's not really a fix, just a patch.
Fix lonely backtick
That backtick won't be sad anymore
…i-obk
Remove unneeded visit_statement definition
r? @oli-obk
…=rkruppe
Update the documented default of -Z mutable-noalias
It has been fully disabled by default since rust-lang#54639.
…ebank
Account for macro invocation in `let mut $pat` diagnostic.
Fixesrust-lang#65122.
r? @estebank
Replace some instances of `as *[const | mut] _` with `.cast()`
While in `codegen_llvm`, noticed some places for the recent `pointer::cast` method.
Fix typo on `now()` comments
Fix typo, update words, and remove some redundant word.
Also rustfmt on the rest of the file (hope this is okay :)
revival of rust-lang#61433
r? @kennytm
… r=petrochenkov
lint: extern non-exhaustive types are improper
This PR makes the `improper_ctype` lint trigger for non-exhaustive types when those types aren't defined in the current crate, as per [this comment](rust-lang#44109 (comment)).
cc @Centril
…r=tmandry
Revert rust-lang#63649 - "Upgrade Emscripten targets to use upstream LLVM backend"
This change caused the runtime of the linux-asmjs builder to nearly double from 2+ hours to about 4 hours, which happens to be the bors timeout. (It made it in barely under 4 hours when it was merged.) This is causing timeouts on all new changes.
This reverts commit 7870050, reversing
changes made to 2e72448.
@tmandry

Copy link
Copy Markdown
MemberAuthor

@bors r+ p=18 rollup=never

@bors

bors commented Oct 6, 2019

Copy link
Copy Markdown
Collaborator

📌 Commit 69598dc has been approved by tmandry

@borsbors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Oct 6, 2019
@tmandrytmandry mentioned this pull request Oct 6, 2019
@bors

bors commented Oct 6, 2019

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 69598dc with merge 5a8fb7c...

bors added a commit that referenced this pull request Oct 6, 2019
Rollup of 18 pull requests
This contains changes from all the successful runs that bors marked as timed out, plus a revert of #63649 which appears to be the immediate cause of the timeouts.
Successful merges:
- #64708 (Stabilize `Option::as_deref` and `Option::as_deref_mut`)
- #64728 (Stabilize UdpSocket::peer_addr)
- #64765 (std: Reduce checks for `feature = "backtrace"`)
- #64909 (When encountering chained operators use heuristics to recover from bad turbofish)
- #65011 (Do not ICE when dereferencing non-Copy raw pointer)
- #65064 (permit asyncawait-ondeck to be added by anyone)
- #65066 ([const-prop] Fix ICE when trying to eval polymorphic promoted MIR)
- #65100 (Replace GeneratorSubsts with SubstsRef)
- #65105 (Split out some passes from librustc)
- #65106 (Allow unused attributes to avoid incremental bug)
- #65113 (Fix lonely backtick)
- #65116 (Remove unneeded visit_statement definition)
- #65118 (Update the documented default of -Z mutable-noalias)
- #65123 (Account for macro invocation in `let mut $pat` diagnostic.)
- #65124 (Replace some instances of `as *[const | mut] _` with `.cast()`)
- #65126 (Fix typo on `now()` comments)
- #65130 (lint: extern non-exhaustive types are improper)
- #65151 (Revert #63649 - "Upgrade Emscripten targets to use upstream LLVM backend")
Failed merges:
r? @ghost
@bors

bors commented Oct 6, 2019

Copy link
Copy Markdown
Collaborator

☀️ Test successful - checks-azure
Approved by: tmandry
Pushing 5a8fb7c to master...

@borsbors added the merged-by-bors This PR was explicitly merged by bors. label Oct 6, 2019
@bors
bors merged commit 69598dc into rust-lang:masterOct 6, 2019
@tmandrytmandry added the rollup A PR which is a rollup label Oct 6, 2019
@tmandry
tmandry deleted the rollup-btn4a01 branch October 6, 2019 18:38
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged-by-borsThis PR was explicitly merged by bors.rollupA PR which is a rollupS-waiting-on-borsStatus: 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.

16 participants

@tmandry@bors@SimonSapin@messense@alexcrichton@estebank@nikomatsakis@csmoe@wesleywiser@Mark-Simulacrum@qwaz@spastorino@cuviper@Centril@memoryruins@davidtwco