Adding missing test project config to .sln file (#33679)

Annotate System.Net.WebClient for nullable reference types (#33670)

Annotate System.Net.Requests for nullable reference types (#33630)

Fix Reading the Currency Formats (#33599)

* Fix Reading the Currency Formats

* Fix the managed format arrays

* Support formats missing #'s

Merge pull request #33602 from vitek-karas/AppHostVersionAlways

Include apphost_version parameter in download URLs for both GUI and CLI apps

Annotate System.Net.WebSockets.Client for nullable reference types (#33625)

Revert "Disable System.Reflection.Emit.Tests in mono (#32201)" (#33641)

This reverts commit 11dcd00c1a77c8da0fe06404609a9c4af9e5da4f.

Enable crossgen2smoke test for ARM64 (#33530)

* Enable crossgen2smoke test for ARM64.
* Re-enable NullableWithExplicitLayoutTest sub-test.
* Restore setting the image base removed by #31663.

GC coverage can now occur in scenarios where a thread doesn't exist. (#33615)

* GC coverage can now occur in scenarios where a thread doesn't exist.
Properly handle this case and assert some invariant.

* Update gccover.cpp

Misspelled function referenced in comment.

* Remove MethodDesc from GC coverage.

* Code review and fix some asserts when running the Debug version of tests.

* Limit JIT flag manipulation to FEATURE_TIERED_COMPILATION

Use compilerName and compilerArg in pipeline configs (#33605)

Fix Win64 exception propagation in NativeCallable methods (#33653)

A recent change has removed the UMThunkStub usage for native callable
methods. That has broken handling of exceptions propagated from the
managed native callable method into the native caller in case there were
explicit frames below the native frame, like during the GC stress 3.

This change fixes it by unwinding the explicit frames upto the native
frame during the 2nd pass of exception handling when exception unwinding
passes through the frame of a native callable method and isn't handled
there.

Simplify Convert (#33659)

* Simplify Convert

* Optimize Number.UInt32ToDecStr

Fix usage of JIT_FLAG_CODE_EXEC_ONLY flag. (#33623)

#19219 didn't convert the JIT flags into options on cfg meaning that code currently checked JIT_FLAG_CODE_EXEC_ONLY never executed. Fix handles the new JIT flag in same way as other JIT flags, converted into bit field on cfg.

With this fix, the following C# method:

```
static double Test()
{
    return 1.0f;
}
```

will now generate the following assembly on amd64, when using MONO_ARCH_CODE_EXEC_ONLY:

```
sub rsp,8
mov r11,qword ptr [180003318h]
movsd xmm0,mmword ptr [r11]
add rsp,8
ret
```

Co-authored-by: lateralusX <lateralusX@users.noreply.github.com>

Delete IExecutionEngine (#33640)

* Delete IExecutionEngine

* Delete InitUtilcode

* Delete cvtres invocation

* Delete cached m_pClrDebugState and fix contract violations

Delete unused and redundant msbuild properties in coreclr build (#33645)

Limit JIT flag manipulation to FEATURE_TIERED_COMPILATION

[merp] Add breadcrumb for StackHash (#33635)

Contributes to https://github.com/mono/mono/issues/18715

<!--
Thank you for your Pull Request!

If you are new to contributing to Mono, please try to do your best at conforming to our coding guidelines http://www.mono-project.com/community/contributing/coding-guidelines/ but don't worry if you get something wrong. One of the project members will help you to get things landed.

Does your pull request fix any of the existing issues? Please use the following format: Fixes #issue-number
-->

Co-authored-by: alexischr <alexischr@users.noreply.github.com>

Annotate System.Net.Http for nullable (#33268)

* Annotate System.Net.Http for nullable

[jit] Transition the SIMD code to use the MONO_CPU_... set of flags instead of the SIMD_VERSION_... flags. (#33267)

Co-authored-by: vargaz <vargaz@users.noreply.github.com>

Update area owners: Setup, Infra-installer (#33650)

Implement simple version of On Stack Replacement (OSR) (#32969)

Add support to runtime and jit to allow switching from unoptimized to
optimized code for a method while the method has active stack frames.

Details in the included document.

Fix COM + dynamic tests (#33642)

Revert "Add example profiler that does stack sampling with ICorProfilerInfo10::SuspendRuntime (#32988)" (#33656)

This reverts commit 7a362a03f6e829828b5276ab2bf3f1d3073cc166.

Code review and fix some asserts when running the Debug version of tests.

Fix timespan wrap-around when multiplying back by inverse scale

When creating a `TimeSpan` by either dividing, multiplying or using one of
the `From...` methods, the ticks are multiplied by an inverse scale.
When converting the double-precision value back to a long, this
currently causes incorrect values.

This change makes MaxValue work and to allow it to roundtrip.

Implement Vector.Ceiling / Vector.Floor (#31993)

* Add Ceil/Floor to S.P.CoreLib and expose publicly

* Make Ceil/Floor JIT intrinsic

* Add missing identifier for closing endif

* Add JIT test for Ceil/Floor

* Add test for Ceil/Floor in S.Numerics.Vectors
* Also fix the old path for the new structure

* Apply jit-format

* Do not attempt to treat Ceil/Floor if SSE4.1 isn't available

* Mark Vector<T>.Ceiling/Floor as Intrinsic

* Add missing underscore

* Revert "Add missing idenfitier for closing endif"
* Oops

* Add missing breaks
* Oops, again

* Simplify switch case

* Per review suggestion

Co-Authored-By: Egor Chesakov <egor.chesakov@microsoft.com>
Co-authored-by: Egor Chesakov <egor.chesakov@microsoft.com>

[CBOR] Implement Map support for CborReader and CborWriter (#33500)

* Implement map support for CborReader and CborWriter

* fix test naming

* move helper functions to nested type

* check for arithmetic overflow in CborReader.ReadStartMap

* throw FormatExceptions on data items whose definite length exceeds remaining buffer size

Changed RuntimeName to the existing RuntimeFlavor property. (#33559)

Change RuntimeName to the exiting RuntimeFlavor property.

Delete redundant LangVersion properties (#33637)

Update the docs with the change

Delete unnecessary ReferenceSystemPrivateCoreLib from CoreCLR tests (#33629)

Replace manually managed TLS slots with regular thread_local statics (#33597)

* Delete TlsIdx_XXX

* Delete EnableTerminationOnHeapCorruption

* Delete redundant cant stop tracking

* Workaround compiler bug

[Loader] Change mono_trace level from info to debug (#33104)

Fixes mono/mono#18795

**Testing:**
Created HelloWorld script
Set environment variable `export MONO_LOG_LEVEL=info`
Compiled and executed script before and after these changes were applied
```
mdhwang:~/Desktop$ ls -al csc_*txt mono_*txt
-rw-r--r--@ 1 mdhwang  staff   4270 Mar  3 10:12 csc_after.txt
-rw-r--r--@ 1 mdhwang  staff  56698 Mar  3 10:14 csc_before.txt
-rw-r--r--@ 1 mdhwang  staff    176 Mar  3 10:12 mono_after.txt
-rw-r--r--@ 1 mdhwang  staff   3617 Mar  3 10:14 mono_before.txt
```
There is a noticeable difference in log output sizes, and the logs created after the changes were applied mainly contain `Loading reference 0 of _ asmctx _` as desired in the issue.

Co-authored-by: mdh1418 <mdh1418@users.noreply.github.com>

[master] Update dependencies from 3 repositories (#33588)

* Update dependencies from https://github.com/mono/linker build 20200313.2

- ILLink.Tasks - 0.1.6-prerelease.20163.2

* Update dependencies from https://github.com/microsoft/vstest build 20200313-06

- Microsoft.NET.Test.Sdk - 16.6.0-preview-20200313-06

* Update dependencies from https://github.com/mono/linker build 20200316.1

- ILLink.Tasks - 0.1.6-prerelease.20166.1

* Update dependencies from https://github.com/microsoft/vstest build 20200316-01

- Microsoft.NET.Test.Sdk - 16.6.0-preview-20200316-01

* Update dependencies from https://github.com/dotnet/llvm-project build 20200316.1

- runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools - 6.0.1-alpha.1.20166.1
- runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk - 6.0.1-alpha.1.20166.1
- runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools - 6.0.1-alpha.1.20166.1
- runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk - 6.0.1-alpha.1.20166.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>

Remove calls to empty RuntimeHelpers.PrepareConstrainedRegions() (#33191)

Fixes #33179

Remove unused variable for System.Net.Requests, as part of #30457 (#33619)

src/libraries/System.Net.Requests/src/System/Net/FtpControlStream.cs, remove unused variable 'request' in L414.

Partially Fix #30457

Add example profiler that does stack sampling with ICorProfilerInfo10::SuspendRuntime (#32988)

Remove MethodDesc from GC coverage.

Don't delete private keys detected from SerializedCert imports

Update gccover.cpp

Misspelled function referenced in comment.

GC coverage can now occur in scenarios where a thread doesn't exist.
Properly handle this case and assert some invariant.

Additional code coverage in System.Text.Json - no public constructors (#33137)

* added tests for concurrent collection private constructors

* added tests for non generic collection private constructors

* added tests for generic collection private constructors

* added additional test for read only dictionary private constructor

* fixed location of compile directive

* added assertion for exception messages

* added additional assertions for exception messages

* using [Theory] tests to reduce LoC

* using [Theory] for additional tests

* reordered files in test project

* added internal ctor tests

Fixing the sourcelink failure for embeded files (#33521)

* fixing sourcelink

* enable sourcelink on official build

* addressing feedback

* removing extra space

* reverting eng common change

* adding comment

* moving comment to correct place

* not embedding files for test assemblies

Print Ibc messages only when --verbose is specified (#33595)

Update links to the design repo (#33606)

Use compilerName and compilerArg in pipeline configs

Update .NET 5 Unicode data to version 13.0.0 (#33538)

[mono] Implement all Ssse3 and Sse42 intrinsics (#33591)

* implement Ssse3

* Implement Sse42

* remove unrelated change

* Fix C compilation error

Add missing early-exit for LastIndexOfOrdinal (#33601)

Include apphost_version parameter in download URLs for both GUI and CLI apps

So far the code only included the `&apphost_version=<version>` URL parameter in the URL used by the error dialog in GUI apps. The error written to stderr/trace would not have that parameter in its URL.

This change include this parameter in all cases. This also means that comhost/ijwhost and so on will also include the `apphost_version` parameter in their error codes and it will be the version of the comhost/ijwhost and so on. This is not 100% correct, but for the purposed of the website this should be enough (for now we don't see a reason to differentiate between the various hosting scenarios).

Adds a new test to cover this case.

[wasm] Add a sample for collecting coverage data. (#33584)

Co-authored-by: vargaz <vargaz@users.noreply.github.com>

Merge pull request #33578 from vitek-karas/HostDownloadUrls

Add a document describing usage of URLs in the hosting layer

Apply suggestions from code review

Co-Authored-By: Elinor Fung <47805090+elinor-fung@users.noreply.github.com>

Allow Utf8String package to work on netstandard 2.0 (#33357)

* Update ref assembly for building netstandard2.0.

Includes System.Text.Rune for netstandard2.0, since that is currently only netcoreapp3.0.

* Get Rune working on netstandard2.0

* Get Utf8Span compiling on netstandard2.0.

* Get Utf8String building on netstandard2.0

* Compile Utf8String/Span.Conversion.cs for netstandard.

* Build Utf8String Comparison and Enumeration for netstandard.

* Enable Index and Range on netstandard2.0

* Moving code around so it lines up with the ref assemblies.
Getting the full build working again.

* Build for netcoreapp3.0.
Get the package building.

* Get tests building on netfx

* Get Utf8String constructors working on netstandard.

* Get all tests running on netfx.

* Fix build for netcoreapp3.0, which doesn't have SerializeToStreamAsync with cancellation.

* Add netstandard2.1 support to Utf8String.Experimental.

* Make Intrinsics IsSupported be const fields.

* Address PR feedback

* Ensure nint and nuint are set correctly when building Utf8String outside of CoreLib.

* Fix bug in construction from long UTF16 string.

Fix indentations of nameof (#33586)

Update artifactName prefix for GCC publish log task (#33573)

[master] Update dependencies from dotnet/arcade dotnet/runtime-assets (#33562)

* Update dependencies from https://github.com/dotnet/arcade build 20200312.3

- Microsoft.DotNet.XUnitExtensions - 5.0.0-beta.20162.3
- Microsoft.DotNet.VersionTools.Tasks - 5.0.0-beta.20162.3
- Microsoft.DotNet.ApiCompat - 5.0.0-beta.20162.3
- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.20162.3
- Microsoft.DotNet.Build.Tasks.Feed - 5.0.0-beta.20162.3
- Microsoft.DotNet.Build.Tasks.Packaging - 5.0.0-beta.20162.3
- Microsoft.DotNet.Build.Tasks.SharedFramework.Sdk - 5.0.0-beta.20162.3
- Microsoft.DotNet.Build.Tasks.TargetFramework.Sdk - 5.0.0-beta.20162.3
- Microsoft.DotNet.CodeAnalysis - 5.0.0-beta.20162.3
- Microsoft.DotNet.GenAPI - 5.0.0-beta.20162.3
- Microsoft.DotNet.GenFacades - 5.0.0-beta.20162.3
- Microsoft.DotNet.Helix.Sdk - 5.0.0-beta.20162.3
- Microsoft.DotNet.RemoteExecutor - 5.0.0-beta.20162.3
- Microsoft.DotNet.XUnitConsoleRunner - 2.5.1-beta.20162.3

* Update dependencies from https://github.com/dotnet/runtime-assets build 20200312.1

- System.Windows.Extensions.TestData - 5.0.0-beta.20162.1
- System.Security.Cryptography.X509Certificates.TestData - 5.0.0-beta.20162.1
- System.Private.Runtime.UnicodeData - 5.0.0-beta.20162.1
- System.Net.TestData - 5.0.0-beta.20162.1
- System.IO.Packaging.TestData - 5.0.0-beta.20162.1
- System.IO.Compression.TestData - 5.0.0-beta.20162.1
- System.Drawing.Common.TestData - 5.0.0-beta.20162.1
- System.ComponentModel.TypeConverter.TestData - 5.0.0-beta.20162.1

* Remove APICompat downgrade

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Santiago Fernandez Madero <safern@microsoft.com>

[mono] Implement missing SSE2 intrinsics (#33574)

* Implement missing SSE2 intrinsics

* remove SN_ConvertScalarToVector128Single case

* Remove unrelated change

* Shuffle < Sqrt (binary search)

Read the image earlier (#33576)

Disable publishing CoreCLR product binaries on GCC leg (#33580)

Prep CompareInfo for spanification (#32385)

Fix dump of common methods diff (#33570)

The DiffMethodsForModule was accidentally trying to get leftSection from
the right dumper.

[Arm64] Vector Load/Store structure instructions (#33461)

This adds support in the JIT emitter for Vector Load/Store structure instructions (C3.2.10 - Arm
Architecture Reference Manual):

- LD1 (1-4 registers)
- LD2
- LD3
- LD4
- LD1R
- LD2R
- LD3R
- LD4R
- ST1 (1-4 registers)
- ST2
- ST3
- ST4

in the following addressing modes:

- Base register only
- Post-indexed by a 64-bit register
- Post-indexed by an immediate, equal to the number of bytes transferred

Also adds support in JitDump for printing of

* A SIMD vector register list.
  For example, ld1     {v5.16b, v6.16b, v7.16b, v8.16b}, [x9]

* A SIMD vector element list.
  For example, st1     {v0.b}[3], [x1],#1

Make NativeCallableAttribute public. (#33005)

* Avoid creating a COM Delegate when calling NativeCallableAttribute.

* Create reverse P/Invoke frame.
Create Preempt and Coop PreStubWorker().

* Limit the exclusion for NativeCallableAttribute to Win-x86.

* Add Reverse P/Invoke JIT helpers to CrossGen2.

* Add test for generic class with NativeCallableAttribute method.

* Implement Unix EH change for NativeCallableAttribute method.

Co-authored-by: Jan Vorlicek <janvorli@microsoft.com>
Co-authored-by: Jan Kotas <jkotas@microsoft.com>

Remove TS_UserSuspendPending enum (#33539)

This PR removes the unused TS_UserSuspendPending enum as well as flag usage.

Add a document describing usage of URLs in the hosting layer

[jit] Add support for emitting r4/r8 constants outside of the text segment. (#33552)

Co-authored-by: vargaz <vargaz@users.noreply.github.com>

HostModel: Remove depricated API (#33545)

Some depricated APIs were maintained in #33413 so that SDK build is green.
Now that the SDK is updated https://github.com/dotnet/sdk/pull/10849, remove the unused APIs.

Also move out a test-only methods from the product to the test helpers.

Disable not working test containing RemoteExecutor (#33560)

Add polymorphic tests for all root APIs (#33391)

[runtime] Coverage profiler improvements. (#33540)

Co-authored-by: vargaz <vargaz@users.noreply.github.com>

Merge pull request #33512 from kouvel/NoQuickJitFix

When QuickJit is disabled, fix assertion failures regarding R2R code

Rename "PolicyProperty" to "PropertyInfoForClassInfo" and update nullability (#33432)

Update doc for exceptions thrown during serialization (#33400)

Fix content of baseline files (#33514)

[mono] Track .dotnet runtime version (#33180)

* Track .dotnet runtime version correctly

* Use BundledNETCoreAppPackageVersion

* Rollback to an error

* Update src/mono/mono.proj

Co-Authored-By: Alexander Köplinger <alex.koeplinger@outlook.com>
Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>

Replace ushort usage with int (#32694)

* Replace pointless ushort usage with int

* Revert extracting name[i] to a local

* Use int instead of byte in IPv4 ParseCanonical

Clean up unused locals in System.Drawing.Common (#32212)

* Clean up unused locals in System.Drawing.Common

* Remove BufferedStream, use MemoryStream.ToArray()

[threads] If shutting_down, don't create new thread, kill current thread (#32968)

If the runtime is shutting down, instead of creating a new thread, or throwing
an exception on the current thread, just shut down the current thread.

This takes advantage of `mono_threads_set_shutting_down`'s behavior that if you
call it after shutdown is already started, it will suspend, detach and exit
from the current thread.

Fixes https://github.com/dotnet/runtime/issues/31842

Fix line endings under src/mono (#33547)

Fixes #2222

Clean `fgMorphPotentialTailCall` for the future changes. (#33457)

* Clean `fgMorphPotentialTailCall` for the future changes.

* Use the original type, because the call was retyped to void.

* another fix for arm64.

[debugger] [coop] assertion when `socket_transport_send` called from the crash reporter (#33489)

If there is a crash and the debugger is attached, we call mini_get_dbg_callbacks ()->send_crash().

That call ends up in mono_debugger_agent_send_crash which does all kinds of questionable stuff like trying to take the loader lock. Eventually it ends up calling socket_transport_send which tries to do a transition to GC Safe mode.
I inserted a MONO_ENTER_GC_UNSAFE in mono_debugger_agent_send_crash function and this fixes the assert.

Fixes mono/mono#18794

Co-authored-by: thaystg <thaystg@users.noreply.github.com>

[wasm][debugger] Continue SS after leaving managed code (#33272)

Continue stepping over/into/out of managed code to native code, without this fix the behavior was resume when a SS leaves the managed code.

Co-authored-by: thaystg <thaystg@users.noreply.github.com>

Extensions: Adding FileProviders and FileSystemGlobbing (#33212)

* - [x] Bulk update all ref csproj except for FileProviders/FileSystemGlobbing
* - Bulk move Descriptions to json file, all except for FileProviders and FileSystemGlobbing
* - Bulk correct test project targetframeworks, all except for FileProviders and FileSystemGlobbing
* - Bulk rename ref cs file and delete the extra duplicates
* - Bulk move/rename Resources file
from: Resources.resx to Resources\Strings.resx

* Enables ref/src/test/pkg/sln on FileProviders projects and FileSystemGlobbing
- [x] Enable on Directory.Build.props
- [x] build ref/src projects and used GenerateReferenceSource
- [x] cleanup src csproj properties
- [x] set proper targetframeworks

[jit] Add more SSE2 opcodes. Enable SSE2. (#33465)

* [jit] Fix the Sse2 supported check.

* Add more SSE2 opcodes. Enable SSE2.

* Use the non-immediate version of the shift opcodes.

* Fix ShiftRightArithmetic.

* Remove a debug printf.

* Implement all SSE3 intrinsics.

* Define all intrinsics using the SimdIntrinsic struct.

* Disable SSE2 support as some intrinsics are still missing.

Remove asserts that were firing during rejit tests (#33492)

The rejit path no longer suspends the runtime so that assert is inaccurate

Reduce fixed overhead of some Utf8Parser.TryParse methods (#33507)

Fix handling thread abort in HelperMethodFrame (#33448)

* Fix handling thread abort in HelperMethodFrame

The thread abort during func eval from a managed debugger on Linux and macOS
was sometimes causing the debuggee to exit with unhandled c++ PAL_SEHException.
The reason is that the thread abort detection that is done in the
HELPER_METHOD_FRAME_BEGIN and ...END macros was done outside of the
INSTALL_MANAGED_EXCEPTION_DISPATCHER / UNINSTALL_MANAGED_EXCEPTION_DISPATCHER
region and so the exception thrown when thread abort request is detected
there was not being caught and translated into a call to DispatchManagedException.
Since the caller frame was a managed function frame, the C++ exception handling
didn't know how to unwind it and so it declared the exception being unhandled.

This fix moves the frame pushing and popping inside of the
INSTALL_MANAGED_EXCEPTION_DISPATCHER / UNINSTALL_MANAGED_EXCEPTION_DISPATCHER
region. That way, the thread abort exception is properly handled.

[test][mono] Capture the ExceptionDispatchInfo when rethrowing from TaskContinuation (#33491)

Regression test for https://github.com/mono/mono/issues/19166

Co-authored-by: lambdageek <lambdageek@users.noreply.github.com>

JIT: fix overly aggressive tail recursive call loop marking (#33517)

If there is a tail recursive call site, the jit will mark all blocks
from method entry to the block with the call as "in a loop," anticipating
a tail recursive call to loop optimization.

Because of some confusing naming we were doing this even for recursive calls
that were not tail calls. Upshot is that blocks were marked as being in loops
when they weren't, and (among other things) this made the inliner more
aggressive for calls in those blocks.

Moved warning disables from warningcontrol.h to configurecompiler.cmake (#33439)

* Moved warning disables from warningcontrol.h to configurecompiler.cmake

* Completely moved the contents of inc/warningcontrol.hto configurecompiler.cmake.

Add CoreCLR build pipeline for GCC (#33446)

* Add CoreCLR build pipeline for GCC

* Fix GCC build issues

* Use empty clangArg in case of GCC

* Revert ilasm's cmake file

* Use GCC to also build native test components

Fixed memory leak of OverlappedData when doing read from server WebSocket stream. (#33480)

Merge pull request #33497 from dotnet/vivpatch

Adding separate 2.1 SDK yaml

Cleanup stale ApiCompatBaseline files (#33519)

[metadata] Disable three icalls on netcore monodroid (#33513)

They aren't used there.

Co-authored-by: akoeplinger <akoeplinger@users.noreply.github.com>

Remove the copy of core_root by using core_root as the corelation dir (#33498)

* Remove the copy of core_root by using core_root as the corelation dir

* Change to remove core_root, it is the workitem_payload

* Fix core_root location

Merge pull request #33494 from maryamariyan/fromoldmaster-bringmissingfiles-1

Bring commit history for more files from Extensions repo

Fix type layout whan Cross OS compiling (#33487)

* Fix cache line size to be a TARGET property

* Fix CCompRC cross OS DAC layout

* Fix cross OS InlineSArray packing

* Add DAC_ALIGNAS()

Align first member of various classes which have shown
inconsistent layout during cross compilation.

* Fix RegMeta cross os layout

* Fix typo

* Remove dead code

* Remove FunctionSigBuilder et. al from DAC

Remove FunctionSigBuilder and classes which depend on its
definition from the DAC.

Remove the FunctionSigBuilder layout correction to make it
cross platform stable.

* Remove more stubgen classes from DAC

* Improve comment

[merp] Remove unnecessary call to msync in mono_state_free_mem (#33395)

Closes https://github.com/mono/mono/issues/19136

Co-authored-by: alexischr <alexischr@users.noreply.github.com>