Skip to content

Conversation

@SwapnilGaikwad
Copy link
Contributor

@SwapnilGaikwad SwapnilGaikwad commented Nov 17, 2025

Contribute towards #121659

This PR fixes the NaN comparisons that were incorrectly handled in ConvertTo* tests as highlighted in the following screenshot.

Failure output.
image

@dotnet/arm64-contrib @a74nh @EgorBo

@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Nov 17, 2025
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Nov 17, 2025
@EgorBo
Copy link
Member

EgorBo commented Nov 17, 2025

@SwapnilGaikwad do you need to run runtime-coreclr outerloop?

@SwapnilGaikwad SwapnilGaikwad changed the title Fix failing NaN comparisons in Sve2.ConvertTo* tests Arm64: [SVE] Fix failing NaN comparisons in Sve2.ConvertTo* tests Nov 17, 2025
@SwapnilGaikwad
Copy link
Contributor Author

runtime-coreclr outerloop

It would be useful, cheers. However, the other Sve2_ShiftArithmeticSaturate* tests would still fail. Up to you if you want to wait until those are fixed.

@EgorBo
Copy link
Member

EgorBo commented Nov 17, 2025

/azp run runtime-coreclr outerloop

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@EgorBo
Copy link
Member

EgorBo commented Nov 17, 2025

@SwapnilGaikwad it looks like this change introduced a lot more new failures from 40-50 to 800+

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution Indicates that the PR has been added by a community member needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants