Skip to content

Conversation

@lewing
Copy link
Member

@lewing lewing commented Mar 20, 2025

Use packsimd intrinsics where they match since the implementations there are more complete

@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @BrzVlad, @kotlarmilos
See info in area-owners.md if you want to be subscribed.

@lewing lewing force-pushed the forward-packed-simd branch 2 times, most recently from 0661597 to 94c11a3 Compare March 20, 2025 22:18
@lewing lewing requested a review from radekdoulik March 20, 2025 22:21
@lewing lewing force-pushed the forward-packed-simd branch 4 times, most recently from 37cd4f6 to 7965046 Compare March 21, 2025 04:46
@lewing lewing force-pushed the forward-packed-simd branch from 4e5befb to f4871ec Compare March 21, 2025 19:04
@lewing lewing marked this pull request as ready for review March 30, 2025 16:38
Copilot AI review requested due to automatic review settings March 30, 2025 16:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Files not reviewed (2)
  • src/mono/mono/mini/interp/simd-methods.def: Language not supported
  • src/mono/mono/mini/interp/transform-simd.c: Language not supported

@lewing
Copy link
Member Author

lewing commented Mar 30, 2025

/ba-g failures are not related

@lewing lewing merged commit 7cc3219 into dotnet:main Mar 30, 2025
68 of 71 checks passed
@lewing lewing deleted the forward-packed-simd branch March 30, 2025 19:54
thaystg pushed a commit to thaystg/runtime that referenced this pull request Apr 1, 2025
In the Wasm interpreter, start using the PackedSimd intrinsics directly where they match the Vector intrinsics, Translate the names where appropriate.
@github-actions github-actions bot locked and limited conversation to collaborators Apr 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants