Fix resolve root IServiceProvider (#45171)

Breaking change: Throw when resolving on a disposed service provider (#45116)

Add documentation on how to disable runtime tests (#45578)

* Update pr-guide.md

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

Return the result of PAL_InjectActivation in Thread::InjectGcSuspension (#44731)

Document Compilation Options and Compilation Metadata References CDIs (#37605)

Add Tuple-based Math.DivRem overloads (#45074)

* Add Tuple-based Math.DivRem overloads

* Remove internal versions of Math.DivRem

* Add AggressiveInlining

* Add comments

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

Fix location of JIT-EE version GUID (#45574)

With #45044 it moved from corinfo.h to jiteeversionguid.h

Update UnmanagedCallersOnly tests to not emit IL (#45572)

* Update UnmanagedCallersOnly tests to emit IL only when needed.

* Move last test into IL and delete test that uses invalid IL that only RefEmit can handle.

Make BinderEventListener not listen to TplEventSource (#45570)

* Make BinderEventListener not listen to TplEventSource

* Remove TasksFlowActivityIds definition

Co-authored-by: Juan Hoyos <juan.hoyos@microsoft.com>

XmlSerializer.Serialize doesn't work when using TrimMode=link (#44772)

* XmlSerializer.Serialize doesn't work when using TrimMode=link

Make XmlSerializer work correctly with ILLinker trimming.

1. Use constant BindingFlags to work around https://github.com/mono/linker/issues/1617
2. Annotate CodeGenerator.CreateTypeBuilder correctly to preserve base class members

Fix #41389

* Update ILLinker suppressions file.

* PR feedback.

Fix ILLinker suppresions.

SuperPMI collection pipeline: fix location of JIT-EE version GUID (#45576)

With #45044 it moved from corinfo.h to jiteeversionguid.h

Various fixes in Microsoft.NETCore.App packs (#45566)

Upgrade StyleCopAnalyzers to v1.2.0-beta.261 (#45562)

* Upgrade StyleCopAnalyzers to v1.2.0-beta.261

https://github.com/DotNetAnalyzers/StyleCopAnalyzers/releases/tag/1.2.0-beta.261

* Disable SA1142

* Enable SA1142 except in tests

Fix dependency for test assets on musl test runs (#45567)

* Fix dependency for test assets on musl test runs

* Use testOsSubGroup on dependency

Disable Long Running JSON tests on Checked CoreCLR (#45505)

* Disable Long Running JSON tests on Checked CoreCLR

* PR Feedback

Don't emit type fixups for primitive types in Crossgen2 (#45555)

We were emitting fixups for primitive types like bool, void, etc. These
are not necessary as all of these types are always loaded and so it was
just a waste of R2R file space.

This change blocks generating fixups for those.

Removed Serialization tests for SQLClient types (#42016)

* Remove Serialization tests for SQLClient types

* Deleted reference to System.Data.SqlClient

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

Fix covariant returns with generic return types (#45275)

* Fix covariant returns with generic return types

When a covariant return type was an uninstantiated generic type, the
ClassLoader::IsCompatibleWith was not working properly. In debug builds,
it was asserting because there was no MethodTable for that type and in
release builds, it resulted in ExecutionEngineException or an internal
CLR error.

This change fixes it by using TypeHandle::CanCastTo instead of
MethodTable::CanCastTo and adds a regression test for two cases where
the problem was observed (Assembly.GetTypes() and creating an instance
of a type with a covariant return with a problematic kind of type).

* Fix issue 45082 too

There were two issues. First, the
ClassLoader::ValidateMethodsWithCovariantReturnTypes was called before
typeHnd.DoFullyLoad and that resulted in an assert down the call chain
of TypeDesc::CanCastTo due to a wrong load level.
Second, the SigTypeContext generation for the current MD in the
ClassLoader::ValidateMethodsWithCovariantReturnTypes requires the same
change for class instantiation as the one that we had for the parent MD.

* Fix the issue 45082 in a correct way

The call to ClassLoader::ValidateMethodsWithCovariantReturnTypes is now
in MethodTable::DoFullyLoad.
I have also added a test case that verifies a case that David Wrighton
has suggested offline, where there are 3 types... A, B and C.
C derives from B which derives from A. B has a bad override which
should produce an error.  Then, cause C to be fully loaded without
otherwise triggering a load of B.

Remove FEATURE_COMPILED_XSL define (#45553)

* Remove FEATURE_COMPILED_XSL define

This is always defined, so there is no need in this define. In the past, it was not defined for UAP builds.

* Remove unused resource

Move return buffer handling from interop to the JIT (#39294)

Co-authored-by: Jan Kotas <jkotas@microsoft.com>
Co-authored-by: Carol Eidt <carol.eidt@microsoft.com>
Co-authored-by: Sergey Andreenko <seandree@microsoft.com>
Co-authored-by: Andy Ayers <andya@microsoft.com>

Update first-class-structs.md (#45512)

* Update first-class-structs.md

* Address feedback and add links to x64 vector ABI branches

Adds support for testing tvOS (#45043)

XHarness already supports tvOS as a target, so this change supplies the right info

Co-authored-by: Steve Pfister <steve.pfister@microsoft.com>

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

[master] Update dependencies from dotnet/xharness dotnet/runtime-assets dotnet/arcade

Correctly apply EventSourceSupport feature switch to NativeRuntimeEventSource (#43602)

`NativeRuntimeEventSource.Process` is called from native code in CoreCLR and as such is rooted in the descriptor. But if event source is disabled via feature switch it should not do anything.

[runtime] Avoid checking for hardened runtime on ios, its not needed, and it causes alerts because of PROT_WRITE|PROT_EXEC. (#45514)

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

Fix issue 45091 (#45409)

Fix overeager call to Validate() - we should call this with a bDeep parameter of FALSE, because the pointers to other objects may not be valid yet, if these other objects have not yet been copied.

Note this issue will only repro if HEAPVERIFY_GC is on - otherwise we won't attempt to verify pointers to other objects.

Temporarily disable Mono Desktop Windows Release x64 (#45529)

DisableFileSystemWatcherTest (#45511)

Remove ActivatorUtilities_In_DependencyInjection #define

Move ActivatorUtilities from the Common directory to DI.Abstractions

This is the only project in the repo that uses this file. There's no reason for it to be in the Common directory.

Fix continue on error conditions (#45519)

fix invalid IL in profiler stub helper (#45453)

Fixes paths (#45392)

Fix IID for System.Runtime.InteropServices.ComTypes.IEnumSTATDATA (#45498)

Delete unnecessary GetNativeHandle methods on reflection types (#45489)

They just return "this" for the most part

WinHttpHandler: fix double -> uint conversions for ARM64 (#45480)

* extend ReceiveDataTimeout functional coverage

* fix double -> uint conversions

* more comments + remove 'unchecked'

* Comment suggestion

Co-authored-by: Jan Kotas <jkotas@microsoft.com>
* Comment suggestion

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

Remove an erroneous assert (#45495)

Redirect default install location in test to avoid using machine-wide install (#45500)

disable restore in vs for projects targeting platform specific old frameworks (#44932)

* disable vs restore in vs for projects targeting platform specific older frameworks

* Add infra to auto-update solution files for restore

* Add NuGet.config files for solutions affected by restore issue

Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>

Fix break in send-to-helix-inner.yml (#45502)

Use the updated URL for FabricBot (#45499)

And fix a couple of formatting nits while editing (removing commas between owners, and aligning table spacing)

Fix Write_DataReadFromDesiredOffset test (#45477)

(Already failing on my goal not to submit PRs while on vacation, but I wanted to help squash this test failure taking out multiple PRs.)

Sporadic null reference exceptions were occurring in CI as part of GZipStream.Dispose.  The call to writeable.Dispose in the writing task can race with the invisible writeable.Dispose that happens as part of the disposal at the end of the block, due to the earlier using statement.  If the writeable.Dispose in the task nulls out the _deflateStream inside of the GZipStream after the implicit Dispose has checked for null but before it calls Dispose on the field, it'll null ref.  The fix is simple and something that should have been there anyway: explicitly wait for the writing task to complete, which guarantees the implicit disposal won't race with the explicit one.

Increase Mac outerloop timeout for AzDO job (#45490)

As seen in https://dnceng.visualstudio.com/public/_build/results?buildId=905703&view=logs&j=8580ecfb-912a-5dbd-35ce-e64d0d51ddbf, OSX test runs can get cancelled as little as 12 minutes before they would have finished (timed out job in question started at 2020-12-02 13:38:06.9640000, finished  2020-12-02 14:59:19.2760000, but the job was cancelled at 2020-12-02T14:48:29.3981590Z.

A small jit changes for Arm64 apple. (#45461)

* Expand the test.

* The jit change that slipped in branches and merges.

Add OSX arm64 jit. (#45467)

* CMake changes.

* use unix_osx.

Use jit interface thunkgen tech for more files (#45044)

- Reduce the boilerplate logic when modifying the jit ee interface
 - 3 new files in superpmi are now autogenerated as well as 2 files in the jit directory are now autogenerated
- Reduce to only having one copy of the jit-ee interface guid

Remove ResolveXHandle indirections (#45292)

* Remove internal redirection for RuntimeTypeHandle

* Remove internal redirection for RuntimeFieldHandle

* Remove internal redirection for RuntimeMethodHandle

* Add null token tests

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

Reduce CacheEntry size (#45410)

* replace 3 boolean fields with one enum flag field

* use auto property for AbsoluteExpiration

* simplify the code by using expression bodies

* don't use internal fields of CacheEntry, the public properties are enough

* expose private methods of MemoryCache and replace the usage of delegates with dependency to MemoryCache

2 fields less for MemoryCache and three for every CacheEntry

Fix the EH-write thru scenario (#45248)

* Fix the EH-write thru

* Fix condition for BBJ_THROW

* Proper fix

Fix deadlock in TestEventListener and re-enable ArrayPool DiagnosticEvent tests (#45469)

* fix deadlock in TestEventListener

* re-enable disabled test

* Apply suggestions from code review

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

Bump timeout for iOS CI jobs (#45484)

We've sometimes seen the job timeout on AzDO because the Helix queue got a little busy, bump the timeout to give it more breathing room.

Updates the RID graph to include Android 30 (#45442)

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

Add Mono Windows x86 builds (#45434)

Also contains initial work to support arm/arm64 builds but those don't build yet.

Update dependencies from https://github.com/dotnet/xharness build 20201201.1 (#45476)

[master] Update dependencies from dotnet/xharness

Fix for R2RDumpTests nullref in composite mode (#45447)

My original conjecture that the R2RDumpTests nullref crash in CG2
composite pipeline had the same cause as the libraries test
failures spotted over the weekend seems wrong. The R2RDumpTests
crash is a plain initialization issue:

In composite mode, native code for the entire framework gets
emitted into the special composite native image (CoreCLR test
scripting uses the name framework-r2r.dll for this image); the
individual component assemblies including SPC only act as holders
for the MSIL and forwarders to the native image.

For this reason, running R2RDumpTests to dump SPC in composite mode
yielded a library with zero native methods; thus we never entered
the conditional clause in EnsureMethods and _methods remained null.
I have reformulated the code by centralizing initialization of
both _instanceMethods and _methods in EnsureMethods.

Thanks

Tomas

Removed thread-safety from SequenceReader (#45437)

As it's a ref-struct and thus stack-only there's no need to have any guarantees for thread-safety and atomicity.

Change AssemblyName formatting not to require static table of KVPs (#45418)

* Change AssemblyName formatting not to require static table of KVPs
to do loop with span operations to replace single characters

Expanded test coverage to actually cover this odd behaviour

Realign the implementation with original code at
https://github.com/dotnet/runtime/blob/master/src/coreclr/src/binder/textualidentityparser.cpp#L690

Fix conditions to include suceeded on send to helix inner (#45432)

* Fix conditions to include suceeded on send to helix inner

* Fix yaml syntax

Disable DiagnosticEvent tests in arraypool (#45451)

[master] Update dependencies from mono/linker (#45108)

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

Microsoft.NET.ILLink.Tasks
 From Version 6.0.0-alpha.1.20561.1 -> To Version 6.0.0-alpha.1.20566.3

* Update tools to 5.0 RTM which is now required by illinker

The update was forced for linker because of https://github.com/dotnet/arcade/pull/6547 automatic distribution. The same change were suppose to land in dotnet/runtime in https://github.com/dotnet/runtime/pull/44835 but was reverted with no explanation in https://github.com/dotnet/runtime/pull/44835/commits/6b863052d7ccae09bc0dd50755414efe29c04805

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

Microsoft.NET.ILLink.Tasks
 From Version 6.0.0-alpha.1.20561.1 -> To Version 6.0.0-alpha.1.20580.1

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

Microsoft.NET.ILLink.Tasks
 From Version 6.0.0-alpha.1.20561.1 -> To Version 6.0.0-alpha.1.20580.2

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

Microsoft.NET.ILLink.Tasks
 From Version 6.0.0-alpha.1.20561.1 -> To Version 6.0.0-alpha.1.20580.3

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

Microsoft.NET.ILLink.Tasks
 From Version 6.0.0-alpha.1.20561.1 -> To Version 6.0.0-alpha.1.20580.4

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

Microsoft.NET.ILLink.Tasks
 From Version 6.0.0-alpha.1.20561.1 -> To Version 6.0.0-alpha.1.20580.5

* Revert "Update tools to 5.0 RTM which is now required by illinker"

This reverts commit 9532380dd2ee77bf50c3834eb798d68447c97d85.

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

Microsoft.NET.ILLink.Tasks
 From Version 6.0.0-alpha.1.20561.1 -> To Version 6.0.0-alpha.1.20601.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Marek Safar <marek.safar@gmail.com>

Remove ZipDownloadDisable for azdo artifacts download (#45427)

[wasm] remove compile timezone data task (#45443)

* [wasm] remove compile timezone data task

* Bump runtime-assets dependency

* Fix merge conflict

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

Enable rolling builds for staging pipeline and fail the build if any failure (#45430)

Handle additional multireg args (#43870)

* Handle additional incoming multireg args

[master] Update dependencies from dotnet/llvm-project dotnet/icu (#45412)

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

runtime.osx.10.12-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.osx.10.12-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.linux-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.linux-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk
 From Version 9.0.1-alpha.1.20573.1 -> To Version 9.0.1-alpha.1.20580.1

* Update dependencies from https://github.com/dotnet/icu build 20201130.2

Microsoft.NETCore.Runtime.ICU.Transport
 From Version 6.0.0-alpha.1.20573.1 -> To Version 6.0.0-alpha.1.20580.2

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

Fix Crossgen2 test concurrency on Windows (#45433)

In my previous attempt at fixing the test concurrency w.r.t. to
multiple "RunOnly" tests exercising the same "BuildAndRun"
executable compiled with Crossgen2,

https://github.com/dotnet/runtime/pull/44492

I have apparently used the wrong escaping in the batch (cmd)
version of the script so that the fix doesn't work on Windows.
This change is fixing this deficiency.

Thanks

Tomas

[tests] Use correct output path when running with helix, so that the log (#45390)

.. files get surfaced to azdo.

Thanks to @akoeplinger for the suggestion.

Suppress StackTrace ILLinker warning in TryResolveStateMachineMethod (#45042)

Remove dependency on PerfCounter.dll (#45142)

* Remove dependency on PerfCounter.dll

This dependency was unnecessary and only for calling through to pdh.dll.

PerfCounter.dll is still needed for the consumption side of performance
counters, but removing from the dependency from the .NET can help enable
some scenarios like reading non-.NET counters.

* Update conditions on performance counter tests.

* Condition more perf counter tests

Update lsra-detail.md (#45235)

* Update lsra-detail.md

Reenable Mono Windows runtime build in runtime-official.yml (#45376)

The binary now includes the proper version header.

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

Fix PlaintextStreamFilter for sync requests (#45388)

* Fix PlaintextStreamFilter for sync requests

As with ConnectCallback, make it usable for sync requests. Also:
- Consolidate blocking to the sync entry point where possible.
- Remove unnecessary async arguments
- Fix XxAsync naming on several methods

* Address PR feedback

Enable the AsnXml build target on non-Windows.

Co-authored-by: Adeel Mujahid <adeelbm@outlook.com>

More MemoryCache perf improvements (#45280)

* don't acces expensive CacheEntryHelper.Current if options can't be propagated anyway, +20k RPS for TechEmpower benchmark!

* inline the hot paths, +7k RPS

* prefer .HasValue over null check for nullables, use better names for private helper methods

* if possible, avoid expensive CacheEntryHelper.Current access when adding cache entries

r0InUse should be true for SoftFP in CodeGen::genProfilingLeaveCallback in src/coreclr/src/jit/codegenarm.cpp (#45377)

Update Interop CRST dependency. (#45372)

Remove unused Remoting CRST.

Add staging pipeline to PRs (#45178)

* Add staging pipeline to PRs

* Condition extraSubsets parameter

Fix issue 45183 (#45307)

Preserve the paramAttrs

Improve R2RDump method diff (#45301)

Based on my investigation of System.Private.CoreLib size growth
after switching over to Crossgen2 I have used R2RDump to analyze
the differences and this is an initial set of improvements I made
to make the diff more useful. Most notably it introduces
a statistic summary of method sizes and counts that were previously
lost in the lengthy tables of individual methods.

Thanks

Tomas

Remove NETStandard2.2 versions and product dependency (#43304)

* Remove NetStandard2.2 versions and product dependency

Fix CSE print (#45314)

Update host-tracing.md (#45389)

Add two readonlys to IcuLocaleData.cs (#45367)

Allow SocketsHttpHandler.ConnectCallback for sync requests (#45300)

HTTP/1.1 support with sync requests currently doesn't work when a ConnectCallback is specified, even though a developer who wanted to make synchronous requests could provide a synchronously-completing callback.

This also consolidates the connect logic across sync/async, avoids an extra delegate, etc.

Make the wasm sample csproj work on non-bash shells. (#45365)

Explicitly check for arrays in Marshal.SizeOf impl. (#45227)

Integrate fixes from dotnet/runtimelab:feature/NativeAOT (#45305)

attempt to stabilize CopyToAsync_AllDataCopied_Large for Tls13 (#44741)

* attempt to stabilize CopyToAsync_AllDataCopied_Large for Tls13

* feedback from review

* swap streams back

* feedback from windows

* remove extra using

optimize DateTimeOffset.UtcNow by removing redundant verification (#45281)

* optimize DateTimeOffset.UtcNow by removing redundant verification, cuts 10% of time on Windows

* Update src/libraries/System.Private.CoreLib/src/System/DateTimeOffset.cs

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

Fix build on Alpine edge (#45352)

The atoll definition in the pal.h was leaking into the PAL implementation and
on Alpine edge, the difference in throws() classification was causing a build
break.

Use AsSpan/AsMemory (start, length)  (#45310)

* Use AsSpan(start, length)

Replace `AsSpan().Slice(start, length)` with `AsSpan(start, length)`

* Use AsMemory(start, length)

Replace `AsMemory().Slice(start, length)` with `AsMemory(start, length)`

* Apply suggestions from code review

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

[master] Update dependencies from dotnet/xharness (#45263)

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

Optimize storage of icu locale data (#45296)

* Optimize storage of icu locale data

to reduce SPC size by about 10k and avoid allocating huge string on the heap

Fixes #45129

* Formatting fixes

Revert "Compile System.Private.CoreLib with Crossgen2 by default (#44618)" (#45320)

* Revert "Compile System.Private.CoreLib with Crossgen2 by default (#44618)"

This reverts commit b79e4d3cd24f76dd1bcb722f589327c827e66632.

* Add a change to ensure libraries are built/tested

Bump emscripten to 2.0.9. (#45299)

Support R2RDump in framework compilation (#45288)

Previously, when --r2r-dump-path was specified, we only applied it
to test app compilations. This change adds support for R2RDump
in framework compilation too.

Thanks

Tomas

Preserve UnmanagedCallersOnlyAttribute on mono, the runtime needs it, and it can be removed when using --used-attrs-only. (#45291)

[Android] Implement AES-GCM,CCM and add pal_ssl (#44989)

Fix stack size allocation during interpreter entry. (#45271)

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

Compile System.Private.CoreLib with Crossgen2 by default (#44618)

After I deleted the Utf8String-related experimental code from the
runtime repo per JanK's suggestion, I can now enable Crossgen2
compilation for System.Private.CoreLib by default.

Thanks

Tomas