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

rustc_const_eval: always require Substs and a ParamEnv. #43505

Merged
merged 2 commits into from
Jul 28, 2017

Conversation

eddyb
Copy link
Member

@eddyb eddyb commented Jul 27, 2017

Fixes #43357 by tracking the Substs and ParamEnv for const-evaluation in generic contexts.

@rust-highfive
Copy link
Collaborator

r? @pnkfelix

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

@nikomatsakis
Copy link
Contributor

r? @nikomatsakis

@nikomatsakis
Copy link
Contributor

@eddyb -- missing a test, I think

@eddyb eddyb force-pushed the poly-const-eval-layout-of branch from e0e095b to 60cf542 Compare July 27, 2017 17:59
Copy link
Contributor

@nikomatsakis nikomatsakis left a comment

Choose a reason for hiding this comment

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

r=me with test

@bors
Copy link
Contributor

bors commented Jul 27, 2017

📌 Commit 60cf542 has been approved by nikomatsakis

@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented Jul 27, 2017

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

@bors
Copy link
Contributor

bors commented Jul 27, 2017

📌 Commit 60cf542 has been approved by nikomatsakis

@eddyb
Copy link
Member Author

eddyb commented Jul 27, 2017

@bors r=nikomatsakis

@alexcrichton alexcrichton added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jul 27, 2017
@bors
Copy link
Contributor

bors commented Jul 28, 2017

⌛ Testing commit 60cf542 with merge 7167843...

bors added a commit that referenced this pull request Jul 28, 2017
rustc_const_eval: always require Substs and a ParamEnv.

Fixes #43357 by tracking the `Substs` and `ParamEnv` for const-evaluation in generic contexts.
@bors
Copy link
Contributor

bors commented Jul 28, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: nikomatsakis
Pushing 7167843 to master...

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.

6 participants