Skip to content

Rollup of 4 pull requests - #161822

Merged
rust-bors[bot] merged 9 commits into
rust-lang:mainfrom
JonathanBrouwer:rollup-ZOtovjn
Aug 26, 2026
Merged

Rollup of 4 pull requests#161822
rust-bors[bot] merged 9 commits into
rust-lang:mainfrom
JonathanBrouwer:rollup-ZOtovjn

Conversation

@JonathanBrouwer

Copy link
Copy Markdown
Member

Successful merges:

r? @ghost

Create a similar rollup

fereidani and others added 9 commits January 5, 2026 13:25
…codes

UEFI status codes are usize-wide with the high bit set for errors
(e.g. DEVICE_ERROR is 0x8000000000000007 on 64-bit). the previous
`as i32` cast silently dropped the upper bits, losing all error
information. stripping the error high bit to extract the error number
and negating it, so errors (negative) are distinguishable from
success/warning codes (non-negative).
Alloc `String::retain` optimization

Hi again!
This uses the exact same algorithm as my other PR: rust-lang#149784

Technically it should improve performance for `String::retain` too, But let's see what bors thinks.
std::process: fix UEFI ExitStatus::code() silent truncation of error …

…codes

UEFI status codes are usize-wide with the high bit set for errors (e.g. DEVICE_ERROR is 0x8000000000000007 on 64-bit). the previous `as i32` cast silently dropped the upper bits, losing all error information. stripping the error high bit to extract the error number and negating it, so errors (negative) are distinguishable from success/warning codes (non-negative).
…jorn3

Rename dlltool helper function

It doesn't check for MinGW toolchain (`windows-gnu` or `windows-gnullvm`), but `windows-gnu` only. All the callsites use that helper to decide whether dlltool should be used.

r? @bjorn3
@rust-bors rust-bors Bot added the rollup A PR which is a rollup label Aug 26, 2026
@rustbot rustbot added A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. 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. labels Aug 26, 2026
@JonathanBrouwer

Copy link
Copy Markdown
Member Author

@bors r+ p=5

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

@rust-bors

rust-bors Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

📌 Commit f7f990f 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 Aug 26, 2026
@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Aug 26, 2026
Rollup of 4 pull requests


try-job: dist-various-1
try-job: test-various
try-job: x86_64-gnu-aux
try-job: x86_64-gnu-llvm-21-3
try-job: x86_64-msvc-1
try-job: aarch64-apple-1
try-job: aarch64-apple-2
try-job: x86_64-mingw-1
try-job: i686-msvc-1
try-job: i686-msvc-2
@rust-bors rust-bors Bot added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Aug 26, 2026
@rust-bors

rust-bors Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

💔 Test for 87beb46 failed: CI

@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 Aug 26, 2026
@rust-bors

rust-bors Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

☀️ Test successful - CI
Approved by: JonathanBrouwer
Duration: 3h 7m 36s
Pushing 3b8ee6c to main...

@rust-bors
rust-bors Bot merged commit 3b8ee6c into rust-lang:main Aug 26, 2026
1 of 2 checks passed
@rustbot rustbot removed the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Aug 26, 2026
@rustbot rustbot added this to the 1.100.0 milestone Aug 26, 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 c241221 (parent) -> 3b8ee6c (this PR)

Test differences

Show 147 test diffs

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

Test dashboard

Run

cargo run --manifest-path src/ci/citool/Cargo.toml -- \
    test-dashboard 3b8ee6c0ca55afb08e2e130003227a3195394425 --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. x86_64-gnu-llvm-21-1: 35m 3s -> 57m 45s (+64.8%)
  2. x86_64-gnu-gcc-core-tests: 17m 9s -> 7m 57s (-53.6%)
  3. dist-ohos-x86_64: 56m 19s -> 1h 25m (+52.2%)
  4. x86_64-gnu-llvm-21-2: 1h 11m -> 1h 42m (+43.0%)
  5. x86_64-gnu-llvm-22-1: 1h 19m -> 49m 24s (-37.6%)
  6. x86_64-gnu-stdlib-semver-check: 17m 11s -> 11m 18s (-34.2%)
  7. arm-android: 1h 53m -> 1h 15m (-33.7%)
  8. x86_64-rust-for-linux: 56m 24s -> 38m 5s (-32.5%)
  9. i686-gnu-nopt-1: 2h 19m -> 1h 37m (-30.3%)
  10. x86_64-gnu-aux: 1h 57m -> 2h 33m (+30.2%)
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.

@rust-timer

Copy link
Copy Markdown
Collaborator

Finished benchmarking commit (3b8ee6c): comparison URL.

Overall result: no relevant changes - no action needed

@rustbot label: -perf-regression

Instruction count

This perf run didn't have relevant results for this metric.

Max RSS (memory usage)

Results (secondary -2.1%)

A less reliable metric. May be of interest, but not used to determine the overall result above.

mean range count
Regressions ❌
(primary)
- - 0
Regressions ❌
(secondary)
- - 0
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
-2.1% [-2.1%, -2.1%] 1
All ❌✅ (primary) - - 0

Cycles

This perf run didn't have relevant results for this metric.

Binary size

This perf run didn't have relevant results for this metric.

Bootstrap: 474.908s -> 473.778s (-0.24%)
Artifact size: 402.63 MiB -> 402.59 MiB (-0.01%)

@rust-bors

rust-bors Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

📌 Perf builds for each rolled up PR:

PR# Message Perf Build Sha
#150067 Alloc String::retain optimization 7bd5e543f2022d93513e77bae21f25ccc65417c8
(link)
#153973 std::process: fix UEFI ExitStatus::code() silent truncation… b1ee8efb1aef36eba7dbcc2377723dcc912f79e1
(link)
#161690 Deny #[inline] on EII declarations 1e41a50566a2ea11ea47628ef149e6d9afbb7e3f
(link)
#161701 Rename dlltool helper function 6f63701ba2a9eec4938139758a7b79a6bf1043a4
(link)

parent commit: c24122146b

In the case of a perf regression, run the following command for each PR you suspect might be the cause: @rust-timer build $SHA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. 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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants