Skip to content

Conversation

arielb1
Copy link
Contributor

@arielb1 arielb1 commented Aug 14, 2017

We started emitting StorageDead, so we better emit the corrseponding
StorageLive to avoid problems.

cc #43772 rust-lang/miri#303

We started emitting StorageDead, so we better emit the corrseponding
StorageLive to avoid problems.
@arielb1
Copy link
Contributor Author

arielb1 commented Aug 14, 2017

r? @nagisa

@rust-highfive rust-highfive assigned nagisa and unassigned pnkfelix Aug 14, 2017
@rust-highfive
Copy link
Contributor

r? @pnkfelix

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

@carols10cents carols10cents added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 14, 2017
@nagisa
Copy link
Member

nagisa commented Aug 14, 2017

@bors r+

@bors
Copy link
Collaborator

bors commented Aug 14, 2017

📌 Commit e3495b2 has been approved by nagisa

@bors
Copy link
Collaborator

bors commented Aug 15, 2017

⌛ Testing commit e3495b2 with merge f6a30bd...

bors added a commit that referenced this pull request Aug 15, 2017
emit StorageLive for box temporaries

We started emitting StorageDead, so we better emit the corrseponding
StorageLive to avoid problems.

cc #43772 rust-lang/miri#303
@bors
Copy link
Collaborator

bors commented Aug 15, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: nagisa
Pushing f6a30bd to master...

@bors bors merged commit e3495b2 into rust-lang:master Aug 15, 2017
@RalfJung
Copy link
Member

Thanks, this fixes the miri testsuite :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants