Skip to content

[MIR] Translate ConstVal::Function #30590

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Dec 31, 2015
Merged

Conversation

nagisa
Copy link
Member

@nagisa nagisa commented Dec 28, 2015

This moves back (essentially reverts #30265) into MIR-specific translation code, but keeps the
funcition split out, since it is expected to eventually become recursive.

Fixes #29572

cc @oli-obk

@rust-highfive
Copy link
Contributor

r? @arielb1

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

This moves back (essentially reverts rust-lang#30265) into MIR-specific translation code, but keeps the
funcition split out, since it is expected to eventually become recursive.
@nagisa nagisa force-pushed the mir-constval-function branch from 62dba7a to e137c2a Compare December 28, 2015 19:27
@luqmana
Copy link
Member

luqmana commented Dec 29, 2015

👍

@bors: r+

@bors
Copy link
Collaborator

bors commented Dec 29, 2015

📌 Commit e137c2a has been approved by luqmana

@bors
Copy link
Collaborator

bors commented Dec 31, 2015

⌛ Testing commit e137c2a with merge 26fed7d...

@bors
Copy link
Collaborator

bors commented Dec 31, 2015

💔 Test failed - auto-mac-32-opt

@eddyb
Copy link
Member

eddyb commented Dec 31, 2015

@bors retry

@bors
Copy link
Collaborator

bors commented Dec 31, 2015

📌 Commit e137c2a has been approved by luqmana

bors added a commit that referenced this pull request Dec 31, 2015
This moves back (essentially reverts #30265) into MIR-specific translation code, but keeps the
funcition split out, since it is expected to eventually become recursive.

Fixes #29572

cc @oli-obk
@bors
Copy link
Collaborator

bors commented Dec 31, 2015

⌛ Testing commit e137c2a with merge ac5db18...

@bors
Copy link
Collaborator

bors commented Dec 31, 2015

💔 Test failed - auto-linux-64-opt

nagisa added a commit to nagisa/rust that referenced this pull request Dec 31, 2015
…mana

This moves back (essentially reverts rust-lang#30265) into MIR-specific translation code, but keeps the
funcition split out, since it is expected to eventually become recursive.

Fixes rust-lang#29572

cc @oli-obk
bors added a commit that referenced this pull request Dec 31, 2015
@bors bors merged commit e137c2a into rust-lang:master Dec 31, 2015
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants