Skip to content

Porting some intrinsics to stdarch-gen-arm#2100

Open
sayantn wants to merge 6 commits intorust-lang:mainfrom
sayantn:gen-arm-ports
Open

Porting some intrinsics to stdarch-gen-arm#2100
sayantn wants to merge 6 commits intorust-lang:mainfrom
sayantn:gen-arm-ports

Conversation

@sayantn
Copy link
Copy Markdown
Contributor

@sayantn sayantn commented May 2, 2026

These intrinsics were implemented in aarch64/neon/mod.rs, so this PR ports them to stdarch-gen-arm. All the edits are by @davidtwco, I have just cherry-picked from his branch and putting up the PR

@rustbot

This comment has been minimized.

Copy link
Copy Markdown
Member

@davidtwco davidtwco left a comment

Choose a reason for hiding this comment

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

I authored these changes so unsurprisingly they look good to me

View changes since this review

davidtwco added 6 commits May 5, 2026 12:38
This enables the `big_endian_inverse` transformation to apply to this
intrinsic.
This enables the `big_endian_inverse` transformation to apply to these
intrinsics.
This enables the `big_endian_inverse` transform to apply to these
intrinsics.
This enables the `big_endian_inverse` transform to apply to this
intrinsic.
This enables the `big_endian_inverse` transform to apply to this
intrinsic.
Manually written tests for `vcopy_laneq`, `vget{,q}_{high,low,lane}`,
`vcombine` and `vaddw_high` don't account for the expected differences
in output for big endian intrinsics and given that they duplicate the
testing of `intrinsic-test`, can be removed.
@davidtwco davidtwco marked this pull request as ready for review May 5, 2026 11:40
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented May 5, 2026

r? @Amanieu

rustbot has assigned @Amanieu.
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

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @Amanieu, @folkertdev, @sayantn
  • @Amanieu, @folkertdev, @sayantn expanded to Amanieu, folkertdev, sayantn
  • Random selection from Amanieu, folkertdev

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.

4 participants