Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Arm64 SVE: Use mask versions of instructions where possible. #103078

Open
a74nh opened this issue Jun 5, 2024 · 2 comments
Open

Arm64 SVE: Use mask versions of instructions where possible. #103078

a74nh opened this issue Jun 5, 2024 · 2 comments
Labels
area-System.Runtime.Intrinsics arm-sve Work related to arm64 SVE/SVE2 support Priority:1 Work that is critical for the release, but we could probably ship without
Milestone
10.0.0

Comments

@a74nh
Copy link
Contributor

a74nh commented Jun 5, 2024

Some SVE instructions have both vector and mask versions.

TRN1 <Zd>.<T>, <Zn>.<T>, <Zm>.<T>
TRN1 <Pd>.<T>, <Pn>.<T>, <Pm>.<T>

Currently if we pass masks into Sve.TransposeOdd() then it will convert the masks to vectors and use the vector version. Instead, this should be detected and the mask version used instead.

TODO: Add a list of all the applicable instructions here
@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jun 5, 2024
@dotnet-policy-service dotnet-policy-service bot added the untriaged New issue has not been triaged by the area owner label Jun 5, 2024
@a74nh a74nh mentioned this issue Jun 5, 2024
Merged
@kunalspathak kunalspathak added the arm-sve Work related to arm64 SVE/SVE2 support label Jun 5, 2024
@kunalspathak
Copy link
Member

@dotnet/arm64-contrib

@kunalspathak kunalspathak removed the untriaged New issue has not been triaged by the area owner label Jun 5, 2024
@vcsjones vcsjones added area-System.Runtime.Intrinsics and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Jun 5, 2024
@dotnet-policy-service Dotnet Policy Service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-system-runtime-intrinsics
See info in area-owners.md if you want to be subscribed.

@kunalspathak kunalspathak mentioned this issue Jun 6, 2024
Closed
@jeffhandley jeffhandley added this to the 10.0.0 milestone Jul 27, 2024
@kunalspathak kunalspathak added the Priority:1 Work that is critical for the release, but we could probably ship without label Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-System.Runtime.Intrinsics arm-sve Work related to arm64 SVE/SVE2 support Priority:1 Work that is critical for the release, but we could probably ship without
Projects
None yet
Milestone
10.0.0
Development

No branches or pull requests
4 participants
@vcsjones @jeffhandley @a74nh @kunalspathak