Merge pull request #14195 from wtgodbe/SignType

Use Real signType to install plugin

Use Real signType to install plugin

Merge pull request #14193 from wtgodbe/SigningInfra

Update publish infra to parameterize PB_SignType

Update publish infra to parameterize PB_SignType

Remove Monitor asm helpers (#14146)

- Removed asm helpers on Windows and used portable C++ helpers instead
- Rearranged fast path code to improve them a bit and match the asm more closely

Perf:
- The asm helpers are a bit faster. The code generated for the portable helpers is almost the same now, the remaining differences are:
  - There were some layout issues where hot paths were in the wrong place and return paths were not cloned. Instrumenting some of the tests below with PGO on x64 resolved all of the layout issues. I couldn't get PGO instrumentation to work on x86 but I imagine it would be the same there.
  - Register usage
    - x64: All of the Enter functions are using one or two (TryEnter is using two) callee-saved registers for no apparent reason, forcing them to be saved and restored. r10 and r11 seem to be available but they're not being used.
    - x86: Similarly to x64, the compiled functions are pushing and popping 2-3 additional registers in the hottest fast paths.
    - I believe this is the main remaining gap and PGO is not helping with this
- On Linux, perf is >= before for the most part
- Perf tests used for below are updated in PR https://github.com/dotnet/coreclr/pull/13670

Merge pull request #14171 from wateret/ryu-arm-putarg-bitcast

[RyuJIT/armarch] Put arguments with GT_BITCAST

[RyuJIT/armarch] Put arguments with GT_BITCAST

Put arguments with GT_BITCAST instead of GT_COPY for arm32/arm64

Fix #14008

[ARM32] Update armel rootfs for Tizen (#14173)

* [Tizen/ARM32] Update armel rootfs for Tizen

- Add lttng-ust-dev to Tizen rootfs for cross build
- Use rootfs of the latest Tizen 4.0

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>
* [Tizen/ARM32] Update Tizen CI script

- Update arm32 CI script for Tizen to use the latest environment

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

Add CI PR triggers for x86_arm and x64_arm64 testing (#14158)

* Add CI PR triggers for x86_arm and x64_arm64 testing

* Fix bug

* Don't do GC stress with altjit

Merge pull request #14041 from sdmaclea/PR-ARM64-SET-FLAGS

[Arm64] Use GTF_SET_FLAGS/GTF_USE_FLAGS

[Arm64] Repsond to feedback

Cleanup gentree per comments

[Arm64] Use GTF_SET_FLAGS/GTF_USE_FLAGS

Merge pull request #14145 from BruceForstall/CleanupTestsLst

Re-enable tests that should now pass

Merge pull request #14080 from hqueue/arm/ryujit/issue_13750

[RyuJIT] Fix assertion condition for interval of constant value in LSRA

[mscorlib] Enable hardware intrinsic in debug and reflection (#14164)

[RyuJIT/ARM32] Fix assertion condition in LSRA for ARM32

Update assertion condition when handling an inactive
constant interval for ARM32, because an inactive constant
interval may be encountered.

Fix #13750

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

Moving parts of `System.Math` and `System.MathF` to be shared with CoreRT. (#14119)

* Moving parts of `System.MathF` to be shared with CoreRT.

* Moving parts of `System.Math` to be shared with CoreRT.

* Updating the new 'Round' named intrinsic to map to the legacy 'Round' intrinsic.

* Adding back the case statement for CORINFO_INTRINSIC_Round, since it is required for Desktop compat.

Merge pull request #14153 from mikedn/x86-newarr-nint

Make GitHub_14116 test work on 32 bit

Make GitHub_14116 test work on 32 bit

newarr expects int32 or native int on the stack. x64 JIT hapilly allows an int64 as well but the x86 JIT asserts.

Also change the constant to 0xFFFFFFF1FFFFFFFF so OverflowException is thrown on 32 bit as well.

Merge pull request #14143 from sdmaclea/PR-FEEDBACK-14139

Respond to Feedback #14139

[Arm64/Arm] Favor emitActualTypeSize for non-memory (#14046)

* [Arm64/Arm] Favor emitActualTypeSize for non-memory

Register ops size is generally referred to
by emitActualTypeSize.  'Actual' size refers to the
size in register as opposed to size in memory

* [Arm64] Remove bogus CQ ToDo

* Fix formatting

Re-enable tests that should now pass

Merge pull request #14109 from hseok-oh/ryujit/fix_13738

[RyuJIT/ARM32] JitStress=1 test: padding local variable

Merge pull request #14118 from rartemev/devdiv_491206

Fixed return constant value cache on 32 bit VMs

Merge pull request #14117 from mikedn/earyprop-neg-length

Don't early-propagate negative array lengths

[Arm64] Add Floor/Ceiling intrinsics (#14132)

* [Arm64] Add Floor/Ceiling intrinsics

* Templatize emitActualTypeSize

Create dotnet deployment and acquisition size on disk test (#13956)

* [WIP] First version of SoDBench. Contains bugs

* [WIP] Move SoDBench files and increase error checking

* [WIP] Add NugetConfig to enable pulling packages from myget

* [WIP] Remove unhelpful templates and add back oses

* [WIP] Add ability to specify channel

* [WIP] Improve CSV writing

* [WIP] Improve options parsing

* Fix syntax error

* [WIP] Add test leg to perf.groovy

* [WIP] Adjust label to target an existing machine pool

* Change label to run on virtual machine

* Use setMachineAffinity

* Add ASP.NET-Core feed and deafult to master as the channel

* Change channel to master in perf.groovy

* Move NuGet.Config up a directory so it only needs to be written once

* Add CommandLine to external dependencies

* Remove CommandLine as it is now in external.depproj

* Adjust wget command to work more consistantly

* Change calculation of relative paths for clarity

* Set job to run daily instead of on push/PR

* Build sodbench on job run

* Remove MSBuild from job

* Fix quote placement

* Change metadata to be more accurate

* Add rollup totals for each measured category

* Refactor to use a tree rather than a dictionary

* Limit report size

* Publish in release configuration

[Arm64] Implement JCC/SETCC nodes (#14101)

* [Arm64] Implement JCC/SETCC nodes

* Fix formatting

Respond to Feedback #14139

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

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

Added IntegralValue() method

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

Fixed test return value

Merge pull request #14126 from jashook/use_new_x64_client

Add support for priority and update x64 client

Fixed formating

Handle long value in MergedReturns optimization

Added microtest

Fixed return constant value cache on 32 bit VMs

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

Don't early-propagate negative array lengths

There's no need for that and if the negative array length is not representable in 32 bit we'll end up producing a GT_CNS_INT node that has TYP_INT and a 64 bit value.

That's because the original type (always TYP_INT) of the GT_ARR_LENGTH is preserved when changing the node to GT_CNS_INT.

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

[RyuJIT/ARM32] JitStress=1 test: padding local variable

JitStress=1 test make padding offset for local variable.
On ARM32, padding value must support stack alignment.

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