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

Add missing OperandPair struct field index adjustments. Fixes #41479. #41529

Merged
merged 2 commits into from
Apr 28, 2017

Conversation

bitshifter
Copy link
Contributor

@bitshifter bitshifter commented Apr 25, 2017

This is a bug fix for a regression in 6d841da.

Some accesses in OperandPairs were missing. Fixes rust-lang#41479.
@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

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

@eddyb
Copy link
Member

eddyb commented Apr 25, 2017

@bors r+

@bors
Copy link
Contributor

bors commented Apr 25, 2017

📌 Commit 72acea0 has been approved by eddyb

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

arielb1 commented Apr 25, 2017

Beta-nominating because 1.18 is now beta.

@eddyb
Copy link
Member

eddyb commented Apr 25, 2017

@bors r+

@bors
Copy link
Contributor

bors commented Apr 25, 2017

📌 Commit a510e1d has been approved by eddyb

@nikomatsakis
Copy link
Contributor

r? @eddyb

@rust-highfive rust-highfive assigned eddyb and unassigned nikomatsakis Apr 25, 2017
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 26, 2017
Add missing OperandPair struct field index adjustments. Fixes rust-lang#41479.

This is a bug fix for a regression in rust-lang@6d841da.
@aidanhs aidanhs added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Apr 26, 2017
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 27, 2017
Add missing OperandPair struct field index adjustments. Fixes rust-lang#41479.

This is a bug fix for a regression in rust-lang@6d841da.
@bors
Copy link
Contributor

bors commented Apr 27, 2017

⌛ Testing commit a510e1d with merge 8cf09be...

@bors
Copy link
Contributor

bors commented Apr 27, 2017

💔 Test failed - status-appveyor

@aidanhs
Copy link
Member

aidanhs commented Apr 27, 2017

@bors retry

Appveyor network issues https://appveyor.statuspage.io/incidents/06gzq846jl9x

@nikomatsakis nikomatsakis added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Apr 27, 2017
@bors
Copy link
Contributor

bors commented Apr 27, 2017

⌛ Testing commit a510e1d with merge a8ebd08...

bors added a commit that referenced this pull request Apr 27, 2017
Add missing OperandPair struct field index adjustments. Fixes #41479.

This is a bug fix for a regression in 6d841da.
@bors
Copy link
Contributor

bors commented Apr 28, 2017

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

@bors bors merged commit a510e1d into rust-lang:master Apr 28, 2017
@bitshifter bitshifter deleted the issue-41479 branch April 28, 2017 12:00
@brson brson mentioned this pull request May 10, 2017
@alexcrichton alexcrichton removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label May 10, 2017
bors added a commit that referenced this pull request May 11, 2017
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. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants