Merge branch 'master' into fix/signed-compare-and-narrowing

Commit migrated from https://github.com/dotnet/coreclr/commit/e948e3c333645ec6fc74763cde095774f67c9b27

GCC compatibility fixes dotnet/coreclr#7 (dotnet/coreclr#22810)

* Use thread_local for thread local storage on non MSVC targets

* Use local copy of visitor rather than function parameter

* Remove extra class qualifier

* Replace hex number representation in ASM files

* Reorder STDAPI and DLLEXPORT

* Suppress conversion

Suppress warning during hash

add casting

* Remove anonymous struct

src/vm/codeversion.h:112:16: warning: ‘struct NativeCodeVersion::<anonymous union>::SyntheticStorage’ invalid; an anonymous union can only have non-static data members [-fpermissive]
         struct SyntheticStorage

* Remove class declaration

Remove extra class declaration

* Remove extern C

* Add implicit paranthesis

src/vm/amd64/virtualcallstubcpu.hpp:735:103: warning: suggest parentheses around ‘-’ in operand of ‘&’ [-Wparentheses]
     resolveInit.toMiss1                = offsetof(ResolveStub,miss)-(offsetof(ResolveStub,toMiss1)+1) & 0xFF;
                                                                                                       ^
src/vm/amd64/virtualcallstubcpu.hpp:741:103: warning: suggest parentheses around ‘-’ in operand of ‘&’ [-Wparentheses]
     resolveInit.toMiss2                = offsetof(ResolveStub,miss)-(offsetof(ResolveStub,toMiss2)+1) & 0xFF;

Add parenthesis

src/vm/dataimage.cpp:631:55: warning: suggest parentheses around ‘&&’ within ‘||’ [-Wparentheses]
                  previousRvaInfo->rva == rvaInfo->rva && previousRvaInfo->size >= rvaInfo->size

Add parenthesis

src/debug/daccess/daccess.cpp:6871:29: warning: suggest parentheses around ‘&&’ within ‘||’ [-Wparentheses]
     _ASSERTE(peFile == NULL && reflectionModule != NULL || peFile != NULL && reflectionModule == NULL);

Add parenthesis

src/vm/dataimage.cpp:631:57: warning: suggest parentheses around ‘&&’ within ‘||’ [-Wparentheses]
                  (previousRvaInfo->rva == rvaInfo->rva) && (previousRvaInfo->size >= rvaInfo->size)

* Initialize member 1

src/ilasm/method.cpp:35:36: warning: operation on ‘((Method*)this)->Method::m_ulColumns[0]’ may be undefined [-Wsequence-point]
     m_ulColumns[0]=m_ulColumns[0]=0;

* Remove unknown compiler option

* Abstract DLLEXPORT

Commit migrated from https://github.com/dotnet/coreclr/commit/cbd672eb2735d583ee6fa46aaf599241fdf6face

Spec for EventCounter changes (dotnet/coreclr#22811)

Commit migrated from https://github.com/dotnet/coreclr/commit/9901aa14bef2006f614d450ff0e45b6081262de7

Add the coreclr blob feed as a restore source for test builds. (dotnet/coreclr#22839)

* Add the coreclr blob feed as a restore source for test builds.
* Use one NuGet.config in the root folder for restore targets.

Commit migrated from https://github.com/dotnet/coreclr/commit/dc6252fa381d370c5feee9820d4cc5884fcbf719

Special-case 1 module per assembly in Assembly.GetTypes/DefinedTypes (dotnet/coreclr#22825)

It's very common for an assembly to have only one module, in which case we can avoid unnecessary allocations and copies in Assembly.GetTypes() and Assembly.DefinedTypes.

Commit migrated from https://github.com/dotnet/coreclr/commit/8509bbd72ed937a626fc436715b962d3f86fa380

Remove buildpipelines (dotnet/coreclr#22835)

* Remove old buildpipeline

* Remove obsolete publish-packages

* Remove obsolete helix scripts

Commit migrated from https://github.com/dotnet/coreclr/commit/af6d24190078f64f892a60967f69c86d838f06c2

Add platforms template parameter to platform-matrix.yml (dotnet/coreclr#22800)

Commit migrated from https://github.com/dotnet/coreclr/commit/265f3c31751ed9f11abbef3a158f45ee2b2e3688

For linux-musl-arm64 use OutputRid (dotnet/coreclr#22840)

This variable was added for linux-musl-x64 in our old official builds,
because our current logic in src/.nuget/dirs.props incorrectly parses
the __DistroRid passed.

Commit migrated from https://github.com/dotnet/coreclr/commit/84f0ae886cc659fabd8d8b499861383bb801fa08

Fix and re-enable some finalizer tests (dotnet/coreclr#22771)

Mark key allocating methods noinline, otherwise they may get inlined
under jit stress and leave jit temps referring to allocations that the test
expects should be collectible.

Resolves dotnet/coreclr#19218.

Commit migrated from https://github.com/dotnet/coreclr/commit/268a6b777febf10e97badede96dcddd54b54c4e1

Remove support for ICLRExecutionManager and pause/resume code for waits (dotnet/coreclr#22834)

Commit migrated from https://github.com/dotnet/coreclr/commit/ea5971a6fe1672f0334c9530d68e8621a9877128

Fix LONG64 definition

Commit migrated from https://github.com/dotnet/coreclr/commit/7c1ee7a5b02d1daa1e7ed4acdcb41913c307551d

Move WaitHandle to shared CoreLib (dotnet/coreclr#22634)

* Move part of WaitHandle to shared CoreLib

* Bring back OpenExistingResult to fix build

* Move reminder of WaitHandle to shared, implement SynchronizationContext logic in managed code, handle SafeHandle referencing in managed code

* Update with changes neeeded for CoreRT

* Span -> ReadOnlySpan

* Remove dead code

* Fix SafeHandle error handling

* Avoid double check on input values in WaitHandle.WaitOne overloads

* Make OpenExistingResult private protected instead of internal

* Make WaitHandle._waitHandle private

* Code style fixes

* Remove unnecessary GC.KeepAlive

* Move ERROR_TOO_MANY_POSTS exception handling to CoreCLR specific code

* Add cache for wait arrays to match previous CoreRT behavior and reduce GC pressure, change unmanaged code to allocate small array on stack

* Address PR feedback

* Reduce allocations/copying by moving stack allocation to managed code

* Address PR feedback

* Minor code reshuffle

* Move thread local access close to each other

* Address code style feedback

* Add const to Wait* constants

Commit migrated from https://github.com/dotnet/coreclr/commit/cf85f56e473b6cdc4a68e9758f57dc571e14c9cb

Merge pull request dotnet/coreclr#22796 from ericstj/splitILAsmCopy

Split out copy of Ilasm and make it optional

Commit migrated from https://github.com/dotnet/coreclr/commit/5ffa6ecefe6b1dcd2418ce7a5524f367e7f865f0

Address janvorli's feedback comments

Commit migrated from https://github.com/dotnet/coreclr/commit/805c8b231e0b8354b3997e81a779604fa1897c4e

Address CR feedback

Commit migrated from https://github.com/dotnet/coreclr/commit/891d7e7a69f4254b13de546b8fd60ded8cde866a

Suppress conversion

Suppress warning during hash

add casting

Commit migrated from https://github.com/dotnet/coreclr/commit/fcdfa70be9af5e7ec8133c3aaf73b142991e2bd4

Fix comparison and narrowing errors reported by GCC

Commit migrated from https://github.com/dotnet/coreclr/commit/dc19f81f05dd2559af329c528c2f04e869d8ef1e

Removed erroneous use of 'PROCESSOR_ARCHITECTURE' Enviroment variable (dotnet/coreclr#22743)

Removed erroneous use of 'PROCESSOR_ARCHITECTURE' Enviroment variable

Changed all uses of 'Enviroment.GetEnviromentVariable("PROCESSOR_ARCHITECTURE")'
to usage of 'System.Runtime.InteropServices.RuntimeInformation.ProcessArchitecture'
and 'enum System.Runtime.InteropServices.Architecture'.

Fix dotnet/coreclr#22694

Commit migrated from https://github.com/dotnet/coreclr/commit/333232d98639df980133205c41791cb9dc7f3d34

fix BSWAP16 intrinsic handling of signed values (dotnet/coreclr#22666)

Commit migrated from https://github.com/dotnet/coreclr/commit/56da27b66c9463f0a28b2427aabbe2af0614efdc

Move AssemblyLoadContext to shared partition (dotnet/coreclr#22685)

* Move AssemblyLoadContext to shared partition

* Move static initializer to DefaultAssemblyLoadContext

and remove stream copying from lock scope

Commit migrated from https://github.com/dotnet/coreclr/commit/2c61b985c0513d9a70eee7221fa15910e7003d16

Make System.Runtime.CompilerServices.Unsafe inbox (dotnet/corefxdotnet/coreclr#35526)

* Make System.Runtime.CompilerServices.Unsafe inbox

Fix a couple places where we wanted to use it from other inbox assemblies but couldn't.

Unsafe remains as a package and newer versions of Unsafe will replace the inbox Version,
permitting us to add API over time.

* Update Rune tests to use TryEncodeToUtf8Bytes directly

* Add temporary suppression to package tests

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/89ecfac9aa06491295e067edaa91193a88ae7b9c

Adding a missing single-quote (dotnet/coreclr#22809)

Commit migrated from https://github.com/dotnet/coreclr/commit/cb20f3982d3d2b1426c477d66221d6887eaec168

Remove 'from HRESULT' (dotnet/coreclr#22782)

Commit migrated from https://github.com/dotnet/coreclr/commit/e3d73ad3c8216118d3fa7962a6d169ba8932902e

Change Environment.Version to return product version (dotnet/coreclr#22664)

* Change Environment.Version to return product version

- Contributes to https://github.com/dotnet/corefx/issues/31099
- Use AssemblyInformationalVersion attribute as fallback

* Add sanity test for Environment.Version

* Disable CodeDom tests

* Fix test assembly name

Commit migrated from https://github.com/dotnet/coreclr/commit/83e9a3959620da5fd3ecdb1ee9d1fa6a80569cf8

Correctly build packages for linux-musl-arm64 (dotnet/coreclr#22793)

* Correctly build packages for linux-musl-arm64

Also add build-package step to PR.

* Correctly add back if

* Fix build-packages

Commit migrated from https://github.com/dotnet/coreclr/commit/3e8cd50bbf3f510ade4a4e8ab889d1cb184ad6eb

Enable FEATURE_BASICFREEZE (dotnet/coreclr#22776)

Commit migrated from https://github.com/dotnet/coreclr/commit/4932d0f6fa87b9a6e3293aacd35cecedde44d87a

Add fatal error details printing (dotnet/coreclr#22789)

Before this change, we were only printing details on fatal errors with
error code COR_E_FAILFAST. For other error codes we were just silently
exiting.
This change enables printing fatal error details for all kinds of fatal
errors.

Commit migrated from https://github.com/dotnet/coreclr/commit/6a392d9084a1afe8b9861138e5d0e5b688deca44

Update CoreFX Windows Arm32 exclusion list. (dotnet/coreclr#22695)

* Update CoreFX Windows Arm32 exclusion list.

* Exclude System.Globalization.Calendars.Tests.

* Disable System.Net.Sockets.Tests

Commit migrated from https://github.com/dotnet/coreclr/commit/ec41a807d2698379e9a1ad0da296898def46c54d

Split out copy of Ilasm and make it optional

Allow for a repository to provide its own copy of ILAsm rather than restoring & copying it its own location.

To use, set the ILAsmToolPath proerty to the directory containing ilasm, similar to CscToolPath.

I noticed that these targets left behind the ILAsm files in temp for each project every time it
built when ToolsDir wasn't defined, so I've fixed that by instead copying to obj.

Commit migrated from https://github.com/dotnet/coreclr/commit/9069830185d02c4753ab03a6d4b8c9036644f825

Ignore EINVAL/EBADF errors from FTruncate in FileStream.Init (dotnet/coreclr#22788)

They indicate we're working with a special file, where failure to truncate is not a fatal condition.

Commit migrated from https://github.com/dotnet/coreclr/commit/187ce49c992aef74cdbf60c445d5f68a60c37f4d

Disable IJW tests on Windows 7. (dotnet/coreclr#22779)

Windows 7 loads desktop mscoree.dll when loading System.Private.CoreLib, before we can load our fake mscoree. So, when IJW modules are loaded, they load the real mscoree.dll, which breaks coreclr's IJW loading. Once we have true .NET Core IJW support in the MSVC++ compiler and .NET Core, then we'll be able to reenable these tests on Win7.

Fixes dotnet/coreclr#21796.

cc: @RussKeldorph @echesakovMSFT

Commit migrated from https://github.com/dotnet/coreclr/commit/3984fc9b98770289def807e1953533bc4a28781a

R2RDump - Gracefully handle inability to parse GC Info

The Amd64 GC Info parser is not complete in its implementation and
sometimes will index to a location beyond the size of the image.

If we fail to parse the GC info for a method, continue dumping the rest
of the image.

Emit a warning when this happens so we don't forget about this.

Commit migrated from https://github.com/dotnet/coreclr/commit/b43650c6b60b47c5823c9bf1806fab0a5ca3f7c1

More GNUC Fixes (dotnet/coreclr#22687)

* Replace __sync_swap with __atomic_exchange_n

__sync_swap() is a clang specific function.

* Remove multiline comment

* Add paranthesis around sum

src/md/hotdata/../inc/streamutil.h:73:34: warning: suggest parentheses around ‘+’ in operand of ‘&’ [-Wparentheses]
     UINT32 aligned = *totalBytes + 3 & ~3;

* Define __int64

* Define windows types for tests

* Remove undefined has_builtin defines and define alloca and inline for GNUC

* Remove __clang__ where possible

* Add implicit casting to help compiler find WCHAR* variant

src/binder/assembly.cpp:294:73: error: no matching function for call to ‘SString::SString(SString)’
         return (pAsmName == nullptr ? nullptr : pAsmName->GetSimpleName());
                                                                         ^
In file included from src/inc/sstring.h:1082:0,
                 from src/inc/ex.h:19,
                 from src/inc/stgpool.h:28,
                 from src/inc/../md/inc/metamodel.h:18,
                 from src/inc/../md/inc/metamodelro.h:19,
                 from src/inc/metadata.h:17,
                 from src/binder/../vm/util.hpp:19,
                 from src/binder/../vm/common.h:110,
                 from src/binder/assembly.cpp:14:
src/inc/sstring.inl:73:8: note: candidate: SString::SString(void*, COUNT_T)
 inline SString::SString(void *buffer, COUNT_T size)
        ^
src/inc/sstring.inl:73:8: note:   candidate expects 2 arguments, 1 provided
src/inc/sstring.inl:436:8: note: candidate: SString::SString(SString::tagLiteral, const WCHAR*, COUNT_T)
 inline SString::SString(tagLiteral dummytag, const WCHAR *literal, COUNT_T count)
        ^
src/inc/sstring.inl:436:8: note:   candidate expects 3 arguments, 1 provided
src/inc/sstring.inl:418:8: note: candidate: SString::SString(SString::tagLiteral, const WCHAR*)
 inline SString::SString(tagLiteral dummytag, const WCHAR *literal)
        ^
src/inc/sstring.inl:418:8: note:   candidate expects 2 arguments, 1 provided
src/inc/sstring.inl:401:8: note: candidate: SString::SString(SString::tagUTF8Literal, const UTF8*)
 inline SString::SString(tagUTF8Literal dummytag, const UTF8 *literal)
        ^
src/inc/sstring.inl:401:8: note:   candidate expects 2 arguments, 1 provided
src/inc/sstring.inl:382:8: note: candidate: SString::SString(SString::tagLiteral, const CHAR*)
 inline SString::SString(tagLiteral dummytag, const ASCII *literal)

* Reorder DLLEXPORT and STDAPI

GNUC wants extern "C" <attribute> format.

* Abstract __FUNCSIG__

* Abstract __debugbreak()

* Move common compiler options out of clang and add Wno-unused-value

* Add paranthesis around || and &&

src/gc/gc.cpp:9084:38: warning: suggest parentheses around ‘&&’ within ‘||’ [-Wparentheses]
                     (!chosen_power2) && (i < free_space_count));

* Set Wno-delete-non-virtual-dtor for CXX files only

* Don't warn on unterminated endif labels

* Suppress unused functions

* Use 0x syntax rather than h syntax on GNU asm files

* Correct constructor call directly

src/ToolBox/superpmi/superpmi-shared/logging.cpp:301:27:   required from here
src/inc/clr_std/string:58:9: error: cannot call constructor ‘std::basic_string<char>::basic_string’ directly
         this->basic_string::basic_string(_Ptr, c_len(_Ptr));

* Suppress NULL used in arithmetic warnings

Commit migrated from https://github.com/dotnet/coreclr/commit/0495f7377fca88999d335757e9680052cfedc51d

Disable tests that fail to build under 3.0.0-preview3 (dotnet/coreclr#22774)

Commit migrated from https://github.com/dotnet/coreclr/commit/1376e5fc5fa4b01a175e26be467a416f1c8765fc

Exclude /Interop/COM/NETClients/Events/NETClientEvent for arm32 Windows. (dotnet/coreclr#22785)

Commit migrated from https://github.com/dotnet/coreclr/commit/ccf62746dcc5455b80df1f8d8638b75c975788a6

Store information about partial ngen in R2R images (dotnet/coreclr#22680)

Commit migrated from https://github.com/dotnet/coreclr/commit/dbf6006da52845624577f88f3200447c4a1d8db2

Update ReJIT/MethodDescBackpatchInfoTracker lock relationship (dotnet/coreclr#22741)

Commit migrated from https://github.com/dotnet/coreclr/commit/df4aa3315f64f43344b4aaf6e57aad9fe0b36cbf

Delete unused FEATURE flags (dotnet/coreclr#22781)

Commit migrated from https://github.com/dotnet/coreclr/commit/eb1aa094dbd32f98fd4ce5dead9f27f9c78a32c5

Fix CoreFX build break. (dotnet/coreclr#22773)

* Fix CoreFX build break.

Introduced by https://github.com/dotnet/corefx/pull/34385.

Commit migrated from https://github.com/dotnet/coreclr/commit/82c4573473ebca2eb59a6cd074c7bef8914798d5

Cleanup unnecessary COUNTER_ONLY (dotnet/coreclr#22759)

Commit migrated from https://github.com/dotnet/coreclr/commit/bd84f468ac3f82847669d49614d01eba8dd2a9a0

Add exception counter (dotnet/coreclr#22734)

* Add exception counter

* Fix build

* Make FCall return UINT32

* Move GetExceptionCount to Exception

* more PR comments, fix x86 build

* Remove global variable initialization

Commit migrated from https://github.com/dotnet/coreclr/commit/521e73c0031c84297088381688d464ce619156a9

Correct __CrossBuild in build-packages to be 0 or 1 (dotnet/coreclr#22769)

Commit migrated from https://github.com/dotnet/coreclr/commit/2c10ccaefa48d3418d1fc4fbc4f49eee5745b623

Merge pull request dotnet/coreclr#22731 from briansull/issue-22436

Fix Issue 22436 - noway_assert  'expTyp == cseLclVarTyp' in optcse.cpp

Commit migrated from https://github.com/dotnet/coreclr/commit/bb0c2df97117060ad21371cfe71fb27e727cd27c

Do not use portablebuild=false for linux-musl (dotnet/coreclr#22760)

Commit migrated from https://github.com/dotnet/coreclr/commit/f7e6da36550bcf847bf80a4601d890c6bf3c0d12

[XUnit] Use ParallelRun==Collections on all platforms. (dotnet/coreclr#22746)

* Use ParallelRun==Collections on all platforms.

* Fix formatting

Commit migrated from https://github.com/dotnet/coreclr/commit/3e140890fd7ba79bd96ee42735c946b14d65d55b

Remove unused perfcounter code (dotnet/coreclr#22742)

* removing references to COUNTER_ONLY, ENABLE_PERF_COUNTERS in src/vm directory

* removing perfcounters* files

* Fix build

* Remove PerfCounterSupportedBuild

* Some more cleanup on the GC side

Commit migrated from https://github.com/dotnet/coreclr/commit/2dfd1779ae9eebe89b223cff6fd77825301971dd

[Unix|*] Rework and share the logic for determining __DistroRid (dotnet/coreclr#22628)

* [Unix|*] Rework and share the logic for determining __DistroRid

This will consolidate all of our different functions which determine
the __DistroRid into one file. In addition, it will remove the unused
__HostDistroRid file.

Lastly, it corrects the rid choice when crossbuilding musl-arm64 as the
cross-rootfs non-portable build will correctly be picked up.

* Fix typo and use __BuildArch in place of __Arch

* Correct non-portable build decision

* Add error for missing redhat-release

* Fix redhat comparison

* Correct previous rhel or

* Fix cross scenerio

* Fix missed $

* Set __RuntimeId for runtest.proj

* Correct naming convention of file

* Address feedback and support -portablebuild=0

With this change:

1. build.sh and build-test.sh should build portable by default.
2. init-distro-rid.sh checks __PortableBuild=1. If so, then it must force rhel 6 and freebsd to __PortableBuild=0 and use the non-portable build
    - For __PortableBuild=0 init-distro-rid.sh sets the non-portable rid correctly to "${ID}.${VERSION_ID}-${buildArch}"

* Correct arguments

* Fix if to include then

* A few more changes to correct linux-musl-arm64

* Add error for non-cross build and correctly pass ROOTFS_DIR

* Correctly exit instead of echo

* Correct isCrossBuild and use rootfsDir only

* Add documentation

* Address feedback

* Rework rhel if

Commit migrated from https://github.com/dotnet/coreclr/commit/a006c44d06d56829a230bbbce17b74b21a0b8842

increasing LocalGC version to 3 for 3.0 (dotnet/coreclr#22751)

Commit migrated from https://github.com/dotnet/coreclr/commit/d559d44e02471854748763a6c7eb52cbdc56168f

Clean up unused parameters around Assembly::Load (dotnet/coreclr#22716)

Commit migrated from https://github.com/dotnet/coreclr/commit/aa542e489034335f3536782f05f5c502d855736b

Mispelling

Commit migrated from https://github.com/dotnet/coreclr/commit/28dcdac7f0251cd3bfff722030e43901688a70ad

Support COM events (dotnet/coreclr#22721)

* Add support for COM events
* Add COM Event tests

Commit migrated from https://github.com/dotnet/coreclr/commit/344004681ace2b49dbc40d83ce422f81aa7efcb5

Added enclosing brances for if/then stmts

Commit migrated from https://github.com/dotnet/coreclr/commit/d5541e44fbae473aefe229de81d56fe238b405e8

JIT: handle preference miss two register xmm return case (dotnet/coreclr#22733)

Fix a bug in codegen when returning a Vector3 result where the target
register is not one of the two return registers.

Re-enable the associated test.

Closes dotnet/coreclr#22401.

Commit migrated from https://github.com/dotnet/coreclr/commit/ee8a0634d77a502f76d700c3a7eaf7cc59d1ceea

Add VS debugging info to our instructions

Commit migrated from https://github.com/dotnet/coreclr/commit/5ec1eeaa95362be9e76f9f79c9b44162527e95ad

Enable the HWIntrinsic extension methods and remove the instance implementations (dotnet/coreclr#22705)

* Removing various S.R.I.Vector instance method APIs, since they should now be exposed as extension methods

* Updating the JIT to recognize the S.R.I.Vector extension methods.

* Updating various S.R.I.Vector test templates

* Regenerating the S.R.I tests that are created from a template.

* Fixing the numArgs for Base_Vector256_GetLower

* Fixing the handling for `Base_VectorXXX_As` to normalize the struct type.

* Adding the Base_Vector128_As intrinsics back for arm64

Commit migrated from https://github.com/dotnet/coreclr/commit/4ba958c9fd412b217e21a1dd508ec466a21aa462

Move Thread to shared CoreLib (dotnet/corefxdotnet/coreclr#35462)

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/feed8ae9df458f336b2dd76cc9abe934bb2a5751

Reduce struct copy by 1 in ValueTask.GetAwaiter (dotnet/coreclr#22738)

* Reduce struct copy by 1 in ValueTask.GetAwaiter

* Bit more explicit

Commit migrated from https://github.com/dotnet/coreclr/commit/f18123aa255793fde4d027f648744af4f2157160

Removed some dead code

Commit migrated from https://github.com/dotnet/coreclr/commit/bbed8b0df6e43de80dbb6570b13bc79ffa01aad4

Fix Isuue 22436 - noway_assert  'expTyp == cseLclVarTyp' in optcse.cpp
We now allow TYP_BYREF to be compatible with TYP_I_IMPL when performing CSE substitutions.
Re-enable the failing BestFitMapping interop tests

Commit migrated from https://github.com/dotnet/coreclr/commit/7221f02352e5699b0f4e3ac0c94cab4fa06d7708

Remove accidentally added Windows.10.Nano.Amd64 from Windows_x86 in platform-matrix.yml (dotnet/coreclr#22723)

Commit migrated from https://github.com/dotnet/coreclr/commit/fae2a56cd032d9b68c58fd4dc960160b7013842a

Fix dblarray tests and reenable for arm32 (dotnet/coreclr#22718)

Run the allocate/gen-check sequence under a forbid GC region, and enable the
tests for arm32.

Closes dotnet/coreclr#22015.

Commit migrated from https://github.com/dotnet/coreclr/commit/33f693194fb6c0fb907e513eab9936397498a9e5

Use ThrowHelper in ManualResetValueTaskSourceCore (dotnet/coreclr#22714)

Just move the helper being used to ThrowHelper.

Commit migrated from https://github.com/dotnet/coreclr/commit/57b0be625346712fd3bafb71a9568edb579971f1

Use Build.DefinitionName for Creator when submitting tests to Helix (dotnet/coreclr#22692)

Commit migrated from https://github.com/dotnet/coreclr/commit/8c03cd9968165287b558d83473b22e3a617ab42c

Fix tizen rootfs building (dotnet/coreclr#22715)

Move tizen-release package from base to unified repo

Commit migrated from https://github.com/dotnet/coreclr/commit/1b136ae2ae550d1085612e73648b9cc376f28d57

Fix bug introduced during cleanup in JapaneseLunisolarCalendar (dotnet/coreclr#22717)

Commit migrated from https://github.com/dotnet/coreclr/commit/4eaa54bcd29217fffadc7c6d6babcc048bb14285

Allow Task.WaitAny completion to run synchronously (dotnet/coreclr#21245)

Commit migrated from https://github.com/dotnet/coreclr/commit/970055f73200974bbc7b5f91a676e16eae2800f7

Abstract deprecated and selectany for GCC and remove LLVM'ism where possible (dotnet/coreclr#22662)

* Abstract selectany

* Fix initializer element is not constant

src/corefx/System.Globalization.Native/pal_icushim.c:58:34: error: initializer element is not constant
 static const int MaxICUVersion = MinICUVersion + 20;

* Enable ms extensions

* Apply LLVM patterns to GCC

* Remove deprecated function

* Fix const conversion error

src/corefx/System.Globalization.Native/pal_calendarData.c:390:16: warning: passing argument 1 of ‘ures_close_ptr’ discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
ures_close(erasResBundle);

src/corefx/System.Globalization.Native/pal_calendarData.c:419:22: warning: initialization discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
char* name = GetCalendarName(calendarId);

* Remove old compiler option

Commit migrated from https://github.com/dotnet/coreclr/commit/1a1bb292ab53c0c30d35dc396c7089c1da5b3c82

Fix race condition in ManualResetValueTaskSourceCore (dotnet/coreclr#22711)

* Fix race condition in ManualResetValueTaskSourceCore

ManualResetValueTaskSourceCore.GetStatus is used from ValueTaskAwaiter.IsCompleted.  If GetStatus returns Success too early, then IsCompleted will also return true too early, which will result in GetResult being called too early.  This doesn't happen today when an MRVTSC is used sequentially.  But if an instance is pooled by the object its stored on getting put back into a pool as part of a call to the wrapper's GetResult, then we can end up in the following situation:
- Thread 1 starts an await on an instance containing an MRVTSC.  It calls IsCompleted.
- Thread 2 starts to complete that instance, getting as far as calling SignalCompletion and setting _completed to true but not yet setting _continuation.
- Thread 1 sees _completed == true and returns true from IsCompleted.  It calls GetResult, and the wrapper extracts the result from the instance, resets it, and puts it back into the pool.
- Thread 3 takes the object out of the pool and starts using it.
- Thread 2 continues SignalCompletion on that instance, and sets _continuation to the sentinel.
- Now Thread 3's instance's _continuation is the sentinel when it should be null.  If it calls SignalCompletion, it'll erroneously find that _continuation is not null and will queue _continuation/_continuationState for execution, resulting in the sentinel getting executed.  If it calls OnCompleted, it'll find that the _continuation is not null, and will queue the supplied continuation/state to execute immediately even though the operation may not yet actually be done.

The fix is simply to check not just _completed but also _continuation.  The operation is considered pending if either _completed is false, meaning SignalCompletion has not yet been called, or if _continuation is still null, meaning it's neither been set to the supplied delegate nor to the sentinel.  We can't just rely on _completed for the above outlined reasons, and we can't just rely on _continuation because it can be non-null if OnCompleted was called to hook up a callback (if we only cared about the await pattern, then we could just check _continuation and wouldn't need _completed, but we also need to support non-await access).

* Address PR feedback

Commit migrated from https://github.com/dotnet/coreclr/commit/3c2e989c61c7466bc6d061e03277dbbd8a7e54e9

Fix CLRDataCreateInstance export. This was breaking SOS. (dotnet/coreclr#22701)

Fix CLRDataCreateInstance export. This was breaking SOS.

Added the right cmake magic so the function is exported properly.

Commit migrated from https://github.com/dotnet/coreclr/commit/e0c2cc567cb7b5072eb71b0ff788da2ffde63a7d

Add ManagedRuntimeEventSource (dotnet/coreclr#22515)

* Adding a dummy ManagedRuntimeEventSource

* Add ManagedRuntimeEventSource

* Renaming ManagedRuntimeEventSource to RuntimeEventSource

* some cleanup

* use Condition instead of ifdefing out the whole file

* some cleanups

* remove unused namespaces

* Move m_RuntimeEventSource to RuntimeEventSource from EventPipeController

* Adding some TODO items

* address more pr comments

* Use GUID to initialize RuntimeEventSource

* Exclude BasicEventSourceTests from CoreFX test CI leg

* Addressing rest of the PR feedback

* dont dispose counters on disable command to avoid races

Commit migrated from https://github.com/dotnet/coreclr/commit/7322038ab453067787c9d8ea2ac9047c76c434ff

Early out from vectored exception handler on uninitialized threads (dotnet/coreclr#22702)

Fixes dotnet/coreclr#22698

Commit migrated from https://github.com/dotnet/coreclr/commit/a054254d2448314f1d24ef319b18dfd7a91747f6

Use out arguments in UmAlQuraCalendar (dotnet/coreclr#22704)

Commit migrated from https://github.com/dotnet/coreclr/commit/d98d827d7d80960e0ffc3c283a8199baf619d94b

Cleanup CoreLib makefiles (dotnet/coreclr#22706)

Commit migrated from https://github.com/dotnet/coreclr/commit/bc430cdd2010bf7790854a60380d00c506c346a6

Adding tannergooding to the arm64Users list. (dotnet/coreclr#22665)

Commit migrated from https://github.com/dotnet/coreclr/commit/4366f1ff7e5956f308762c89cb838e5fb921a901

Merge pull request dotnet/coreclr#22656 from briansull/fix_arm64_assert

ARM64: Correct the assert to add R8 as a potential valid arg register

Commit migrated from https://github.com/dotnet/coreclr/commit/89c33a222cc5798d02b6d0b7c4b98562da2c212c

Fix formatting

Commit migrated from https://github.com/dotnet/coreclr/commit/5782f75f52370c332cae9eb270e8c1fb945b3f30

Use RunContinuationsAsynchronously in SemaphoreSlim.WaitAsync (dotnet/coreclr#22686)

SemaphoreSlim.Release shouldn't invoke arbitrary continuations as part of its invocation, so when it dequeues a task waiter and goes to complete it, rather than just calling TrySetResult, it queues the task to the thread pool to have TrySetResult invoked there.  Now that we have RunContinuationsAsynchronously, though, we can just use that functionality instead.  This has two main benefits:
1. We avoid queueing a work item entirely if there are no continuations from the task.  This might happen, for example, if the semaphore is released so quickly after waiting on it that the WaitAsync caller hasn't yet hooked up a continuation, in which case the await on the WaitAsync task will just see IsCompleted as true and continue running.
2. We avoid queueing a work item when the task represents a synchronous Wait, which happens if there's already a pending WaitAsync when Wait goes to block.  The main benefit here is avoiding potential thread pool starvation, if threads in the pool are blocked in such Waits, and previously another thread pool thread would have been needed to run the queued work item to complete the synchronous Wait.

Commit migrated from https://github.com/dotnet/coreclr/commit/e2081d0e67a1d7fedaf6303f576acef316c7bd66

Correct the assert to add R8 as a potential valid register
Also fix the calls to RemoveElemD to use lvVarIndex
Re-enable the interop tests that were failing on ARM64

Commit migrated from https://github.com/dotnet/coreclr/commit/b44f11e57491f813ab6c7e760c01a902e35bcdf4

Cleanup Calendars code (dotnet/coreclr#22582)

Commit migrated from https://github.com/dotnet/coreclr/commit/7a6b440b5503e38e1847af68b3fc59692d9d2770

Fixing extension method overload resolution for some S.R.I.Vector methods (dotnet/coreclr#22696)

Commit migrated from https://github.com/dotnet/coreclr/commit/50c44bec9c38f4d347c945d2034eb6a933680e36

Use testGroup for defining test jobs (dotnet/coreclr#22608)

This adds a mapping between testGroup string and values of the following parameters:

* scenarios
* priority
* timeoutTotalInMinutes
* timeoutPerTestCollectionInMinutes
* timeoutPerTestInMinutes

and uses testGroup on a level of azure-pipeline.yml to set these values.

Commit migrated from https://github.com/dotnet/coreclr/commit/1216c9751f306033809f9699f6b33b7ce68bc499

Enable telemetry for official builds

Commit migrated from https://github.com/dotnet/coreclr/commit/f26ea2a830ad11a25e6bb15dae6de6a0ea721693

Optimize Vector128/256<T>.Get/WithElement (dotnet/coreclr#22353)

Commit migrated from https://github.com/dotnet/coreclr/commit/d466bdbd47ab7a861046b99340e16e037098fb2a

Adding an update-dependencies.cmd script, like CoreFX has. (dotnet/coreclr#22672)

Commit migrated from https://github.com/dotnet/coreclr/commit/dd5a4952942fbdda01ad779d648d21e5a91a87a6

Fail loading types with default interface methods for fragile ngen (dotnet/coreclr#22603)

We are not generating the right data structures for this. Since fragile crossgen is no longer a mainstream scenario, it's not worth the effort to add default interface methods support for it.

This avoids `!"Precode::GetPrecodeFromEntryPoint: Unexpected code in precode"` and similar issues.

Commit migrated from https://github.com/dotnet/coreclr/commit/388a7d415d8f5b45c3ca6e74d8b0885a20d0b962

Adjust gc stress compatibility settings for two tests (dotnet/coreclr#22633)

* enable eventsourcetrace for gc stress. Closes dotnet/coreclr#17188.
* disable GitHub_12224 for gc stress (relies on timeout). Closes dotnet/coreclr#19086.

Commit migrated from https://github.com/dotnet/coreclr/commit/4ac07f7e396e19043baf42c3e38bc50b51c5bff3

Use correct version for Microsoft.NETCore.Platforms (dotnet/coreclr#22683)

Commit migrated from https://github.com/dotnet/coreclr/commit/7d9c1ef1d5c879082285af0c0015faecccdedfa3

Remove resetting of unmanaged thread locale from thread pool APIs (dotnet/coreclr#22682)

The cost of doing this is showing up in the TechEmpower profiles. There is a lot of thread locale state in unmanaged libraries and OS APIs that the threadpool friendly code has to be aware off. It does not make sense to keep reseting just the Locale.

Commit migrated from https://github.com/dotnet/coreclr/commit/281f69036a435d11349fffdee6bb2f0c20f818eb

Fix constrained call corner cases (dotnet/coreclr#22464)

Fixes dotnet/coreclr#22423. I'm still unclear on how JitStress manages to get itself into that situation, but I was able to write a repro that triggers the `!pMDAfterConstraintResolution->IsInterface()` assert using regular IL, so we need to handle that either way. The repro for that is constrained3.il.

While figuring out the repro, I wrote a bunch of other test code and found another bug (constrained2), where we would box in a situations that doesn't require boxing (canonically ambiguous situation where there's a suitable default interface implementation and a valuetype implementation of the constrained method that does not requires boxing once we no longer deal with __Canon).

Commit migrated from https://github.com/dotnet/coreclr/commit/88249918cd01e8729c60c0f6c2cf8145d0bc3c55

Fix warning causing error in warning as error (Windows) (dotnet/coreclr#22660)

BOOL -> bool conversion

Commit migrated from https://github.com/dotnet/coreclr/commit/a8d58d62d1009b41e423923ba66a041a99bea7cc

Fix Integer Rounding issue and duplicate definition of __deref in sal.h (dotnet/coreclr#22645)

* Fix large integer truncation

pal/pal.h:4988:43:
warning: large integer implicitly truncated to unsigned type
[-Woverflow]
     static const SIZE_T NoTargetFrameSp = SIZE_MAX;
                                                ^~~~~~~~

* Fix compilation error with GCC

/usr/include/c++/7.3.0/debug/functions.h:452:15: error: expected unqualified-id before ')' token
       __deref();
               ^
/usr/include/c++/7.3.0/debug/functions.h:455:34: error: expected primary-expression before '<' token
       typename = decltype(__deref<_It>() < __deref<_It>())>
                                  ^
/usr/include/c++/7.3.0/debug/functions.h:455:38: error: expected primary-expression before '>' token
       typename = decltype(__deref<_It>() < __deref<_It>())>
                                      ^
/usr/include/c++/7.3.0/debug/functions.h:455:40: error: expected primary-expression before ')' token
       typename = decltype(__deref<_It>() < __deref<_It>())>
                                        ^

Commit migrated from https://github.com/dotnet/coreclr/commit/f522e39d4a1da2a4b4859c5db5c741193d098984

Delete RtlMoveMemory and tests (dotnet/coreclr#22668)

Commit migrated from https://github.com/dotnet/coreclr/commit/6252e7d5b428b17e46fc4424bdf6fad38ebd1c35

Fix the build of Loader/NativeLib test (dotnet/coreclr#22648)

The native test assets were not build correctly on Unix platforms.
The native library exports were generated as private symbols by clang.

This change fixes the export declaration so that the global symbols are
correctly generated for exported symbols.

Fixes dotnet/coreclr#22549

Commit migrated from https://github.com/dotnet/coreclr/commit/390d002ac1ff919fcaae047eb09217ddeac78779

Only import the RoslynPropsFile if it exists. (dotnet/coreclr#22657)

Commit migrated from https://github.com/dotnet/coreclr/commit/217b3b11923444cba8e33b6881ce1e3520f5c161

Fix visibility and signed comparison issues for GCC (dotnet/coreclr#22586)

* Fix visibility and signed comparison issues for GCC

* Fix hidden _CLRDataCreateInstance warned by macOS

* Fix indentation in vswprintf/test1

* Change void* to PVOID in implementation files

Commit migrated from https://github.com/dotnet/coreclr/commit/7760228f08059e6f3ed90e56dbae74eac495aa98

Avoid pulling in Lazy<T> during startup (dotnet/coreclr#22659)

Environment constructor runs on every startup path. Remove dependency on Lazy<T> from it.

Commit migrated from https://github.com/dotnet/coreclr/commit/98956dd6b9450bbcafe7c6934e532b557ae38918

Merge pull request dotnet/coreclr#22528 from mikedn/idx-addr

Fix genCodeForIndexAddr

Commit migrated from https://github.com/dotnet/coreclr/commit/e3d4b9c7e355ca67b4717e212497fe4d8a05eb57

[master] Update dependencies from dotnet/corefx (dotnet/coreclr#22595)

* Update dependencies from https://github.com/dotnet/corefx build 20190213.23

This change updates the following dependencies
- Microsoft.NETCore.Platforms - 3.0.0-preview4.19113.23
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview4.19113.23

* Update dependencies from https://github.com/dotnet/corefx build 20190215.1

This change updates the following dependencies
- Microsoft.NETCore.Platforms - 3.0.0-preview4.19115.1
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview4.19115.1

* Fixing up the tests dependent on the S.R.I.Vector API changes

* Regenerating the tests dependent on the S.R.I.Vector API changes

* Disabling an outdated CoreFX test.

Commit migrated from https://github.com/dotnet/coreclr/commit/990dd2c14972f07001ff6021528697df51c8e5e6

Set the 'PLATFORM_OSX' constant during OSX build (dotnet/coreclr#22639)

* Set the 'PLATFORM_OSX' constant during OSX build

* Remove PLATFORM_OSX define from SPCL

Commit migrated from https://github.com/dotnet/coreclr/commit/a4cad5d4d3a7bbb9fa98b8aa3dc8167328065ea7

Fix bug in FindDispatchSlot usage

Commit migrated from https://github.com/dotnet/coreclr/commit/b9b349decc9cedbc1be4cc0b918b8b190ab0e59d