Use GArray instead of GList when getting custom attributes from an image (#32974)

Xamarin.Forms calls this _constantly_ during startup, to the point where the malloc calls were actually showing up in the profile. This should help with that, and gives us some ms back during startup.
There are also a bunch of calls that return empty arrays, which in theory could be created in managed for a perf boost. However, it seems the Xamarin.Android team has mostly gotten rid of those calls and that optimization would complicate the code and require changes to our managed CustomAttribute implemented for both legacy and netcore, so I've opted to not check it in for now. If the empty calls continue to show up down the line, I can dig the code back up from my local branch and PR it separately.

Merge pull request #32912 from maryamariyan/primitives-total

Extensions consolidation

[Debug] IndexOutOfRangeException when extension data doesn't have two generic parameters (#33014)

* added generic param tests

* added proposed fix

* fixed debug statements to use underlying dictionary type

* removed baseline comment

* removed casts to idictionary

* using explicitly typed variable

Co-Authored-By: Layomi Akinrinade <layomia@gmail.com>
Co-authored-by: Layomi Akinrinade <layomia@gmail.com>

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

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

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

* Update dependencies from https://github.com/microsoft/vstest build 20200226-03

- Microsoft.NET.Test.Sdk - 16.6.0-preview-20200226-03

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

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

* fix the libraries build

* Update Directory.Build.props

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

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

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

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

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

- ILLink.Tasks - 0.1.6-prerelease.20127.1

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

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

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

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

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

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

Co-authored-by: Anirudh Agnihotry <anirudhagnihotry098@gmail.com>

Delete lock accounting (#33048)

This is left-over from CLR SQL support. We do not need it in CoreCLR.

Update CONTRIBUTING.md (#33054)

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

[loader] Free some leaked strings in NativeLibrary resolution (#32985)

[threading] Switch domain assemblies lock from OS to Coop (#32502)

I don't think you can create a root domain without reaching the thread info attach call, so this should be safe to change outright.

Fix relative path (#33018)

Update build scripts to enable Microsoft.Extensions projects incrementally
- BinPlacing for Extensions projects
- Ignore building Microsoft.Extensions. projects
- Use correct StringNameKeyId for Extensions projects
- Separate packaging path for extensions projects
- Add PreReleaseVersionIteration

Merge branch 'runtime-consolidation' from Extensions into 'primitives-total'

Remove unused locals in System.Runtime.WindowsRuntime (#33035)

Added two unittests that test whether or not opaque is an empty string, the qop value is read to incorporate the correct hash algorithm for the digest challenge. Also adjusted the conditional in the challenge parse function to allow the opaque value to be an empty string (#32983)

Synchronize shared crossgen2 files (#33026)

Specify entrypoint name for LoadLibrary import (#33025)

The Widechar probing is unnecessary.

Revert "Change debug info default variable tracking system" (#33021)

Replace Marshal.SizeOf for primitive types (#33020)

Handle offset better in DateTimeFormat (#33016)

Eliminate static field access where possible

Update PGO and IBC data versions. (#32978)

Code clean ups (inline out var, remove unnecessary assignments) (#2004)

* Inline out variable declarations

* Remove unnessary assignments

* Update src/libraries/System.Private.CoreLib/src/System/Collections/Generic/ArraySortHelper.cs

Co-Authored-By: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.Private.CoreLib/src/System/IO/Path.Windows.cs

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

Lazily generate native type layout information (#1828)

* Rename and generalize native field categories.

Remove COM-specific blocks on NativeFieldFlags since they don't do anything except require extra categories.

Update unix-x86-only code branch.

Classify each field in EEClassLayoutInfo as used for managed or native layout

Add fast blittability detector to prep for moving interop data calculations out of the type loader. (Blittability is needed in the type loader, so we need to calculate it early).

Heavily simplify native field categories now that they don't need to track blittability.

Update new field blittability checker to also validate the MarshalAs metadata (since if it is blittable, we'll never load the native type and do validation there).

Fix up issues in blittability check.

Remove some dependencies on EEClassLayoutInfo where appropriate.

Collect native layout info in a separate function from type loading.

Initialize NFDs from FieldDescs instead of reading directly from metadata.

Load native type information at a delayed time.

Don't initalize native type information unless the data is requested and the type is non-blittable.

Clean up new field handling of Windows.UI.Xaml.Interop.TypeName.

Fix contract violations and Marshal.SizeOf.

Remove some dead code.

Clean up the blittable checker to also note that KeyValuePair<T,V> in WinRT is non-generic.

Remove special cases that allowed illegal behavior because field marshal data initalization was in the type loader.

alignof -> sizeof

Add comment on why Decimal in particular is never blittable.

Seperate out the native layout info into a separate type.

Get runtime and dac building with it. Crossgen needs some more updates to get fragile NGen working with the new design.

Update fragile NGen to understand the new layout. Get the product to build.

Fixes for Linux build.

Update the loading of fields in layout types to not recurse into the native layout logic. The new logic doesn't support being run as part of type-load.

Remove unneeded "EnsureNativeLayoutInfoInitalized" calls.

Fix build breaks on ARM.

Rename GetNativeFieldFlags->GetCategory and m_flags->m_category.

Don't use the native layout info to get the offset of a field on a blittable type.

Fix indentation.

Fix x86 build.

Fix GC-mode contracts.

Fix thead safety issues. Move methods of EEClassLayoutInfo and EEClassNativeLayoutInfo to a new classlayoutinfo.cpp file.

Fix more contracts.

Move CheckForNativeHFA to classlayoutinfo.cpp

Fix x86 Windows build break.

Track if a MethodTable is marshalable in the native field layout metadata.

Fix case where an explicit structure has a field of non-layout struct, which itself has fields that would disqualify it from managed-sequential.

Fix incorrect calculation of managed size.

Fix accidental recursion into native layout initialization when calculating the SystemV eightbyte classification.

Fix Windows x86 failure.

Wrap MarshalNative::OffsetOfHelper in a helper method frame to fix contract violation.

Fix usage of CONSISTENCY_CHECK to CONSISTENCY_CHECK_MSG.

* Calculate NativeFieldDescriptor::IsNestedType from the category instead of including a bool.

* Change IsUnmarshalable() to return a C++ bool instead of a BOOL.

* PR Feedback.

* Remove native layout info from fragile ngen images/ibc.

* Initialize m_fd in NativeFieldDescriptor constructor.

* Cleanup after removing NGen image save/restore.

* PR Feedback.

* Initialize variable to satisfy compiler.

* Fix FieldDesc assignments.

* Lookup intrinsic vector types by typedef instead of name.

* Mark GetNativeHFATypeRaw as LIMITED_METHOD_CONTRACT.

* Clean up returns and only use return buffers when required.

* Use CQuickArray instead of _alloca.

* Remove out of date comment.

* Remove unused field.

* Make m_nativeLayoutInfo volatile.

* PR Feedback.

* Fix typo that broke non-blittable HFAs.

* Add test validating that we don't marshal Auto structs.

* Add test for returning an auto-struct.

* PR Feedback.

Additional tests for System.Text.Json (#32705)

* added multi-segment DateTime tests

* added missing null argument test

* added null argument tests and missing check for SerializeAsync

* removed unreachable defensive code block

* sealed class type for dictionary converters and removed unreachable code

* added additional test permutation

* added slices to DateTime conversion tests

* Revert "sealed class type for dictionary converters and removed unreachable code"

This reverts commit 1e8e33cdb2309ed37558c37b76454dbcaec0a1fa.

* sealed class type for internal json converters

Guard against NRE when deserializing extension data into dictionary without parameterless ctor (#32952)

* changed exception behavior for immutable dictionary

* refactored tests for deserialize

* using full generic dictionary for tests

* moved where exception is thrown

* added immutable serialization test

* added custom immutable dictionary extension data test

* refactored private ctor dictionary to inherit

* removed unused static create

Ensure Process.MainWindowHandle is refreshed (#32695)

* Ensure Process.MainWindowHandle is refreshed

This will set _haveMainWindow to true only if
_mainWindowHandle is non-zero. It also forces
the latter to be reset during a Process.Refresh()
call, hence allowing the re-evaluation of both.

Fix #32690

* Mark test for local outerloop testing only

* Rename exePath constant to ExePath

Update libraries workflow (#32939)

* Update README.md

Minor readability improvements
Unified camelCasing

* Update README.md

Enable nullable annotations for Utf8String. (#32967)

Clean up unnecessary things in the .csproj files.

Replace Marshal.SizeOf in System.Drawing.Common (#32962)

* Replace Marshal.SizeOf in System.Drawing.Common

* Inline size locals

* Remove out of memory checks

* Replace Marshal operations

Up the timeout for the cross-crossgen job to 3 hours (#32986)

System.IO.sln hangs (#32995)

* System.IO.sln hangs

* Apply suggestions from code review

Fix another 58 URL's (#33003)

Honor converters for underlying types of Nullable<T> specified with JsonConverterAttribute (#32006)

* FIxed exception being thrown when JsonConverterAttribute is used on Nullable<T> when converter can handle T.

* Expanded test coverage.

* Code review feedback.

* Code review feedback.

* Updated to use JsonValueConverterNullableFactory directly instead of fall-through logic.

* Null response is no longer possible.

* Code review.

* Updated for refactoring.

* Code review.

* Code review #2.

Updating labels to reflect the -coreclr postfix (#32997)

Add Server-Timing to KnownHeaders in System.Net.HTtp (#32989)

Fix RegexBoyerMoore.IsMatch case-insensitive (regression from 2.1 to 3.0) (#32984)

As part of a larger change, a change was made for .NET Core 3.0 that switched a character-by-character comparison loop to instead use string.Compare.  This represents a regression, however, when a culture-aware character-by-character comparison yields different results from a multiple-character comparison.

process more TLS frames at once when possible (#32897)

* process more TLS frames at once when possible

* style update

* stop on first FrameType.AppData

Only output type parameter constraint blank line if type parameter has a constraint (#32024)

Fixes #31614

Remove FEATURE_COMPILED from System.Text.RegularExpressions (#32919)

Update ProjectUrl in props (#32599)

I believe this should now be `/runtime` instead of `/corefx` so that it properly attributes the project location when shown in NuGet and elsewhere that relies on the ProjectUrl data.

removed unused local for System.Numerics.Tensor (#32948)

Adding some additional coverage to areas. (#32976)

Remove non-functional spawn code.

Subject to configuration, spawn has three widely divergent behaviors:

succeed but do nothing
print and exit
do what it sounds like
Rather than mislead the callers, provide only option 3 and let the caller decide what to do otherwise.

Co-authored-by: Jay Krell <jay.krell@cornell.edu>

Fix JIT traces for the case where a tier 1 compilation switches to MinOpts (#32928)

Updated to clear the tier 1 flag when switching to MinOpts. Clearing the flag avoids printing "tier 1" in traces and to print more accurate information around here:
https://github.com/dotnet/runtime/blob/c727a412bf2c415bb2b94eeef0939aa07b87591c/src/coreclr/src/jit/compiler.cpp#L4044

Package libcoreclrtraceptprovider.so in shared framework (#32949)

Delete unused libdl interop (#32959)

Reduce number of build libraries for Mono MSVC netcore build. (#32854)

Current mono-netcore.sln builds more than needed by Mono netcore build.
The only library currently needed is mono-2.0-sgen.dll build by
libmono-dynamic-vcxproj. In order to build that library, we also need to
build eglib as static library as well as genmdesc.vcxproj. Build also
build mono-sgen.exe to simplify use of older Mono CLI support and testing.

[mono] Use ALC and throw FileNotFound in RuntimeAssembly.InternalLoad (#32933)

https://github.com/dotnet/runtime/commit/7c6ec46a637017971d3eaa210fd2b6e4d4a6b737#diff-53c474033a1751601becf0eb958fcb5dL287-R287 broke a bunch of tests, this should fix them all.

Fixes #22110
Fixes #32436
Fixes #32437
Fixes #32435
Fixes #32439
Fixes #32434
Fixes #32433
Fixes #32432

[loader] Remove unnecessary newline in loader mono_trace call (#32934)

Enable test fixed already (#32946)

[runtime] Use pointer size gchandles (#32850)

This would enable in the future to use CoreCLR's GC with mono. This doesn't change the current embedding api, but, in the future, we will end up aligning with coreclr on an embedding api that uses pointer sized gchandles.

Co-authored-by: Vlad Brezae <brezaevlad@gmail.com>

Invert Environment.UserInteractive (fix) (#32947)

Update windows-requirements.md (#32938)

Clarify few minor things in initial setup workflow.

Re-re-enable Regex tests on .NET Framework (#32931)

I'd previously done this in the Configuration.props, but then those were deleted and my change was apparently lost in the transition.

Fix .* handling of RegexOptions.Singleline (#32930)

We're accidentally not setting the iteration variable to 0, which then messes up later computation.

Add binlogs to coreclr's build process (#32842)

Mark HostModel package as non-shipping (#32901)

The Microsoft.NET.HostModel nuget package is for internal consumption by the SDK.
Therefore mark it as non-shipping, so that it is not published to nuget.org

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

Incorrect alias in area owners (#32924)

Enable workflow of building Mono and Libraries without CoreCLR (#32881)

* Build libraries against Mono runtime

This change allows you to build libraries against the Mono runtime
without having to first build CoreCLR. There isn't a specifically
supported build option for this but can be forced with the following
command line:

```
.\build.cmd -subsetcategory libraries /p:RuntimeFlavor=mono
```

Fix jit build break (#32927)

Two changes to codegencommon.cpp were merged in short order, and
git/github did not spot the merge conflict.

Fixes #32926.

Update RID graph to include recent Android version and support for emulators (#32914)

Added missing InteropServices.WindowsRuntime types to mono SPC (#32905)

Addresses https://github.com/dotnet/runtime/issues/32898

Annotate System.Net.Security for nullable reference types (#32541)

* annotate System.Net.Security for nullable reference types

* address task nullability issue

* add nullable directives to common files

* address feedback

* add missing nullable directive

* make ProcessAuthentication method explicitly nullable

* fix build after rebase

* add missing nullable directives to common source files

* fix build

* fix build

* make APM method annotations consistent

* address feedback

* address feedback

* address feedback

* address feedback

* address feedback

* fix rebase build issues

Make JitDump label output multi-thread safe (#32578)

* Make JitDump label output multi-thread safe

`s_compMethodsCount` is a static variable and incrementing it when
two JITs are running at the same time can result in inconsistent labels. This PR fixes it by adding an instance variable `compMethodID` on `Compiler` object and setting it to the value of `s_compMethodsCount` after incrementing it using `InterlockedIncrement()`.

Changed the type of `s_compMethodID` to `LONG` to match the signature of `InterlockedIncrement()` and made it private.

Fixes #10984

Clean up libmono-dynamic.vcxproj dependency for netcore. (#32856)

Co-authored-by: Johan Lorensson <lateralusx.github@gmail.com>

JIT: convert codegen over to new style phases (#32899)

Convert codegen over to new style phases. To facilitate this, some
of the state that was passed around in locals in `genGenerateCode`
are now fields on the codegen object.

Also add a final "pseudo phase" in case we see very late asserts.

Fixes #32497.

RyuJIT: Optimize `obj.GetType() != typeof(X)` for sealed classes (#32790)

An array type with value type or final ref type elements is exact.

Use this to optimize more cases of type equality checking.

Disable force_install warning produced by CMake for libraries native build. (#32853)

* Use VS2019 Preview for libraries official build

This lets us enable warn as error in libraries build because this pools contain a newer CMake.

* Revert changes in runtime-official.yml

* Remove usages of useVsPreviewPool in runtime.yml

* Revert useVsPreviewPool usages in xplat-setup.yml

* Revert useVsPreviewPool changes in perf.yml

* Disable force_install warning in libraries build.

* Enable warnaserror in global live build.

* Remove empty line in perf.yml

Detect build config mismatch for MSVC Windows LLVM build. (#32868)

If LLVM build is done using different build configuration, Mono build will fail to static link LLVM libraries due to incompatibility between release and debug C++ runtime library, generating a lot of linker errors like this:

error LNK2038: mismatch detected for 'RuntimeLibrary': value 'MD_DynamicRelease' doesn't match value 'MDd_DynamicDebug' in mini-llvm-cpp.obj

error LNK2038: mismatch detected for '_ITERATOR_DEBUG_LEVEL': value '0' doesn't match value '2' in mini-llvm-cpp.obj

This is normally not a problem when doing local builds of LLVM but when using an external provided build, like netcore Mono build, this problem is more likely to occur.

This fix adds an additional check to make sure build configuration reported by llvm-config.exe --build-mode matches current Mono build configuration and if not give a more specific msbuild error:

error : LLVM build Configuration=Release doesn't match current Mono build Configuration=Debug

Co-authored-by: Johan Lorensson <lateralusx.github@gmail.com>

Reduce number of build libraries for Mono MSVC netcore build.

Current mono-netcore.sln builds more than needed by Mono netcore build.
The only library currently needed is mono-2.0-sgen.dll build by
libmono-dynamic-vcxproj. In order to build that library, we also need to
build eglib as static library as well as genmdesc.vcxproj. Build also
build mono-sgen.exe to simplify use of older Mono CLI support and testing.

[wasm][debugger] Always pass failed breakpoint requests to target (#32906)

Originally we had a simple way to detect that breakpoints were
meant for the runtime but that is no longer true.  Now if we
can't locate a breakpoint we need to pass the request on to the
target so it can make an attempt.

Co-authored-by: Larry Ewing <lewing@microsoft.com>

[wasm][debugger] Show the typename for valuetypes (#32874)

- no details of the valuetype are accessible

Fixes mono/mono#18232

Co-authored-by: Ankit Jain <radical@gmail.com>

[aot] Avoid `mono_defaults` when creating trampolines. (#32904)

Fixes https://github.com/mono/mono/issues/19009.

https://github.com/mono/mono/pull/18505 made PLT initialization occur at
image load time. When initializing the PLT for mscorlib,
`mono_defaults.corlib` is not yet initialized.

This change works around this by using `mscorlib_aot_module`--which is
initialized relatively early--instead of `mono_defaults.corlib`. PLT
initialization is also moved slightly later: it now happens after
`use_page_trampolines` has been initialized. The `USE_PAGE_TRAMPOLINES`
macro has also been changed to dereference `mscorlib_aot_module`.

This commit also removes an unused parameter from
`mono_aot_create_specific_trampoline`.

Co-authored-by: imhameed <imhameed@microsoft.com>

Update shared code (#32779)

Annotate System.Net.Sockets for nullable reference types (#32675)

* All AsyncResult classes have nullable enable.

* Most ancillary files annotated.

* UdpClient annotated

* Annotate TcpClient

* Annotate TcpListener, SafeSocketHandle, SocketTaskExtensions

* Annotate NetworkStream and SendPacketsElement

* Annotate SocketPal.

* Annotate SocketAsyncEventArgs

* Annotate SocketAsyncEngine and Context on Unix

* Annotate Socket.Tasks

* Finish annotating the Socket class.

* Remove inline #nullable lines

* Annotate all public methods.

* Sync up to latest code and cleanup.

* Fix .NET Framework build for WinHttpHandler.

Small fixup in System.Net.Http.WinHttpHandler.csproj (#32890)

removed unused local variables (#32896)

Suppress the GrayListedLeaf error (#32895)

Fix comment: "ThreadLocal" -> "ThreadStatic" as used in code (#32887)

Comment aligned with actual attribute name in code

Enable crossgen tests (#32330)

This turns on benchmarking for a few binaries to track crossgen.

Add ValueNumbering support for GT_SIMD and GT_HWINTRINSIC tree nodes (#31834)

* Added ValueNumbering support for GT_SIMD and GT_HWINTRINSIC tree nodes

* Allow SIMD and HW Intrinsics to be CSE candidates

* Correctness fix for optAssertionPropMain
  - Zero out the bbAssertionIn values, as these can be referenced in RangeCheck::MergeAssertion
    and this is shared state with the CSE phase: bbCseIn

* Improve the VNFOA_ArityMask

* Update to use the new TARGET macros

* Include node type when value numbering SIMDIntrinsicInit
Mutate the gloabl heap when performing a HW_INTRINSIC memory store operation
Printing of SIMD constants only support 0

* Disable CSE's for some special HW_INTRINSIC categories

* Code review feedback

* Record csdStructHnd; // The class handle, currently needed to create a SIMD LclVar in PerformCSE

* Instead of asserting on a struct handle mismatch, we record it in csdStructHndMismatch and avoid making the candidate into a CSE

* Fix the JITDUMP messages to print the CseIndex

* add check for (newElemStructHnd != NO_CLASS_HANDLE)

* Additional checks for SIMD struct types when setting csdStructHnd
Added Mismatched Struct Handle assert in ConsiderCandidates

* fix GenTreeSIMD::OperIsMemoryLoad for ARM64
Removed ismatched Struct Handle assert

* Fix the printing of BitSets on Linux, change the printf format specifier

* Added check for simdNode->OperIsMemoryLoad()) to fgValueNumberSimd
Added bool OperIsMemoryLoad() to GenTreeSIMD, returns true for SIMDIntrinsicInitArray
Added valuenumfuncs.h to src/coreclr/src/jit/CMakeLists.txt

* Avoid calling gtGetStructHandleIfPresent to set csdStructHnd when we have a GT_IND node

* Fix check for (newElemStructHnd != hashDsc->csdStructHnd)

* added extra value number argument VNF_SimdType for Most SIMD operations
added VNF_SimdType // A value number function to compose a SIMD type
added vnDumpSimdType

* Added bool methods vnEncodesResultTypeForSIMDIntrinsic and vnEncodesResultTypeForHWIntrinsic
these return true when a SIMD or HW Instrinsic will use an extra arg to record the result type during value numbering
Changes the ValueNumFuncDef to set the arity to zero when a -1 value is passed in
Updated InitValueNumStoreStatics to fixup the arity of SIMD or HW Instrinsic that have an extra arg to record the result type
Allow a type mismatchj when we have a GT_BLK as the lhs of an assignment, as it is used to zero out Simd structs

* Fix for SIMD_WidenLo arg count

* fix typo

* Fix x86 build breaks
Fix SIMD_WidenHi

* Added method header comment for vnEncodesResultTypeForHWIntrinsic
Added & VNFOA_ArityMask when assigning to vnfOpAttribs[]

* Codereview feedback and some more comments

* fix typo

* Moved the code that sets the arg count for the three SIMD intrinsics

* clang-format

* Adjust CSE for SIMD types that are live across a call

* Proposed fix for #32085

* Revert "Proposed fix for #32085"

This reverts commit 169c24eeafb072a79ffbdb2d1d36960ebc0c17b4.

* Added better comments for optcse SIMD caller saved register heuristics

* Added CONFIG_INTEGER: JitDisableSimdVN,
   Default 0, ValueNumbering of SIMD nodes and HW Intrinsic nodes enabled
   If 1, then disable ValueNumbering of SIMD nodes
   If 2, then disable ValueNumbering of HW Intrinsic nodes
   If 3, disable both SIMD and HW Intrinsic nodes

* Moved JitDisableSimdVN from DEBUG to RETAIL

Fix reference to VS 2015 (#32886)

The FakeSign instructions reference Visual Studio 2016 Update 2
instead of Visual Studio 2015 U2.

Fix #32885

Update Utf8String APIs (#32892)

Merge pull request dotnet/Extensions#3028 from sharwell/mit-license

Switch to MIT license

Commit migrated from https://github.com/dotnet/Extensions/commit/f4066026ca06984b07e90e61a6390ac38152ba93

For large allocation tests, check for available memory on Windows too (#32545)

* For large allocation tests, check for available memory on Windows too

Previously, we had logic to get the physical memory on non-Windows,
but nothing on Windows.
This means that on windows machines with low memory, we would allocate
too much and the test would fail.
This PR logic to get physical memory on Windows too.

On Windows, it invokes the `systeminfo` command.
It shares parsing logic with the non-windows case, since they both need
to look for a line with a particular start and end.

Tested this change by manually increasing the allocation size in
`largeexceptiontest.cmd` and seeing that it allocated 300MB instead.
Also did the same test on Ubuntu since non-Windows code was affected too.

Fix #27098

Remove duplicate MemCheck and LargeObject code

Accept more uint formats

Use GlobalMemoryStatusEx on Windows instead of systeminfo

* simplify meminfo parsing

Co-authored-by: Andy Hanson <andy.ms.2019@gmail.com>

Restore blank lines

These lines were removed unexpectedly by the automated file header code
fix.

Commit migrated from https://github.com/dotnet/Extensions/commit/738f2d6cfa40bbf3f21f2580903850414c7e6bae

Update license in reference assemblies

Commit migrated from https://github.com/dotnet/Extensions/commit/d5255396ab10f90c15a01d7622c3a2b785817913

Switch file headers to the MIT license

Commit migrated from https://github.com/dotnet/Extensions/commit/321a30c4d74b387db587d92f965391235fe33770

Normalize all file headers to the expected Apache 2.0 license

Commit migrated from https://github.com/dotnet/Extensions/commit/cec6e75f0ce3145c76acd4c8a8c14093696a03ba

[wasm][debugger] Fixes to get async method locals (#32807)

.. and cleans up tests.

Co-authored-by: Ankit Jain <radical@gmail.com>

Merge pull request #2107 from BrianBohe/master

Change debug info default variable tracking system

JIT: use new phase objects more extensively (#32688)

Convert the mainline compiler phases over so they are run via compiler or
action phases.

To keep jit dump size and checked jit throughput the same as it is now,
add phase whitelists to opt into pre phase dumps and checks triggered by the
phase objects on a per-phase basis. This allow us to gradually remove explicit
dumps and check invocations phase by phase, over time.

[runtime] Remove MONO_LLVM_INTERNAL. (#32796)

Co-authored-by: Zoltan Varga <vargaz@gmail.com>

Re-disable CA2010 (#32872)

Apparently there were missed warnings that are now being treated as errors due to a PR that raced with mine yesterday to update the CodeAnalysis.ruleset file.

Plural TargetFrameworks replaced with singluar TargetFramework (#32860)

Stress test build fails because we have a plural tag <TargetFrameworks>, but only one nested element. PR converts it to a singular <TargetFramework>.

Fixes #32813

Send QUIT on SmtpClient.Dispose() (#683)

* send QUIT on Dispose()

* add try/finally

* simplify null check

* address PR feedback

* send quit in non-blocking mode

* resolve conflicts

Fix Mono netcore TPA preload hook to work with Windows CoreRun.exe. (#32789)

Current hook compares TPA list file names using case sensitive compare.
On Windows, the TPA list, when passed from CoreRun.exe makes all file
paths lower case, meaning that our checks fails to find assemblies in
TPA list.

Since CoreRun.exe is used for both CoreCLR and Mono, removing the tolower
on file paths added to the TPA list in CoreRun.exe could be an option,
but that might introduce regressions in CoreCLR, so this fix isolate the
change to Mono's netcore preload hook.

Nullable annotate System.Drawing.Common (#31716)

* Nullable annotate System.Drawing.Common

* Annotate unix files

* add removed attributes

* fix rebase issue

* address feedback

* add missing annotation

* address feedback

* address feedback

* address more feedback

Add support in crossgen2 for 32-byte alignment (#32602)

Use VS2019 Preview for libraries build which contains a newer CMake (#32626)

* Try Windows Server pools with VS2019 Preview

* Pass parameter to use vs preview pool.

* Use VS2019 Preview Pool when parameter is true

* Update xplat-setup.yml

* Update global-build-job.yml

* PR Feedback

* Update perf.yml to use right pools.

* Fix casing of live runtime configuration.

* Libraries build configuration upper case

Fix Windows CoreRun.exe to work with CoreCLR/Mono xplat hosting API. (#32788)

CoreRun.exe is shared between both CoreCLR and Mono. On Windows,
CoreRun.exe is still using ICLRRuntimeHost4 hosting API but Mono
doesn't implement that API, just the xplat hosting API. Fix change
CoreRun.exe on Windows to use xplat hosting API, so it works with
both CoreCLR and Mono on Windows. CoreRun on Unix already used
xplat hosting API.

Behaviour on CoreRun.exe Windows CoreCLR should be identical, since
implementation of xplat hosting API on CoreCLR uses ICLRRuntimeHost4
internally and calls to CreateAppDomainWithManager, ExecuteAssembly,
UnloadAppDomain2 and Stop look identical to what done in CoreRun.exe
before this fix.

Reg struct copy. (#32362)

* Fix `gsMarkPtrsAndAssignGroups` for struct assignment.

The old code was expecting `ASG` in a form that doesn't exist anymore.

* Fix SPMI replay with `repCompareTypesForEquality` misses.

* Reg struct copies.

Keep `ASG(LCL_VAR struct, LCL_VAR or INIT_VALUE)` representation in morph
and rationalize and change it to STORE_LCL_VAR later.

* Don't do retyping for `ASG(LCL_VAR struct, init val)` .

* Add a comment about morph tranformation.

and fix format error after merge.