Skip to content

Conversation

@ZuseZ4
Copy link
Member

@ZuseZ4 ZuseZ4 commented Dec 24, 2025

Subtree update of stdarch to rust-lang/stdarch@6111906.

Created using https://github.com/rust-lang/josh-sync.

r? @sayantn

My first josh sync, it lgtm, but let me know if I missed something.
I'm especially looking forward to the amd GPU module, which we want to use for the offload project.

CrooseGit and others added 30 commits November 24, 2025 15:12
This fixes build issues associated with failing LLVM const param assertions
The intrinsics test was flagging differences in aarch64_be between rust in debug and clang in O2.
It was found that rust was correct in debug, but incorrect in release,
and in both cases were being compared against clang in O2 which was also
incorrect.

The vdot intrinsics were fixed and are now correct in rust for both
release and debug. However the vcmla ones could not be as the issue lies
with LLVM. Both the vdot and vcmla intrinsics were added to the skiplist
as clang is still incorrect for both.
LLVM issue: llvm/llvm-project#166190
…rinsics-debug

Fixes issue with arm intrinsics not compiling in dev profile
Use generic SIMD intrinsics for cvtepi intrinsics
Correct the docs for WASM relaxed_simd's `relaxed_min`/`relaxed_max` operations
Format `f16` values correctly in intrinsic-test
Make `cpuid` safe and update docs
use `wrapping_add` for `_kadd` intrinsics
…-test-updates

`intrinsic-test`: Coalescing all intrinsic tests into a single run of the test-file binaries
@bors
Copy link
Collaborator

bors commented Dec 26, 2025

📌 Commit 62b5c88 has been approved by sayantn

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 Dec 26, 2025
@bors
Copy link
Collaborator

bors commented Dec 26, 2025

⌛ Testing commit 62b5c88 with merge 0301e3f...

bors added a commit that referenced this pull request Dec 26, 2025
stdarch subtree update

Subtree update of `stdarch` to rust-lang/stdarch@6111906.

Created using https://github.com/rust-lang/josh-sync.

r? `@sayantn`

My first josh sync, it lgtm, but let me know if I missed something.
I'm especially looking forward to the amd GPU module, which we want to use for the offload project.
@rust-log-analyzer

This comment has been minimized.

@bors
Copy link
Collaborator

bors commented Dec 26, 2025

💔 Test failed - checks-actions

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Dec 26, 2025
@ZuseZ4 ZuseZ4 force-pushed the zusez4-stdarch-push branch from 62b5c88 to 6f06d38 Compare December 26, 2025 07:46
@rustbot
Copy link
Collaborator

rustbot commented Dec 26, 2025

The Miri subtree was changed

cc @rust-lang/miri

Some changes occurred in src/tools/rustfmt

cc @rust-lang/rustfmt

@rustbot rustbot added the T-rustfmt Relevant to the rustfmt team, which will review and decide on the PR/issue. label Dec 26, 2025
@ZuseZ4
Copy link
Member Author

ZuseZ4 commented Dec 26, 2025

@bors2 try jobs=dist-various-2

@rust-bors

This comment has been minimized.

rust-bors bot added a commit that referenced this pull request Dec 26, 2025
stdarch subtree update

try-job: dist-various-2
@rust-bors
Copy link

rust-bors bot commented Dec 26, 2025

☀️ Try build successful (CI)
Build commit: d16d763 (d16d76398507db7765d67ac7ec447a2e3dc24320, parent: fabece9e9491d0a3c3655dba488881968b7c5f7a)

@ZuseZ4
Copy link
Member Author

ZuseZ4 commented Dec 26, 2025

Ok, it looks like I found the missing unsafe cpuid calls now (no other changes), let's try again:
@bors r=sayantan

@bors
Copy link
Collaborator

bors commented Dec 26, 2025

📌 Commit 6f06d38 has been approved by sayantan

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 Dec 26, 2025
@bors
Copy link
Collaborator

bors commented Dec 26, 2025

⌛ Testing commit 6f06d38 with merge 2e854a9...

@bors
Copy link
Collaborator

bors commented Dec 26, 2025

☀️ Test successful - checks-actions
Approved by: sayantan
Pushing 2e854a9 to main...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Dec 26, 2025
@bors bors merged commit 2e854a9 into rust-lang:main Dec 26, 2025
13 checks passed
@rustbot rustbot added this to the 1.94.0 milestone Dec 26, 2025
@github-actions
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 4c383cd (parent) -> 2e854a9 (this PR)

Test differences

Show 44 test diffs

44 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 2e854a9344154564259de51385e9ec9506c0f3b7 --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. aarch64-apple: 11756.3s -> 10011.1s (-14.8%)
  2. dist-apple-various: 3740.9s -> 4211.9s (+12.6%)
  3. dist-aarch64-msvc: 5336.2s -> 5953.4s (+11.6%)
  4. x86_64-mingw-1: 9163.8s -> 10148.4s (+10.7%)
  5. aarch64-gnu: 7180.4s -> 7829.8s (+9.0%)
  6. dist-x86_64-msvc-alt: 8857.3s -> 9626.7s (+8.7%)
  7. pr-check-1: 1531.8s -> 1663.9s (+8.6%)
  8. x86_64-gnu: 7534.4s -> 8129.0s (+7.9%)
  9. dist-x86_64-llvm-mingw: 7137.3s -> 6632.5s (-7.1%)
  10. dist-various-2: 2223.5s -> 2081.3s (-6.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.

@ZuseZ4 ZuseZ4 deleted the zusez4-stdarch-push branch December 26, 2025 14:08
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For future reference rustfmt formatting tests shouldn't really need to be updated

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh sorry @ytmimi, I hope I didn't cause too much extra work by that.
I had missed that the last CI failure here didn't include the rustfmt tests, I'll be more careful with cleanups in my next sync.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall it's not a big deal. I just wanted to point out that we don't need to make any changes for existing formatting test. Hopefully it saves a little time for you in the future.

@rust-timer
Copy link
Collaborator

Finished benchmarking commit (2e854a9): comparison URL.

Overall result: ❌✅ regressions and improvements - no action needed

@rustbot label: -perf-regression

Instruction count

Our most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.

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

Max RSS (memory usage)

Results (primary -2.5%, secondary -2.3%)

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

mean range count
Regressions ❌
(primary)
1.0% [1.0%, 1.0%] 1
Regressions ❌
(secondary)
2.0% [1.8%, 2.1%] 3
Improvements ✅
(primary)
-3.6% [-7.0%, -0.8%] 3
Improvements ✅
(secondary)
-4.5% [-5.1%, -4.1%] 6
All ❌✅ (primary) -2.5% [-7.0%, 1.0%] 4

Cycles

Results (secondary 2.9%)

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)
4.1% [2.1%, 6.5%] 4
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
-2.0% [-2.0%, -2.0%] 1
All ❌✅ (primary) - - 0

Binary size

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

Bootstrap: 483.337s -> 482.393s (-0.20%)
Artifact size: 392.36 MiB -> 390.46 MiB (-0.48%)

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

Labels

A-compiler-builtins Area: compiler-builtins (https://github.com/rust-lang/compiler-builtins) 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. T-libs Relevant to the library team, which will review and decide on the PR/issue. T-rustfmt Relevant to the rustfmt team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.