Merge pull request dotnet/coreclr#8645 from dotnet-bot/master-UpdateDependencies

Update CoreFx to beta-24815-01 (master)

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

Update CoreFx to beta-24815-01

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

Merge pull request dotnet/coreclr#8546 from alsemenn/debuggertests_generate_scripts_win

Add script generator and generate test scripts to adapt debuggertests…

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

Merge pull request dotnet/coreclr#8640 from pgavlin/VSO359734

Fix ref count adjustment in `fgMorphBlockStmt`.

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

Add a regression test.

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

Merge pull request dotnet/coreclr#8582 from dotnet-bot/master-UpdateDependencies

Update CoreClr, CoreFx to beta-24814-03, beta-24814-02, respectively (master)

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

Fix ref count adjustment in `fgMorphBlockStmt`.

LclVar ref counts must be incremented before attempting to remove the
morphed statement, since doing so decrements ref counts (and thus
requires refcounts to be conservatively correct).

Fixes VSO 359734.

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

[Linux][GDB-JIT] Add simple C++ mangling of method names in GDBJIT DWARF (dotnet/coreclr#8638)

* Add simple C++ mangling of method names in GDBJIT DWARF

Example:
    Namespace1.Class1.Method -> void Namespace1_Class1::Method()

* Do not convert a name during mangling if target buffer is NULL

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

Merge pull request dotnet/coreclr#8601 from pgavlin/gh7963

Fix consume-order checking in codegen.

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

Add support for R2R ldvirtftn helpers (dotnet/coreclr#8608)

The codegen for the non-readytorun path (used on CoreCLR) requires the
runtime to support general purpose virtual method resolution based on a
method handle. CoreRT doesn't have such helpers.

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

Update CoreClr, CoreFx to beta-24814-03, beta-24814-02, respectively

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

Move RUNTIME_FUNCTION__BeginAddress into clrnt.h (dotnet/coreclr#8632)

RUNTIME_FUNCTION__BeginAddress is defined in corcompile.h for x86, but
is defined in clrnt.h for all the other architectures.

This commit moves RUNTIME_FUNCTION__BeginAddress defines for x86 into
clrnt.h to make it consistent.

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

[x86/Linux] Adds Dummy Exception Handler (dotnet/coreclr#8613)

This commit adds a dummy exeption handler to enable x86/Linux build.

This commit also reverts 7b92136d5ee to make it easy to enable
WIN64EXCEPTIONS in x86/Linux.

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

[ARM32/Linux] Initial bring up of FEATURE_INTERPRETER (dotnet/coreclr#8594)

* Bring up FEATURE_INTERPRETER for ARM32/Linux
* Add a disclaimer message for GC preemption workaround

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

Adding method implementations to fast span/readonlyspan  (dotnet/coreclr#8607)

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

[x86/Linux] Enforce 16-byte stack alignment (dotnet/coreclr#8587)

Clang (and GCC) requires 16-byte stack alignment, but the current
implementation of CallDescrInternal and ThePreStub does not provide any
guarantee on stack alignment.

This commit adds 16-byte stack alignment adjust code inside these functions.

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

[x86/Linux] Fix incorrect __fastcall definition (dotnet/coreclr#8585)

In x86/Linux, __fastcall is defined as __stdcall which causes stack
corruption issue for the following functions:
 - HelperMethodFrameRestoreState
 - LazyMachStateCaptureState

This commit removes __fastcall definition as clang recognize __fastcall.

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

[x86/Linux] Fix "Bad opcode" assert in unwindLazyState (dotnet/coreclr#8609)

* [x86/Linux] Fix "Bad opcode" assert in unwindLazyState

This commit suppresses "Bad opcode" assert while runing "Hello, World" example.

This commit address the following three code patterns discovered while
digging the assert failure:
 - and $0x1, %al
 - xor $0xff, %al
 - stack protection code:
   mov %gs:<off>, <reg>
   cmp <off>(%esp), <reg>
   mov <reg>, <off>($esp)
   jne <disp32>

This commit revises LazyMachState::unwindLazyState to handle the first two patterns,
and revises compile options not to emit the third pattern.

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

Merge pull request dotnet/coreclr#8617 from hqueue/arm/test20161210/IsContainableImmed

Ryujit/ARM32 Initial Lowering::IsContainableImmed for ARM

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

[x86/Linux] Port jithelp.asm (dotnet/coreclr#8491)

* [x86/Linux] Port jithelp.asm

This commit ports jithelp.asm for x86/Linux

The following Tailcall helpers are marked as NYI:
 - JIT_TailCall
 - JIT_TailCallReturnFromVSD
 - JIT_TailCallVSDLeave
 - JIT_TailCallLeave

* Revise macro and indentation

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

Fix the ARM32 build.

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

Remove no-op file security (dotnet/coreclr#8611)

Deletes FileSecurityState and pulls redundant methods. Also
removes DriveInfo, which isn't in use in Core.

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

Fix consume-order checking in codegen.

The switch to LIR invalidated the correspondence between a node's
sequence number and the order in which it must be consumed with respect
to other nodes. This in turn made the consume-order checks during code
generation incorrect.

This change introduces a new field, `gtUseNum`, that is used during code
generation to check the order in which nodes are consumed. Re-enabling
these checks revealed a bug in code generation for locked instructions
on x86, which were consuming their operands out-of-order; this change
also contains a fix for that bug.

Fixes dotnet/coreclr#7963.

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

Merge pull request dotnet/coreclr#8560 from AlexRadch/Deconstruction

Supporting C# 7 deconstruction of KeyValuePair and DictionaryEntry

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

Merge pull request dotnet/coreclr#8621 from mikedn/cmp-cast-drop

Fix incorrect compare narrowing in TreeNodeInfoInitCmp

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

Fix incorrect compare narrowing in TreeNodeInfoInitCmp

TreeNodeInfoInitCmp attempts to eliminate the cast from
`cmp(cast<ubyte>(x), icon)` by narrowing the compare to ubyte. This should
only happen if the constant fits in a byte so it can be narrowed too,
otherwise codegen produces an int sized compare. (or a byte sized compare
with a truncated constant if we try to use GTF_RELOP_SMALL).

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

Enable interop debugging for Windows amd64 and x86. (dotnet/coreclr#8603)

Found sos portable pdb problem on x86.  Fixed interop problems between the native sos and SOS.NetCore managed helper assembly.

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

Synchronize src\mscorlib\corefx with the CoreRT fork (dotnet/coreclr#8619)

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

Remove managed environment cache (dotnet/coreclr#8604)

The PAL already caches- no need to do it in managed.
Fold the code back into Environment.cs.

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

Delete HostProtection attributes (dotnet/coreclr#8610)

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

Ryujit/ARM32 Initial Lowering::IsContainableImmed for ARM

Initial implementation of IsContainableImmed for ARM.

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/04689b1b87d1cca78649066b79e8317f1e0cfd92

Merge pull request dotnet/coreclr#8520 from hqueue/arm/test20161208/lowercast

Ryujit/ARM32: Implement Lowering::LowerCast for ARM

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

Ryujit/ARM32 Implement Lowering::LowerCast for ARM

Simple integer to interger type conversion is passed.
Add comment for LowreCast with C++ comment style.

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/fd68b6fe29f18d69093c7a007fbc8fb9e5eaaa90

Merge pull request dotnet/coreclr#8510 from DrewScoggins/CommitNameFix

Add ability to give a name to a manual PR run

When using the PR leg of the Jenkins performance jobs it is now possible to specify a name that will be used when inserting the data into the perf database.

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

Add ability to give a name to a PR run

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

Rename Contract.Assert to Debug.Assert (dotnet/coreclr#8600)

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

[x86/Linux] implement TheUMEntryPrestub (dotnet/coreclr#8589)

Initial code for x86 TheUMEntryPrestub, UMThunkStub

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

Fix longname DAC to enable arm[64] symbol packages (dotnet/coreclr#8574)

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

Merge pull request dotnet/coreclr#8532 from sivarv/fixedRegFix

Fix to issue 8286.

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

Merge pull request dotnet/coreclr#8598 from adiaaida/disableGetGeneration

Disable GetGenerationWR2 for GCStress on x86

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

Merge pull request dotnet/coreclr#8541 from briansull/vso-287671

Fix missing flags on GT_DYN_BLK node

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

// BLOCKED (do not add now): [EditorBrowsable(EditorBrowsableState.Never)]

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

Disable GetGenerationWR2 for GCStress on x86

Disable this test for GCStress on x86 until the cause for its failure
can be investigated.

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

Merge pull request dotnet/coreclr#8421 from hseok-oh/enable_arm_linux_nupkg

First step to create nuget packages for ARM32/Linux

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

Span<T> api update (dotnet/coreclr#8583)

* Changing method/property order to match CoreFX impl

To make diffing the files easier

* Added other missing methods to match CoreFX impl

Added:
- public void CopyTo(Span<T> destination)
- public static bool operator ==(Span<T> left, Span<T> right)
- public static bool operator !=(Span<T> left, Span<T> right)
- public override bool Equals(object obj)
- public override int GetHashCode()

Also removed 'public void Set(ReadOnlySpan<T> values)' and it's no
longer part of the Span<T> public API, see

https://github.com/dotnet/apireviews/tree/master/2016/11-04-SpanOfT#spantset

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

[x86/Linux][SOS] Don't include utilcode.h in gcdumpx86.cpp and gcdump.cpp (dotnet/coreclr#8475)

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

[x86/Linux][SOS] Add CALLBACK (aka stdcall) to function declarations (dotnet/coreclr#8476)

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

Local GC: Decouple write barrier operations between the GC and EE (dotnet/coreclr#8568)

* Decouple write barrier operations between the GC and EE

* Address code review feedback

* Address code review feedback

* Repair the standalone GC build

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

Fix typo in clang-format directive and reformat end of flowgraph.cpp (dotnet/coreclr#8573)

Last 4K or so lines of flowgraph.cpp were not being formatted because
the clang-format on directive had a typo.

Fix the typo and reformat the latter part of the file.

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

Merge pull request dotnet/coreclr#8581 from stephentoub/cwt_unsafecast

Use JitHelpers.UnsafeCast in ConditionalWeakTable

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

Use JitHelpers.UnsafeCast in ConditionalWeakTable

We know the types and can use UnsafeCast when reading out the objects as TKey and TValue.  This improves reading speed by ~30%.

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

Merge pull request dotnet/coreclr#8420 from dotnet-bot/master-UpdateDependencies

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

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

Improve ConditionalWeakTable.Remove (dotnet/coreclr#8580)

Clear the key of the deleted entry to allow GC collect objects pointed to by it

Fixes dotnet/coreclr#8577

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

TimeZoneInfo: Avoid cloning privately-created ArgumentRule[] arrays (dotnet/coreclr#8575)

TimeZoneInfo currently always creates a defensive copy of the specified
ArgumentRule[] array when created. This makes sense for the public
static factory methods. However, there's no need to create a defensive
copy of arrays created privately as part of its implementation (e.g.
reading the rules from the registry/disk). This change avoids the
unnecessary cloning.

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

Fix misguided lock in CurrentTimeZone (dotnet/coreclr#8569)

CurrentTimeZone locks against a static lock object when modifying a
non-static Hashtable. Instead, use the Hashtable instance itself as the
lock object.

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

Removes final FEATURE_CORECLR defines (dotnet/coreclr#8555)

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

Adding API ConditionalWeakTable.AddOrUpdate (dotnet/coreclr#8490)

* Added ConditionalWeakTable.AddOrUpdate

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

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

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

Fix to issue 8286.

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

Merge pull request dotnet/coreclr#8576 from stephentoub/cwt_clear

Change ConditionalWeakTable.Clear

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

Change ConditionalWeakTable.Clear

The original CoreRT implementation just dropped the current table and replaced it with a new one.  In the process of porting the CoreRT implementation to CoreCLR, I'd changed it to instead remove each item from the table (by setting its hashcode to -1, as does Remove), in order to work around some code in the finalizer that would null out te parent's reference to the container, and that would cause problems with dropping this table... but that code in the finalizer changed before it got merged, and in its current form, the old CoreRT clear implementation was fine.  It's also likely better, as it'll let the handles be cleaned up earlier, and it's simple.  So reverting back to it.

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

Merge pull request dotnet/coreclr#8572 from stephentoub/cwt_gpas

Change CWT use of GetPrimaryAndSecondary to GetPrimary

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

Strip some security related attributes (dotnet/coreclr#8571)

Strips SecurityCritical, SecuritySafeCritical, SecurityPermission,
EnvironmentPermission, and PermissionSet attributes.

Also removes empty defines these left behind.

Patterns used:

^.*\[(System\.Security\.)?SecurityCritical\](\s*//.*|\s*)$[\r\n]*
^.*#if FEATURE_CORECLR[\s\r\n]*(#else)?[\s\r\n]*#endif.*[\r\n]*
^.*\[(System\.Security\.Permissions\.)?SecurityPermission(Attribute)?\([^)]*\)\](\s*//.*|\s*)$[\r\n]*

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

Change CWT use of GetPrimaryAndSecondary to GetPrimary

This snuck in as part of my previous ConditionalWeakTable changes.  We don't need the secondary here, and it's more expensive to get than just the primary.

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

Fix path separator in CrossGen help on Linux

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

Make it easier to iterate through an ArraySegment (dotnet/coreclr#8559)

* Make it easier to iterate through an ArraySegment. See Make it easier to iterate through an ArraySegment

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

Merge pull request dotnet/coreclr#8544 from sivarv/moveRegFix

Fix to issue 8287.

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

Merge pull request dotnet/coreclr#8507 from stephentoub/portcorert_cwt

Make ConditionalWeakTable reads lock-free

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

StringBuilder.AppendJoin (appending lists to StringBuilder) (dotnet/coreclr#8350)

Adding StringBuilder.AppendJoin

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

Merge pull request dotnet/coreclr#8563 from xoofx/TPA_glossary

Add TPA/Trusted Platform Assemblies description to the glossary

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

Add TPA/Trusted Platform Assemblies description to the glossary (From https://github.com/dotnet/coreclr/issues/6470#issuecomment-235161459 )

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

model.xml

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

[x86/Linux] Port ResolveWorkerAsmStub (dotnet/coreclr#8557)

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

[x86/Linux] Use Portable FastGetDomain (dotnet/coreclr#8556)

FastGetDomain function (with  __declspec(naked)) causes segmentation
fault.

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

Fix incremental build when dummy version.cpp is generated (dotnet/coreclr#8547)

This change fixes a problem with incremental build on Unix. When the
version.cpp is generated by the build.sh as a dummy one with no real
version stamp in it, it is recreated every time the build.sh is run.
That means that build needs to rebuild that file and also re-link
all the components that include it.
This change tests the file presence and contents before actually
regenerating it.

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

[x86/Linux] Fix getcpuid calling convention (dotnet/coreclr#8552)

Fix getcpuid(), getextcpuid() with STDCALL
Fix xmmYmmStateSupport() with STDCALL

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

Strip more defines from CoreLib (dotnet/coreclr#8545)

* Strip more defines from CoreLib

Removes the rest of

FEATURE_CAS_POLICY
FEATURE_REMOTING
FEATURE_MACL

And another significant chunk of

!FEATURE_CORECLR

* Address feedback

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

Fix perf regression with lots of objects in a ConditionalWeakTable

The CoreRT implementation of ConditionalWeakTable that was ported back to CoreCLR uses a special scheme to make reads lock-free.  When the container needs to grow, it allocates new arrays and duplicates all of the dependency handles from the previous array, rather than just copying them.  This avoids issues stemming from a thread getting a dependency handle in an operation on the container, then having that handle destroyed, and then trying to use it; the handle won't be destroyed as long as the container is referenced.

However, this also leads to a significant cost in a certain situation.  Every time the container grows, it allocates another N dependency handles where N is the current size of the container.  So, for example, with an initial size of 8, if 64 objects are added to the container, it'll allocate 8 dependency handles, then another 16, then another 32, and then another 64, resulting in significantly more handles than in the old implementation, which would only allocate 64 handles total.

This commit fixes that by changing the scheme slightly.  A container still frees its handles in its finalizer.  However, rather than duplicating all handles, that responsibility for freeing is transferred from one container to the next.  Then to avoid issues where, for example, the second container is released while the first is still in use, a reference is maintained from the first to the second, so that the second can't be finalized while the first is still in use.

The commit also fixes a race condition with resurrection and finalization, whereby dependency handles could be used while or after they're being freed by the finalizer. It's addressed by only freeing handles in a second finalization after clearing out state in the first finalization to guarantee no possible usage during the second.

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

Port ConditionalWeakTable from CoreRT

The CoreRT ConditionalWeakTable was modified to support lock-free reads.  This ports the implementation back to coreclr.

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

Fix to issue 8287.

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

Merge pull request dotnet/coreclr#8539 from sandreenko/fix-unix-unwind-info

Fix unix unwind info

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

[x86/Linux] Fix PAL unit test paltest_pal_sxs_test1 (dotnet/coreclr#8522)

Fix unit test error for x86/Linux
- fix fail of exception_handling/pal_sxs/test1/paltest_pal_sxs_test1

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

[x86/Linux] Revise asmhelper.S using macro (dotnet/coreclr#8523)

* [x86/Linux] Revise asmhelper.S using macro

This commit revises asmhelper.S using macros that inserts CFI
directives.

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

Merge pull request dotnet/coreclr#8537 from pgavlin/VSO299202

Disable special put args for LIMIT_CALLER on x86.

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

Merge pull request dotnet/coreclr#8543 from pgavlin/VSO299207

Correct an assertion in LSRA.

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

Make TimeZoneInfo fields readonly (dotnet/coreclr#8526)

TimeZoneInfo is immutable. Help enforce this by making its fields
readonly.

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

TimeZoneInfo: Use string.Concat instead of string.Format (dotnet/coreclr#8540)

It's more efficient to concatenate the strings.

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

Make TimeZoneInfo.TransitionTime fields readonly (dotnet/coreclr#8529)

TransitionTime is immutable. Help enforce this by making its fields
readonly.

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

[x86/Linux] Port Several Stubs as NYI (dotnet/coreclr#8515)

This commit adds SinglecastDelegateInvokeStub and VSD-related Stubs as NYI.

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

Make TimeZoneInfo.AdjustmentRule fields readonly (dotnet/coreclr#8528)

AdjustmentRule is immutable. Help enforce this by making its fields
readonly.

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

Preallocate the TimeZoneInfo.Utc instance (dotnet/coreclr#8530)

There doesn't appear to be a good reason why the TimeZoneInfo.Utc
instance needs to be cleared when TimeZoneInfo.ClearCachedData() is
called. Instead, we can pre-allocate and reuse a singleton instance,
obviating the need for the lazy-initialization/locking mechanics.

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

Simplify TimeZoneInfo.AdjustmentRule.Equals (dotnet/coreclr#8527)

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

Remove private TimeZoneInfoComparer (dotnet/coreclr#8512)

Use Comparison<T> instead of IComparer<T> to sort the list of
TimeZoneInfos, which moves the comparison code to the sole place where
it is used, and now that Array.Sort is implemented in terms of
Comparison<T> instead of IComparer<T>, avoids some unnecessary
intermediate allocations.

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

Add script generator and generate test scripts to adapt debuggertests repo for coreclr infrastructure

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

Fix unix unwind info

Windows uses offset from stack pointer, when unix has to use offset from
caninical frame address,

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

Remove sscanf and sprintf usage (dotnet/coreclr#8508)

* Remove sscanf
* Remove sprintf

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

Correct an assertion in LSRA.

`verifyFinalAllocation` asserts that if a non-BB interval RefPosition
that is either spilled or is the interval's last use does not have a
register, then that ref position must be marked `AllocateIfProfitable`.
However, this situation can also arise in at least one other situation:
an unused parameter will have at least one ref position that may not be
allocated to a register. This change corrects the assertion to check
`RefPosition::RequiresRegister` rather than
`RefPosition::AllocateIfProfitable`.

Fixes VSO 299207.

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

The fix is to set the GTF_EXCEPT and GTF_GLOB_REF for every GT_DYN_BLK node that we create.
We typically don't have any information about the address supplied to a GT_DYN_BLK so we should
conservatively allow that it can either be a null pointer or could point into the GC heap.

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

Disable special put args for LIMIT_CALLER on x86.

On x86, `LSRA_LIMIT_CALLER` is too restrictive to allow the use of special
put args: this stress mode leaves only three registers allocatable--eax,
ecx, and edx--of which the latter two are also used for the first two
integral arguments to a call. This can leave us with too few registers to
succesfully allocate in situations like the following:

    t1026 =    lclVar    ref    V52 tmp35        u:3 REG NA <l:$3a1, c:$98d>

            /--*  t1026  ref
    t1352 = *  putarg_reg ref    REG NA

     t342 =    lclVar    int    V14 loc6         u:4 REG NA $50c

     t343 =    const     int    1 REG NA $41

            /--*  t342   int
            +--*  t343   int
     t344 = *  +         int    REG NA $495

     t345 =    lclVar    int    V04 arg4         u:2 REG NA $100

            /--*  t344   int
            +--*  t345   int
     t346 = *  %         int    REG NA $496

            /--*  t346   int
    t1353 = *  putarg_reg int    REG NA

    t1354 =    lclVar    ref    V52 tmp35         (last use) REG NA

            /--*  t1354  ref
    t1355 = *  lea(b+0)  byref  REG NA

Here, the first `putarg_reg` would normally be considered a special put arg,
which would remove `ecx` from the set of allocatable registers, leaving
only `eax` and `edx`. The allocator will then fail to allocate a register
for the def of `t345` if arg4 is not a register candidate: the corresponding
ref position will be constrained to { `ecx`, `ebx`, `esi`, `edi` }, which
`LSRA_LIMIT_CALLER` will further constrain to `ecx`, which will not be
available due to the special put arg.

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

Avoid allocating in TimeZoneInfo.GetHashCode() (dotnet/coreclr#8513)

Avoid the intermediate ToUpper string allocation.

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