Merge pull request #9054 from kouvel/SpanGcReporting

Fix GC info for `ByReference<T>`

Merge pull request #9123 from CarolEidt/FixCoreFx15309

Fix SIMD field issues

PAL Tests: Centralize common code (#9151)

Create a single COMMON_TEST_LIBRARIES variable which contains all libraries
with which the tests link.

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

Merge changes from TFS

Merge pull request #9155 from dotnet/revert-9152-SpeedUpSync

Revert "Speed up test native binary syncing in pipeline"

Revert "Speed up test native binary syncing in pipeline"

Merge pull request #9152 from wtgodbe/SpeedUpSync

Speed up test native binary syncing in pipeline

Merge pull request #9153 from adityamandaleeka/jitdisasm_fix

Fix another JIT-disasm check

Merge pull request #9141 from pgavlin/RemoveDeadCode

Remove dead code in the JIT.

Don't require write barrier for assignment to ByRef

The legacy code gen was hitting an assert when trying to use a write barrier for ByRef to ByRef assignment. RyuJIT is also generating an unnecessary write barrier but the assert is not there. As ByRefs are never in GC locations, don't require a write barrier.

Fix another JIT-disasm check.

[GDB-JIT][Linux] Fix incorrect DWARF offset for primitive types with typedef node (char, byte and sbyte) (#9106)

Clean up PAL functions (#9149)

Clean up PAL functions to enable cross-compiling for Android
* Rename wcsnlen to PAL_wcsnlen, avoids a name collision on Android
* Remove unuwed _getw and _putw functions

Speed up test native binary syncing in pipeline

Merge pull request #9142 from AndyAyersMS/FixAsserts

JIT: fix some bad asserts

Merge pull request #9129 from RussKeldorph/defineenum

Disable assertions in ModuleBuilder.DefineEnum

Fix encoding of native signatures in the presence of Enum.GetHashCode optimization (#9138)

Fixes #9118

Fix CodeGenMirror build break due to 347243f80d62d5aaa5359ef8c8d. This includes the new file unwindx86.cpp to be compiled.

[tfs-changeset: 1645771]

JIT: fix some bad asserts

Fix a few asserts that use = instead of ==.

Remove dead code in the JIT.

All code under `LONG_ASG_OPS` and `FANCY_ARRAY_OPT` is statically dead
(these symbols are never defined as anything besides 0). This change
removes this cruft.

Merge pull request #9132 from BruceForstall/FixBuildTest

Fix build-test.cmd

Merge pull request #9125 from BruceForstall/ImproveAddrModeJitDump

Improve address mode detection JitDumps

Merge pull request #9126 from BruceForstall/CleanUnusedTailcallNodes

Clean up unused tailcall via helper nodes

Merge pull request #9133 from BruceForstall/ImproveSimdTests

Improve SIMD tests

Merge pull request #9120 from dotnet-bot/master-UpdateDependencies

Update CoreClr, CoreFx to beta-24925-08, beta-24926-02, respectively (master)

Update CoreClr, CoreFx to beta-24925-08, beta-24926-02, respectively

Extract GetStackParameterSize from ESPIncrOnReturn (#9135)

Show register name in UPDATEREG log (#9134)

Implement PAL_ProbeMemory with write/read to pipe (issue #4094) (#9122)

The recommended way to validate/probe memory on Linux/xplat is to use the write/read file apis to a pipe instead
of the hacky fragile hardware exception catching based probing we have now.

This builds on previous cleanup work in the master branch that removed the PAL Read/WriteProcessMemory
functions and added/replaced them with PAL_ProbeMemory. The DbgTransportSession::CheckBufferAccess that
issue mentions was also replaced.

Merge pull request #9114 from erozenfeld/SecurityPermissionAttribute

Remove System.Security.Permissions.SecurityPermissionAttribute from System.Private.CoreLib.

Merge pull request #9124 from BruceForstall/FixSimdDetection

Fix non-deterministic AVX2 detection

Reduce number of branches and IL size of string.Trim methods (#9115)

Fix GC info for `ByReference<T>`

When `ByRefernece<T>` is directly on the stack, not inside a `Span<T>`, ComputeGCLayout was missing to include in the GC info. This was causing many span tests to fail in GCStress=c mode.

Improve SIMD tests

1. Since the F1/F2 tests of VectorReturn found an issue Vector2
on x86, I added variants of the same test for Vector3 and Vector4
(by basically copying F1/F2 and altering appropriately).
2. I noticed VectorMin for Vector4 wasn't checking the 'W' member,
so I fixed that.

Fix build-test.cmd

Move set of __TestWorkingDir so it is available even when
__BuildAgainstPackages is not defined.

Disable assertions in ModuleBuilder.DefineEnum

The assertions fire when the enum name contains embedded null characters,
and we don't want to take a breaking change to fix that.

Fixes #7669

Fix SIMD field issues

First, when a GT_FIELD is transformed to a GT_IND, if its GTF_FLD_NULLCHECK flag is set, it must be cleared. Having this set leads to assertions or worse in value numbering. It is not entirely clear why this is only causing issues for SIMD fields.

Second, when a SIMD assignment is re-morphed, the dest may have already been changed to a GT_IND, in which case no further morphing is required (it is already a simple assignment).

Remove System.Security.Permissions.SecurityPermissionAttribute from System.Private.CoreLib.

When compiling with /unsafe+ Roslyn emits
  .permissionset reqmin
             = {class 'System.Security.Permissions.SecurityPermissionAttribute, System.Private.CoreLib,
             Version=4.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e' =
             {property bool 'SkipVerification' = bool(true)}}
if System.Security.Permissions.SecurityPermissionAttribute is present.

In particular, System.Private.CoreLib.dll is compiled with /unsafe+ and has the above in the manifest.

https://github.com/dotnet/coreclr/commit/e516211b2d3a3ae2dbd80f2c94b129910a9e280d
made BclRewriter remove System.Security.Permissions.SecurityPermissionAttribute
from System.Private.CoreLib.dll but the above reference remained.

This change removes System.Security.Permissions.SecurityPermissionAttribute completely
to avoid confusion when tools are analyzing msil and try to resolve the manifest
of System.Private.CoreLib.dll.

Clean up unused tailcall via helper nodes

For x86 tailcall via helper, delete the unused placeholder target
location nodes when they are replaced.

Improve address mode detection JitDumps

Fix `Span<T>` GCStress issue on x86 (#9116)

x86 by default treats the type of `ByReference<T>` as the actual type of its IntPtr field as an optimization. `ByReference<T>` is a special case where the struct needs to be treated as a value type so that its field can be considered as a by-ref pointer. This fixes `GCStress=f` issues on span tests on x86.

Fix non-deterministic AVX2 detection

The caller required ECX to be zero when calling CPUID, but wasn't calling
the getextcpuid() function to cause that to happen.

Introduce REGDISPLAY_BASE (#9095)

* Introduce REGDISPLAY_BASE

Merge pull request #9119 from wtgodbe/fixGroovyFx

Fix CoreFX Binary copying in netci.groovy

Fix CoreFX Binary copying in netci.groovy

This will stop building frankenjit.dll for x86.

[tfs-changeset: 1645686]

Merge pull request #9111 from adiaaida/fixCompatJitLeg

Fix x86compatjit and x86lb legs

Merge pull request #9099 from wtgodbe/CoreRootRunTest

Reintroduce building Core_Root to runtest.cmd

Merge pull request #9113 from wtgodbe/groovyRelease

Always use release folder for CoreFxBinDir

Fix x86compatjit and x86lb legs

Change 7eacdd7 moved Exclude from runtest.cmd to build-test.cmd, which
broke the x86compatjit and x86lb legs in the lab. This change splits the
builds for these jobs into build.cmd skiptests and build-tests.cmd Exclude
<exclusion file> and removes the Exclude option from the runtest command.

Always use release folder for CoreFxBinDir

Merge pull request #9112 from wtgodbe/fixbuildagainstpackages

Fix arg parsing for buildagainstpackages argument

Fix arg parsing for buildagainstpackages argument

Merge pull request #9100 from adiaaida/vso370233

Update isUsedFromMemory to include CNS_DBLs

Update isUsedFromMemory to include CNS_DBLs

Contained GT_CNS_DBLs should also be considered as isUsedFromMemory.

Fixes VSO 370233.

Merge pull request #9024 from stephentoub/fix_taskwhenany

Fix race condition in Task.WhenAny

Merge pull request #9102 from dotnet/revert-8969-encoding

Revert "Fast-path for ASCII & UTF8 Encoding ASCII data"

Merge pull request #9104 from dotnet/revert-9091-perf-CompareOrdinalIgnoreCaseHelper

Revert "Improve CompareOrdinalIgnoreCaseHelper performance for matching chars"

Merge pull request #9084 from hqueue/arm/ryujit/fp_sqrt

[Ryujit/ARM32] Implement GT_INTRINSIC_Sqrt for floating point type

Revert "Improve CompareOrdinalIgnoreCaseHelper performance for matching chars"

Revert "Fast-path for ASCII & UTF8 Encoding ASCII data"

Merge pull request #9073 from DrewScoggins/DevEngUbuntuFix

Fix Ubuntu Perf Runs

Fix Ubuntu Perf Runs

After the dev/eng work the Ubuntu perf runs were broken because of
layout changes in the bin drop of CoreFX.  We relied on these for the
CoreFX that we used during perf testing.  I have made changes on the
CoreFX side to tar up the new runtime folder instead of a collection of
other bin folder.  This change allows the infrastructure to consume
this.

Merge pull request #9096 from AndyAyersMS/RevComplement

Add BenchmarkGames revcomp benchmark

Merge pull request #9020 from briansull/jumpstub-logging

Add logging for jumpsStubs and ClrVirtualAllocWithinRange

Merge pull request #9092 from wtgodbe/fixRunTest

Fix runtest.sh build of Coreoverlay to account for new directory structure in CoreFX

Reintroduce building Core_Root to runtest.cmd

Fix runtest.sh build of Coreoverlay to account for new directory strucure in CoreFX

Add BenchmarkGames revcomp benchmark

This is based on version 1 of the C# variants. Am not including
the largest input file since it is ~250MB. Instead we iterate
on a smaller file.

Note the text input files are opened as binary and the app looks for
Unix line endings, hence the update to .gitattributes.

The benchmark is setup to look for its input next to the benchmark exe
as run in both normal test and xunit-perf test modes, keying off the
CORE_ROOT environment variable. In the xunit-perf mode the input files
are subsequently copied into the sandbox.

Also updated the ending copyright date on the Benchmark Games
license page.

Document jumpStub usage in Codeman.cpp
Added new STRESS_LOG calls that do the following:
 logging for JumpStubs
      log the total count of jump stub lookups: JumpStubLookup
      log the number of actual jump stub thunks created: JumpStubUnique
      log the number of jump stub blocks allocated: JumpStubBlockAllocCount
      log the number jump stub blocks that are completely filled: JumpStubBlockFullCount
 logging for ClrVirtualAllocWithinRange
      log the total number of calls to ClrVirtualAllocWithinRange
      log the number of VirtualQuery operations used by the current call
      log the range that was requested
      log the return value
      log any reasons for failure.

Improve CompareOrdinalIgnoreCaseHelper performance for matching chars (#9091)

Merge pull request #9065 from pgavlin/FixPhaseListOrder

Fix the ordering of phases in compphases.h.

Merge pull request #8973 from wtgodbe/HelixTestRuns

Add support for running CoreCLR Tests in Helix

Merge pull request #9090 from rahku/arm64fix

Fix arm64 build break

Fast-path for ASCII & UTF8 Encoding ASCII data (#8969)

* ASCII Encoding fast-path

* Add skipp for BIGENDIAN

* fixes

* ascii  GetBytes(char[] chars) fix

* feedback

* Clean up

* Reuse exception block

* Add debug Asserts

Merge pull request #8944 from DmitryDzuba/patch-1

Add version increment into Sort(Comparison<T>)

Add support for running CoreCLR Tests in Helix

Merge pull request #9071 from gkhanna79/FixVer200

Update CoreCLR nuget package versions to 2.0.0

Fix arm64 build break

Merge pull request #9076 from dotnet-bot/master-UpdateDependencies

Update CoreClr, CoreFx to beta-24924-01, beta-24924-02, respectively (master)

Delete unused files (#9082)

- *.vrg files (registry entries)
- *.ini files (performance counters)
- *.vcproj/vcxproj/vspscc/sln (out of date)
- mscorrc.rc2 (not referenced from anywhere)

Update CoreClr, CoreFx to beta-24924-01, beta-24924-02, respectively

[x86/Linux] Port CheckRegDisplaySP (#9081)

Remove duplicated declarations of CheckRegDisplaySP, and enable it for x86.

Fix codegen for GT_NEG to handle floating point type

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

[x86/Linux] Fix EH Region Mismatch (#9043)

Simplify #ifdef related with UnwindStackFrame (#9052)

Implement GT_INTRINSIC_Sqrt for floating-point

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Clean up #else in EECodeManager::FixContext (#9048)

* Clean up #else in EECodeManager::FixContext

* Remove FixContext decl from EECodeManager

Clean up LeaveFinally/LeaveCatch (#9050)

* Hide LeaveFinally/LeaveCatch when WIN64EXCEPTION is defined

Merge pull request #9074 from BruceForstall/FixGenTreeHash

Fix GenTree hash function for commutative operators

Disallow statics of spans and class instance members of span (#9061)

Functional fix for #8516 to cover some additional cases. IL tests will be coming later in a separate PR.

[x86/Linux] Port two preconditions in ExceptionTracker::ResetThreadAbortStatus (#9079)

Rename Esp in x86 REGDISPLAY as SP (#9070)

* Rename Esp in x86 REGDISPLAY as SP

* Fix x86/Windows build error

Merge pull request #8672 from mikedn/cmp-lower-refactoring

Refactor TreeNodeInfoInitCmp

Merge pull request #9072 from CarolEidt/DevDiv_1206929

Add a test case for a test optimization

Port GC fixes from CoreRT (#9078)

Remove PALAPI attribute from PAL Callback (#9075)

Clean up #else in EECodeManager::GetAmbientSP (#9049)

Merge pull request #8910 from AndyAyersMS/SpanBench

Add a simple span benchmark

[Local GC] Provide an implementation of GCToOSInterface for Unix-like platforms (#8976)

* Add way to build with FEATURE_STANDALONE_GC from build.sh
* Make CMake changes to build the GC 'PAL' as its own build target (to avoid -nostdinc).
In addition, introduce a "GC PAL" that provides an implementation of
GCToOSInterface on Unix-like platforms, for use with
FEATURE_STANDALONE_GC.