Merge pull request #12580 from wtgodbe/disableSync

Disable sync.cmd -n for Windows binaries

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>

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)

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.

create GetAssertionIndex to convert from unsigned to AssertionIndex (#12554)

Fix a bug in LIR dead store removal.

This came in with the recent operand visitor changes: instead of marking
the operands of a dead indirect store unused, DSE would mark the operands
of its address unsued.

[x86/Linux] Restore PTR_MethodTable in FastPrimitiveArrayAllocator (#12569)

[RyuJIT/armel] Passing stack double arguments

Fix #11927

Move the bitset iterator for bitsetasuint64inclass inside the bitset. (#12553)

It is how it is done for others bitsets.

Improve fgAddRefPred (#12585)

* Improve fgAddRefPred

Only do one search through the predecessor list looking for an
existing predecessor edge, or looking for the proper location to
insert a new predecessor edge. Previously, we traversed the list
twice (or part of twice) for new edges. For new edges, stop the
search when we reach the correct location in sorted order.

This reduces x86 release minopts instruction count by 0.42% and
full opts instruction count by 0.33% (of a superpmi replay of
the tests).

Fixes #12582

* Update for feedback

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

Merge nmirror to master

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

Reducing diffs with CoreRT (#12579)

* Rename m_firstChar to _firstChar to reduce CoreRT diffs
* Reducing diffs with CoreRT
* Delete dead code

Merge pull request #12568 from hseok-oh/ryujit/fix_11781

[RyuJIT/ARM32] Fix using fgMorphMultiregStructArg for mkrefany

Remove unnecessary update of RegRecord in LSRA (#12474)

* Remove unnecessary update of RegRecord

We don't need to clear assignedInterval of regRec,
because it was already cleared by checkAndClearInterval() above.

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

- Add a assertion and comment for the assertion

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

[RyuJIT/ARM32] Return HFA struct (#12453)

* [RyuJIT/ARM32] Return HFA struct

- Enable return HFA struct
- Merge ARM32/ARM64 CodeGen::isStructReturn(GenTreePtr treeNode)
- Merge ARM32/ARM64 CodeGen::genStructReturn(GenTreePtr treeNode)

* Add comment

Add TODO comment to optimize two-float loading