Skip to content

Sve2 AddHighNarrowing (Even/Odd) #116848

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.

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

Merged
merged 6 commits into from
Jun 30, 2025
Merged

Conversation

jacob-crawley
Copy link
Contributor

@a74nh @kunalspathak

Contributing towards #115479

Copy link
Contributor

@kunalspathak kunalspathak left a comment

Choose a reason for hiding this comment

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

added some comments around test.

Copy link
Contributor

@kunalspathak kunalspathak left a comment

Choose a reason for hiding this comment

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

LGTM

@kunalspathak kunalspathak enabled auto-merge (squash) June 25, 2025 16:21
@a74nh
Copy link
Contributor

a74nh commented Jun 30, 2025

@dotnet/arm64-contrib @amanasifkhalid

auto-merge was automatically disabled June 30, 2025 12:54

Head branch was pushed to by a user without write access

@jacob-crawley jacob-crawley changed the title Sve2 AddHighNarrowing (Upper/Lower) Sve2 AddHighNarrowing (Even/Odd) Jun 30, 2025
Copy link
Member

@amanasifkhalid amanasifkhalid left a comment

Choose a reason for hiding this comment

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

LGTM too, thanks for the test cleanup

@amanasifkhalid
Copy link
Member

/ba-g hit #111922 on x64 config

@amanasifkhalid amanasifkhalid merged commit 8ceabe8 into dotnet:main Jun 30, 2025
155 of 162 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jul 31, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Runtime.Intrinsics community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants