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

rustc: Remove `used_mut_nodes` from `TyCtxt` #45283

Merged
merged 1 commit into from Oct 16, 2017

Conversation

Projects
None yet
7 participants
@alexcrichton
Copy link
Member

alexcrichton commented Oct 14, 2017

This updates the borrowck query to return a result, and this result is then used
to incrementally check for unused mutable nodes given sets of all the used
mutable nodes.

Closes #42384

@alexcrichton

This comment has been minimized.

Copy link
Member Author

alexcrichton commented Oct 14, 2017

@rust-highfive

This comment has been minimized.

Copy link
Collaborator

rust-highfive commented Oct 14, 2017

r? @pnkfelix

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

rustc: Remove `used_mut_nodes` from `TyCtxt`
This updates the borrowck query to return a result, and this result is then used
to incrementally check for unused mutable nodes given sets of all the used
mutable nodes.

Closes #42384

@alexcrichton alexcrichton force-pushed the alexcrichton:used-mut-nodes branch from 522601e to 4df1278 Oct 14, 2017

@arielb1

This comment has been minimized.

Copy link
Contributor

arielb1 commented Oct 15, 2017

@bors r+

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Oct 15, 2017

📌 Commit 4df1278 has been approved by arielb1

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Oct 16, 2017

⌛️ Testing commit 4df1278 with merge fdec805...

bors added a commit that referenced this pull request Oct 16, 2017

Auto merge of #45283 - alexcrichton:used-mut-nodes, r=arielb1
rustc: Remove `used_mut_nodes` from `TyCtxt`

This updates the borrowck query to return a result, and this result is then used
to incrementally check for unused mutable nodes given sets of all the used
mutable nodes.

Closes #42384
@bors

This comment has been minimized.

Copy link
Contributor

bors commented Oct 16, 2017

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

@bors bors merged commit 4df1278 into rust-lang:master Oct 16, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details

@alexcrichton alexcrichton deleted the alexcrichton:used-mut-nodes branch Oct 16, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.