Skip to content

Rollup of 6 pull requests - #162687

Merged
rust-bors[bot] merged 21 commits into
rust-lang:mainfrom
JonathanBrouwer:rollup-nWh5YpQ
Sep 12, 2026
Merged

Rollup of 6 pull requests#162687
rust-bors[bot] merged 21 commits into
rust-lang:mainfrom
JonathanBrouwer:rollup-nWh5YpQ

Conversation

@JonathanBrouwer

Copy link
Copy Markdown
Member

Successful merges:

r? @ghost

Create a similar rollup

DanielEScherzer and others added 21 commits September 11, 2026 12:13
After the separation from `DenseBitSet` and the removal of unused methods,
growable bitsets have no need to track an explicit domain size separate from
the length of their storage vec.
The bitset entries tracked by this code don't have a naturally fixed domain, so
using GrowableBitSet makes more sense than repeatedly resizing a DenseBitSet.
The direct dependency from `rustc_codegen_ssa` to `rustc_mir_transform` allowed
codegen to reuse some MIR-transform code for identifying unnecessary landing
pads.

However, we can achieve the same result by calling through a hook on `TyCtxt`,
eliminating the crate dependency.
Use GrowableBitSet in the main loop of `rustc_mir_transform::sroa`

- Follow-up to rust-lang#161957
---

The first commit is a cleanup based on rust-lang#161957 (comment) by @panstromek. After having disconnected GrowableBitSet from DenseBitSet and removed unused methods, there is no longer any reason to keep track of a separate `domain_size` in GrowableBitSet.

After that, I wanted to get rid of `DenseBitSet::enlarge`, since having it around is not a good fit for the intended purpose of DenseBitSet as a fixed-domain set. After looking more closely at the code in `rustc_mir_transform::sroa` that uses it, I concluded that that code would be better off using GrowableBitSet instead. GrowableBitSet is just as dense, but avoids the need to set a fixed domain size in advance.

The changes to SROA require adding `GrowableBitSet::union`, which is easier after having removed `domain_size`.

(I am not deeply familiar with the SROA pass, but its usage of bitsets seems fairly straightforward in this case.)

There should be no change to compiler output.
…uppe

Disconnect `rustc_codegen_ssa` from `rustc_mir_transform`

The direct dependency from `rustc_codegen_ssa` to `rustc_mir_transform` allowed codegen to reuse some MIR-transform code for identifying unnecessary landing pads (for rust-lang#143208).

However, we can achieve the same result by calling through a hook on `TyCtxt`, eliminating the crate dependency.

---

There should be no change to compiler output.
…yUwU

Remove unused arguments of functions

That these arguments are unused was hidden by the `#[trace::instrument]` macro, which uses the arguments.
I don't see an easy way to avoid this from happening in the future.
…ypo, r=jieyouxu

Rustdoc issue template: fix typo in comment ("thorugh")
…umeGomez

Add `E0747` explanation for `type/const` mismatch case

Fixes rust-lang#162320

r? @GuillaumeGomez
…nna-kruppe

Stabilize `Vec::from_fn`

Tracking issue: rust-lang#149698.
@rust-bors rust-bors Bot added the rollup A PR which is a rollup label Sep 12, 2026
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. WG-trait-system-refactor The Rustc Trait System Refactor Initiative (-Znext-solver) labels Sep 12, 2026
@JonathanBrouwer

Copy link
Copy Markdown
Member Author

@bors r+ p=5

Trying commonly failed jobs
@bors try jobs=dist-various-1,test-various,test-x86_64-gnu-aux,test-x86_64-gnu-llvm-21-3,test-x86_64-msvc-1,test-aarch64-apple-1,test-aarch64-apple-2,test-x86_64-mingw-1,test-i686-msvc,test-armhf-gnu

@rust-bors

rust-bors Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

📌 Commit f91d429 has been approved by JonathanBrouwer

It is now in the queue for this repository.

@rust-bors rust-bors Bot 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 Sep 12, 2026
@rust-bors

This comment has been minimized.

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Sep 12, 2026
Rollup of 6 pull requests


try-job: dist-various-1
try-job: test-various
try-job: test-x86_64-gnu-aux
try-job: test-x86_64-gnu-llvm-21-3
try-job: test-x86_64-msvc-1
try-job: test-aarch64-apple-1
try-job: test-aarch64-apple-2
try-job: test-x86_64-mingw-1
try-job: test-i686-msvc
try-job: test-armhf-gnu
rust-bors Bot pushed a commit that referenced this pull request Sep 12, 2026
…uwer

Rollup of 6 pull requests

Successful merges:

 - #162623 (Use GrowableBitSet in the main loop of `rustc_mir_transform::sroa`)
 - #162678 (Disconnect `rustc_codegen_ssa` from `rustc_mir_transform`)
 - #162641 (Remove unused arguments of functions)
 - #162663 (Rustdoc issue template: fix typo in comment ("thorugh"))
 - #162684 (Add `E0747` explanation for `type/const` mismatch case)
 - #162685 (Stabilize `Vec::from_fn`)
@JonathanBrouwer

Copy link
Copy Markdown
Member Author

@bors yield

@rust-bors

rust-bors Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

Auto build was cancelled. Cancelled workflows:

The next pull request likely to be tested is #160377.

@rust-bors

rust-bors Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

☀️ Try build successful (CI)
Build commit: a1f535e (a1f535e3a625fd0856c63cfe81f3ad47bbab524b)
Base parent: 9c99d05 (9c99d05505bccb67912d68e05fe7fc7c58afcb41)

@rust-bors

This comment has been minimized.

@rust-bors rust-bors Bot added merged-by-bors This PR was explicitly merged by bors. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Sep 12, 2026
@rust-bors

rust-bors Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

☀️ Test successful - CI
Approved by: JonathanBrouwer
Duration: 3h 13m 46s
Pushing 809936e to main...

@rust-bors
rust-bors Bot merged commit 809936e into rust-lang:main Sep 12, 2026
15 checks passed
@rustbot rustbot added this to the 1.100.0 milestone Sep 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor
What is this? This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.

Comparing adcc5f3 (parent) -> 809936e (this PR)

Test differences

Show 393 test diffs

Stage 0

  • bit_set::tests::growable_union: [missing] -> pass (J0)

Stage 1

  • bit_set::tests::growable_union: [missing] -> pass (J1)

Additionally, 391 doctest diffs were found. These are ignored, as they are noisy.

Job group index

Test dashboard

Run

cargo run --manifest-path src/ci/citool/Cargo.toml -- \
    test-dashboard 809936eac66c547a5127ce1da805f0d3a6789b98 --output-dir test-dashboard

And then open test-dashboard/index.html in your browser to see an overview of all executed tests.

Job duration changes

  1. test-x86_64-gnu-stdlib-semver-check: 11m 12s -> 18m 26s (+64.6%)
  2. test-x86_64-gnu-gcc-core-tests: 11m 9s -> 18m 10s (+62.9%)
  3. test-x86_64-mingw-2: 1h 53m -> 2h 50m (+50.1%)
  4. test-x86_64-gnu-gcc: 58m 51s -> 1h 26m (+46.2%)
  5. test-x86_64-gnu-debug: 1h 32m -> 2h 11m (+42.1%)
  6. dist-powerpc64le-linux-musl: 1h 12m -> 1h 40m (+38.9%)
  7. test-i686-gnu-1: 1h 40m -> 2h 18m (+38.4%)
  8. test-x86_64-gnu-llvm-22-3: 1h 26m -> 1h 55m (+33.4%)
  9. test-x86_64-gnu-llvm-21-2: 1h 44m -> 1h 11m (-32.0%)
  10. dist-x86_64-msvc: 2h 27m -> 3h 12m (+30.8%)
How to interpret the job duration changes?

Job durations can vary a lot, based on the actual runner instance
that executed the job, system noise, invalidated caches, etc. The table above is provided
mostly for t-infra members, for simpler debugging of potential CI slow-downs.

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 T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. WG-trait-system-refactor The Rustc Trait System Refactor Initiative (-Znext-solver)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants