Fix for a bug in tail recursion elimination.

Tail recursion elimination transforms a tail call into a loop.
If compInitMem is set, we may need to zero-initialize some locals. Normally it's done in the prolog
but the loop we are creating can't include the prolog. The fix is to insert zero-initialization
for all non-parameter non-temp locals in the loop. Liveness phase will remove unnecessary initializations.

We never hit this case with normal C# code since C# definite assignment rules ensure that there are
no uninitialized locals in the generated msil. In the repro the method with tail recursion is a dynamic method
and it has an uninitialized local.

Merge pull request #12722 from CarolEidt/DumpFixes

A couple more fixes to JitDump

Merge pull request #12674 from CarolEidt/LowerStoreLoc

Move StoreLoc type widening to Lowering

Change heading size

Merge pull request #12628 from hseok-oh/ryujit/fix_12602

[RyuJIT/ARM32] Clear assertion for refany struct and struct type temp

Put in a warning that the instructions no longer work properly

Remove  another reference to System.Private.Corelib.ni.dll

Remove reference to System.Private.Corelib.ni.dll

The build now only has the NGENed image (called but called System.Private.Corelib.dll.   Remove references to the *.ni.dll version

Merge pull request #12580 from wtgodbe/disableSync

Disable sync.cmd -n for Windows binaries

A couple more fixes to JitDump

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

Merge changes from TFS

Speed up fgLocalVarLiveness in minopts. (#12665)

In particular:
- Do not re-sort lclVars
- Do not run LVA
- Do not set `mustInit`
- Do not set last uses
- Do not perform DSE

Merge pull request #12717 from wtgodbe/disableJITTestArm

disable hijacking test for Arm32

Merge pull request #12712 from helloguo/errorHandle

Fix error handle of PAL API SetThreadGroupAffinity

disable hijacking test for Arm32

Add symbol signing to published packages. (#12291)

* Add symbol signing to published packages.
- Enable MicroBuild signing steps.
- Update build to call new BuildTools symbol signing targets.

Disable sync.cmd -n for Windows binaries

Fix optdata bootstrap test script & Linux skiprestoreoptdata behavior (#12541)

Fixes #12171.

* bootstrap.py was printing an improperly-cased path to put PGO/IBC
  training data, which breaks on case-sensitive file systems, i.e., on
  Linux.

* On Linux, `skiprestoreoptdata` was improperly skipping PGO even if
  optdata was already present due to some of the conditional checks in
  build.sh. This cleans that up to better match the intent of the
  switch, and to fix Linux behavior to better match Windows.

Merge pull request #12710 from hseok-oh/ryujit/fix_12703

[RyuJIT/ARM32] Fix register allocation for split GT_FIELD_LIST struct

Update the telemetry logging to use MEASURES instead of TELEMENTRY

[tfs-changeset: 1665164]

[RyuJIT/ARM32] Fix register allocation for split GT_FIELD_LIST struct

To fix register allocation for split GT_FIELD_LIST struct,
set source register for each node of GT_FIELD_LIST

fix error handle

Merge pull request #12572 from wateret/armel/dblarg_stack

[RyuJIT/armel] Passing stack double arguments

[RyuJIT/ARM32] Avoid promoting structures which can be handled by HFA (#12604)

* [RyuJIT/ARM32] Avoid promoting structures which can be handled by HFA

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>
* [RyuJIT/ARM32] Add comments for ARM32

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

Note that tests don't have to be built to view JIT dumps (#12699)

Unify LazyInitializers with CoreRT (#12697)

CoreCLR implementation cached s_activatorFactorySelector delegate, but CoreRT did not cache it - it got commented out during earlier AOT binary size optimization efforts. I have unified it by avoiding s_activatorFactorySelector delegate altogether to keep the best of both.

TypeForward cleanup for Culture=neutral (#12698)

Move files to shared CoreLib partition (#12694)

Use R2R helper for runtime lookups only when USEHELPER was specified (#12669)

The CoreRT compiler would like to be able to do indirections-based
lookup even when ReadyToRun codegen is specified.

Avoid a large local in a recursive visitor. (#12693)

This local of type `CORINFO_CALL_INFO` bloats the frame size of its
caller when the function is inlined and ends up aggravating stack size
issues in deep recursion. This change refactors the offending method a
bit in order to avoid the local entirely.

Hopefully this fixes #12691.

Update CoreClr, CoreFx to preview2-25508-04, preview2-25508-04, respectively (#12633)

Remove redundant call to TimeSpan.TotalMilliseconds (#12692)

Ensure that ZeroMemoryInGCHeap writes in pointer-sized increments (#12689)

by adding the volatile keyword (which disables memset optimization).
Fixes #12207

Merge pull request dotnet/corert#4110 from jkotas/nmirror-merge

Merge nmirror to master

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

Merge pull request dotnet/corert#4100 from dotnet/nmirror

Merge nmirror to master

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

Fix Unix debug assert to not use instance handle (#12687)

Fix GitHub_3449 test (#12683)

* Fix GitHub_3449 test

This test was looking for one answer for ARM and ARM64 but another
for other architectures. This didn't work for ARM32 on Windows
(compared to ARM32 on Linux).

Instead, simply allow either answer. This doesn't seem like a
terrible loss of precision, since we already allowed different
answers based on the platform.

* Update comment

Minor fix added to mirrored changes

Merge pull request dotnet/corert#4086 from tarekgh/CleanupSerializationInEncoding

Clean up serialization from Encoding

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

Move StoreLoc type widening to Lowering

Merge pull request #12682 from adityamandaleeka/arm64_dump_hang

Temporarily block core dump printing on arm64

Merge pull request #12675 from mikedn/range-disp

Dump tree IDs instead of pointers in RangeCheck

Merge pull request #12673 from ruben-ayrapetyan/fix-issue-12643

Fix copying of FieldMarshaler structures

Merge pull request #12681 from BruceForstall/RemoveNonexistentVarargsTests

Remove varargs test references

Restore missing native *.ni.pdb file from the Microsoft.NETCore.Runtime.CoreCLR (#12677)

* Restore missing native *.ni.pdb file from the Microsoft.NETCore.Runtime.CoreCLR package

DLLs that have native code, need both the *.pdb (for IL information) and the *.ni.pdb (for
native information).

When System.Private.Corlib was renamed from System.Private.Corlib.ni.pdb to System.Private.Corlib,
the logic that decides what goes into the  Microsoft.NETCore.Runtime.CoreCLR package lost
the *.ni.pdb for System.Private.Corlib.

This change basically causes this logic to search both for *.ni.pdb as well as the *.pdb file and
thus fixes this issue.

* Remove unnecessary condition (review feedback)

Temporarily block core dump printing on arm64 to unblock CI.

Revert change that always publish runtime layout (#12679)

Merge pull request #12500 from adiaaida/throughputExclusionList

Update throughput exclusion list

Remove varargs test references

varargs tests were removed with 0a427c6d6d87e69ce0f87dafd2f965ee989e6d29

Dump tree IDs instead of pointers in RangeCheck

Fix skipcrossgen option (#12650)

* Fix skipcrossgen option

* Use return instead of exit

Merge pull request #12429 from mikedn/dead-qmark

Remove QMARK related dead code

Merge pull request #12623 from hqueue/arm/ryujit/issue_12615

[RyuJIT/ARM32] Unassign double register properly

Merge pull request #12597 from hseok-oh/ryujit/struct_split/lsra

[RyuJIT/ARM32] Codegen: split struct argument less than 16 bytes

Fix copying of FieldMarshaler structures in EEClassLayoutInfo::CollectLayoutFieldMetadataThrowing.

Related issue: #12643

[RyuJIT/ARM32] Codegen: split struct argument less than 16 bytes

Enable passing split struct (less than 16 bytes)
- LSRA phase
- Codegen phase

Merge pull request #12653 from wateret/arm/lvadump

[RyuJIT/arm] Fix double type local variable dump

[RyuJIT/ARM32] Apply review feedback

Rename vairable and rewrite code to use less #ifdef

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

[RyuJIT/armel] Rewrite changes for readibility

Pin GC threads to corresponding processors on Linux (#12486)

* fix SetThreadGroupAffinity, add SetThreadAffinityMask and pin GC threads to processors on Linux

* fix loop count and error handler

Merge pull request #12667 from rartemev/issue_12465_1

Added explicit cast float -> double for arguments

Merge pull request #12634 from rartemev/issue_12466_1

Fix misprint in complex condition

Added explicit cast float -> double for arguments

Use SUB instead of XOR to clear fat pointer bit (#12658)

This makes the code generated for fat calli smaller because of the clearing of the fat pointer bit gets fused into address mode.

Merge pull request #12666 from mskvortsov/UnalignedLdSt

[RyuJIT/Arm32] Support unaligned floating point loads/stores

Merge pull request #12655 from YongseopKim/ryujit/arm32/add_nyi_promoted_struct

[RyuJIT/ARM32] Add NYI_ARM for handling promoted struct

Update throughput exclusion list

There are many assemblies in our throughput benchmark package that are
reference or forwarding assemblies. There is no jit work in these
assemblies, so we want to exclude them.

This change also adds an iterations parameter so users can specify a
specific number of iterations for testing.

Merge pull request #12053 from mikedn/xcopy-bindir

Do not copy bin subdirectories to test core_root

Merge pull request #12652 from hseok-oh/ryujit/fix_split_lower

[RyuJIT/ARM32] Fix register type setting for split struct arg

Support unaligned floating point loads/stores

[RyuJIT/ARM32] Add NYI_ARM for handling promoted struct

- In RyuJIT/ARM32, promoted struct on stack can't be handled like
  splitted struct arg's one.
- Until implementing handling it, make it entering the NYI_ARM.

Fix DumpLog to be compatible with CoreRT (#12636)

build runtime layout when skipping tests (#12587)

[x86/Linux] Enable thread-abort reraise loop prevention (#12646)

[RyuJIT/arm] Fix double type local variable dump

[RyuJIT/ARM32] Fix register type setting for split struct arg

Fix register type setting for split struct argument using GT_FIELD_LIST node

[RyuJIT/ARM32] Fix assertion failed 'gcPtrCount == 0' (#12621)

* [RyuJIT/ARM32] Fix assertion failed 'gcPtrCount == 0'

Change wrong gcPtrCount's value from gtNumSlots to gtNumberReferenceSlots of GenTreePutArgStk

* Fix a formatting error

Merge pull request #12647 from dotnet/mirror-merge-9210244

Mirror changes from dotnet/corert

Fix the merge from the corert changes

Merge pull request dotnet/corert#4080 from dotnet/nmirror

Merge nmirror to master

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

Merge pull request #12606 from adityamandaleeka/mask_activation_on_altstack

Mask INJECT_ACTIVATION_SIGNAL while on alternate signal stack

Mask INJECT_ACTIVATION_SIGNAL while on alternate signal stack.

Update gtDispRegVal() for multi register nodes (#12573)

* Update gtDispRegVal() for multi register nodes

* [RyuJIT/armel] Fix

Cleanup officialbuild warnings (#12578)

* Cleanup officialbuild warnings

* Re-enable "sync test native binaries" step

Merge pull request #12619 from parjong/fix/merge_jit_and_engine

Fix missing references in FEATURE_MERGE_JIT_AND_ENGINE

Merge pull request #12637 from parjong/fix/skipcrossgen

Add skipcrossgen option

Merge pull request #12575 from wateret/armel/floatret

[RyuJIT/armel] Move returned float to float reg

Add skipcrossgen option

Support interprter build when INTERP_TRACING is not set (#12625)

Remove unused global variable (#12620)

Hide methods in IdDispenser (instead of using DacNotImpl) (#12624)

[RyuJIT/ARM32] Clear assertion for refany struct and struct type temp

- Change refany struct as like x64 and arm64
- Make struct type temp argument as GT_OBJ

Fix misprint in complex condition

Update to latest System.Memory

Update CoreClr, CoreFx to preview2-25504-01, preview2-25504-01, respectively

Delete unnecessary code (#12607)

[RyuJIT/armel] Move returned float to float reg

Returned float values are in r0(float) or r0:r1(double).
These values must be moved to proper float register after call.

Fix #12574

[RyuJIT/ARM32] Unassign double register properly

In LinearScan::tryAllocateFreeReg(), unassign a double register properly
when a found float register is a part of a double register.

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

Fix missing refernces in FEATURE_MERGE_JIT_AND_ENGINT

Merge pull request #12608 from pgavlin/FixDSE

Fix a bug in LIR dead store removal.