Return exit code 1 in android test runner if a test failed (#36319)

* Return exit code 1 in android test runner if a test failed

* Exit instrumentation with runner exit code

Remove some `= null!`s from corelib (#36175)

* Remove some `= null!`s from corelib

In some cases, this was achieved by:
- Using [MemberNotNull] when it could be used to enable the compiler to appropriately follow the initialization
- Making the field nullable and using `!`s at usage sites in cases where the `= null!` wasn't appropriate because the field could actually be null
- Making the field nullable and adding null checks sparingly
- Suppressing warnings in cases where fields weren't accessed in managed code
- Removing stale cases where it wasn't necessary anymore anyway
- Removing unnecessary empty internal/private ctors

* Address PR feedback

* Disable test on mono

Emit DWARF debug_abbrev offset for compile units as a label instead of 0 (#36320)

When outputting DWARF code to start a compilation unit in .debug_info, the standard expect a 4-byte offset from the .debug_abbrev code. Mono has always output an offset of 0.

However, this doesn't work in every cases. When we have linux+fullaot, we link two object files (one from Mono, one from LLVM). Both have their .debug_abbrev section. If we use 0 as an offset, it seems possible that the linker will keep thinking that our offset is 0, no matter the circumstances. Since the offset is always 0, it can be using the wrong abbreviation table (i.e. the one from the LLVM assembly instead of the one from the Mono assembly). The consequence of this is that the linked file is not valid DWARF (dwarfdump and objdump will complain about invalid offsets). At best, some tools will be able to work with a part of what we have, but any program requiring entirely valid DWARF will fail.

To fix this, we generate a label for the start of our debug_abbrev section and we instead generate it by generating a long with that label. This matches existing behavior seen in the LLVM generated code, and makes dwarfdump and objdump react properly to the linked product.

Fixes mono/mono#8806

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

[runtime] Move delegate_bound_static_invoke_cache to MonoWrapperCaches. (#36309)

<!--
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: vargaz <vargaz@users.noreply.github.com>

Add MonoAOTCompiler msbuild task (#35961)

Remove several unnecessary arrays used with Split (#36304)

Now that we have a Split overload that takes a single char, the previous optimization to cache an array and reuse that array with the array-based overload isn't needed.

Add POH size counter (#36150)

* Add POH size counter

* Add 'Pinned Object Heap' to DisplayName

Flow exit code from RunTests.sh in iOS (#36315)

Remove unsed variable in MsQuicApi (#36302)

Prevent build scripts from letting random values through (#35642)

* Arch, OS, and Config no longer let random values through when building the repo. Also, removed the casing restrictions on the bash version.

* Fixed the bash build script! Also, casing restrictions are gone!

* Added safeguards for bogus values on -rc and -lc on the sh build script.

* Fixed the merge conflict on build.sh

* Used awk instead of declare for increased compatibility on build.sh

* Restored execute permissions to build.sh

Fix Microsoft.Extensions.DependencyModel AssemblyVersionInPackageVersion (#36301)

Set ExactSpelling=true on P/Invokes in System.Private.CoreLib that already specify the unicode W suffix (#36257)

* Set ExactSpelling=true on P/Invokes in System.Private.CoreLib that
already specify the unicode W suffix

* PR feedback: refactor NDirectMethodDesc::FindEntryPoint

Fix HttpClient SSL3.0 test on newer Windows OS (#36308)

When I upgraded my dev machine to Windows 10 Version 2004 Preview, I noticed that the
HttpClient GetAsync_SupportedSSLVersion_Succeeds test was failing when using SSL 3.0.

This new version of Windows changes the default setting of SSL 3.0. So, by default,
SSL 3.0 is no longer enabled.

The problem was that this specific test wasn't using our PlatformDetection.SupportsSsl30
logic. Fixed the test so that it will properly detect whether SSL 3.0 is enabled on
the machine.

Add ctor to JsonSerializerOptions that takes serializer defaults (#36073)

* add ctor to JsonSerializerOptions that takes serializer defaults

* add out of range handling; enhance xml documentation

* test serializer option construction via serializer defaults

* adjust PredefinedSerializerOptions tests

Add QCALL_CONTRACT_NO_GC_TRANSITION (#36294)

New QCall macro for APIs that use SuppressGCTransitionAttribute.

Fix nullable annotations on DangerousAcceptAnyServerCertificateValidator (#36305)

It should match ServerCertificateCustomValidationCallback.

Comment typos (#36296)

Use Publish target for self contained test apps (#36290)

Remove PlatformAbstractions from runtime tests. (#36243)

* Remove PlatformAbstractions from runtime tests.

Contributes to #3470

Ignore the private key handle cert property for persisted keys

When a CERT_CONTEXT value has both property 2 (prov info) and property 78
(ncrypt key handle), prefer to load based on the property 2 state.
This avoids a scenario where calling Get[Algorithm]PrivateKey sets the
'CLR IsEphemeral' property on a persisted key, preventing future loads of that key.

An alternative approach of preferring the loaded key over the cold-load was
not selected to avoid value contamination of ephemeral properties set on the
CngKey object directly after the caller calls Get[Algorithm]PrivateKey.

Fix build break by bumping DependencyModel package version (#36285)

Minor MethodTable building optimization (#36283)

When learning on how virtual method overriding is implemented in the
runtime, I've bumped into two places where we were calling
MapToImplSlotNumber twice in a row without any reason. Since this method
isn't trivial, it seems worth fixing.

Apply CA1835 - Prefer memory overloads for Stream.ReadAsync/WriteAsync (#35941)

* Apply CA1835 Roslyn fixes on .NET Libraries
- Prefer the Stream.ReadAsync and Stream.WriteAsync memory overloads.
- Unit test fix suggested by buyaa-n to adjust the usage of these overloads.

* Additional missing unit test overload for ReadOnlyMemory

[mono] Don't throw inheritance error on interfaces in IsDefined (#35516)

Issue is very similar to the one fixed in https://github.com/dotnet/runtime/commit/3df21ae60ec7de12abc77f4cc329903abe9ea542. When adding the error messages, we were too strict on when to throw. This should hopefully avoid all that by being significantly more lax.

Test moved to be in a location better suited to both examples.

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

Use CopyTo(arr) instead of creating List<T> first (#36228)

* Use LINQ ToArray

* Update RoType.cs

* Update RoType.cs

* Update src/libraries/System.Reflection.MetadataLoadContext/src/System/Reflection/TypeLoading/Types/RoType.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Fix whitespace causing analyzer warnings

Co-authored-by: Stephen Toub <stoub@microsoft.com>

Update testing.md (#36286)

[master] Update dependencies from 4 repositories (#36279)

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

- Microsoft.DotNet.XUnitExtensions: 5.0.0-beta.20256.5 -> 5.0.0-beta.20258.8
- Microsoft.DotNet.VersionTools.Tasks: 5.0.0-beta.20256.5 -> 5.0.0-beta.20258.8
- Microsoft.DotNet.ApiCompat: 5.0.0-beta.20256.5 -> 5.0.0-beta.20258.8
- Microsoft.DotNet.Arcade.Sdk: 5.0.0-beta.20256.5 -> 5.0.0-beta.20258.8
- Microsoft.DotNet.Build.Tasks.Feed: 5.0.0-beta.20256.5 -> 5.0.0-beta.20258.8
- Microsoft.DotNet.Build.Tasks.Packaging: 5.0.0-beta.20256.5 -> 5.0.0-beta.20258.8
- Microsoft.DotNet.Build.Tasks.SharedFramework.Sdk: 5.0.0-beta.20256.5 -> 5.0.0-beta.20258.8
- Microsoft.DotNet.Build.Tasks.TargetFramework.Sdk: 5.0.0-beta.20256.5 -> 5.0.0-beta.20258.8
- Microsoft.DotNet.CodeAnalysis: 5.0.0-beta.20256.5 -> 5.0.0-beta.20258.8
- Microsoft.DotNet.XUnitConsoleRunner: 2.5.1-beta.20256.5 -> 2.5.1-beta.20258.8
- Microsoft.DotNet.Helix.Sdk: 5.0.0-beta.20256.5 -> 5.0.0-beta.20258.8
- Microsoft.DotNet.RemoteExecutor: 5.0.0-beta.20256.5 -> 5.0.0-beta.20258.8
- Microsoft.DotNet.GenFacades: 5.0.0-beta.20256.5 -> 5.0.0-beta.20258.8

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

- Microsoft.NET.ILLink.Tasks: 5.0.0-preview.3.20257.4 -> 5.0.0-preview.3.20261.2

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

- runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools: 6.0.1-alpha.1.20261.1 -> 6.0.1-alpha.1.20261.3
- runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools: 6.0.1-alpha.1.20261.1 -> 6.0.1-alpha.1.20261.3
- runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk: 6.0.1-alpha.1.20261.1 -> 6.0.1-alpha.1.20261.3
- runtime.osx.10.12-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools: 6.0.1-alpha.1.20261.1 -> 6.0.1-alpha.1.20261.3
- runtime.osx.10.12-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk: 6.0.1-alpha.1.20261.1 -> 6.0.1-alpha.1.20261.3
- runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk: 6.0.1-alpha.1.20261.1 -> 6.0.1-alpha.1.20261.3

* Update dependencies from https://github.com/dotnet/xharness build 20200511.4

- Microsoft.DotNet.XHarness.Tests.Runners: 1.0.0-prerelease.20257.4 -> 1.0.0-prerelease.20261.4

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

Ensure Process.ProcessName doesn't change when setting Thread.Name on Linux (#34064)

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

This issue is a side-effect of adding support for setting thread names on Linux in https://github.com/dotnet/coreclr/pull/27182.

Co-authored-by: Dan Moseley <danmose@microsoft.com>
Co-authored-by: Aleksey Kliger (λgeek) <akliger@gmail.com>

Add configurations for SunOS CoreCLR managed build (#36266)

Build tests as self contained apps for mobile targets (#36262)

This uses the SDK to publish tests as self contained apps before building the app bundle using the built runtime pack.

I moved the `RunTests.sh` output path to `BundleDir` as whenever we start sending to helix, there is no need to .zip up the whole `OutDir` as we can just send the bundle + `RunTests.sh` script.

Merge pull request #35683 from krwq/nullable-xml-2

Nullable: System.Xml, part 2

Build WASM runtimes (#35566)

Fix vs testing link (#36261)

Handle struct marshal stubs in ILStubManager::TraceManager. (#36249)

Fixes #36248

disable ServerAsyncAuthenticate_MismatchProtocols_Fails again (#36250)

[ARM64] Implement Duplicate and DuplicateSelectedScalar (#36144)

* Adding support for DuplicateToVector64 and DuplicateToVector128 to Arm.AdvSimd and Arm.AdvSimd.Arm64

* Adding tests for DuplicateToVector64 and DuplicateToVector128

* Adding support for DuplicateSelectedScalarToVector64 and DuplicateSelectedScalarToVector128 to Arm.AdvSimd and Arm.AdvSimd.Arm64

* Adding tests for DuplicateSelectedScalarToVector64 and DuplicateSelectedScalarToVector128

* Regenerating tests for the newly added HWIntrinsics

* Ensure parentheses exist

* Removing the AdvSimd.Arm64 hwintrinsiclist entry for DuplicateToVector64

* Removing the rest of the NI_AdvSimd_arm64_DuplicateToVector64 usages and clarifying a comment

* Moving break outside the braces for consistency

* Removing another Arm64.DuplicateToVector64

* Removing the AdvSimd.Arm64.DuplicateToVector64 tests

* Adding tests covering DuplicateToVector64/DuplicateToVector128 immediate inputs

* Regenerating the ARM64 HWIntrinsic tests

* Fixing the DuplicateToVector immediate tests

* Regenerating the DuplicateToVector immediate tests

Delete outdated todo comment (#36224)

update branding to preview6 (#36247)

Reduce DebugInfoTable size (#36174)

The DebugInfoTableNode contains an array of native format vertices that currently are not unified. Many methods have identical debug info blobs so they can be collapsed to save output binary size. For identical debug info blobs, pass the same byte array reference to the native format writer which will unify on the object reference.

In the framework composite image for example, 70% of the debug infos are duplicates. Removing the duplicates reduces the composite image size by 5%.

Fix empty ParamName in ArgumentException (#36208)

* Update reflectioninvocation.cpp

* Update TypeInfoTests.cs

* Update EnumConverterTest.cs

Fix up nint code after latest changes to use compiler nint. (#36220)

[testing] Rename Prop AppleTestBuilderDir to AppleAppBuilderDir (#36227)

Co-authored-by: Mitchell Hwang <mihw@microsoft.com>

Remove EmbeddedResourceUseDependentUponConvention (#36140)

Underlying issue is fixed: https://github.com/microsoft/msbuild/issues/4740

Use already defined generateAssemblyReferenceSource rather defining a new target (#35818)

* use arcade target

* extra space

* fixes generation of reference src from runtime projects

* run target to get genapi

* use referencePath to get references

* adding  GenerateReferenceSource target

* added the target name

Setup pr and batch runs for dev/infrsatructure (#36218)

Ensure we mark op2 as delayFree if it is rmw and the parent node returns a non-SIMD type (#36226)

Update the mscorlib.md document. (#36225)

Rename mscorlib.md to corelib.md.
Remove no longer relevant details.

Fix iOS and Android samples (#36223)

Make global ComWrappers for marshalling respect requested IID (#36054)

Include Content-Type in exception in ReadFromJsonAsync (#36186)

* Update Strings.resx

* Update HttpContentJsonExtensions.cs

Expose the parsed signature in structured format (#36190)

Use TargetFramework conditions consistently in libraries (#35956)

* Use TargetFramework condition consistently in libs

* Add docs section to describe the guideline

* Update Microsoft.Extensions.DependencyInjection.csproj

* Outline TargetFramework DesingTimeBuild issue

* Update Microsoft.Extensions.DependencyInjection.csproj

Update dependencies from https://github.com/dotnet/llvm-project build 20200511.1 (#36210)

- runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools: 6.0.1-alpha.1.20254.1 -> 6.0.1-alpha.1.20261.1
- runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools: 6.0.1-alpha.1.20254.1 -> 6.0.1-alpha.1.20261.1
- runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk: 6.0.1-alpha.1.20254.1 -> 6.0.1-alpha.1.20261.1
- runtime.osx.10.12-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools: 6.0.1-alpha.1.20254.1 -> 6.0.1-alpha.1.20261.1
- runtime.osx.10.12-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk: 6.0.1-alpha.1.20254.1 -> 6.0.1-alpha.1.20261.1
- runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk: 6.0.1-alpha.1.20254.1 -> 6.0.1-alpha.1.20261.1

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

Add invalidation to DTFI's SetAllDateTimePatterns (#34438)

Invalidate derived patterns to match the behavior of invoking the corresponding property setters

Revert "Revert "Remove superflous CheckReleased call which will slow down Event Log Reading up to 20x times. (#35911)" (#36143)" (#36188)

This reverts commit ca8c9dea8e30a9701160e4cd61b5b26f209499ff.

Preempt Watson exception handling (#36179)

* switch to preemp GC before DW version lookup

* move preempt to PopulateBucketParameters

* Adding GC_TRIGGERS to callchain of CaptureUnhandledInfoForWatson

* explictly adding PREEMPTIVE to all DwGetFileVersionInfo codepaths.

Add some upcoming distro RIDs

This adds some RIDs that we know are current/upcoming:

- Fedora 33 is in development, and the version after this is most likely
  going to be Fedora 34. Adding this RID now, so we can get this RID in
  all current releases when it comes time to build on Fedora 34

- RHEL has generally committed to doing a major release every 3 years.
  With RHEL 8 released in 2019, RHEL 9 is under development. There's
  even a plan on how to create that from Fedora here:
  https://fedoraproject.org/wiki/Changes/ELN_Buildroot_and_Compose

- If there's a RHEL 9, there will most likely be an equivalent CentOS 9.
  Lets add that too.

- CentOS 7 has an aarch64 port. As does CentOS 8.

Improve deserialization perf for case-insensitive and missing-property cases (#35848)

Return the correct OSVersion on OSX like systems. (#36029)

* Return the correct OSVersion on OSX like systems.

Fix #34977

* Rename files to not have Linux when they aren't Linux specific.

* Assert OSVersion Build number is valid.

* PR feedback

Move interop code to Common\src\Interop.
Specify the full path to libobjc.dylib.
Fix objc_msgSend_stret for ARM64.

* Specify full path to libproc.dylib.

Fix #24095

Port GC changes from dotnet/corert (#36197)

[mono] Fix musl PowerPC build (#36162)

PT_NIP and friends are in this kernel header, but glibc includes it
somewhere in the chain implicitly. musl doesn't. Fix that.

Update ryujit-tutorial.md (#36191)

Adding unit test to provoke exception using Sch_WhiteSpaceRestriction… (#35013)

* Adding unit test to provoke exception using Sch_WhiteSpaceRestriction1 as its message.

[tests] Adjustments to interp disabled tests (#36178)

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

Fix sync call in XContainer.ReadContentFromAsync (#35789)

* Fix sync call in XContainer.ReadContentFromAsync

Add tests for changes in #34438 (#36166)

Remove CompareOrdinalEx (#36184)

Update dependencies from https://github.com/dotnet/runtime-assets build 20200508.1 (#36161)

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

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

Fix mobile testing and do some extra cleanup in variables (#36157)

Remove unnecessary static array from FrameworkName (#36176)

Now that there's a char overload of Split, we don't need the cached array.

Reenable iOS_arm builds (#34942)

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

Fix buffer overrun in JIT for Vector256<T> types on ARM64. (#35864)

Stop recognizing Vector256<T> types as intrinsic for ARM64 in Crossgen2.

Enable Common Runtime Tests for Mono on Linux-Arm64 in CI (#35366)

Enable runtime tests on Linux-amd64 running Mono. Also add issues to issues.targets for the same.

Make Uri Thread-Safe (#33042)

* Do not lock on string in Uri

* Make Uri thread-safe without locks

*A lock may still be used for custom parsers

* Update comments

* Use existing style for enum values

* Add a comment about locking for custom parsers

* Add a comment about DebugSetLeftCtor usage

* Add trailing dot in exception message

* Fix typos

* Fix typo

Co-authored-by: Stephen Toub <stoub@microsoft.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>

Use C# compiler provided nint/nuint (#36159)

increase FailingTestTimeoutMiliseconds for Ssl server tests. (#36101)

* increase FailingTestTimeoutMiliseconds

* use standard ITestOutputHelper helper

* set timeout to to 500ms

* rework protocol mismatch

Co-authored-by: Tomas Weinfurt <furt@Shining.local>

add platform to two tests to avoid test skip on Unix (#36082)

Apply Regex starting loop optimization to non-atomic loops as well (#35936)

* Apply Regex starting loop optimization to non-atomic loops as well

* Remove min iteration restriction

The node.N > 0 restriction isn't necessary, and prevents this optimization from being used with * loops.    Worst case, the loop doesn't match anything, and we pay to overwrite the starting position with itself.  Best case, we eliminate a ton of cost.

[interp] Fix sdks build (#36121)

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

Fixing test failing on arm64 (#36145)

Finish the other part of the GCTotalPhysicalMemory config implementation (#36152)

* Finish the other part of the GCTotalPhysicalMemory config implementation

The total physical memory specified by the GCTotalPhysicalMemory config
needs to communicated over to the VM side for memory load calculation.
Otherwise inside GC we are using the correct physical memory specified,
but when we ask for the memory load it's still based on the total available
memory.

* Delete unnecessary delay-loading of kernel32.dll

* Avoid mutable restricted memory limit inside OS PAL

Co-authored-by: Maoni0 <maonis@microsoft.com>

React to latest Roslyn nullability changes (#36104)

Also updates the compiler toolset to the latest version

Naricc/test mono is mono (#36079)

Add test to test if we are running mono.

Revert "Remove superflous CheckReleased call which will slow down Event Log Reading up to 20x times. (#35911)" (#36143)

This reverts commit 4602a61526631a5b29d451b40be3e3e321b86c00.

Enable VS Test Explorer without the -vs switch (#36126)

* Enable VS Test Explorer without the -vs switch

Add windows 8 and windows 7 testing for PRs in a reduced way (#36096)

Document F5 and Test Explorer prerequisites (#36117)

* Document F5 and Test Explorer prerequisites

* Update docs/workflow/testing/visualstudio.md

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

Fix reverse marshalling of refs to blittable types (#8140) (#36119)

Hit in selfhosted ilc.exe. In JitInterface we have a couple cases where we do reverse p/invoke with a delegate that has a ref parameter. Sometimes the `ref` is null (and we don't touch it).

The marshaller was trying to make a copy of the pointed to value, NullRefing in the process because the pointer was null. We don't actually need to make a copy because the value is blittable.

Co-authored-by: Michal Strehovský <MichalStrehovsky@users.noreply.github.com>

Merge pull request #35826 from layomia/converter_handle_null

Add opt-in for converters to handle null

Merge pull request #32107 from YohDeadfall/fix-hidden-props

Fixed serialization of hidden base class members

Add mono performance runs (#34825)

* Add mono perf job

* Move runtimeFlavor to the right level

* chnage variable access

* Change condition

* Add steps to download mono and build the patched dotnet

* Change variable access

* Pass variables directly

* Plumb through mono path to runner

* Fix parameter reference

* Fix variables access?

* Fix typo

* Fix pathing issue

* Add log saving for publish to helix step

* Fix variable in shell script

* Fix up mv command and publish logs step

* Change runkind and add Windows runs

* Add Windows mono build

* Fixup parsing

* Remove Windows runs for now

* Switch to using build command to generate mono testhost

* Fixup parens

* Forgot to add a slash

* Add Windows runs and fix pathing issue

* Add backslash for Windows and -r for cp on Linux

* Switch to using corerun argument

* Switch to monopath

* Use CoreRun

* Remove mono property and use corerun instead

* Fixing closing tags

* Change xcopy to copy

* Add llvm config flag

* Add AOT and Interpreter config flags

* Change order to ensure config gets added

* Change order

* Add exlusion filter for Perf_Image_Load tests

* Add more to exclusion list

* Remove Windows legs for now

[master] Update dependencies from 4 repositories (#36111)

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

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

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

- Microsoft.NET.ILLink.Tasks: 5.0.0-preview.3.20256.5 -> 5.0.0-preview.3.20257.4

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

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

* Update dependencies from https://github.com/dotnet/xharness build 20200507.4

- Microsoft.DotNet.XHarness.Tests.Runners: 1.0.0-prerelease.20256.2 -> 1.0.0-prerelease.20257.4

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

Modified PR for reverse diagnostics server (#35850)

Remove attempts to do too much cleanup on process shutdown, e.g., don't attempt to close handles or cancel overlapped IO

Optimize Vector128 and Vector256.Create methods (#35857)

* Updating Vector128.Create(T value) to be intrinsic

* Updating Vector128.Create(T, ..., T) to be intrinsic

* Updating Vector256.Create(T) and Vector256.Create(T, ..., T) to be intrinsic

* Applying formatting patch

* Adding additional comments explaining how the Vector128.Create and Vector256.Create calls are lowered

* Add an assert that argCnt is as expected for LowerHWIntrinsicCreate

* Applying formatting patch

* Use unsigned rather than signed types in LowerHWIntrinsicCreate

* Have HWIntrinsicInfo::lookupId take the number of arguments into account

* Adjusting Vector128.Create(T, ..., T) to not be recursive on ARM64

* Fixing MyICJI::allocMem in superpmi to respect certain CorJitAllocMemFlag

* Avoid an implicit downcast

remove unused locals: System.Private.DataContractSerialization (#35958)

Struct Improvements (#35544)

* Struct Improvements

- If we have a struct with a single field we can reference it as the full size of the struct.
- Eliminate old code for an integer zero RHS with a SIMD12 LHS, and unify the codegen for zero-init for SIMD8 and SIMD12 (between block init and assignment of SIMD init of 0)

[interp] Add x86 support (#35966)

This mainly consists of a trampoline that is used when calling from interp into native code, as well as the call convention logic that initializes the CallContext structure (which is passed to the trampoline). On other platforms we also have a single trampoline used when entering interpreter code. That trampoline works in a similar way and its purpose is to avoid compilation of all possible interp entry signatures, which is problematic on full aot scenarios. Given the interp x86 support is currently provided for android, we can avoid using that trampoline for now and use the per signature jit compiled interp_in_wrappers instead.

The few disabled interp tests are the same as on amd64.

We only support pinvoke into cdecl functions.

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

Remove MinHandles and OutstandingHandles logic from SocketAsyncEngine (#36019)

* remove MinHandles logic from SocketAsyncEngine (always use Round-robin)

* remove _outstandingHandles

* add Debug.Assert to make sure that we always succeed in adding a new context to the dcitionary

* remove duplicated Debug.Assert (checked few lines above)

disable sslv2 tests for now (#36102)

Co-authored-by: Tomas Weinfurt <furt@Shining.local>

Migrating Apple and Android AppBuilder and TestRunner binaries to Artifacts/Bin (#35909)

This PR is a part of #35123.

This PR achieves two things:
1. Migrates Apple and Android AppBuilder and TestRunner binaries to `<repo-root>/artifacts/bin`
2. Add property paths for Apple and Android AppBuilder and TestRunner projects and binaries

After changes, when building `./build.sh -os iOS -subset Mono+Libs+Libs.Tests`, binary files are no longer produced in `<repo-root>/src/mono/msbuild/`. Instead, they can be found in `<repo-root>/artifacts/bin/`