Skip to content

Conversation

@kunalspathak
Copy link
Contributor

@kunalspathak kunalspathak commented Apr 25, 2024

Implement the following math operations:

  • Sve.Divide
  • Sve.Multiply
  • Sve.Subtract

I have removed the *DivR and *SubR from the comments because the APIs do not support it and I am not sure if there are any APIs for those instructions.

Contributes to #99957

@ghost
Copy link

ghost commented Apr 25, 2024

Note regarding the new-api-needs-documentation label:

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.

@kunalspathak
Copy link
Contributor Author

@kunalspathak
Copy link
Contributor Author

@dotnet/arm64-contrib

Copy link
Contributor

@a74nh a74nh left a comment

Choose a reason for hiding this comment

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

This LGTM.

@kunalspathak
Copy link
Contributor Author

/ba-g The failure is #101634 which for some reason is not tagged by BuildAnalysis

@kunalspathak kunalspathak merged commit 7031052 into dotnet:main Apr 29, 2024
@kunalspathak kunalspathak deleted the sve_math2 branch April 29, 2024 18:11
matouskozak pushed a commit to matouskozak/runtime that referenced this pull request Apr 30, 2024
* Add Divide/Multiply/Subtract

* Add hwintrinsic support for Divide,Multiply and Subtract

* remove space

* Add test cases

* trim space

* handle the scenario for 2 operands predicated-only instruction

* Remove *DIVR and *SUBR from comments

* fix jit format

* fix merge conflict
michaelgsharp pushed a commit to michaelgsharp/runtime that referenced this pull request May 9, 2024
* Add Divide/Multiply/Subtract

* Add hwintrinsic support for Divide,Multiply and Subtract

* remove space

* Add test cases

* trim space

* handle the scenario for 2 operands predicated-only instruction

* Remove *DIVR and *SUBR from comments

* fix jit format

* fix merge conflict
Ruihan-Yin pushed a commit to Ruihan-Yin/runtime that referenced this pull request May 30, 2024
* Add Divide/Multiply/Subtract

* Add hwintrinsic support for Divide,Multiply and Subtract

* remove space

* Add test cases

* trim space

* handle the scenario for 2 operands predicated-only instruction

* Remove *DIVR and *SUBR from comments

* fix jit format

* fix merge conflict
@github-actions github-actions bot locked and limited conversation to collaborators May 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

arm-sve Work related to arm64 SVE/SVE2 support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants