Merge pull request #13167 from alpencolt/ryu-arm-remove-lea-0-offset

[RyuJIT/ARM32] Remove LEA[b+0]

Merge pull request #14139 from sdmaclea/PR-ARM64-EMIT-CBxZ-TBxZ

[Arm64] Add emitters for cbz, cbnz, tbz, and tbnz

Merge pull request #14140 from jashook/pri0_arm_jobs

Set default arm jobs to pri0

Fix incorrect use of local var

Fix build logic as well

Disable tracing tests on Windows at runtime, not build (#14127)

FileStream should not truncate files when opening fails. Fixes dotnet/corefx#24081 (#14043)

Merge pull request #14142 from sdmaclea/PR-ARM64-FIX-TYPO

[Arm64] Fix typo

[Arm64] Fix typo

Merge pull request #14141 from jashook/fix_post_build_typo

fix arm64_post_build typo

fix arm64_post_build typo

Merge pull request #14050 from mikedn/earlyprop-range-check

Remove constant range checks in earlyprop

Set default arm jobs to pri0

Remove ResId class (#14135)

Merge pull request #14129 from BruceForstall/FixArm64TriggerString

Fix ARM64 trigger strings to match job names

[Arm64] Add emitters for cbz, cbnz, tbz, or tbnz

Fixing `FloatingPointUtils.round` to include the fix for #12137 (#14133)

Fix SIGSEGV in EventPipe on Shutdown (#14123)

* Fix a crash that occurs when a provider is registered after the configuration object has been destroyed.

* Code review feedback.

Improve thread pool worker thread's spinning for work (#13921)

Improve thread pool worker thread's spinning for work

Closes https://github.com/dotnet/coreclr/issues/5928

Replaced UnfairSemaphore with a new implementation in CLRLifoSemaphore
- UnfairSemaphore had a some benefits:
  - It tracked the number of spinners and avoids waking up waiters as long as the signal count can be satisfied by spinners
  - Since spinners get priority over waiters, that's the main "unfair" part of it that allows hot threads to remain hot and cold threads to remain cold. However, waiters are still released in FIFO order.
  - Spinning helps with throughput when incoming work is bursty
- All of the above benefits were retained in CLRLifoSemaphore and some were improved:
  - Similarly to UnfairSemaphore, the number of spinners are tracked and preferenced to avoid waking up waiters
  - For waiting, on Windows, a I/O completion port is used since it releases waiters in LIFO order. For Unix, added a prioritized wait function to the PAL to register waiters in reverse order for LIFO release behavior. This allows cold waiters to time out more easily since they will be used less frequently.
  - Similarly to SemaphoreSlim, the number of waiters that were signaled to wake but have not yet woken is tracked to help avoid waking up an excessive number of waiters
  - Added some YieldProcessorNormalized() calls to the spin loop. This avoids thrashing on Sleep(0) by adding a delay to the spin loop to allow it to be more effective when there are no threads to switch to, or the only other threads to switch to are other similar spinners.
  - Removed the processor count multiplier on the max spin count and retuned the default max spin count. The processor count multiplier was causing excessive CPU usage on machines with many processors.

fix checkLclVarSemantics (#14053)

* extract CheckLclVarSemantics from CheckLIR.

* add a test that shows the silent bad execution.

* fix the checker.

* add the test to the exclude list.

* rename consumed to used

Merge pull request #13105 from stephentoub/asyncmem

Reduce allocations when async methods yield

Merge pull request #14126 from jashook/use_new_x64_client

Add support for priority and update x64 client

Fix ARM64 trigger strings to match job names

In particular, add "Cross" to the trigger strings.

Merge pull request #14108 from jashook/spmi_get_helper

Fix superPMI getHelperFunction

Add support for priority and update x64 client

Update stub dispatch hidden argument convention for CoreRT (#14120)

[Jit] fgMorphCast does not reset GTF_ASG (#14102)

add repro for DevDiv_495792.
fix the issue.

Merge pull request #14107 from sdmaclea/PR-ARM64-REMOVE-DEAD

[Arm64] Remove LegacyBackend defines

Remove constant range checks in earlyprop

[RyuJIT] Regression test #13057

LEA[b+0] was not eliminated on ARM which cause assertion on code generation

Fix formatting

Merge branch 'master' of https://github.com/dotnet/coreclr

Merge pull request #14099 from JosephTremoulet/LiteralBase

Change base of literals in k-nucleotide-9

Fix superPMI getHelperFunction

[Arm64] Remove LegacyBackend defines

Merge pull request #14039 from OpenVisionLabs/typos

Correction of typos in several infrastructure files

Change base of literals in k-nucleotide-9

This keeps it in sync with the copies in the release branches (the
release/1.1.0 branch required this since older C# compilers don't accept
the binary literals), and improves readability.

Merge pull request #13541 from hqueue/arm/ryujit/issue_12614_enable_unrolling_for_cpblk

[RyuJIT/ARM32] enable unrolling for cpblk

Merge pull request #14089 from BredPet/switch_generation

ARM32: fix switch jump table generation bug for RyuJIT

Merge pull request #14048 from wtgodbe/RIDs

Restore packages for every Windows architecture by default

Merge pull request #14086 from sdmaclea/PR-ARM-REMOVE-OBS-ASSERT

[Arm] Remove obsolete assert

Improve instructions for installing VS2017 and building for ARM (#14073)

* Improve instructions for installing VS2017 and building for ARM

* Add section on cross-building ARM for Windows

mark argplace node as no_lir (#14044)

mark argplace node as no_lir

ARM32: fix switch jump table generation bug for RyuJIT

Signed-off-by: Petr Bred <bredpetr@gmail.com>

When building as standalone GC the free object is initialized with 0 flags.  This leads to the free list plugs not having the right size on heap traversal (often leading to a crash).   The fix is for the free object to be an array with a compenent size of 1 byte.

Merge pull request #14085 from qmfrederik/marshal-utf8-intptr-zero

Fix marshaling IntPtr.Zero to UTF8 strings, add unit tests

[Arm] Remove obsolete assert

Assert should have been removed with #14018

Fixes #14082

Reset static state per iteration for k-nucleotide-9 (#14081)

Otherwise iterations keep getting slower and slower.

Also bump inner iteration count to 10 to restore the nominal one second
duration per iteration.

16-byte Stack Aligned StubDispatchFixupStub (#14078)

Test marshaling IntPtr.Zero to UTF8 strings, add unit tests

Merge pull request #14075 from BruceForstall/AllowMoreStressModes

Refine ARM64 CI stress modes allowed

Refine ARM64 CI stress modes allowed

For JitStressRegs, allow 0x10, 0x80, 0x1000; don't use simple 10 or 80
in names (to match netci.groovy).

Remove "tieredcompilation" from allowed modes; it is not supported in the
CI smarty helper currently.

Merge pull request #14068 from JosephTremoulet/HelixExclude

Update exclusions for moved tests

Merge pull request #14069 from BruceForstall/FixFormatting

Fix jit-format error

Fix rlimit setting of RLIM_NOFILE on OSX (#14054)

This change fixes an issue with rlimit setting of RLIM_NOFILE. The problem
is that the rlim_max that we get from getrlimit is too large and so setting
the rlimit_cur to that value fails. The OSX man page for rlimit has a compat
note about it, stating that the rlimit_cur needs to be limited to
min(OPEN_MAX, rlim_max) if one wants to set it to rlim_max.

Fix jit-format error

Support PR trigger of ARM JIT stress jobs (#14055)

Add a list of jobs known to the ARM64 CI, and allow those to be triggered.

Also, allow Release triggered jobs to run tests (before, the label said
"Build and Test", but no test was done).

For the newly added jobs, make them weekly triggered, not daily triggered,
for the periodic jobs, to reduce machine usage.

Update exclusions for moved tests

Change #13994 moved some tests that were excluded from Helix runs, but
failed to update the exclusion list; fix that oversight and exclude the
tests in their new locations.

Fixes #14034.

Merge pull request #14059 from hseok-oh/ryujit/fix_13760

[RyuJIT/ARM32] Fix TreeNodeInfoInit for STOREIND with writebarrier

Move initialization of YieldProcessorNormalized to the finalizer thread (#14058)

Move initialization of YieldProcessorNormalized to the finalizer thread

Fixes https://github.com/dotnet/coreclr/issues/13984
- Also moved relevant functions out of the Thread class as requested in the issue
- For some reason, after moving the functions out of the Thread class, YieldProcessorNormalized was not getting inlined anymore. It seems to be important to have it be inlined such that the memory loads are hoisted out of outer loops. To remove the dependency on the compiler to do it (even with forceinline it's not possible to hoist sometimes, for instance InterlockedCompareExchnage loops), changed the signatures to do what is intended.

Add span-based Guid.{Try}Parse{Exact} methods (#14062)

This commit does two things:
- Changes the existing Guid.{Try}Parse{Exact} methods to work in terms of spans rather than strings.
- Then adds new overloads that accept spans and that uses the same underlying span-based support.

Due to the first change, performance actually improves for several of the guid formats, in particular due to substring allocations that are no longer incurred.  For example, this program:
```C#
using System;
using System.Diagnostics;

class Program
{
    static void Main()
    {
        var sw = new Stopwatch();
        const int Iters = 1000000;

        Console.WriteLine("Pattern\tBytes\tTime");
        foreach (string pattern in new[] { "D", "B", "P", "N", "X" })
        {
            string input = Guid.NewGuid().ToString(pattern);
            Guid.Parse(input);

            long a = GC.GetAllocatedBytesForCurrentThread();
            sw.Restart();
            for (int i = 0; i < Iters; i++) Guid.Parse(input);
            sw.Stop();
            a = GC.GetAllocatedBytesForCurrentThread() - a;

            Console.WriteLine($"\"{pattern}\"\t{a / Iters}\t{(int)(sw.Elapsed.TotalMilliseconds / Iters * 1000000)}ns");
        }
    }
}
```
on my machine previously output:
```
Pattern Bytes   Time
"D"     0       218ns
"B"     0       217ns
"P"     0       210ns
"N"     168     388ns
"X"     744     765ns
```
and with this change outputs:
```
Pattern Bytes   Time
"D"     0       213ns
"B"     0       192ns
"P"     0       196ns
"N"     0       372ns
"X"     0       577ns
```

Merge pull request #14018 from sdmaclea/PR-ARM-Remove-extra-cast

[Arm/Arm64] LowerCast remove small int widen cast

Merge pull request #13941 from sdmaclea/PR-ARM64-CMP-EXTENDED

[Arm64] Do not extend within Compare/Test nodes

Merge pull request #14063 from JosephTremoulet/Fannkuch-5

Fix expected values in fannkuch-redux-5

Restore packages for every Windows architecture by default

Fix expected values in fannkuch-redux-5

The validation logic was testing against `chksum`, which actually can
vary depending on the number of processors (as that is used to determine
the number of threads across which the work is partitioned, and the
checksum is sensitive to the bucketing).  Change it to test against
`maxflips` instead, which is stable.

Fixes #14040.

Merge pull request #14061 from JosephTremoulet/BenchmarkReadme

Update BenchmarksGames README.txt

Merge pull request #13753 from CarolEidt/FixSpillWeights

Sane spill weights

Merge pull request #14057 from ahsonkhan/UpdateProperties

Change AsSpan() to property Span and rename AsMemory to Memory

Update BenchmarksGames README.txt

Reflecting recent updates to the snapshot of these tests.

Merge pull request #14052 from stephentoub/parsenumbers_managed

Move ParseNumbers to shared partition

[RyuJIT/ARM32] Fix TreeNodeInfoInit for STOREIND with writebarrier

Set source count for GT_STOREIND with writebarrier.
It's same value with ARM64

Disable VSD Stub Kind Prediction on H/W Exception (#14051)

* Disable VSD Stub Kind Prediction on H/W Exception

* Add a comment on a false argument

* Use BOOL (instead of bool) for consistency

* Replace remaining bool/true/false with BOOL/TRUE/FALSE

Change AsSpan() to property Span and rename AsMemory to Memory

Address PR feedback

Merge pull request #14029 from mikedn/cmp-op-swap

Remove useless TODO from genCompareInt

Merge pull request #14003 from noahfalk/fix_commitstack

Fix test StackCommitCommon

Move ParseNumbers to shared partition

Remove now dead ParseNumbers code in runtime

Clean up ParseNumbers and fix perf regressions

- Fix formatting
- Replace char[] allocations with stackalloc
- Replace StringBuilders with FastAllocateString
- Tweak a few comparisons to make them leaner
- Tweak some tight loops to make them a bit leaner
- Help JIT to eliminate bounds checks on target spans
- Walk ptr destinations rather than indexing

Use the calculated size (offset) instead of estimated (size) (#14049)

Change jit notifications so that they pass the native code address.  (#14021)

* Change jit notifications so that they pass the native code address. This fixes !bpmd so that it will set the correct breakpoint on tiered jitted methods.

* code review feedback

* don't handle OnCodeGenerated

Merge pull request #13664 from BruceForstall/WinRyuArm32CI

First step to enable Windows RyuJIT/arm32 testing

Sane spill weights

The existing spill weight computation is comparing non-normalized bbWeight to normalized lvRefCntWtd, and is also using the weight of whatever the last block turns out to be (often EH zero-weight block) for the `RefTypeParamDef`s.

The straightforward fix resulted in regressions:
- It turns out that it is useful to give extra weight to tree temps, as all their references are very localized.
- Also, it is helpful to reduce the relative weight of a lclVar if it has already been spilled.

As is often the case, more tuning would be useful, but this is a more sane algorithm.

Fix #13721

Disable tests that fail in RyuJIT/arm32 testing

First step to enable Windows RyuJIT/arm32 testing

Renames arm legacy backend jobs to "armlb" and keeps "arm"
for RyuJIT/arm32 testing.

Fix RuntimeTypeHandle underlying type for CoreRT (#14026)

RuntimeTypeHandles on CoreRT are backed by IntPtr today. This fix is not complete. There are likely more places that make assumption about the underlying runtime handle types, but this is enough to fix the immediate problem hit by @sergign60.

Copy corert's ParseNumbers implementation to coreclr

Correct typos in reporoot/perf.groovy

Correct typos in reporoot/build.cmd

Correct reporoot/CMakeList.txt typos

Correct typos in reporoot/dir.props

Merge pull request #13878 from sdmaclea/PR-ARM64-LowerCompare-Cast-Const

[Arm64] LowerCompare Cast-Bool/UBYTE CMP immediate #0 to TST immediate

Merge pull request #13926 from sdmaclea/PR-ARM64-BOUND-CHECK

[Arm64] Enable Lowering GT_ARR_BOUNDS_CHECK

Merge pull request #14025 from mikedn/cmp-dead-code

Remove dead code from TreeNodeInfoInitCmp

Remove useless TODO from genCompareInt

gtSetEvalOrder already does this. And even if it doesn't it's not the job of genCompareInt to swap operands, that should happen in LowerCompare.

Attempting to do this in LowerCompare resulted in 0 diffs.

Merge pull request #13994 from JosephTremoulet/BenchmarksGame

Update BenchmarksGame benchmarks to latest

Remove dead code from TreeNodeInfoInitCmp

Mark removed tests EXPECTED_FAIL

The next update to the Tests.lst files will need to include the new
variants of these tests.

[RyuJit] fix the inconsistency between setContained and isContained. (#13991)

* show the problem with contained arg_place

We set contained on PUTARG_REG, but it doesn't pass IsContained check.

* Fix problem with gtControlExpr

* fix problem with ARGPLACE

* additional improvements1

We should never have a contained node that is the last node in the
execution order.

* additional impovement2 for xarch.

It is redundant, do not need to set as contained.

* additional improvement2 for arm

`GenTree* ctrlExpr = call->gtControlExpr;` was unused.

* additional improvement3: unify CheckLir.