Skip to content

Remove ty_bot from the type system#17603

Closed
ghost wants to merge 3 commits into
masterfrom
unknown repository
Closed

Remove ty_bot from the type system#17603
ghost wants to merge 3 commits into
masterfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Sep 28, 2014

Copy link
Copy Markdown

We now instead use a fresh variable for expressions that diverge.

Closes #14973.
Closes #13847.

[Work in progress]

cc @nikomatsakis

@rust-highfive

Copy link
Copy Markdown
Contributor

warning Warning warning

  • These commits modify unsafe code. Please review it carefully!

@nikomatsakis

Copy link
Copy Markdown
Contributor

@jakub- assigning myself, sorry for delay, it's been a busy week or so.

@nikomatsakis nikomatsakis self-assigned this Oct 8, 2014
@ghost

ghost commented Oct 11, 2014

Copy link
Copy Markdown
Author

@nikomatsakis Thanks. This PR isn't compiling yet, AFAIR, due to some unconstrained variables in rustc but the general direction is mostly set. I will rebase soon.

@alexcrichton

Copy link
Copy Markdown
Member

ping @jakub-, any updates here?

@ghost

ghost commented Oct 24, 2014

Copy link
Copy Markdown
Author

@alexcrichton @nikomatsakis This now bootstraps so I'd appreciate your review.

@ghost

ghost commented Oct 25, 2014

Copy link
Copy Markdown
Author

This is green now.

bors added a commit that referenced this pull request Oct 28, 2014
We now instead use a fresh variable for expressions that diverge.

Closes #14973.
Closes #13847.

[Work in progress]

cc @nikomatsakis
@bors bors closed this Oct 29, 2014
@ghost ghost deleted the ty_bot branch October 29, 2014 17:15
@nikomatsakis

Copy link
Copy Markdown
Contributor

Nice work.

@ghost

ghost commented Oct 30, 2014

Copy link
Copy Markdown
Author

@nikomatsakis Cheers!

edwardw added a commit to edwardw/rust that referenced this pull request Feb 6, 2015
PR rust-lang#17603 introduced bottom type fallback but did it a bit too
eagerly. This patch makes the fallback lazy so that `typeck` can run
its cause and detect as many type errors as possible with regard to
diverging types.

Closes rust-lang#21878
lnicola pushed a commit to lnicola/rust that referenced this pull request Jul 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

5 participants