Skip to content

Conversation

GuillaumeGomez
Copy link
Member

Successful merges:

Failed merges:

r? @ghost
@rustbot modify labels: rollup

Create a similar rollup

ssomers and others added 11 commits December 13, 2020 11:27
Move intrinsics lowering pass from the optimization phase (where it
would not run if -Zmir-opt-level=0), to the drop lowering phase where it
runs unconditionally.

The implementation of those intrinsics in code generation and
interpreter is unnecessary. Remove it.
`crate.name` is already set by `tcx.crate_name`, there's no need to
override it.
…ulacrum

BTreeMap: more expressive local variables in merge

r? ```````@Mark-Simulacrum```````
…ulacrum

BTreeSet: simplify implementation of pop_first/pop_last

…and stop it interfering in rust-lang#79245.
r? ```````@Mark-Simulacrum```````
…Dylan-DPC

Always run intrinsics lowering pass

Move intrinsics lowering pass from the optimization phase (where it
would not run if -Zmir-opt-level=0), to the drop lowering phase where it
runs unconditionally.

The implementation of those intrinsics in code generation and
interpreter is unnecessary. Remove it.
Use more symbols in rustdoc

Builds on rust-lang#80044 and should not be merged before.

I want to test if this is actually faster before merging it, there was a lot of `to_string()` calls so I'm not sure it will actually help. That means I have to wait for 80044 to get merged before running perf.

r? `@ghost`
@rustbot rustbot added the rollup A PR which is a rollup label Dec 17, 2020
@GuillaumeGomez
Copy link
Member Author

@bors: r+ p=5

@bors
Copy link
Collaborator

bors commented Dec 17, 2020

📌 Commit 5873fe8 has been approved by GuillaumeGomez

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Dec 17, 2020
@bors
Copy link
Collaborator

bors commented Dec 17, 2020

⌛ Testing commit 5873fe8 with merge caeb333...

@bors
Copy link
Collaborator

bors commented Dec 17, 2020

☀️ Test successful - checks-actions
Approved by: GuillaumeGomez
Pushing caeb333 to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. 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.

7 participants