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

make redundant StorageLive UB #79931

Merged
merged 1 commit into from
Dec 12, 2020

Conversation

RalfJung
Copy link
Member

The interesting behavior of StorageLive in loops (#42371) has been fixed, so we can now finally make it a hard error to mark a local as live that is already live. :)

r? @oli-obk
Fixes #42371

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Dec 11, 2020
@RalfJung RalfJung changed the title make redundant StorageLive/StorageDead UB make redundant StorageLive UB Dec 11, 2020
@RalfJung RalfJung force-pushed the no-redundant-storage-live branch from 0eb906c to 41c56f5 Compare December 11, 2020 10:31
@oli-obk
Copy link
Contributor

oli-obk commented Dec 11, 2020

r=me with rustfmt

@RalfJung RalfJung force-pushed the no-redundant-storage-live branch from 41c56f5 to 78deacc Compare December 11, 2020 12:19
@RalfJung
Copy link
Member Author

@bors r=oli-obk

@bors
Copy link
Contributor

bors commented Dec 11, 2020

📌 Commit 78deacc has been approved by oli-obk

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Dec 11, 2020
@bors
Copy link
Contributor

bors commented Dec 11, 2020

⌛ Testing commit 78deacc with merge b65fa51fe6201a73dbf25f2d0bc99e65faa4175d...

@jyn514 jyn514 added A-const-eval Area: Constant evaluation, covers all const contexts (static, const fn, ...) T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Dec 11, 2020
@bors
Copy link
Contributor

bors commented Dec 11, 2020

💥 Test timed out

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Dec 11, 2020
@RalfJung
Copy link
Member Author

@bors retry

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Dec 11, 2020
@rust-log-analyzer
Copy link
Collaborator

Your PR failed (pretty log, raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log.

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @rust-lang/infra. (Feature Requests)

@bors
Copy link
Contributor

bors commented Dec 12, 2020

⌛ Testing commit 78deacc with merge 602899c...

@bors
Copy link
Contributor

bors commented Dec 12, 2020

☀️ Test successful - checks-actions
Approved by: oli-obk
Pushing 602899c to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Dec 12, 2020
@bors bors merged commit 602899c into rust-lang:master Dec 12, 2020
@rustbot rustbot added this to the 1.50.0 milestone Dec 12, 2020
@RalfJung RalfJung deleted the no-redundant-storage-live branch December 13, 2020 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-const-eval Area: Constant evaluation, covers all const contexts (static, const fn, ...) merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Semantics of StorageLive in loops
7 participants