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

Fix logic error in test guarding prototype MIR borrowck code. #44032

Merged
merged 1 commit into from
Aug 23, 2017

Conversation

pnkfelix
Copy link
Member

@pnkfelix pnkfelix commented Aug 22, 2017

Fix logic error in test guarding prototype MIR borrowck code.

tl;dr: 🤦

(This crept in during the shift from a transform to a query (#44009); I didn't notice because my muscle memory was still always passing -Z mir-borrowck, while my test cases also had the #[rustc_mir_borrowck] attribute attached to them.)

(This crept in during the shift from a transform to a query; I didn't
notice because my muscle memory was still always passing `-Z
mir-borrowck`, while my test cases *also* had the
`#[rustc_mir_borrowck]` attribute attached to them.)
@rust-highfive
Copy link
Collaborator

r? @arielb1

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

@eddyb
Copy link
Member

eddyb commented Aug 22, 2017

@bors r+

@bors
Copy link
Contributor

bors commented Aug 22, 2017

📌 Commit fbb099e has been approved by eddyb

@bors
Copy link
Contributor

bors commented Aug 22, 2017

⌛ Testing commit fbb099e with merge 1682f9d...

bors added a commit that referenced this pull request Aug 22, 2017
Fix logic error in test guarding prototype MIR borrowck code.

Fix logic error in test guarding prototype MIR borrowck code.

tl;dr: 🤦

(This crept in during the shift from a transform to a query (#44009); I didn't notice because my muscle memory was still always passing `-Z mir-borrowck`, while my test cases *also* had the `#[rustc_mir_borrowck]` attribute attached to them.)
@bors
Copy link
Contributor

bors commented Aug 23, 2017

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

@bors bors merged commit fbb099e into rust-lang:master Aug 23, 2017
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.

5 participants