Merge pull request #14627 from sdmaclea/PR-ARM64-SIMD-EMITTERS

[Arm64] Add SIMD emitters

Merge pull request #14638 from BruceForstall/FixSimdMinOpts

Allow GT_CALL as BYREF operand for SIMD intrinsics

Update profiling API status (#14644)

We've been making progress testing APIs and fixing issues. The description in this doc had gotten well out of date.

Merge pull request #14620 from CarolEidt/RegSelectFix

Fix Register selection refactor bugs

Revert "Reenable PGO on Linux Release builds" (#14645)

Merge pull request #14625 from BruceForstall/FixDynBlkAssert

Remove incorrect assert for DynBlk codegen

Rename MemoryHandle PinnedPointer to Pointer and add property HasPointer. (#14604)

Allow GT_CALL as BYREF operand for SIMD intrinsics

This is an extension of #13965: in the MinOpts case, we don't
have GT_RET_EXPR -- we have GT_CALL instead.

We now see IR like:
```
[000429] --C-G--N----              |  |  \--*  BLK(32)   simd32
[000413] --C-G-------              |  |     \--*  CALL      byref  System.Runtime.CompilerServices.Unsafe.AsRef
[000410] ------------              |  |        |     /--*  CNS_INT   int    2
[000411] ------------              |  |        |  /--*  MUL       int
[000409] ------------              |  |        |  |  \--*  CAST      int <- int
[000408] ------------              |  |        |  |     \--*  CNS_INT   int    16 Vector<T>.Count
[000412] ------------ arg0         |  |        \--*  ADD       int
[000407] ------------              |  |           \--*  LCL_VAR   int    V01 arg1
```

Whereas in the optimizing case, we see:
```
[000060] ------------              *  STMT      void  (IL   ???...  ???)
[000058] I-C-G-------              \--*  CALL      byref  System.Runtime.CompilerServices.Unsafe.AsRef (exactContextHnd=0x028FAFF8)
[000055] ------------                 |     /--*  CNS_INT   int    2
[000056] ------------                 |  /--*  MUL       int
[000054] ------------                 |  |  \--*  CAST      int <- int
[000053] ------------                 |  |     \--*  CNS_INT   int    16 Vector<T>.Count
[000057] ------------ arg0            \--*  ADD       int
[000052] ------------                    \--*  LCL_VAR   int    V01 arg1

...

[000076] --C----N----              |  |  \--*  BLK(32)   simd32
[000065] --C---------              |  |     \--*  RET_EXPR  byref (inl return from call [000058])
```

Fixes #14301

[Arm64] Add SIMD emitters

[Arm64] Add more SIMD instructions

Remove incorrect assert for DynBlk codegen

The assert specified a particular register for the `size` argument
to the memset helper call. However, it preceded the genConsumeBlockOp()
function which ensured that would be the case.

Fixes #14544

[RyuJit/arm32] Do nothing if double is on stack. (#14603)

Fix Register selection refactor bugs

Fix #14617
Fix #14618

Merge pull request #14606 from CarolEidt/Fix14591

LSRA Arm64 consistent reg sets

Merge pull request #14609 from hseok-oh/ryujit/fix_14377_linux

[RyuJIT/ARM32] Remove NYI: struct return from multi-reg GT_CALL

[RyuJIT/ARM32] Fast tail call: code generation (#14445)

* Codegen for fast tail call

Codegen call and epilog for fast tail call

* Implementation for GT_START_NONGC

This implementation removes two NYI_ARM
Code generation for GT_START_NONGC which is used to prevent GC in fast tail call

* Define fast tail call target register and mask on ARMARCH

Define REG_FASTTAILCALL_TARGET and RBM_FASTTAILCALL_TARGET on ARMARCH
Modify lsra init and codegen to use these definition

* Merge genFnEpilog

Merge genFnEpilog for ARM32 and ARM64

* Fix bug in getFirstArgWithStackSlot

Fix bug in getFirstArgWithStackSlot: AMD64 and X86

[Local GC] Add a Standalone GC loader design document (#14435)

* Add a Standalone GC loader design document

* First round of feedback:
        1. Remove some stuff from the preamble that doesn't belong in a
        design document
        2. Simplify the three-variable loading approach to just use one
        variable that contains the path of a GC to load.
        3. Clean up the wording in a few places.
        4. Remove the "state machine" section and reference to state
        machines in general.

Update comments in TieredCompilationManager (#14610)

We've continued plugging away and the status in the comment was out-of-date.

[RyuJIT/ARM32] Remove NYI: struct return from multi-reg GT_CALL

Remove useless NYI

Merge pull request #14600 from BruceForstall/FixArmBadGCInfoForCpObj

Fix ARM bad GC info for CpObj

Merge pull request #14602 from benaadams/jit-formatting

Fix jit fromatting

LSRA Arm64 consistent reg sets

tryAllocateFreeReg() uses the RegOrder array to iterate over the available registers. This needs to be consistent with the available registers of the given type. Otherwise, allocateBusyReg() will assert when it finds a free register that should have been allocated in tryAllocateFreeReg().
Fix #14591

Fix jit fromatting

Fix ARM bad GC info for CpObj

In the case where the dst lives on the stack, after the first
gcref/byref was copied, we never set the type back to non-GC
for subsequent copies using the same temp register.

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

Merge changes from TFS

Merge pull request #14576 from fiigii/knobs

Add configuration knobs to control "IsSupported" and fix a bug

Make ContractException public in impl assembly to support corert typeforwarding (#14589)

* Make ContractException public in impl assembly to support corert typeforwarding

Fix condition for mustExpand flag (#14590)

The flag was getting overwritten when both CORINFO_FLG_INTRINSIC and CORINFO_FLG_JIT_INTRINSIC are set

fix clang format complaint

[tfs-changeset: 1678578]

Add configuration knobs to control IsSupported

Fix Desktop build break - warning for unreachable code

[tfs-changeset: 1678568]

Fix warning: potentially uninitialized local variable 'recentAssignedRef' used

[tfs-changeset: 1678567]

Merge pull request #14582 from briansull/cleanup-iconhdl

Cleanup IconHandle nodes

Merge pull request #14570 from sdmaclea/PR-ARM64-FLAG_FEATURE_SIMD

[Arm64] Support flag FEATURE_SIMD

[RyuJIT/ARM32] Support for CFI unwind info (#14447)

* [RyuJIT/ARM32] Support for CFI unwind info

Use ascii * instead of Chinese * character. (#14584)

Currently we have Chinese `*` characters at `src\classlibnative\bcltype\number.cpp` L289-294. That would cause a `C4819: Non-ASCII character in source` warning when building on Windows via Visual Studio (version: 15.4.0 (msvc 19.11.25547)).

This PR replaces the Chinese * characters with ascii *.

Fix #14556

Merge pull request #14534 from CarolEidt/RegSelectCleanup

Refactor register selection heuristics

Fix hill climbing float overflow (#14505)

Fix hill climbing float overflow

- When hill climbing finds that it wants to decrease the thread count but can't because the thread count is already the minimum, it instead tries to increase the sampling interval by a factor of up to 10 depending on how much it wanted to decrease the thread count
- The ratio was being used incorrectly (used max instead of min), and sometimes the ratio can be so large that the conversion to int after the float math overflows
- If something in the process enabled floating point exceptions, it may also crash
  - There doesn't appear to be a clean way to disable hill climbing, added a config variable that disables it in case a workaround is necessary for some other reason in the future
- Fixed to avoid overflow in the math to what was probably intended
- There may be another bug in GetWaveComponent() that causes values of such high magnitude to be generated, I'll leave that investigation for when that in particular becomes a real issue

Merge pull request #14573 from CarolEidt/Fix13751

Fix multiple issues in LSRA for TYP_DOUBLE:

JIT: optimize stelem (ref) like we optimize stelem.ref (#14580)

In particular, avoid the array store check when storing null or values from
the same array.

Addresses first part of #14574.

Removed hnd1 and hnd2 arguments to gtNewIconEmb*HndNode methods

Clang format fixes

Ifdef out legacy uses of GT_ASG_op (#14384)

* Ifdef out legacy uses of GT_ASG_op

GT_ASG_op nodes are only generated when the legacy backend is used.

* Address feedback

* Cleanup gtOverflow/gtOverflowEx

Removed unused handle args
from gtNewIconHandleNode and gtNewIcomEmbHndNode

Fix BinaryWriter/Reader span parameter names (#14577)

Per API review, they should match the corresponding overloads' existing parameter names.

Allow both CORINFO_FLG_INTRINSIC and CORINFO_FLG_JIT_INTRINSIC to be set on potential intrinsics (#14566)

* Allow both CORINFO_FLG_INTRINSIC and CORINFO_FLG_JIT_INTRINSIC to be set on potential intrinsics

* Call getIntrinsicID for CORINFO_FLG_INTRINSIC only

Removed unused fields of Icon nodes
The union of gtIconHdl1, gtIconHdl2 and gtIconCPX, gtOIconCls is not used

Merge pull request #14572 from sdmaclea/PR-ARM64-SIMD-INSTRS

[Arm64] Add instructions needed by SIMD

Fix multiple issues in LSRA for TYP_DOUBLE:
- updateAssignedInterval() needs to handle the case where the previously assigned interval was TYP_DOUBLE.
- Similarly, at block boundaries, if we are setting the register for a double interval, and the assigned interval is either null or is TYP_FLOAT ,we also need to unassign the other half of the register.
- LSRA only considers the valid double registers when allocating. When setting the candidates for the source of a return, it should only set the bit for the valid double register.

Fix #13751

[Arm64] Support flag FEATURE_SIMD

[Arm64] Add instructions needed by SIMD

Merge pull request #14569 from wtgodbe/eqNe

Replace ne with eq in publish to azure steps

Replace ne with eq in publish to azure steps

Merge pull request #14554 from adiaaida/reenablePGOLinux

Reenable PGO on Linux Release builds

Improve thread statics performance (#14560)

- Disable code to handle multiple appdomains
- Use more efficient object array accessor

Update BuildTools, CoreClr, CoreFx to prerelease-02118-01, preview1-25818-02, preview1-25818-01, respectively (#14558)

Merge pull request #14547 from CarolEidt/Fix14539

Arm64: Use op2 type for LOCKADD

format spmi sources. (#14545)

format spmi sources.

Merge pull request #14553 from briansull/fix-warn

Fix for x86 desktop build break

Arm64: Use op2 type for LOCKADD

Fix #14539

Merge pull request #14555 from wtgodbe/AzureBlobSymPkg

Use C#-friendly path for indexing symbol packages

Use C#-friendly path for indexing symbol packages

Reenable PGO on Linux Release builds

Now that all the machines have been updated to have llvm 3.9 with PGO
support, reenable PGO builds.

Fix for x86 desktop build
Fix warning C4389: '==' : signed/unsigned mismatch

Merge pull request #14541 from stephentoub/taskbeginwait

Avoid TaskWaitBegin/End events in Task.Wait on already completed task

GetEnvironmentVariable: Avoid StringBuilder marshaling overhead (#14502)

* GetEnvironmentVariable: Avoid StringBuilder marshaling overhead

Try a stack allocated buffer first, then fallback to using the shared
array pool.

CLRLifoSemaphore cleanup (#14535)

- Removed volatile loads. They don't actually help with anything. Should help a bit on arm, haven't tested.
  - I had added them initially out of pattern and could have left them out. There was some concern before that without a volatile load a compiler could replace local uses with a memory load and that would change the meaning of what is intended, but that doesn't apply to these kind of loops because it would be incorrect to do so, the concern may apply to loops like the following:
    ```c#
    while(true)
    {
        Counts counts = m_counts;
        if(InterlockedCompareExchange(&m_counts, counts + 1, counts) == counts)
            break;
    }
    ```
    Where if the 3rd argument is replaced by the compiler with m_counts, it may yield an incorrect result. It's an invalid optimization but anyway that can't be done in the type of loops used in this code because the initial value used for subsequent loop iterations is the result of the compare-exchange operation and not the memory location.
- No need to decrement the count of waiters woken upon timeout. I had initially copied that part of the code from SemaphoreSlim, which needs it because it uses a Monitor that doesn't provide the same guarantees as the wait objects used here. It's not needed here.
- No change to perf on x64, I plan on testing arm as part of issue https://github.com/dotnet/coreclr/issues/14067 once I have some time and get a machine.

Avoid TaskWaitBegin/End events in Task.Wait on already completed task

We are unnecessarily firing the TaskWaitBegin/End EventSource event when the task has already completed if it's faulted/canceled.  We should only fire it if the task hasn't completed by the time we check.

Merge pull request #14493 from alpencolt/armel-cross-build

[RyuJIT/armel] cross build armel on x86 host

Merge pull request #14334 from BruceForstall/AddTestEnvToWindows

Fix corefx testing invocation

move ReplaceWith to cpp

to make DEBUG_DESTROY_NODE visible.

Refactor CopyFrom

add check that each tree in the method has its own unique gtTreeId.

extract fgDebugCheckStmtsList

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

Update CoreFx to preview1-25817-02 (master)

Merge pull request #14311 from ViktorHofer/SerializationExceptions

Make coreclr exceptions serializable and add typeforwards

Update CoreFx to preview1-25817-02

[RyuJIT/ARM32] Add the optimization case on CreateDictionaryLookupHelper (#13933)

* Implement optimization case for CreateDictionaryLookupHelper

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>
* Reenable mainv1/mainv2 tests

[RyuJIT/armel] cross build armel on x86 host.

Compile ARM soft-fp version of libclrjit.so on x86 platform.
For running cross-jit execute as:
COMPlus_AltJit=* COMPlus_AltJitName=libprotononjit.so ./corerun ...

protect strlen from nullptr (#14533)

Merge pull request #14350 from CarolEidt/LsraInfoCleanup

Cleanup of Lowering & LsraInfo

Merge pull request #14527 from stephentoub/timer_partition

Reduce Timer lock contention

Refactor register selection heuristics

This is in preparation for tuning these for both throughput and code quality.

Update BuildTools to prerelease-02116-01 (#14530)

Merge pull request #14529 from adityamandaleeka/r2r_exception_race

Fix deadlock in ReadyToRun scenarios between EH and code heap deletion

Merge pull request #14501 from briansull/FitsIn

Use FitsIn8 when setting GT_INT_CNS costs

Fix optEarlyPropRewriteTree (#14480)

Return ans use the right tree from optEarlyPropRewriteTree
Forbid large dst nodes.

* fix typo

Fixed stack probing for RHEL 6.9 (Linux kernel 2.6.32) (#14481)

Fixed stack probing for RHEL 6.9 (Linux kernel 2.6.32) (#14481)

Updated GenTree::IsIntCnsFitsInI32() tio use FitsInI32()

Fix deadlock caused by R2R method info lookups during code heap
deletion.

Fix stepping with tiered jitting

Fix #14426. Added a more flexible IL master breakpoint that can:
   a) bind to native offset 0 of each jitted code body
   b) use a MethodDescFilter so that only jitted code for one generic instance receives breakpoints

The remaining change is simply to switch step-in and trace stepping to use that new breakpoint binding behavior instead of the code version specific binding behavior that was used before. This ensures that even if the code version changes after the trace occurs we will still have a breakpoint on the alternate code versions and complete the stepping operation.

Merge branch 'master' into FitsIn

Merge pull request #14522 from dotnet/wtgodbe-patch-1

Parameterize PublishFlatContainer in x86 build

Review feedback changes

spill GT_RET_EXPR inside fat calli candidates. (#14520)

spill ret expr

Delete dead code (#14521)

PR Feedback

Optimize native build of coreclr repo - 3x less memory , 11% build time improvement  (#14509)

* Fix native build overparallelism

* Fix native build parallelization problems

* Allow overriding all build workaround parameters

* Fix cleanup ommisions and local environment variable naming

* Address code review feedback

* Fix nul spelling and add standard message prefix

* Simplify settings for CL compiler parallelism

* Address code review feedback - remove whitespace

Address PR feedback

Adjusting field name changes in native code