ignore GT_ARGPLACE before gtUseNum check. (#14233)

Add the capability to force JIT options on the SPMI command line. (#14209)

expand the grammar for jit{,2}option to -jitoption [force] key=value

[x86/Linux] Fix typo in SetDebuggerREGDISPLAYFromREGDISPLAY (#14221)

Fix typo in SetDebuggerREGDISPLAYFromREGDISPLAY: Esi -> Eax.

Merge pull request #14225 from fiigii/falsedependency

[RyuJIT] Fix bad VEX encoding to avoid false register dependency

JIT: have gtClone handle long constants (#14201)

The `Enum.HasFlag` opt needs to clone the enum values, and they
can be long constants. Make sure `gtClone` handles this case.

Closes #14149.

Merge pull request #14232 from BruceForstall/Fix14196

Fix issue with HFA altjit api

fix bad VEX.vvvv to avoid false register dependency

Merge pull request #14192 from sdmaclea/PR-ARM64-JCMP

[Arm64] Add GT_JCMP node

Merge pull request #14231 from fiigii/fixintrinsic

fix bugs from #14164

Merge pull request #14234 from dotnet/ConditionLoc

Move Condition location in dotnet-trusted-publish

Move Condition location in dotnet-trusted-publish

Fix issue with HFA altjit api

One case in the Interop\ArrayMarshalling\ByValArray\MarshalArrayByValTest\MarshalArrayByValTest.cmd
test marshals an array of strings as members of a struct. The !FEATURE_HFA
code for IsHFA() needs to check for value type before calling CheckForHFA(),
which will assert if it is not a value type.

Fixes #14196.

[Arm64] Respond to feedback

fix bug from #14164

Adding {RO}Memory<T> debugger view (#14130)

* Adding memory debugger view and fixing ToArray impl.

* Addressing PR feedback and adding span debugger display string.

* Adding link to specific VSTS instance.

* Removing redundant info in comments.

Merge pull request #14230 from dotnet/CondInjection

Condition Injection on SignType

Condition Injection on SignType

Merge pull request #14229 from dotnet/PB_Fix

Change PB_ConfigurationGroup to ConfigurationGroup

Update DotNet-Trusted-Publish.json

Fix modf and fmod casing for consistency

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

[Jit] fix DevDiv_499435 (#14179)

fix DevDiv_499435

Merge pull request #14226 from dotnet/SymbolPkgInject

Conditionally Inject signed symbol packages on SignType

Conditionally Inject signed symbol packages on SignType

[ARM64/Unix] Enable FEATURE_EVENT_TRACE (#14176)

* [ARM64/Unix] Enable FEATURE_EVENT_TRACE

* [Arm64] Remove _PlatformDoesNotSupportEventTrace

Additional parameter validation for StartNoGCRegion (#14088)

* Additional parameter validation for StartNoGCRegion

* Fix the clang build by moving some defs before the first goto

* Guard against size_t overflow when scaling allocation budgets

* Move some parameter validation for managed code, throw better exceptions on parameter validation errors

* Remove stray comma

* Address code review feedback - unify code branches that branch to the failure case. Also keep data types as uint64_t as long as possible to avoid size_t overflow

* Fix the clang build

* Code review feedback:
 1. Remove useless asserts
 2. Remove a dead code path
 3. Add a LocalGC TODO to comment

Merge pull request #13897 from hseok-oh/ryujit/ftc_arg_count

Argument register & stack size calculation for fast tail call

[Arm64] Add JCMP comments per review

[Arm64] Fix JCMP lsra binding

Merge pull request #14207 from CarolEidt/Fix14028

Handle GT_SIMD in GenTree::Compare

Merge pull request #14090 from BredPet/interface_dispatch_cell

ARM32: fix interface dispatch cell address transfer

JIT: devirtualization support for EqualityComparer<T>.Default (#14125)

Mark `EqualityComparer<T>.Default`'s getter as `[Intrinsic]` so
the jit knows there is something special about it. Extend the jit's
named intrinsic recognizer to recognize this method.

Add a new jit interface method to determine the exact type returned
by `EqualityComparer<T>.Default`, given `T`. Compute the return type by
mirroring the logic used in the actual implementation. Bail out when
`T` is not final as those cases won't simplify down much and lead to
code bloat.

Invoke this interface method when trying to devirtualize calls where
the 'this' object in the call comes from `EqualityComparer<T>.Default`.

The devirtualized methods can then be inlined. Since the specific comparer
`Equal` and `GetHashCode` methods look more complicated in IL than they
really are, mark them with AggressiveInlining attributes.

If devirtualization and inlining happen, it is quite likely that the value
of the comparer object itself is not used in the body of the comparer. This
value comes from a static field cache on the comparer helper.

When the comparer value is ignored, the jit removes the field access since it
is non-faulting. It also removes the the class init helper that is there to
ensure that the (no-longer accessed) field is properly initialized. This helper
has relatively high overhead even in the fast case where the class has been
initialized aready.

Add a perf test.

Closes #6688.

[GDBJIT] Provide full file names in generated DWARF (#14172)

* Fix compile unit info

Provide full path to source in SequencePoints for GDBJIT
Use source file and directory path for compile unit info in DWARF instead of module name.

* Add a class for building Directory Table and File Table

* Fix code inconsistencies and add comments

Merge pull request #14214 from stephentoub/quwi_preferlocal

Add ThreadPool.QueueUserWorkItem(..., bool preferLocal)

ARM32: fix interface dispatch cell address transfer

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

Use FCDECL2_VI instead of FCDECL_VV (#14215)

Permit to disable FEATURE_EVENT_TRACE on Tizen (#14217)

Remove useless variable

Remove compArgRegCount and compFlatArgRegCount in Info struct

Fix typo

Fix typo in lclvars.cpp and register_arg_convention.h

Enhancement caller register & stack calculation for fast tail call

Save caller register count & stack size using argument varDsc initialization
It remove duplicate calculation for register count
and add stack size count in varDsc initialization

[RyuJIT/ARM32] Fast tail call: Init LSRA to use R12 to save jump target (#14184)

* [RyuJIT/ARM32] Fast tail call: use R12 to save jump target

To save jump target of fast tail call on ARM32, we use r12.

* Use pre-defined mask and fix comment

Use pre-defined mask: RBM_R12
fix comment: mov pc -> br

Add ThreadPool.QueueUserWorkItem(..., bool preferLocal)

Merge pull request #14178 from stephentoub/async_avoid_delegate

Avoid async method delegate allocation

Merge pull request #14197 from rartemev/fixed_test_build_restore

Fixed Linux test build

Handle GT_SIMD in GenTree::Compare

Fix #14028

Add SetThreadIdealProcessorEx PAL API (#13606)

* add SetThreadIdealProcessorEx PAL API

* fix lpPreviousIdealProcessor

Update linux-performance-tracing.md (#14194)

Add documentation on how to resolve framework symbols on Linux performance traces.

Enable event tracing for Tizen. (#14137)

Fix for a special CoreRT helper CORINFO_HELP_JIT_PINVOKE_BEGIN (#14147)

Merge pull request #14202 from wtgodbe/SigningDisableInstall

Install signing plugin with Real signing

Install signing plugin with Real signing

[Arm64] Fix GT_JCMP lsra

Replace Contract.Assumes and Contract.Assert with Debug.Assert/Debug.Fail (#14136)

* Remove use of Contract.Assert and Contract.Assumes in favor of Debug.xx

* Remove dead IA64 blocks

* Remove use of Contract.Requires in favor of Debug.Assert

* Remove Contract.EndContractBlock()

* Dead comments

* Straggler EndcontractBlock

* Remove all Contract.Ensures

* Remove [Pure] attribute and using statements

* Remove using statements for M.D.Contracts.Internal.Contract

* Rmove CA suppressions for Contracts

* Remove M.D.Contracts.Internal stub

* Comments and extra using

* Revert accidentallly removed #if

* Unix build

* Merge Math*

* Fix assert not updated since desktop

* Asserts add no value: remove

* Avoid infinite recursion in StringBuilder assert

* Remove asserts that should be public parameter validation

* Fix comment

* Remove blank line after open curly

Fixed Linux test build

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

[Arm64] Add GT_JCMP node

Create new node type GT_JCMP to represent a
fused Relop + JTrue which does not set flags

Add lowering code to create GT_JCMP when
Arm64 could use cbz, cbnz, tbz, or tbnz

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

Avoid async method delegate allocation

Previously when a task-returning async method would yield for the first time, there would be four allocations: the task, the state machine object boxed to the heap, a context "runner" object, and a delegate that points to the boxed state machine's MoveNext method.  A recent PR changed this to avoid the separate box object and the runner, but that still left the task and the delegate.

This PR avoids the delegate as well in a common case.  For async methods that only ever await Task/Task`1, that aren't using a custom sync context/scheduler, and for which tracing isn't enabled, we know the inner workings of both the builder and the awaiter and can thus bypass the awaiter's pattern APIs; instead of creating the delegate that gets passed to the awaiter and then stored in the wrapped task's continuation slot/list, we can instead just store the boxed state machine directly in the slot/list.

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