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

Do not require semantic types for all syntactic types when there are errors #44945

Merged
merged 1 commit into from
Oct 1, 2017

Conversation

petrochenkov
Copy link
Contributor

@petrochenkov petrochenkov commented Sep 30, 2017

Fixes #44814
Fixes #44858
Fixes #44946
r? @nikomatsakis

@petrochenkov
Copy link
Contributor Author

I've tweaked the patch slightly and moved the is_tainted_by_errors condition directly into fn node_ty.

@arielb1
Copy link
Contributor

arielb1 commented Oct 1, 2017

@bors r+

@bors
Copy link
Contributor

bors commented Oct 1, 2017

📌 Commit 8037c28 has been approved by arielb1

@bors
Copy link
Contributor

bors commented Oct 1, 2017

⌛ Testing commit 8037c28 with merge 3651cdd...

bors added a commit that referenced this pull request Oct 1, 2017
Do not require semantic types for all syntactic types when there are errors

Fixes #44814
Fixes #44858
Fixes #44946
r? @nikomatsakis
@bors
Copy link
Contributor

bors commented Oct 1, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: arielb1
Pushing 3651cdd to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants