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

Rollup of 19 pull requests #47748

Merged
merged 99 commits into from
Jan 26, 2018
Merged

Rollup of 19 pull requests #47748

merged 99 commits into from
Jan 26, 2018

Conversation

pftbest and others added 30 commits January 15, 2018 16:42
Old implementation called the assembler once per crate, but we need to call
it for each object file instead, because a single crate can now have more
than one object file.

This patch fixes issue rust-lang#45836 (Can't compile core for msp430 in release mode)
This addresses the concern of rust-lang#47316 by estimating CGU size based on
the size of its MIR. Looking at the size estimate differences for a
small selection of crates, this heuristic produces different orderings,
which should more accurately reflect optimisation time.

Fixes rust-lang#47316.
Arrays of uninhabited types were considered to also be uninhabited if
their length had not been evaluated, causing unsoundness.
Unstable sort was added recently, and the code that is being modified is 3 years old. As quicksort doesn't allocate it will likely perform as well as, or better than linear search.
It turns out that this *can* happen after all, if the region is only
used in projections from the input types.
@bors
Copy link
Contributor

bors commented Jan 26, 2018

⌛ Testing commit c915e3a with merge d0121328dabef6bda79acb845d233b2c3cc0dc7e...

@alexcrichton
Copy link
Member Author

@bors: retry

@bors
Copy link
Contributor

bors commented Jan 26, 2018

⌛ Testing commit c915e3a with merge cfe81f291317a8c9d6d3044cbe55fa1773004782...

@alexcrichton
Copy link
Member Author

@bors: r+

@bors
Copy link
Contributor

bors commented Jan 26, 2018

📌 Commit 9bb1823 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Jan 26, 2018

⌛ Testing commit 9bb1823 with merge e91e190e7a65f688106f1cea6686a1b8f836d973...

@bors
Copy link
Contributor

bors commented Jan 26, 2018

💔 Test failed - status-travis

@alexcrichton
Copy link
Member Author

@bors: r+

@bors
Copy link
Contributor

bors commented Jan 26, 2018

📌 Commit a06d333 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Jan 26, 2018

⌛ Testing commit a06d333 with merge bacb5c5...

bors added a commit that referenced this pull request Jan 26, 2018
@bors
Copy link
Contributor

bors commented Jan 26, 2018

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup 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.