Note regarding thenew-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, to 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.

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly onearea label.

Tagging subscribers to this area: @dotnet/area-system-numerics
See info inarea-owners.md if you want to be subscribed.

Marking asblocked until we confirm this won't break WPF. CC.@jeffhandley

Rebased to remove conflict

/azp run runtime-extra-platforms

@akoeplinger could you please check/confirm if the mono failures here are related to Static Abstracts in Interfaces?

@akoeplinger could you please check/confirm if the mono failures here are related to Static Abstracts in Interfaces?

The android failures look infra related and the wasm ones I believe are flaky. Not sure about the windows build failure:

LINK : fatal error LNK1181: cannot open input file 'ijwhost.lib'

Not sure about the windows build failure:

LINK : fatal error LNK1181: cannot open input file 'ijwhost.lib'

It was hit on the rolling build as well -#66380

The android failures look infra related

@steveisok, any way we could confirm this? I'd prefer to not have to back this out after merging since its unblocking a lot of work related to generic math and it shipping in .NET 7.

Most of the failures look to be showing up as:

System.AggregateException : One or more errors occurred. (Failed to install mobile app.
Expected: True
Actual: False) (The following constructor parameters did not have matching fixture data: _Global globalVar)
---- Failed to install mobile app.
Expected: True
Actual: False
---- The following constructor parameters did not have matching fixture data: _Global globalVar

Also seeing a few failures like

[21:17:57] info: Attempting to install /datadisks/disk1/work/A1EA0909/w/B8F90A60/e/System.Linq.Tests.apk
[21:17:57] dbug: Executing command: '/datadisks/disk1/work/A1EA0909/p/microsoft.dotnet.xharness.cli/1.0.0-prerelease.22153.2/runtimes/any/native/adb/linux/adb -s emulator-5556 install /datadisks/disk1/work/A1EA0909/w/B8F90A60/e/System.Linq.Tests.apk'
[21:17:57] fail: Error:
Exit code: 1
Std out:
Performing Streamed Install

            Std err:            adb: failed to install /datadisks/disk1/work/A1EA0909/w/B8F90A60/e/System.Linq.Tests.apk: cmd: Can't find service: package

And then for WASM:

DevOpsReportFailure: Failed to upload results: Error occurred in request., RetryError: HTTPSConnectionPool(host='dev.azure.com', port=443): Max retries exceeded with url: /dnceng/public/_apis/test/Runs/45564706/Results (Caused by ResponseError('too many 503 error responses',))

@tannergooding I'm aware of both problems and they are unrelated to your change.

Thanks!

@agocke,@marek-safar,@jeffhandley are there any other tests or things that could block this from merging on your end?

Everything looks to be passing with the Mono failures being unrelated as indicated above. I don't see any of the tests disabled for NAOT/Mono underhttps://github.com/dotnet/runtime/blob/main/src/tests/issues.targets either.

@tannergooding check outhttps://github.com/dotnet/runtime/blob/main/src/libraries/tests.proj

check outhttps://github.com/dotnet/runtime/blob/main/src/libraries/tests.proj

Likewise don't see any disablement here. Notably the tests also moved fromSystem.Runtime.Experimental toSystem.Runtime so I think existing disablement (if it had existed) would no longer be working.

We'll hold off on merging this until VS 17.2 Preview 2 is available. 🔜

Merging as per offline conversation with@jeffhandley. Now that VS 17.2 Preview 2 is publicly available and contains the C++/CLI fixes, external contributors will no longer be blocked.

dotnet/arcade#8633 tracks the helix support and should only impact WPF for our known dependents.

Visual Studio 17.2 Preview 2 is available here:
https://docs.microsoft.com/en-us/visualstudio/releases/2022/release-notes-preview#1720-pre20--visual-studio-2022-version-172-preview-2