Skip to content

[MIR] Translate ConstVal::Function#30590

Merged
bors merged 1 commit into
rust-lang:masterfrom
nagisa:mir-constval-function
Dec 31, 2015
Merged

[MIR] Translate ConstVal::Function#30590
bors merged 1 commit into
rust-lang:masterfrom
nagisa:mir-constval-function

Conversation

@nagisa

@nagisa nagisa commented Dec 28, 2015

Copy link
Copy Markdown
Member

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
Copy Markdown
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

luqmana commented Dec 29, 2015

Copy link
Copy Markdown
Contributor

👍

@bors: r+

@bors

bors commented Dec 29, 2015

Copy link
Copy Markdown
Collaborator

📌 Commit e137c2a has been approved by luqmana

@bors

bors commented Dec 31, 2015

Copy link
Copy Markdown
Collaborator

⌛ Testing commit e137c2a with merge 26fed7d...

@bors

bors commented Dec 31, 2015

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-mac-32-opt

@eddyb

eddyb commented Dec 31, 2015

Copy link
Copy Markdown
Contributor

@bors retry

@bors

bors commented Dec 31, 2015

Copy link
Copy Markdown
Collaborator

📌 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

bors commented Dec 31, 2015

Copy link
Copy Markdown
Collaborator

⌛ Testing commit e137c2a with merge ac5db18...

@bors

bors commented Dec 31, 2015

Copy link
Copy Markdown
Collaborator

💔 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
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.

6 participants