Skip to content

Conversation

@CrooseGit
Copy link
Contributor

  • This patch begins by fixing build failures that occurred due to failing LLVM assertions when building arm neon intrinsics in debug.
  • It then addresses some differences the intrinsic tests flagged between C and rust on big endian aarch64 caused by an issue in in both clang and rust.
  • Finally it reworks some of the changes from previous step as they were causing instr assert test failures on armv7 in release due to an llvm bug.

This is part 1/2 of this PR which was asked to be split in two.

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
@rustbot
Copy link
Collaborator

rustbot commented Nov 27, 2025

r? @folkertdev

rustbot has assigned @folkertdev.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Copy link
Contributor

@folkertdev folkertdev left a comment

Choose a reason for hiding this comment

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

Nice, thanks!

@folkertdev folkertdev added this pull request to the merge queue Nov 27, 2025
Merged via the queue into rust-lang:main with commit 6209f35 Nov 27, 2025
73 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants