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

introduce a fudge_regions_if_ok to address false region edges #37659

Merged
merged 1 commit into from
Nov 12, 2016

Conversation

nikomatsakis
Copy link
Contributor

Fixes #37655.

r? @eddyb
cc @sfackler

/// Assumptions:
/// - no new type variables are created during `f()` (asserted
/// below); this simplifies our logic since we don't have to
/// check for escaping type variables
Copy link
Member

Choose a reason for hiding this comment

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

This will break down once lazy normalization happens, right? Not that it blocks the PR.

@eddyb
Copy link
Member

eddyb commented Nov 9, 2016

@bors r+

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Nov 10, 2016

📌 Commit c428535 has been approved by alexcrichton

@alexcrichton
Copy link
Member

er,

@bors: r=eddyb

@bors
Copy link
Contributor

bors commented Nov 10, 2016

💡 This pull request was already approved, no need to approve it again.

@bors
Copy link
Contributor

bors commented Nov 10, 2016

📌 Commit c428535 has been approved by eddyb

@alexcrichton alexcrichton added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Nov 10, 2016
eddyb added a commit to eddyb/rust that referenced this pull request Nov 11, 2016
…eddyb

introduce a `fudge_regions_if_ok` to address false region edges

Fixes rust-lang#37655.

r? @eddyb
cc @sfackler
eddyb added a commit to eddyb/rust that referenced this pull request Nov 12, 2016
…eddyb

introduce a `fudge_regions_if_ok` to address false region edges

Fixes rust-lang#37655.

r? @eddyb
cc @sfackler
bors added a commit that referenced this pull request Nov 12, 2016
@bors bors merged commit c428535 into rust-lang:master Nov 12, 2016
@nikomatsakis nikomatsakis deleted the sfackler-36340-fix branch April 14, 2017 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants