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

try to fix lvalue ops for real #41828

Merged
merged 1 commit into from
May 9, 2017
Merged

try to fix lvalue ops for real #41828

merged 1 commit into from
May 9, 2017

Conversation

arielb1
Copy link
Contributor

@arielb1 arielb1 commented May 8, 2017

Hopefully this is the last PR needed.

Fixes #41726.
Fixes #41742.
Fixes #41774.

Hopefully this is the last PR needed.

Fixes rust-lang#41726.
Fixes rust-lang#41742.
Fixes rust-lang#41774.
@arielb1
Copy link
Contributor Author

arielb1 commented May 8, 2017

r? @eddyb

@rust-highfive rust-highfive assigned eddyb and unassigned pnkfelix May 8, 2017
@rust-highfive
Copy link
Collaborator

r? @pnkfelix

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

@eddyb
Copy link
Member

eddyb commented May 8, 2017

@bors r+

@bors
Copy link
Contributor

bors commented May 8, 2017

📌 Commit 052d071 has been approved by eddyb

frewsxcv added a commit to frewsxcv/rust that referenced this pull request May 9, 2017
try to fix lvalue ops for real

Hopefully this is the last PR needed.

Fixes rust-lang#41726.
Fixes rust-lang#41742.
Fixes rust-lang#41774.
bors added a commit that referenced this pull request May 9, 2017
Rollup of 8 pull requests

- Successful merges: #41293, #41520, #41827, #41828, #41833, #41836, #41838, #41842
- Failed merges:
@bors bors merged commit 052d071 into rust-lang:master May 9, 2017
@bors
Copy link
Contributor

bors commented May 9, 2017

⌛ Testing commit 052d071 with merge bedd7da...

@arielb1 arielb1 added the beta-nominated Nominated for backporting to the compiler in the beta channel. label May 10, 2017
@arielb1
Copy link
Contributor Author

arielb1 commented May 10, 2017

Beta-nominating because this fixes regression introduces in #41578 which is going into beta.

@arielb1 arielb1 mentioned this pull request May 10, 2017
@nikomatsakis nikomatsakis added the beta-accepted Accepted for backporting to the compiler in the beta channel. label May 17, 2017
@nikomatsakis
Copy link
Contributor

Accepting for beta. Small-ish patch, serious soundness regressions.

cc @rust-lang/compiler

@nikomatsakis nikomatsakis added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label May 17, 2017
@nikomatsakis
Copy link
Contributor

Backport in #42053

bors added a commit that referenced this pull request May 22, 2017
Beta backport various arielb1 patches

Backport of the following PRs:

- #41651
- #41828

cc @arielb1 @rust-lang/compiler
@brson brson removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label May 23, 2017
@brson
Copy link
Contributor

brson commented May 23, 2017

Looks like this has been backported. Removed the nominated tag.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. 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.

7 participants