-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Arm64: [SVE] Fix failing NaN comparisons in Sve2.ConvertTo* tests #121706
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
base: main
Are you sure you want to change the base?
Arm64: [SVE] Fix failing NaN comparisons in Sve2.ConvertTo* tests #121706
Conversation
|
@SwapnilGaikwad do you need to run |
It would be useful, cheers. However, the other |
|
/azp run runtime-coreclr outerloop |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@SwapnilGaikwad it looks like this change introduced a lot more new failures from 40-50 to 800+ |
Apologies. Now limited the changes to only ConvertTo* tests using separate templates. Let's see how this goes, didn't show any failures locally. |
|
Hi @EgorBo, should we run |
|
/azp run runtime-coreclr outerloop |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch |
|
Seems majority of the failures are related to |
I though that the changes made here for eng/pipelines/coreclr/hardware-intrinsics.yml meant that the hardware instrinsic tests would all automatically be run for any PR that touches |
@jkoritzinsky @dotnet/jit-contrib any YML experts? |
Contribute towards #121659
This PR fixes the NaN comparisons that were incorrectly handled in ConvertTo* tests as highlighted in the following screenshot.
Failure output.

@dotnet/arm64-contrib @a74nh @EgorBo