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

Add regression test for #61452. #61627

Merged
merged 1 commit into from Jun 8, 2019

Conversation

Projects
None yet
4 participants
@davidtwco
Copy link
Member

commented Jun 7, 2019

Fixes #61452.

Turns out this ICE had already been fixed, so this PR only adds a regression test.

@rust-highfive

This comment has been minimized.

Copy link
Collaborator

commented Jun 7, 2019

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton

This comment has been minimized.

Copy link
Member

commented Jun 7, 2019

@bors: r+ rollup

@bors

This comment has been minimized.

Copy link
Contributor

commented Jun 7, 2019

📌 Commit 5604d9a has been approved by alexcrichton

Centril added a commit to Centril/rust that referenced this pull request Jun 7, 2019

Rollup merge of rust-lang#61627 - davidtwco:ice-async-fn-lint, r=alex…
…crichton

Add regression test for rust-lang#61452.

Fixes rust-lang#61452.

Turns out this ICE had already been fixed, so this PR only adds a regression test.

@Centril Centril referenced this pull request Jun 7, 2019

Closed

Rollup of 7 pull requests #61630

bors added a commit that referenced this pull request Jun 7, 2019

Auto merge of #61630 - Centril:rollup-2u0y4h9, r=Centril
Rollup of 7 pull requests

Successful merges:

 - #61130 (Add std::mem::take as suggested in #61129)
 - #61597 (Bump dirs, rand and redox_users)
 - #61615 (syntax: Treat error literals in more principled way)
 - #61616 (parser: Remove `Deref` impl from `Parser`)
 - #61621 (Clarify when we run steps with ONLY_HOSTS)
 - #61625 (Rename remaining "Eval" to "Interp")
 - #61627 (Add regression test for #61452.)

Failed merges:

r? @ghost

Centril added a commit to Centril/rust that referenced this pull request Jun 7, 2019

Rollup merge of rust-lang#61627 - davidtwco:ice-async-fn-lint, r=alex…
…crichton

Add regression test for rust-lang#61452.

Fixes rust-lang#61452.

Turns out this ICE had already been fixed, so this PR only adds a regression test.

@Centril Centril referenced this pull request Jun 7, 2019

Closed

Rollup of 6 pull requests #61637

bors added a commit that referenced this pull request Jun 7, 2019

Auto merge of #61637 - Centril:rollup-41hiuip, r=Centril
Rollup of 6 pull requests

Successful merges:

 - #61597 (Bump dirs, rand and redox_users)
 - #61615 (syntax: Treat error literals in more principled way)
 - #61616 (parser: Remove `Deref` impl from `Parser`)
 - #61621 (Clarify when we run steps with ONLY_HOSTS)
 - #61625 (Rename remaining "Eval" to "Interp")
 - #61627 (Add regression test for #61452.)

Failed merges:

r? @ghost

Centril added a commit to Centril/rust that referenced this pull request Jun 7, 2019

Rollup merge of rust-lang#61627 - davidtwco:ice-async-fn-lint, r=alex…
…crichton

Add regression test for rust-lang#61452.

Fixes rust-lang#61452.

Turns out this ICE had already been fixed, so this PR only adds a regression test.

@Centril Centril referenced this pull request Jun 7, 2019

Closed

Rollup of 7 pull requests #61644

bors added a commit that referenced this pull request Jun 8, 2019

Auto merge of #61644 - Centril:rollup-tjx5kx9, r=Centril
Rollup of 7 pull requests

Successful merges:

 - #61223 (Document tuple's Ord behavior as sequential)
 - #61597 (Bump dirs, rand and redox_users)
 - #61615 (syntax: Treat error literals in more principled way)
 - #61616 (parser: Remove `Deref` impl from `Parser`)
 - #61621 (Clarify when we run steps with ONLY_HOSTS)
 - #61627 (Add regression test for #61452.)
 - #61641 (Revert "Make LocalAnalizer visitor iterate instead of recurse")

Failed merges:

r? @ghost

Centril added a commit to Centril/rust that referenced this pull request Jun 8, 2019

Rollup merge of rust-lang#61627 - davidtwco:ice-async-fn-lint, r=alex…
…crichton

Add regression test for rust-lang#61452.

Fixes rust-lang#61452.

Turns out this ICE had already been fixed, so this PR only adds a regression test.

@Centril Centril referenced this pull request Jun 8, 2019

Merged

Rollup of 7 pull requests #61649

bors added a commit that referenced this pull request Jun 8, 2019

Auto merge of #61649 - Centril:rollup-b4nx9k9, r=Centril
Rollup of 7 pull requests

Successful merges:

 - #61223 (Document tuple's Ord behavior as sequential)
 - #61615 (syntax: Treat error literals in more principled way)
 - #61616 (parser: Remove `Deref` impl from `Parser`)
 - #61621 (Clarify when we run steps with ONLY_HOSTS)
 - #61627 (Add regression test for #61452.)
 - #61641 (Revert "Make LocalAnalizer visitor iterate instead of recurse")
 - #61647 (Use stable wrappers in f32/f64::signum)

Failed merges:

r? @ghost

@bors bors merged commit 5604d9a into rust-lang:master Jun 8, 2019

1 check passed

Travis CI - Pull Request Build Passed
Details

@davidtwco davidtwco deleted the davidtwco:ice-async-fn-lint branch Jun 9, 2019

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.