Fix a build break.

[tfs-changeset: 1665563]

Implement a new approach for SIMD8/LONG interactions. (#12590)

SIMD8 values need to be converted to longs under a small number of
situations on x64/Windows:
- SIMD8 values are passed and returned as LONGs
- SIMD8 values may be stored to a LONG lclVar

Currently, LSRA performs some gymnastics when building use positions in
order to ensure that registers are properly allocated. This change is a
stab at a different approach: rather than pushing this work onto the RA,
lowering inserts `GT_BITCAST` nodes where necessary to indicate
that the source long- or SIMD8-typed value should be retinterpreted as
a SIMD8- or long-typed value as necessary. The RA performs one specific
optimization wherein it retypes stores of `GT_BITCAST` nodes to
non-register-candidate local vars with the type of the cast's operand
and preferences the cast to its source interval.

This approach trades slightly larger IR for some functions that
manipulate SIMD8 values for tighter code in buildRefPositions.

Merge pull request #12654 from wateret/armel/gtcopy_putargreg

[RyuJIT/armel] Do not update reg for PutArgReg copy

Merge pull request #12762 from wateret/fix-legacy-12686

[LegacyJIT/ARM] Fix RegSet assert failure

Merge pull request #12708 from YongseopKim/ryujit/arm32/add_nyi_promoted_struct_in_genPutArgStk

[RyuJIT/ARM32] Add NYI_ARM for promoted struct to genPutArgStk

Merge pull request #12741 from pgavlin/IRDocs

Update the RyuJIT IR documentation.

Address PR feedback.

[LegacyJIT/ARM] Fix RegSet assert failure

Fix RegSet assert failure while codegen for CpBlk and InitBlk

Fix #12686

[RyuJIT/ARM32] Add NYI_ARM for promoted struct to genPutArgStk

- Before solving the issue(#12657), add NYI_ARM to genPutArgStk().
- This commit's code's objective is same to the PR(#12655).
- This is from the issue(#12622).

Merge pull request #12749 from stephentoub/remove_cts_throw_switch

Remove ThrowExceptionIfDisposedCancellationTokenSource switch

Merge pull request #12752 from dotnet-maestro-bot/master-UpdateDependencies

Update CoreClr, CoreFx to preview2-25511-05, preview2-25512-01, respectively (master)

Update CoreClr, CoreFx to preview2-25511-05, preview2-25512-01, respectively

Merge pull request #12680 from CarolEidt/RefactorContain

Extract containment analysis

Merge pull request #12735 from erozenfeld/RecursiveTailCallBug

Fix for a bug in tail recursion elimination.

PR feedback

Merge pull request #12730 from briansull/remove-ibc-error-string

Remove m_pIBCErrorNameString and IBCErrorNameString()

Merge pull request #12746 from crummel/updateBuildToolsForSymbolFix

Update BuildTools to fix output path for signing.

Remove ThrowExceptionIfDisposedCancellationTokenSource switch

This is a legacy switch from desktop; we used to throw exceptions in certain cases, now we don't, and for whatever reason it was deemed an app compat issue on desktop, so a switch was added to continue throwing that exception if the switch was set.  This was all then inherited into coreclr. But there's no reason anyone would want to opt-in to the behavior on core, and the checks for it are actually measurable in scenarios that stress registering/unregistering cancellation tokens.

I'm simply deleting the switch.

Merge pull request #12732 from wateret/fix-runtestdesc

Remove duplicated description in runtest.sh

Update BuildTools to fix output path for signing.
This takes dotnet/buildtools#1603 which fixes the IntermediateOutputPath -
this is required by MicroBuild for signing.

Delete bitset iterators defines (#12642)

Delete BLOCKSET_ITER_INIT

Rename blkNum to bbNum.
We usually use bbNum (basic block number), rather than blkNum(block
number).
This change allows to grep for iterator and etc easier.

Delete BITVEC_ITER_INIT and BITVEC_INIT_NOCOPY.

Delete VARSET_ITER_INIT.

Rename some variables around varSet iterators.

Merge pull request #12729 from CarolEidt/Fix12724

Extract Lowering of SIMD12 locals

Fix typo in method-descriptor.md (#12742)

Merge pull request #12743 from pgavlin/RemoveUseDef

Remove `GTF_VAR_USEDEF`.

Fix crossgen debug directory generation problems. (#12715)

* Fix crossgen debug directory generation problems.

The first problem was that when the existing/incoming PDB debug
directory entry was a portable PDB (MinorVersion == 0x504d), the
ngen/native PDB added had the same MinorVersion indicating that
it was a portable PDB (but it never can be).

This was fixed by setting MinorVersion to 0 when creating the
ngen PDB debug directory entry.

The second problem was that the ngen PDB entry was being created
even when crossgen was run on linux/mac, etc.

The fix was to ifdef NO_NGENPDB the save ngen PDB entry code.

Merge pull request #12695 from dotnet-maestro-bot/master-UpdateDependencies

Update CoreClr, CoreFx to preview2-25510-01, preview2-25511-02, respectively (master)

Remove `GTF_VAR_USEDEF`.

Nothing sets this flag.

Fix build.sh break by poorly timed dotnet first time experience (#12727)

Depending on machine state, calling `dotnet` to dump the PGO and IBC
package version might trigger the first time experience message. This
breaks build.sh where it tries to parse the package version from `dotnet
msbuild`'s output. The fix is to disable the first time experience path
in these two invocations of `dotnet`.

Extract Lowering of SIMD12 locals

Fix #12724

Update the RyuJIT IR documentation.

The most important parts of these changes update the description of the
IR to cover HIR, LIR, and the differences between the two. Other updates
include some minor restructuring, typo fixes, rephrasings, and JIT dump
refreshes.

Update CoreClr, CoreFx to preview2-25510-01, preview2-25511-02, respectively

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.

Remove duplicated description in runtest.sh

Remove m_pIBCErrorNameString and IBCErrorNameString()

Merge pull request #12670 from parjong/fix/SOS_StackWalk

[ARM/Linux] Fix DacStackWalk

Extract containment analysis

In preparation for moving containment analysis to the 1st phase of Lowering, refactor most of it into separate methods. For some cases, e.g. block stores, it is more efficient to keep the logic integrated, as it will move to a similarly integrated place in Lowering.

Also, remove some dead/unused code for ARM.

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

[ARM/Linux] Enable DacStackWalk

For ARM/Linux, DacUnwindStackFrame failed to unwind stack frame due to
the mismatch between RUNTIME_FUNCTION and IMAGE_ARM_RUNTIME_FUNCTION_ENTRY.

ClrStack SOS command currently does not work due to this bug.

This commit fixes this mismatch and re-enables ClrStack SOS command.

[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.