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.

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

[MIR] Fix MIR trans edge cases that showed up on crater. #34128

Merged
merged 3 commits into from
Jun 7, 2016

Conversation

eddyb
Copy link
Member

@eddyb eddyb commented Jun 6, 2016

These fixes cover all of the regressions found by crater (for #34096).

Two of them were Pair edge cases (ZSTs and constants) causing LLVM assertions, the other one was causing stack overflows in debug scripts compiled in debug mode, due to the fn_ret_cast alloca ending up in a loop.

@rust-highfive
Copy link
Contributor

r? @arielb1

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

@luqmana
Copy link
Member

luqmana commented Jun 7, 2016

\o/ Nice!
@bors r+

@eddyb eddyb changed the title [MIR] Fix two Pair edge cases that showed up on crater. [MIR] Fix MIR trans edge cases that showed up on crater. Jun 7, 2016
@luqmana
Copy link
Member

luqmana commented Jun 7, 2016

@bors r+

@bors
Copy link
Collaborator

bors commented Jun 7, 2016

📌 Commit 02cbc0e has been approved by luqmana

@pnkfelix
Copy link
Member

pnkfelix commented Jun 7, 2016

@bors p=2

pumping up priority so that this beats #34109 to landing

@bors
Copy link
Collaborator

bors commented Jun 7, 2016

⌛ Testing commit 02cbc0e with merge 9b2beca...

bors added a commit that referenced this pull request Jun 7, 2016
[MIR] Fix MIR trans edge cases that showed up on crater.

These fixes cover all of the [regressions found by crater](https://gist.github.com/nikomatsakis/88ce89ed06ef7f7f19bfd1e221d7f7ec) (for #34096).

Two of them were `Pair` edge cases (ZSTs and constants) causing LLVM assertions, the other one was  causing stack overflows in debug scripts compiled in debug mode, due to the `fn_ret_cast` `alloca` ending up in a loop.
@bors bors merged commit 02cbc0e into rust-lang:master Jun 7, 2016
@eddyb eddyb deleted the mir-trans-fixes branch June 7, 2016 19:19
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