Skip to content

Conversation

dpaoliello
Copy link
Contributor

@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. labels Jul 9, 2025
@dianqk
Copy link
Member

dianqk commented Jul 10, 2025

Unlock #140136.
@bors r+ rollup=never p=1

@bors
Copy link
Collaborator

bors commented Jul 10, 2025

📌 Commit 65dc474 has been approved by dianqk

It is now in the queue for this repository.

@bors bors 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 Jul 10, 2025
@bors
Copy link
Collaborator

bors commented Jul 10, 2025

⌛ Testing commit 65dc474 with merge 32cd911...

@bors
Copy link
Collaborator

bors commented Jul 10, 2025

☀️ Test successful - checks-actions
Approved by: dianqk
Pushing 32cd911 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Jul 10, 2025
@bors bors merged commit 32cd911 into rust-lang:master Jul 10, 2025
12 checks passed
@rustbot rustbot added this to the 1.90.0 milestone Jul 10, 2025
Copy link
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 607fbd8 (parent) -> 32cd911 (this PR)

Test differences

Show 3 test diffs

Stage 2

  • [run-make] tests/run-make/compressed-debuginfo-zstd: pass -> ignore (ignored if LLVM wasn't build with zstd for ELF section compression (we want LLVM/LLD to be built with zstd support)) (J0)

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

Job group index

  • J0: x86_64-gnu-nopt
Test dashboard

Run

cargo run --manifest-path src/ci/citool/Cargo.toml -- \
    test-dashboard 32cd9114712a24010b0583624dc52ac302194128 --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-rust-for-linux: 2527.0s -> 3929.4s (55.5%)
  2. x86_64-apple-1: 5716.5s -> 8880.1s (55.3%)
  3. pr-check-1: 1527.6s -> 2358.2s (54.4%)
  4. pr-check-2: 2113.3s -> 3231.9s (52.9%)
  5. x86_64-gnu-tools: 3298.4s -> 4726.1s (43.3%)
  6. dist-apple-various: 6251.7s -> 4154.3s (-33.5%)
  7. aarch64-apple: 4327.5s -> 5690.4s (31.5%)
  8. x86_64-gnu-miri: 4603.8s -> 5978.1s (29.8%)
  9. dist-aarch64-linux: 8016.2s -> 5682.7s (-29.1%)
  10. armhf-gnu: 4909.3s -> 6303.1s (28.4%)
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.

@bors bors mentioned this pull request Jul 10, 2025
8 tasks
@rust-timer
Copy link
Collaborator

Finished benchmarking commit (32cd911): comparison URL.

Overall result: no relevant changes - no action needed

@rustbot label: -perf-regression

Instruction count

This benchmark run did not return any relevant results for this metric.

Max RSS (memory usage)

Results (secondary 0.7%)

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)
1.9% [1.1%, 2.8%] 4
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
-1.7% [-1.8%, -1.7%] 2
All ❌✅ (primary) - - 0

Cycles

Results (secondary 3.5%)

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)
3.5% [3.5%, 3.5%] 1
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
- - 0
All ❌✅ (primary) - - 0

Binary size

Results (secondary -0.0%)

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)
-0.0% [-0.0%, -0.0%] 1
All ❌✅ (primary) - - 0

Bootstrap: 464.395s -> 465.513s (0.24%)
Artifact size: 374.48 MiB -> 374.51 MiB (0.01%)

@dpaoliello dpaoliello deleted the llvmupate branch August 5, 2025 00:09
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. 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.

6 participants