-
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
Count16BitElements() intrinsic for SVE #94845
Conversation
* Add encodings for SVE * Adjust the offsets * jit format * fix the build for riscV64 * Address review feedback from Bruce * Rename files to remove _ * forgot to rename file names in other places Change-Id: I54932d16104a2582d9afc3fca6844183aa8c3536
Change-Id: Ie54c83275cced2f4fbc0f93e8cb687084547ac45
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 DetailsExtracted from #94765 For now, this is based off both #94529 and #94791 For now it emits an
|
using System.Numerics; | ||
using Xunit; | ||
|
||
namespace JIT.HardwareIntrinsics.Arm._AdvSimd |
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.
Should this be _Sve?
This needs rebasing. But still can't progress into until skeleton library files are committed. |
Draft Pull Request was automatically closed for 30 days of inactivity. Please let us know if you'd like to reopen it. |
Extracted from #94765
For now, this is based off both #94529 and #94791
All the new code starts from 3cb75c5
For now it emits an
MRS
instruction. This needs updating to use an SVECNTH
, however, the codegen parts need adding alongside. Will wait until I can use the tool.JitDump
ing the test program, I see theMRS