Skip to content

Stabilize the stdarch_neon_dotprod feature#2120

Merged
folkertdev merged 1 commit into
rust-lang:mainfrom
lupodevelop:stabilize-neon-dotprod
Jun 3, 2026
Merged

Stabilize the stdarch_neon_dotprod feature#2120
folkertdev merged 1 commit into
rust-lang:mainfrom
lupodevelop:stabilize-neon-dotprod

Conversation

@lupodevelop

@lupodevelop lupodevelop commented May 18, 2026

Copy link
Copy Markdown
Contributor

Closes rust-lang/rust#117224

Stabilizes the NEON dot product intrinsics (vdot*), tracking issue rust-lang/rust#117224.

Gated behind stdarch_neon_dotprod since 2023. The dotprod target feature and is_aarch64_feature_detected!("dotprod") are already stable, so this is an attribute-only change, same pattern as the fp16 stabilization in 0f9516b.

The vdot*_laneq* blocks had a plain unstable with no arch split.
I gave them the not(arm) split like the others for consistency.

Regenerated with stdarch-gen-arm, the diff is the stability attribute only.

Needs an FCP.

Stabilizes the NEON dot product intrinsics tracked by
rust-lang/rust#117224:

  vdot{q}_{s,u}32
  vdot{q}_lane_{s,u}32
  vdot{q}_laneq_{s,u}32

These have been available unstably since 2023. The `dotprod` target
feature and `is_aarch64_feature_detected!("dotprod")` are already
stable, so this only flips the intrinsics' stability attribute on
aarch64 and arm64ec.

On AArch32 the intrinsics remain unstable under
`stdarch_arm_neon_intrinsics` (rust-lang/rust#111800), consistent with
the rest of the AArch32 NEON surface.
@rustbot

rustbot commented May 18, 2026

Copy link
Copy Markdown
Collaborator

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

Why was this reviewer chosen?

The reviewer was selected based on:

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

@folkertdev

Copy link
Copy Markdown
Contributor

This needs a green light from T-libs-api, I've nominated the tracking issue so this will be discussed in one of their next weekly meetings.

@lupodevelop

Copy link
Copy Markdown
Contributor Author

Perfect! I "gave this push" because I had exchanges with several people who are waiting for this

@folkertdev folkertdev added the S-waiting-on-fcp Status: PR is in FCP and is awaiting for FCP to complete. label May 25, 2026
@folkertdev folkertdev removed the S-waiting-on-fcp Status: PR is in FCP and is awaiting for FCP to complete. label Jun 3, 2026

@folkertdev folkertdev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@folkertdev folkertdev added this pull request to the merge queue Jun 3, 2026
Merged via the queue into rust-lang:main with commit 05ced9d Jun 3, 2026
74 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.

Tracking Issue for NEON dot product intrinsics

3 participants