Fix GC reporting for slow tail call arguments of type `Span<T>` (#14826)

Fix GC reporting for slow tail call arguments of type `Span<T>`

Fixes https://github.com/dotnet/coreclr/issues/9032:
- Refactored by-ref-like method table walking to find offsets of by-ref pointers in siginfo.hpp/cpp
- Reused that for appending GC layout when creating the copy-args helper for a slow tail call

Merge pull request #14024 from mikedn/long-cast-comm

Move genLongToIntCast call to codegenlinear

Merge pull request #14735 from mikedn/emit-const-cleanup

Cleanup const data emission

Cleanup BCLDebug (#14868)

- Delete BCLDebug and related types since it was replaced by the public System.Diagnostic.Debug
- Preserve commented out or conditionally compiled logging in a few place where it seemed potentially useful

Fixes #11389

Reenable PGO optimization on release builds (#14885)

Merge pull request #14855 from briansull/improved-jitdump

Improved the dump when fgOptimizeBranch clones statements

Add .NET Core 2.0 performance post (#14875)

Allow any string length in Environment.SetEnvironmentVariable (#14872)

Fix for CoreFX #16766

Merge pull request #14873 from BruceForstall/ImproveCorefxAltjitTesting

Improve altjit corefx testing

Merge pull request #14506 from mikedn/alloc-mess

Streamline JIT memory allocation

Merge pull request #14871 from mikedn/vn-dead-divmod

Remove unused VNF_DIV_UN and VNF_MOD_UN

Merge pull request #14840 from CarolEidt/ShrinkLSRADump

LSRA Dump: don't print unused regs

Merge pull request #14857 from sdmaclea/PR-ARM64-SIMD12-IND

[Arm64] SIMD12 Indirect Load/Store

Improve altjit corefx testing

For altjit, we can't run managed code from .NET installs that
are not under test. The XSLT Compiler tests invoke an XSLT
Compiler from the machine installed desktop framework. When it
sees the AltJit variables, but no altjit compiler dll, it dies.
So, don't run those tests. They have been annotated with a new
attribute to allow this exclusion.

Remove unused VNF_DIV_UN and VNF_MOD_UN

Integer division operators do not use GTF_UNSIGNED. There are distinct unsigned operators (GT_UDIV and GT_UMOD) and VN already handles those directly.

Limit min threads in thread pool to 1 (#14864)

Related to https://github.com/dotnet/coreclr/issues/14239

Merge pull request #14851 from CarolEidt/Fix14372

Unassign both floats if assigning double

[Infrastructure] Use MSBuild v15 extension point to control CL parallelism, abstract MSBuild /maxcpucount switch (#14578)

This commit frees -ExtraParameters and uses MSBuild v15 extension point to control CL compiler parallelism.

If MSBuild 15.0 is available - installed with Visual Studio 2017 - CL parallelism control is achieved by using extension point in Microsoft.Common.props file which allows to include Directory.Build.props - for details see [Microsoft.Common.props] [1] file in MSBuild repo. https://github.com/Microsoft/msbuild/blob/b38e4ceeaaec36c5237ae698041e9b9f18c84876/src/Tasks/Microsoft.Common.props#L36.

MSBuild parallelism is controled via abstracted /maxcpucount command line switch
which now is available as -MSBuildNodeCount pass through run.cmd/run.sh command line
argument with default value equal to /maxcpucount (what defaults to number of logical processors).
This allows to control MSBuild parallelism on all supported platforms.

It is possible to control MSBuild parallelism programatically via MSBuild public
API using MaxNodeCount properties available on the following APIs:

Microsoft.Build.Evaluation.ProjectCollection.MaxNodeCount
Microsoft.Build.Execution.BuildParameters.MaxNodeCount
It's use in current build infrastructure would require creating unnecessary
complexity and was ruled out.

Merge pull request #14859 from justinvp/rom

ReadOnlyMemory: Fix copy/paste mistake in doc comment

Fix perf-prep for throughput (#14845)

There is a missing ! before checking to see if the throughput benchmarks
have already been downloaded. Currently, we download them if they
already exist, where we want to download them if they don't already
exist. This change fixes that.

Print duration to xunit output from `runtest.sh` (#14489)

Merge pull request #14858 from jashook/turn_off_publish_coreclr

[BuildPipeline] Turn off publishing to myget

[BuildPipeline] Turn off publishing to myget

Do not publish to myget, unless the 'myget' PB_PublishType
is passed.

Merge pull request #14849 from sdmaclea/PR-ARMARCH-genRangeCheck-gtThrowKind

[Arm64/Arm] genRangeCheck gtThrowKind

ReadOnlyMemory: Fix copy/paste mistake in doc comment

Formatting

[Arm64] SIMD12 Indirect Load/Store

Unassign both floats if assigning double

If the even half is unassigned, or a float, we need to check and unassign the odd half if it is assigned.

Improved the dump when fgOptimizeBranch clones statements

Update BuildTools, CoreFx, PgoData to prerelease-02203-01, preview1-25902-08, master-20171103-0222, respectively (#14834)

LSRA Dump: don't print unused regs

Also, eliminate the non-terse dump mode.

[Arm64/Arm] genRangeCheck gtThrowKind

Merge pull request #14836 from stephentoub/valuetaskinterfaces

Move I{Configured}ValueTaskAwaiter interfaces to correct location

Merge pull request #14824 from CarolEidt/Fix14798

Fix copyOrMoveRegInUse

Mark applicable structs as readonly (#14789)

In a few cases (e.g. nullable), I added readonly to fields in order to allow readonly on the type.

Merge pull request #14838 from BruceForstall/Fix14551

Handle TryGetUse case with GT_PUTARG_SPLIT of GT_FIELD_LIST

Merge pull request #14841 from BruceForstall/FixArm64Build

Fix arm64 build

Fix arm64 build

to have broken the arm64 build. Fix one file by removing non-ASCII
characters.

fix (#14837)

GTF_RET means that this flag is valid only for GTF_RET node, but this check was missed.
The issue was in collision between GTF_RET_MERGED and GTF_CALL_UNMANAGED.

Merge pull request #14801 from briansull/newfix

Fixes Assertion failed 'firstILBlock->hasProfileWeight()

Handle TryGetUse case with GT_PUTARG_SPLIT of GT_FIELD_LIST

Move I{Configured}ValueTaskAwaiter interfaces to correct location

They need to be in the shared partition.

JIT: Merge legacy inlining policies (#14815)

Merge the LegacyPolicy and EnhancedLegacyPolicy into a unified policy that
behaves like the EnhancedLegacyPolicy. Rename this policy to the DefaultPolicy
since it is in fact the default inline policy.

We had been keeping the LegacyPolicy around in case we ever needed to revert
back to the initial RyuJit inline behavior, but that safeguard no longer seems
necessary.

Remove some of the checks in flowgraph.cpp that alter behavior based on policy
as they are no longer needed.

Remove the jit config setting that allowed selection of the LegacyPolicy.

This is the first stage in fixing #14441.

Allow coredumps inside docker (#14814)

Merge pull request #14811 from jashook/signing_var_changes

Orchestrated build Config signing.

Update BuildTools, CoreClr, PgoData to prerelease-02202-02, preview1-25901-06, master-20171102-0019, respectively (#14766)

Orchestrated build Config signing.

Sign if Real or Test is passed.

Merge pull request #14796 from sdmaclea/PR-SIMD-lower-bounds-test

Add SIMD get_Item lower bounds test

Merge pull request #14809 from sdmaclea/PR-ARM64-genSIMDIntrinsicGetItem

[Arm64] SIMDIntrinsicGetItem

Merge pull request #14820 from sdmaclea/PR-ARM64-SIMDIntrinsicRelOp

[Arm64] genSIMDIntrinsicRelOp

Merge pull request #14827 from jashook/wrong_eq

Invert eq

Invert eq

Remove invalid assert in CLRLifoSemaphore (#14805)

After https://github.com/dotnet/coreclr/pull/14535, the assert is no longer valid. A thread that times out does not decrement the count of waiters signaled to wake because a timed-out thread does not observe a signal to the object that was waited upon. If there were no other waiters to observe the signal, the count of waiters signaled to wake can become greater than the waiter count. This is valid, and tracks how many signals are pending. I don't think there is a good way to update the assert to reflect this, removed the assert.

[Build Pipeline] Add ability to skip testing (#14813)

Merge pull request #14804 from dotnet/tfUrlUpdate

TransportFeed URL update

Switch to /utf-8 for MSVC (#14822)

* Add /source-charset:utf-8 to force MSVC to compile source as UTF-8.

* Fix the illegal character in siginfo.cpp.

Fix copyOrMoveRegInUse

When checking whether a `copyReg` or `moveReg` is still in use, only consider it in use if it is assigned to the register we care about.
We can have strings of `copyReg`s to different registers, especially in stress modes.

Fix #14798

Merge pull request #14797 from BruceForstall/FixBuildCmdPriorityHandling

Fix handling of `-priority=N` argument in build.cmd

Fixes Assertion failed 'firstILBlock->hasProfileWeight()

When creating a new top block we need to use inheritWeight to properly initialize the the block weight and the profile data flag
The fix now handles multiple predecessor blocks that each contribute profile weights to the newT block.

[Arm64] genSIMDIntrinsicRelOp

Handle SIMD12 special case

[Arm64] SIMDIntrinsicGetItem

Fix srcCount
Fix item containment
Implement [Arm64] genSIMDIntrinsicGetItem w/ non const index
Handle out of range const index

Move Dictionary to shared CoreLib partition (#14795)

Add perf pipeline job full groovy (#14397)

This change converts our perf testing to use pipeline jobs. Pipeline
jobs allow us to do the following:

1) Test on the same commit for each of the test legs
2) Parallelize the build and test steps.
3) Separate the build and test steps from one another. This gives us the
ability to use the same build assets for all of the test legs of the
same configuration. It also allows us to build on virtual machines and
test on perf machines, so we only use the perf resources for testing.
4) Have different test scenarios for PRs and rolling. This isn't
strictly a benefit of pipeline jobs, but certainly is made easier by
them.
5) Allows us to have one trigger for PR jobs which will get us all the
perf testing scenarios.

This change also cleans up the groovy scripting for perf testing.

Merge pull request #14791 from sdmaclea/PR-ARM64-impSIMDSelect

[Arm64] Fix impSIMDSelect

Fix sync's location. (#14817)

Merge pull request #14808 from sdmaclea/PR-ARM64-genSIMDIntrinsicDotProduct

[Arm64] Fix genSIMDIntrinsicDotProduct

Merge pull request #14812 from BruceForstall/AddPeriodicAltjitTriggers

Enable periodic triggers for `x86_arm_altjit` and `x64_arm64_altjit` jobs

[Local GC] Fix (another) ScanContext layout issue when building without FEATURE_REDHAWK (#14777)

Enable periodic triggers for x86_arm_altjit and x64_arm64_altjit jobs

These jobs are getting clean enough where having them regularly run
will be valuable.

JIT: fix overly aggressive box-unbox.any assert (#14799)

A recently added assert was trying to ensure that `compareTypesForEqualty`
caught all the box-unbox.any cases we got before by direct handle comparison.
But there are cases where identical handles cannot resolve equality
comparisons, eg when the handles represent shared types like __Canon, and
the assert was not screening for those.

Seems simplest just to remove the assert, it was really only of value when
coding up the new jit interface method.

Closes #14780.

JIT: convert fixed-sized locallocs to locals, enable inlining (#14623)

Optimize fixed sized locallocs of 32 bytes or less to use local buffers,
if the localloc is not in a loop.

Also "optimize" the degenerate 0 byte case.

Allow inline candidates containing localloc, but fail inlining if any
of a candidate's locallocs do not convert to local buffers.

The 32 byte size threshold was arrived at empirically; larger values did
not enable many more cases and started seeinge size bloat because of
larger stack offsets.

We can revise this threshold if we are willing to reorder locals and see
fixed sized cases larger than 32 bytes.

Closes #8542.

Also add missing handler for the callsite is in try region, this was
an oversight.

[Arm64] Fix genSIMDIntrinsicDotProduct

Revert sync test sync binaries (#14807)

Updating xUnit Benchmark Scenarios to capture Etw with Pmc support. (#14734)

- Adding 'tiered' jitting
- Enable JitBench to capture Pmc data (Profile=On)
- Fixed some xUnit performance warnings
- Enforce DOTNET_MULTILEVEL_LOOKUP environment to be turned off
- Specify `dotnet publish` output directory.

Publishing to the transport feed main feed

Supporting publishing to the main feed where the relative path is empty

Merge pull request #14792 from sdmaclea/PR-SIMD-negative-get_Item

SIMD Handle negative indicies in get_Item

Merge pull request #14788 from sdmaclea/PR-ARM64-9b-INSTR

[Arm64] Instruction requires 9 bits now

Merge pull request #14781 from fiigii/x86reg

[RyuJIT/X86] Limit byte register candidates for CRC32

TransportFeed URL update

Updating URL to point to the new main feed

[RyuJit/ARMARCH] lower arg with list of float fields. (#14753)

apply bitcast on reg fields
rename ReplaceArgWithPutArgOrBitcast

Merge pull request #14761 from hseok-oh/ryujit/fix_13746

[RyuJIT/ARM32] Fix double type PutArgStk check bug on LSRA init

Fix handling of `-priority=N` argument in build.cmd

This argument is intended to be passed through directly to msbuild,
but CMD doesn't handle `=` well, so using `-priority=N` with
`build all` didn't work, for instance.

Add special handling for the `-priority` argument to get around
this problem.

Fix (#14778)

skip commas and nops.

Add SIMD get_Item lower bounds test

Merge pull request #14779 from fiigii/fixclone

Fix #14773

Merge pull request #14629 from sdmaclea/PR-ARM64-SIMD-CODEGEN

[Arm64] Add SIMD codegen

Merge pull request #14616 from sdmaclea/PR-ARM64-SIMDINTRINSICLIST

[Arm64] Implement simdintrinsiclist.h

[Arm64] Fix impSIMDSelect

Merge pull request #14790 from sdmaclea/PR-ARM64-insEncodeDatasizeLS

[Arm64] Revise insEncodeDatasizeLS

SIMD Handle negative indicies in get_Item

[Arm64] Revise insEncodeDatasizeLS

[Arm64] Instruction requires 9 bits now

Update URLs to use for formatter job (#14785)

Need 2.0 SDK now. Not sure of the best URLs but these seem to work.

Also shorten path so we don't run into windows pathlength issues.

Limit byte register candidates for CRC32

Merge pull request #14767 from CarolEidt/Fix14373

Fix checking for reg in use

Remove intermediate sync for test build (#14775)

Fix slot size for transformed long

Fix slot size as 2 for transformed long from double
Use expected slot size for assertion check

fix #14773

PR feedback & fix an error in `isRegInUse`

Merge pull request #14705 from hseok-oh/ryujit/fix_14666

[RyuJIT/ARMARCH] Use ActualType for PutArgStk size