-
-
Notifications
You must be signed in to change notification settings - Fork 14.3k
stdarch subtree update #150357
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
stdarch subtree update #150357
Conversation
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
s390x: use `simd_shuffle!` macro
use `wrapping_add` for `_kadd` intrinsics
instead of len(intrinsic) output streams
…-test-updates `intrinsic-test`: Coalescing all intrinsic tests into a single run of the test-file binaries
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.
This comment has been minimized.
This comment has been minimized.
|
💔 Test failed - checks-actions |
62b5c88 to
6f06d38
Compare
|
The Miri subtree was changed cc @rust-lang/miri Some changes occurred in src/tools/rustfmt cc @rust-lang/rustfmt |
|
@bors2 try jobs=dist-various-2 |
This comment has been minimized.
This comment has been minimized.
stdarch subtree update try-job: dist-various-2
|
Ok, it looks like I found the missing unsafe cpuid calls now (no other changes), let's try again: |
|
☀️ Test successful - checks-actions |
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 differencesShow 44 test diffs44 doctest diffs were found. These are ignored, as they are noisy. Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard 2e854a9344154564259de51385e9ec9506c0f3b7 --output-dir test-dashboardAnd then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
|
Finished benchmarking commit (2e854a9): comparison URL. Overall result: ❌✅ regressions and improvements - no action needed@rustbot label: -perf-regression Instruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
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.
CyclesResults (secondary 2.9%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 483.337s -> 482.393s (-0.20%) |
Subtree update of
stdarchto 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.