Skip to content

[beta] backports #37549

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 10 commits into from
Nov 3, 2016
Merged

[beta] backports #37549

merged 10 commits into from
Nov 3, 2016

Conversation

brson
Copy link
Contributor

@brson brson commented Nov 3, 2016

This is all outstanding backports for 1.13. Don't merge yet. Not all are accepted yet, nor tested on master. cc @rust-lang/compiler please review those noted below for acceptance.

After this beta is built we need to update master to pick up the new 1.13 cargo rev set in rust-lang-deprecated/rust-packaging#64.

brson and others added 7 commits November 3, 2016 01:19
There used to be only a global cache, which led to uncached calls to
trait selection when there were type parameters.

I'm running a check that there are no adverse performance effects.

Fixes rust-lang#37106 (drop elaboration times are now ~half of borrow checking,
so might still be worthy of optimization, but not critical).
This will make it easier for packagers to bootstrap rustc when they happen
to have a bootstrap compiler with a slightly different version number.

It's not ok for anything other than the build system to set this environment variable.
The `specializes()` function was trying to normalize the impl trait in
an empty environment. This could lead to inexplicable failures.
@rust-highfive
Copy link
Contributor

r? @Aatch

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

@rust-highfive
Copy link
Contributor

warning Warning warning

  • Pull requests are usually filed against the master branch for this repo, but this one is against beta. Please double check that you specified the right target!

@brson
Copy link
Contributor Author

brson commented Nov 3, 2016

Oh, I'm also still testing this locally.

@alexcrichton
Copy link
Member

r=me when approved

@brson
Copy link
Contributor Author

brson commented Nov 3, 2016

Tests pass locally.

@brson brson merged commit 5f7cfd1 into rust-lang:beta Nov 3, 2016
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants