Skip to content

Handle nested pairs in MIR trans. #34679

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

Merged
merged 1 commit into from
Jul 8, 2016
Merged

Conversation

eddyb
Copy link
Member

@eddyb eddyb commented Jul 6, 2016

Found while trying to compile the latest Servo master.

cc @shinglyu

@rust-highfive
Copy link
Contributor

r? @nrc

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

@dotdash
Copy link
Contributor

dotdash commented Jul 6, 2016

@bors r+

@bors
Copy link
Collaborator

bors commented Jul 6, 2016

📌 Commit fe5ce97 has been approved by dotdash

@eddyb
Copy link
Member Author

eddyb commented Jul 7, 2016

@bors p=1 Servo needs to rustup to a nightly with this fix for -Zorbit to work for them.

@bors
Copy link
Collaborator

bors commented Jul 7, 2016

⌛ Testing commit fe5ce97 with merge dc17348...

@bors
Copy link
Collaborator

bors commented Jul 8, 2016

👀 Test was successful, but fast-forwarding failed: 422 Update is not a fast forward

@alexcrichton
Copy link
Member

@bors: retry

  • manually merged to master

cc @brson, I guess bors needs to be manually kicked when a manual merge happens by accident?

@bors
Copy link
Collaborator

bors commented Jul 8, 2016

⌛ Testing commit fe5ce97 with merge 3fa1cdf...

bors added a commit that referenced this pull request Jul 8, 2016
Handle nested pairs in MIR trans.

Found while trying to compile the latest Servo master.

cc @shinglyu
@Manishearth
Copy link
Member

Manishearth commented Jul 8, 2016

@bors p=20

(should be higher pri than the rollup, but rollup should be higher pri than everything else)

@bors
Copy link
Collaborator

bors commented Jul 8, 2016

💔 Test failed - auto-win-gnu-32-opt-rustbuild

@eddyb
Copy link
Member Author

eddyb commented Jul 8, 2016

@bors retry

Looks spurious but I haven't seen this exact one before.

@bors bors merged commit fe5ce97 into rust-lang:master Jul 8, 2016
@eddyb eddyb deleted the mir-nested-pairs branch July 8, 2016 13:23
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.

7 participants