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

Remove extra calls to kill_loans_out_of_scope_at_location. #50891

Merged
merged 1 commit into from May 23, 2018

Conversation

Projects
None yet
5 participants
@davidtwco
Copy link
Member

davidtwco commented May 19, 2018

r? @nikomatsakis

Remove extra calls to kill_loans_out_of_scope_at_location - keep only…
… before_statement_effect and before_terminator_effect.
@pnkfelix

This comment has been minimized.

Copy link
Member

pnkfelix commented May 21, 2018

@bors r+

@bors

This comment has been minimized.

Copy link
Contributor

bors commented May 21, 2018

📌 Commit 280c6fa has been approved by pnkfelix

@bors

This comment has been minimized.

Copy link
Contributor

bors commented May 23, 2018

⌛️ Testing commit 280c6fa with merge e249596...

bors added a commit that referenced this pull request May 23, 2018

Auto merge of #50891 - davidtwco:nll-kill-loans-out-of-scope-at-locat…
…ion-reduce-calls, r=pnkfelix

Remove extra calls to kill_loans_out_of_scope_at_location.

r? @nikomatsakis
@bors

This comment has been minimized.

Copy link
Contributor

bors commented May 23, 2018

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

@bors bors merged commit 280c6fa into rust-lang:master May 23, 2018

2 checks passed

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

@davidtwco davidtwco deleted the davidtwco:nll-kill-loans-out-of-scope-at-location-reduce-calls branch May 26, 2018

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.