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

Don't ICE when computing a layout of a generator tainted by errors #81008

Merged
merged 1 commit into from
Jan 15, 2021

Conversation

tmiasko
Copy link
Contributor

@tmiasko tmiasko commented Jan 14, 2021

Fixes #80998.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 14, 2021
@jonas-schievink
Copy link
Contributor

Fixes #80998, presumably

@tmandry
Copy link
Member

tmandry commented Jan 14, 2021

Thanks!

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Jan 14, 2021

📌 Commit 5ea1d0e has been approved by tmandry

@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 Jan 14, 2021
JohnTitor added a commit to JohnTitor/rust that referenced this pull request Jan 15, 2021
…ndry

Don't ICE when computing a layout of a generator tainted by errors

Fixes rust-lang#80998.

type F = impl Future;
// Check that statics are inhabited computes they layout.
static POOL: Task<F> = Task::new();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there should be a second error here, that Task::new isn't const and therefore can't be used in a static expression, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This check is skipped if there are type errors, so it will only appear only after resolving the first error.

bors added a commit to rust-lang-ci/rust that referenced this pull request Jan 15, 2021
Rollup of 5 pull requests

Successful merges:

 - rust-lang#80254 (Don't try to add nested predicate to Rustdoc auto-trait `ParamEnv`)
 - rust-lang#80834 (Remove unreachable panics from VecDeque::{front/back}[_mut])
 - rust-lang#80944 (Use Option::map_or instead of `.map(..).unwrap_or(..)`)
 - rust-lang#81008 (Don't ICE when computing a layout of a generator tainted by errors)
 - rust-lang#81023 (Remove doctree::Variant)

Failed merges:

 - rust-lang#81033 (Remove useless `clean::Variant` struct)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit ce06df2 into rust-lang:master Jan 15, 2021
@rustbot rustbot added this to the 1.51.0 milestone Jan 15, 2021
@tmiasko tmiasko deleted the generator-layout-err branch January 16, 2021 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

ICE in github build
7 participants