[Local GC] FEATURE_EVENT_TRACE 1/n: Tracking Event State (#15873)

* [Local GC] FEATURE_EVENT_TRACE 1/n: Add infrastructure for keeping event state within the GC and plumbing to communicate event state changes

* Code review feedback: use a load without a barrier in IsEnabled and put debug-only code under TRACE_GC_EVENT_STATE

* Address code review feedback: add EventPipe callback and comments

* Fix the non-FEATURE_PAL build

* Fix an issue where the GC fails to react to ETW callbacks to occur before the GC is initialized (e.g. on startup when an ETW session is already active)

* Simplify callback locking scheme

* Add a separate callback for each EventPipe provider and funnel them all through a common handler

* Fix non-FEATURE_PAL build

Add Unsafe.IsAddressGreaterThan / IsAddressLessThan (#15988)

Catch ambiguous interface method resolution exceptions (#15978)

Default interface methods might end up being ambiguous. This thows an exception we need to catch.

Do not devirtualize shared default interface methods (#15979)

The result of the devirtualization for a method on a generic type should be an instantiating stub, but this doesn't seem to work right.

Fixing that is a perf issue (that can be triaged/punted - #15977). This commit is a correctness fix to avoid bad codegen for that case.

Resolves #15591.

Merge pull request #15984 from dagood/remove-conversion/master

Never attempt Portable PDB conversion

Never attempt Portable PDB conversion

The inline script detected whether the branch name contained "release/" and ran a different target. That target tries to convert Portable PDBs into Windows PDBs, which currently hits errors in CoreCLR.

Remove this logic from CoreCLR: it is no longer necessary because orchestrated builds will have a post-build symbol publish step.

Remove hardcoded paths in linkbench scripts (#15939)

* Remove hardcoded paths in linkbench scripts

Linkbench has hardcoded paths based off of VS140COMNTOOLS, which not all
machines will have (ie, machines with only VS2017 installed). This
change removes the hardcoded paths, and replaces them with checks to
make sure the tool is on the path (which they will be if we are in a VS
environment of any kind - which we would have been in if
VS140COMNTOOLS was already set).

Update BuildTools, CoreClr, CoreFx, PgoData to prerelease-02422-01, preview2-26123-01, preview2-26123-01, master-20180122-0034, respectively (#15882)

Misc Span/Memory changes (#15941)

* Misc Span/Memory additions
- Add MemoryMarshal.CreateSpan to replace DangerousCreate
- Add MemoryMarshal.CreateReadOnlySpan to replace DangerousCreate
- Add MemoryMarshal.Cast to replace NonPortableCast
- Add ToString override to Span and ReadOnlySpan
- Add ToEnumerable function to MemoryMarshal that takes a ReadOnlyMemory

Disable EventPipe tests which fair in stress runs (#15972)

Disable EventPipe tests which fair in stress runs

Addresses: #15924
Follow-up: #15927

Rename Pin offset to byteOffset and clean up Retain test (#26518)

Signed-off-by: dotnet-bot-corefx-mirror <dotnet-bot@microsoft.com>

Remove .NET limit on env var name and value length

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Merge pull request #15966 from weshaggard/MoveTo21Preview2

Move from preview1 to preview2 in master branches

Move TimeZoneInfo.Win32.cs to shared CoreLib partition (#15953)

Reconcile deltas with CoreRT and refactor interop to follow coding conventions

Move from preview1 to preview2 in master branches

preview1 will be produced from the release/2.1 branch

Merge pull request #15944 from BruceForstall/CorefxTestUseCorrectVersion

For corefx testing, use a matching corefx repo version git hash

Merge pull request #15942 from tannergooding/no-multireg-simd

Updating the VM to no longer treat the SIMD HWIntrinsic types as HFA or MultiReg structs

Fix Windows ARM release build break (#15958)

Merge pull request #15959 from jashook/remove_arm_pr_trigger

Turn off arm pr trigger

Turn off arm pr trigger

Stop the SIMD hardware intrinsics types from undergoing crossgen.

Remove AppDomainLeaks configuration option (#15956)

Removed all usages of AppDomainLeaks configuration option and
CHECK_APP_DOMAIN_LEAKS feature

Fix #12094

Rename Libraries.GlobalizationInterop to Libraries.GlobalizationNative

Consistency with CoreFX and coding conventions.

Also delete a few other pieces of unused code.

Define Interop.Kernel32.MAX_PATH (#15952)

* Define Interop.Kernel32.MAX_PATH

For consistency with CoreFX and coding conventions.

Merge pull request #15948 from fiigii/fix-ifdef

Fix a minor bug

Fix timezone lookup to be consistently case insensitive (#15951)

The timezone ids used case insensitive comparisons everywhere, except in the dictionary used to cache timezones.

Fixes dotnet/corefx#26479

Merge pull request #15936 from GrabYourPitchforks/levib/memory_nullref

Allow implicit conversion of null arrays to empty Memory instances

Add optional integer offset to OwnedMemory Pin (#15946)

#if defined(_TARGET_XARCH_) || defined(_TARGET_ARM64_)

Merge pull request #15945 from dotnet-bot/from-tfs

Merge changes from TFS

Break current culture initialization cycle (#15938)

Fixes #15931

Merge pull request #15903 from CarolEidt/Fix545505

ARM: Fix morphing of struct passed on stack

Fix desktop build

1. Fix `LEGACY_BACKEND`
2. `#if FEATURE_HW_INTRINSICS` => `#ifdef FEATURE_HW_INTRINSICS`

[tfs-changeset: 1686599]

Updating the VM to no longer treat the SIMD HWIntrinsic types as HFA or MultiReg structs.

Add CI support for updated run-corefx-tests.py

Change CI to build a "GenerateLayoutOnly" drop for corefx testing,
specifically to cause the corefx package to be restored, so we
can determine its version.

Merge pull request #15836 from tannergooding/emitInsBinary

Refactoring emitInsBinary

CharUnicodeInfo.GetUnicodeCategory(int codePoint) (#15911)

* CharUnicodeInfo.GetUnicodeCategory(int codePoint)

https://github.com/dotnet/corefx/issues/26173

- renamed UnicodeCategory.InternalGetUnicodeCategory to GetUnicodeCategory
- renamed its parameter ch to codePoint
- made it public

* fix build break

Merge pull request #15798 from sdmaclea/PR-ARM64-Set-Instruction-Set-Flags

[Arm64] Set Instruction set flags

Merge pull request #15933 from sdmaclea/PR-ARM64-EMIT-COMPARE-ZERO

[Arm64] Add compare vector to zero emitters

Merge pull request #15935 from sdmaclea/PR-ARM64-IRREGULAR-NOT

[Arm64] Remove irregular vector not handling

Convert perf.groovy to use run-xunit-perf.py (#15881)

* Convert perf.groovy to use run-xunit-perf.py

Also removes run-xunit-perf.cmd and run-xunit-perf.sh. This change moves us to using build-tests.sh to generate the layout (ie pull down corefx), so this change fixes #12143.

Merge pull request #15930 from mikedn/useless-code

Remove some redundant code from TreeNodeInfoInit

For corefx testing, use a matching corefx repo version git hash

If the user doesn't specify a specific corefx commit hash to use,
try to find the matching commit hash in the coreclr repro. If the
matching hash can't be found, use 'HEAD'.

We find the matching corefx commit hash by first parsing file
'dependencies.props' at the root of the coreclr repro, looking for this:
   <MicrosoftPrivateCoreFxNETCoreAppPackageVersion>4.5.0-preview1-26112-01</MicrosoftPrivateCoreFxNETCoreAppPackageVersion>
This determines the corefx package version that matches. Next, we look for the version.txt
file in the package cache, e.g.,
   <coreclr_root>\packages\microsoft.private.corefx.netcoreapp\4.5.0-preview1-26112-01\version.txt
The contents of this file is exactly the git commit hash
we need to use, e.g.:
   197a0699b08087ea85581679afdd9fd7b5c465c3
The version.txt file is created when the corefx package is restored,
which happens when doing one of:
   Windows: tests\runtests.cmd GenerateLayoutOnly
   non-Windows: build-test.sh generatelayoutonly

It would also be possible to not depend on the package already
being downloaded, but instead download the correct package here,
using the determined "MicrosoftPrivateCoreFxNETCoreAppPackageVersion"
package version, e.g.:
   https://dotnet.myget.org/F/dotnet-core/api/v2/package/Microsoft.Private.CoreFx.NETCoreApp/4.5.0-preview1-26112-01
and then extracting the ZIP archive to find the version.txt file.

This might get easier if the corefx commit hash is added directly to dependencies.props, as
discussed in https://github.com/dotnet/buildtools/issues/1141.

Stop treating all calls to instance interface methods as callvirt (#15925)

Fixes #15827.

Remove Redundant Resource entry (#15928)

Move platform-neutral part of TimeZone to shared CoreLib partition (#15926)

Prep work for moving the platform specific parts

Merge pull request #15749 from fiigii/tabledrive

Table-driven Intel hardware intrinsic

Allow implicit conversion of null arrays to empty Memory instances

[Arm64] Set Instruction set flags

[Arm64] Remove irregular vector not handling

Disable eventpipetrace test due to JIT stress failure (#15927)

* Disable eventpipetrace test due to JIT stress failure

* Use double wildcard to properly include the right files

[Arm64] Add compare vector to zero emitters

Revert "Change just-in-time debugging registry keys. (#15722)" (#15921)

This reverts commit 8b2cac974db3edde38e1a555455f53592d03b68e.

Remove some redundant code from TreeNodeInfoInit

[armel tizen] Fixed CoreRT issue #4626 unwinding support (#15913)

Merge SSE intrinsics into the table-driven framework

Removing an incorrect assert from emitInsBinary

Cache Invariant CompareInfo (#15902)

The Invariant CompareInfo is used in number of places. It is fetched via virtual method on CultureInfo. Caching it avoids the virtual method calls and extra indirections.

Merge pull request #15920 from BruceForstall/AllowArmJitHaltPopup

Allow COMPlus_JitHalt to display a pop-up on ARM/ARM64

Do not throw IndexOutOfBounds when AM/PM designator is empty (#15904)

* do not throw IndexOutOfBounds when AM/PM designator is empty

* store am/pm designator in local

Mirror changes from dotnet/corefx (#15909)

* Consolidate System.Memory code to shared folder (dotnet/corefx#26393)

* Consolidate System.Memory code to shared folder

This change is removing the duplicate codes from System.Memory and keep only one copy under the shared folder to be easier to edit such code in one place and get reflected on the other repos.

* Address the review feedback

* Addressing more feedback

* More cleanup

* remove empty line and added a comment

Signed-off-by: dotnet-bot-corefx-mirror <dotnet-bot@microsoft.com>
* Add missing throw helper methods used in the code we got from corefx

* Update the exception helper

* fix the break

Enable EventPipe and EventSource Functional Tests (#15917)

Refactoring emitInsBinary

Convert BulkTypeLogger Allocations From Stack to Heap (#15907)

table drive Intel hardware intrinsic

Merge pull request #15901 from tannergooding/hwintrin-fixup

Resolving a few issues with the HWIntrinsic code

Allow COMPlus_JitHalt to display a pop-up on ARM/ARM64

Note that you need to `set COMPlus_NoGuiOnAssert=0` for the
pop-up to be displayed.

PR Feedback

ARM: Fix morphing of struct passed on stack

If a struct is passed on the stack, it must live on the stack, unless/until we support `GT_FIELD_LIST` for these args. This is unlikely to represent a significant code quality issue, since ARM supports many register args, and this has gone undetected thus far.
This was exposed by tailcall stress on desktop.
I've added a test that exposes the issue without tailcall stress (though it gets a different assert than the desktop failure).

It seemed that `fgMorphMultiregStructArg()` was the best place to fix this - and I noted that this is called for any struct that is larger than a single register. So I updated the comments to reflect that.

I thought about putting the test in the JIT\Regressions test directory, but I consider that it is addressing basic missing test coverage, so I added it to JIT\Methodical\structs.

 Update clr-configuration-knobs.md, add new C# csi.exe based script for xplat document generation (#15858)

* C# csx script to generate CoreCLR and Jit configuration knobs doc

* Normalize category names, fix typos and white space in clrconfigvalues.h

* Normalize category names, knobs descrptions and fix typos in jitconfigvalues.h

* Remove clr-complus-conf-docgen.sh

* Remove from clrconfigvalues.h CONFIG_ declarations which are already present in jitconfigvalues.h

* Update clr-configuration-knobs.md with clr-configuration-knobs.csx script

Update perflab tests to execute in a fixed order

Goal is to pin class init overhead on one test consistently. Also we were
not picking up the "optimize for benchmark" attribute and so likely risking
having tests execute concurrently.

Updating TYP_SIMD locals to no longer undergo struct promotion for HWIntrinsic nodes.

Disabling the Math.Round, Math.Floor, and Math.Ceiling intrinsics on non-AVX machines

Fixing the `LoadAlignedVector128` HWIntrinsic test to ensure that we always read from an aligned address.

Fixing some bad merge conflicts in the `emitIns_R_A_I`, `emitIns_R_C_I`, and `emitIns_R_S_I` methods

Merge pull request #15869 from sdmaclea/PR-ARM64-is16Byte

[Arm64] Use is16Byte

Add environment variable (COMPlus_EnableDiagnostics) to disable debugging and profiling. (#15878)

To disable the named pipes and semaphores created on linux execute "export COMPlus_EnableDiagnostics=0" before start the .NET Core program.

On Windows execute "set COMPlus_EnableDiagnostics=0" and on Linux execute "export "COMPlus_EnableDiagnostics=0"

Removed the "Telesto" registry entry (old unnecessary Silverlight code) and Watson (always true) checks.

For issues #11769 and #8844.

Merge pull request #15897 from tannergooding/hwintrin-debuggerdisplay

Adding a DebugView for Vector64<T>, Vector128<T>, and Vector256<T>

Merge pull request #15900 from tannergooding/sse-intrinsics

Updating emitIns_R_R_A_I to not be defined for the legacy backend.

Skip EDI boundary for async (#15781)

Adding a DebugView for Vector64<T>, Vector128<T>, and Vector256<T>

Move public StreamReader/Writer to CoreLib (#15884)

Updating emitIns_R_R_A_I to not be defined for the legacy backend.

Add MDA (Managed Debugging Assistant) (#15875)

* Add MDA (Managed Debugging Assistant)

* Info about MDA's in .NET Core

Merge pull request #15893 from BruceForstall/FixR2RCrossgen

Fix non-Windows R2R testing in CI

Merge pull request #15894 from BruceForstall/DisableGetGenerationInGCStress

Disable GetGeneration from GCStress runs

Merge pull request #15538 from tannergooding/sse-intrinsics

Implement the SSE hardware intrinsics.

Add missing config.h.in entry for HAVE_LIBINTL_H (#15887)

In my change yesterday, I have added HAVE_LIBINTL_H to the
configure.cmake file, but forgotten to add the corresponding define to
the config.h.in file. This fixes it.

Matching non-shared Decimal changes

Misc Decimal optimizations

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Update corelib shared README.md (dotnet/corefx#26366)

Signed-off-by: dotnet-bot-corefx-mirror <dotnet-bot@microsoft.com>

Disable GetGeneration from GCStress runs

This test has exhibited "random" failures during GCStress runs
(which are believed to be due to the test itself, not some
product issue). So, disable it from GCStress runs.

Fix non-Windows R2R testing in CI

The xunit.performance.api.dll assembly depends on
Microsoft.3rdpartytools.MarkdownLog, which cannot
be found. Add xunit.performance.api.dll to the
list of assemblies that are not crossgen'ed
during R2R testing.

Update filetimes to nsec (#15872)

Resolving PR feedback

Adding tests for the SSE Load, LoadAligned, LoadHigh, LoadLow, and LoadScalar intrinsics

Adding support for the SSE Load, LoadAligned, LoadHigh, LoadLow, and LoadScalar intrinsics

Updating most of the SSE Compare intrinsics to support containment

Updating the SSE HWIntrinsics to share code where possible.

Adding tests for the SSE MoveMask intrinsic