-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Remove the rest of the SimdAsHWIntrinsic support #106594
Conversation
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch |
98d5353
to
5c7a91b
Compare
This one is for .NET 10 only, it does not need backport |
/azp run runtime-coreclr jitstress-isas-x86, jitstress-isas-arm64, Fuzzlyn |
Azure Pipelines successfully started running 2 pipeline(s). |
CC. @dotnet/jit-contrib This is ready for review. This removes the last of the handling that was specific to This results in some fairly significant size and throughput wins for both x64 and Arm64, mostly stemming from places that the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice clean up! I've noticed some missing ternary ops (avx512) among the regressions but I guess it's not a big deal
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This removes quite a bit of complexity and ensures that
Vector<T>
handling is identical to the handling for the fixed sized vector types on exposed APIs.