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.

Sign up for GitHub

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

Jump to bottom

Use HIR map instead of tcx in constant evaluator #31962

Merged
merged 1 commit into from
Mar 1, 2016
Merged

Use HIR map instead of tcx in constant evaluator #31962

merged 1 commit into from
Mar 1, 2016

Conversation

sanxiyn
Copy link
Member

@sanxiyn sanxiyn commented Feb 29, 2016

Fix #31910.

@sanxiyn
Use HIR map instead of tcx in constant evaluator
33bd144 
Constant evaluator can be called while tcx is in construction.
@rust-highfive
Copy link
Collaborator

r? @nrc

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

@arielb1
Copy link
Contributor

arielb1 commented Feb 29, 2016

@bors r+

@bors
Copy link
Contributor

bors commented Feb 29, 2016

📌 Commit 33bd144 has been approved by arielb1

@bors
Copy link
Contributor

bors commented Mar 1, 2016

⌛ Testing commit 33bd144 with merge bbc0238...

bors added a commit that referenced this pull request Mar 1, 2016
@bors
Auto merge of #31962 - sanxiyn:const-eval-map, r=arielb1
bbc0238 
Fix #31910.
@bors
Copy link
Contributor

bors commented Mar 1, 2016

☀️ Test successful - auto-linux-32-nopt-t, auto-linux-32-opt, auto-linux-64-debug-opt, auto-linux-64-nopt-t, auto-linux-64-opt, auto-linux-64-x-android-t, auto-linux-cross-opt, auto-linux-musl-64-opt, auto-mac-32-opt, auto-mac-64-nopt-t, auto-mac-64-opt, auto-mac-ios-opt, auto-win-gnu-32-nopt-t, auto-win-gnu-32-opt, auto-win-gnu-64-nopt-t, auto-win-gnu-64-opt, auto-win-msvc-32-opt, auto-win-msvc-64-opt

@bors bors merged commit 33bd144 into rust-lang:master Mar 1, 2016
@bors bors mentioned this pull request Mar 1, 2016
Merged
@sanxiyn sanxiyn deleted the const-eval-map branch March 2, 2016 13:56
@llogiq llogiq mentioned this pull request Mar 6, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nrc nrc

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@sanxiyn @rust-highfive @arielb1 @bors @nrc