Skip to content

Permit node types to be missing if type errors occurred previously #20897

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

Merged
merged 1 commit into from
Jan 12, 2015

Conversation

barosl
Copy link
Contributor

@barosl barosl commented Jan 10, 2015

If the type of a node cannot be determined due to a previous type error, a no type for node ICE occurs. This commit makes it return ty_err instead in such a case.

Fixes #20401.
Fixes #20506.
Fixes #20614.
Fixes #20752.
Fixes #20829.
Fixes #20846.
Fixes #20885.
Fixes #20886.
Fixes #20952.
Fixes #20970.

@rust-highfive
Copy link
Contributor

r? @nick29581

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

@barosl barosl force-pushed the no-type-for-node-ice branch from 67b3dab to 7b6f52a Compare January 11, 2015 00:16
If the type of a node cannot be determined due to a previous type error,
a "no type for node" ICE occurs. This commit makes it return ty_err
instead in such a case.

Fixes rust-lang#20401.
Fixes rust-lang#20506.
Fixes rust-lang#20614.
Fixes rust-lang#20752.
Fixes rust-lang#20829.
Fixes rust-lang#20846.
Fixes rust-lang#20885.
Fixes rust-lang#20886.
@barosl barosl force-pushed the no-type-for-node-ice branch from 7b6f52a to a015544 Compare January 11, 2015 10:42
bors added a commit that referenced this pull request Jan 12, 2015
If the type of a node cannot be determined due to a previous type error, a `no type for node` ICE occurs. This commit makes it return `ty_err` instead in such a case.

Fixes #20401.
Fixes #20506.
Fixes #20614.
Fixes #20752.
Fixes #20829.
Fixes #20846.
Fixes #20885.
Fixes #20886.
Fixes #20952.
Fixes #20970.
@bors bors merged commit a015544 into rust-lang:master Jan 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment