-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Expose AVX512 Compare and BlendVariable APIs #87404
Conversation
Note regarding the This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change. |
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch Issue DetailsThis makes progress towards #87097
|
CC. @dotnet/jit-contrib, @dotnet/avx512-contrib This is the first of the APIs that required VectorMask under the old design and which abstract that away in lowering instead under the new design. There should be 2-3 more PRs to finish out the public API surface for them. That will cover That will then finish the public API surface for the new design. There would still be further possible work, but it would be just polish/light-up in lowering to better handle the vector mask like scenarios. |
This is still ready for review, just merging in upstream since its been a couple days since CI completed. |
This makes progress towards #87097