Skip to content

Rollup of 6 pull requests - #51334

Merged
bors merged 19 commits into
rust-lang:masterfrom
Mark-Simulacrum:rollup
Jun 4, 2018
Merged

Rollup of 6 pull requests#51334
bors merged 19 commits into
rust-lang:masterfrom
Mark-Simulacrum:rollup

Conversation

@Mark-Simulacrum

Copy link
Copy Markdown
Member

Successful merges:

Failed merges:

faernand others added 19 commits June 2, 2018 12:03
Instead of only for booleans. This means that if let also becomes
a br.
Apart from making the IR slightly simpler, this is supported by
FastISel.
Rebase of rust-lang#50783 has accidentally revived the flag (which should be
renamed to `-Z codegen-time-graph` by rust-lang#50615).
…=eddyb
Remove rustdoc-specific is_import field from HIR
Fixesrust-lang#47100.
I believe that there is no need to check for the name being the same, as this
part of rustdoc seems to be strictly interested in exploring "public modules."
Re-exports from the same module cannot visit another module; and, re-exports
cannot export items with a greater visibility than that item declares.
Therefore, I think this code is either sufficient, or in fact does more than
is necessary, depending on whether rustdoc cares about the re-export itself.
r? @eddyb
const fn integer operations
A follow up to rust-lang#51171Fixesrust-lang#51267
Makes a lot of the integer methods (`swap_bytes`, `count_ones` etc) `const fn`s. See rust-lang#51267 for a discussion about why this is wanted and the solution used.
…Mark-Simulacrum
Allow enabling incremental via config.toml
r? @QuietMisdreavus
Generate br for all two target SwitchInts
Instead of only for booleans. This means that `if let` also becomes a br.
Apart from making the IR slightly simpler, this is supported by FastISel (rust-lang#4353).
Various minor slice iterator cleanups
See individual commits
Remove the unused `-Z trans-time-graph` flag.
Rebase of rust-lang#50783 has accidentally revived the flag (which should be renamed to `-Z codegen-time-graph` by rust-lang#50615).
@rust-highfive

Copy link
Copy Markdown
Contributor

r? @aidanhs

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

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 4, 2018
@Mark-Simulacrum

Copy link
Copy Markdown
MemberAuthor

@bors r+ p=10

r? @Mark-Simulacrum

@bors

bors commented Jun 4, 2018

Copy link
Copy Markdown
Collaborator

📌 Commit dd1096f has been approved by Mark-Simulacrum

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 4, 2018
@bors

bors commented Jun 4, 2018

Copy link
Copy Markdown
Collaborator

⌛ Testing commit dd1096f with merge 6232478...

bors added a commit that referenced this pull request Jun 4, 2018
Rollup of 6 pull requests
Successful merges:
- #51288 (Remove rustdoc-specific is_import field from HIR)
- #51299 (const fn integer operations)
- #51317 (Allow enabling incremental via config.toml)
- #51323 (Generate br for all two target SwitchInts)
- #51326 (Various minor slice iterator cleanups)
- #51329 (Remove the unused `-Z trans-time-graph` flag.)
Failed merges:
@bors

bors commented Jun 4, 2018

Copy link
Copy Markdown
Collaborator

☀️ Test successful - status-appveyor, status-travis
Approved by: Mark-Simulacrum
Pushing 6232478 to master...

@bors
bors merged commit dd1096f into rust-lang:masterJun 4, 2018
@borsbors mentioned this pull request Jun 4, 2018
@CentrilCentril added the rollup A PR which is a rollup label Oct 2, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rollupA PR which is a rollupS-waiting-on-borsStatus: 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.

10 participants

@Mark-Simulacrum@rust-highfive@bors@Centril@aidanhs@faern@nikic@oli-obk@sdroege@kennytm