-
Notifications
You must be signed in to change notification settings - Fork 5k
Arm64/Sve: Missing test coverage for invalid immediate #104809
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
Labels
area-CodeGen-coreclr
CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI
arm-sve
Work related to arm64 SVE/SVE2 support
in-pr
There is an active PR which will close this issue when it is merged
Priority:2
Work that is important, but not critical for the release
Milestone
Comments
Assigning to @TIHan for now and we will re-distribute it if needed. |
@dotnet/arm64-contrib |
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch |
priority:2 for RC1 snap : Missing testing |
Assign to myself because Mikhail is working on it. |
amanasifkhalid
pushed a commit
that referenced
this issue
Aug 8, 2024
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
area-CodeGen-coreclr
CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI
arm-sve
Work related to arm64 SVE/SVE2 support
in-pr
There is an active PR which will close this issue when it is merged
Priority:2
Work that is important, but not critical for the release
All the test templates that verify the invalid immediate, is missing this piece of code. Thanks @amanasifkhalid for bringing this to my attention in #104697 (comment).
There are also certain templates like
_SveImmBinaryOpTestTemplate.template
that doesn't even have this test method.*Imm*.template
files that were added/updated as part of SVE work and make sure we add test coverage for it.{Imm}
Imm
ArgumentOutOfRangeException
ArgumentOutOfRangeException
The text was updated successfully, but these errors were encountered: