-
Notifications
You must be signed in to change notification settings - Fork 5.2k
SVE2 FP API's #118332
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
SVE2 FP API's #118332
Conversation
src/libraries/System.Private.CoreLib/src/System/Runtime/Intrinsics/Arm/Sve2.cs
Show resolved
Hide resolved
75a72cb
to
0e704e4
Compare
0e704e4
to
1715a20
Compare
1715a20
to
7707044
Compare
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.
Thanks for all the revisions on this! Looks like some of the conditional select scenarios for FloatingPointExponentialAccelerator
failed (console log). Could you please take a look?
Contains implementations for: - AddRotateComplex - AddSaturateRotateComplex - ConvertToDoubleOdd - ConvertToSingleEvenRoundToOdd - Log2 - MultiplyAddRotateComplex - MultiplyAddRotateComplexBySelectedScalar - MultiplyAddRoundedDoublingSaturateHighRotateComplex* - ReciprocalEstimate - ReciprocalSqrtEstimate
Change-Id: I79efe0da0fd205b2856fa339f08daa3a5bacc200
7707044
to
2e68387
Compare
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
/ba-g mono test timed out |
@dotnet/arm64-contrib @amanasifkhalid @a74nh
Contributing towards #115479
Contains implementations for the set of SVE2 FP API's, aside from
ConvertToSingleOdd
andConvertToSingleOddRoundToOdd
. This is due to concerns I have about the API proposal for these intrinsics being incorrect, when I have a resolution to my comments on this they will be implemented.