Skip to content

[MIR] Fix translation of ConstVal::{Struct, Tuple} - #30774

Merged
bors merged 1 commit into
rust-lang:masterfrom
nagisa:mir-fix-constval-adts
Jan 12, 2016
Merged

[MIR] Fix translation of ConstVal::{Struct, Tuple}#30774
bors merged 1 commit into
rust-lang:masterfrom
nagisa:mir-fix-constval-adts

Conversation

@nagisa

Copy link
Copy Markdown
Member

Fixes#30772

We used to have a untested special case which didn’t really work anyway, because of lacking casts. This PR removes the case in question.

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @arielb1

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

@arielb1

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Jan 8, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit ea52d9e has been approved by arielb1

@bors

bors commented Jan 8, 2016

Copy link
Copy Markdown
Collaborator

⌛ Testing commit ea52d9e with merge 1aca1fd...

bors added a commit that referenced this pull request Jan 8, 2016
Fixes#30772
We used to have a untested special case which didn’t really work anyway, because of lacking casts. This PR removes the case in question.
@bors

bors commented Jan 8, 2016

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-linux-musl-64-opt

@nagisa

Copy link
Copy Markdown
MemberAuthor

Wow, this kind of failure (multiple buildbots were “interrupted” although they all compiled successfully) is new one.

@eddyb

eddyb commented Jan 8, 2016

Copy link
Copy Markdown
Contributor

@bors retry

nagisa added a commit to nagisa/rust that referenced this pull request Jan 9, 2016
…elb1
Fixesrust-lang#30772
We used to have a untested special case which didn’t really work anyway, because of lacking casts. This PR removes the case in question.
@nagisanagisa mentioned this pull request Jan 9, 2016
nagisa added a commit to nagisa/rust that referenced this pull request Jan 11, 2016
…elb1
Fixesrust-lang#30772
We used to have a untested special case which didn’t really work anyway, because of lacking casts. This PR removes the case in question.
@nagisanagisa mentioned this pull request Jan 11, 2016
bors added a commit that referenced this pull request Jan 11, 2016
@bors
bors merged commit ea52d9e into rust-lang:masterJan 12, 2016
Sign up for freeto 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.

5 participants

@nagisa@rust-highfive@arielb1@bors@eddyb