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

Speed up test native binary syncing in pipeline

Merge pull request #9170 from CarolEidt/Fix9159

Fix GT_LCL_FLD case in gtGetStructHandleIfPresent

Merge pull request #9117 from jamesqo/copy.names

Fix parameter names for Array.Copy

Merge pull request #8438 from JonHanna/fix_1764

Support filter and fault blocks in DynamicILGenerator

Merge pull request #8564 from lucenticus/x86-gdbjit

[x86/Linux] Initial support of GDB JIT interface

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

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

Support building lldbplugin on ARM64 (#9195)

Remove allocations from Environment.GetResourceString (#9194)

* Remove delegate allocations from ResourceHelper.GetResourceString

* Remove state object allocation from GetResourceString

Now that we're no longer using ExecuteCodeWithGuaranteedBackout, we don't need the try/catch/finally functionality separated out into separate methods, and thus we don't need to allocate an object to pass into them.

Delete unused encodings from CoreLib (#9196)

JIT: fixup finally target flag after finally opts run (#9171)

The finally target flag can't be incrementally maintaned during finally
opts, so flag if any of these opts run and then run a post-opts fixup pass
to properly determine the flag value.

Only impacts arm targets.

Closes #9015.

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

Add support for lldb-3.9 (#9190)

[GDB-JIT][Linux] Add support of base classes introspection   (#9083)

* [GDB-JIT] Initial support of inherited fields displaying for classes

* [GDB-JIT] Fix link to already existed parent class

* [GDB-JIT] Fix incorrect processing of links to parent class with RefTypeInfo

* [GDB-JIT] Ignore inheritance from System.ValueType

* [GDB-JIT] Insert additional check whether we already dump parent class

Merge pull request #9181 from dotnet/revertevent

Revert recent changes of #if blocks in Eventing code

Remove invalid asserts on unsupported underlying enum type (#9192)

The underlying enum type can be floating point type or native integer for historic reasons.
However, not all framework methods support these special enum types. They should throw exception
for them instead of asserting.

Fix issue #7670

Merge pull request #9184 from AndyAyersMS/RegexDna

Add BenchmarkGames regexdna test

Update glossary.md

Add BenchmarkGames regexdna test

Based on the "version 6" test base. As with revcomp, keeping the input
files as unix line ending, and relatively small.

Merge pull request #9164 from justinvp/string_split

Remove unnecessary private String.SplitInternal method

Update glossary.md

Revert `Remove #if false blocks` just for Eventing

Revert `Remove  FEATURE_MANAGED_ETW_CHANNELS`

Revert `Remove disabled FEATURE_ADVANCED_MANAGED_ETW_CHANNELS`

Merge pull request #9163 from BruceForstall/FixAddrModeBuilding

Stop address mode building in TreeNodeInfoInit pass

Fix GT_LCL_FLD case in gtGetStructHandleIfPresent

This method was incorrectly returning the parent struct handle instead of the handle for the field. This is showing up in remorphing of a SIMD field that was assigned in a field-by-field assignment.

Fix #9159

JIT: Remove empty try regions (#8949)

In runtimes that do not support thread abort, a try-finally with
an empty try can be optimized to simply the content of the finally.

See documentation update for more details.

Closes #8924.

Enable IsInPrologOrEpilog only if GC Info Decoder is not used (#9136)

* Enable IsInPrologOrEpilog only if GC Info Decoder is not used

Merge pull request #9130 from RussKeldorph/corefx

Support new corefx workflow

Use a constant to define the location of '/tmp' (#9150)

* The temp folder is in /data/local/tmp on Android, not /tmp

Support platforms where pthread is part of libc (#9154)

* Make threading code compatible with Android

Adding trivial implementations of Span Fill and Clear (#9162)

Remove unnecessary private String.SplitInternal method

Avoid the unnecessary intermediate method call.

Merge pull request #9054 from kouvel/SpanGcReporting

Fix GC info for `ByReference<T>`

Stop address mode building in TreeNodeInfoInit pass

During Lower, TryCreateAddrMode() will try to find address
modes and convert them to GT_LEA by calling genCreateAddrMode().
Later, the TreeNodeInfoInit() pass walks the IR annotating
nodes with register requirements. It was also calling
genCreateAddrMode() and then clearing various previously
created register requirements, but not actually changing the
node to a GT_LEA. In some cases (in new code I have not checked
in yet), this was leading to code that codegen couldn't handle.
It turns out this second call to genCreateAddrMode() is
unnecessary (and undesireable), so I've removed it. This causes
no asm diffs on amd64 or x86 (run on desktop: superpmi
and ngen, respectively).

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

Revert "Revert "Speed up test native binary syncing in pipeline""

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.

Merge branch 'copy.names' of github.com:jamesqo/coreclr into copy.names

Fix corelib.txt message

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 parameter names for Array.Copy

Fix build-test.cmd

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

Support new corefx workflow

Update the run-corefx-tests.py script and enable it to run CI jobs
against corefx HEAD again.  Also update the instructions for running
corefx tests with a private coreclr.

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]

Use FCThrowArgumentVoid

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"