Merge pull request #14287 from tannergooding/benchmarks-game

Adding the new fastest mandelbrot implementation to benchmarks-game.

Delete AllTestProjects.sln (#14894)

Fixes #14074

Temporarily add back old TryParse signatures (#14893)

My previous change updated the primitive TryParse signatures based on
the new design, but until corefx consumes that change, live corefx bits
won't work with the latest coreclr bits.  Until it can be consumed, I'm
putting back the old signatures as well.

Merge pull request #14415 from mikedn/rc-cleanup

RangeCheck cleanup

Merge pull request #14891 from CarolEidt/Fix14626

ARM64: Fix two register selection issues

Add test case for jit dup stloc optimization (#14803)

See #14784 for details.

Fix the flat publishing to blob feed

Delete OldStyleClearD. (#14874)

* Delete OldStyleClearD from flowgraph

BlockSetOps::Assign requires sets to have the same size, there is no an
additional risk to use ClearD then.

* delete OldStyleClearD from regalloc

regalloc doesn't create new local variables and doesn't change epoch.

* Delete OldStyleClearD from copyprop

CopyProp doesn't create new vars.
Also `VarSetOps::Assign(this, compCurLife, block->bbLiveIn); ` before
the loop requires epoch to be the same.

* Delete OldStyleClearD from assertionpop

Assertion prop doesn't change epoch.

* Delete OldStyleClearD from the emmit

Because it doesn't create new local vars.

* Delete declarations

Merge pull request #14854 from sdmaclea/PR-ARM64-Slot-Immediate-Size

ARM64 slot access immediate register size

Merge pull request #14888 from briansull/fix-break

Fix build break

Merge pull request #14281 from mikedn/mixed-type-relop

Cleanup some mixed type relops the JIT produces

ARM64: Fix two register selection issues

On ARM64 IP0 and IP1 are not in the register selection order, though there are some cases where they must be allocated. See #14607. So we may see them as free when looking for a register to spill.
Also, V15 was missing from the selection order (V16 was in the order twice).

Fix #14626

Update CoreFx, PgoData to preview1-25906-03, master-20171105-0123, respectively (#14865)

Fix TryParse overloads using optional arguments (#14877)

When we originally added Parse and TryParse overloads for `ReadOnlySpan<char>`, we used optional arguments to minimize the number of new overloads needed.  This worked decently well for Parse, but for TryParse it necessitated reordering the parameters from the string counterparts, so that the out result argument would come before the optional parameters.  This makes it more difficult to port string-based calls to span-based calls, and we agreed to add the missing overloads so that we could order the parameters "correctly" to match the existing string-based overloads.  This does so.

Shorten delay in some monitor tests to avoid timeout (#14852)

Shorten delay in some monitor tests to avoid timeout

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

Fix build break

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

RangeCheck cleanup

* Remove unused function paramter `stmt`
* Remove unused function parameter `path`
* Remove unused function parameter `block`
* Use GenTreeLclVarCommon* instead of GenTree* where possible
* Track RangeCheck memory allocations
* Remove useless noway_asserts from MergeEdgeAssertions
* Stop passing the search path via a parameter
* Eliminate unnecessary search path lookup
* Cleanup some local variable uses
* Remove redundant hashtable lookup
* Remove unused keSsaVar
* Pass the actual binary node to BinOp functions
* Fix broken assert
* Remove unused keBinOp
* Remove redundant check

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

[Arm64] Prefer EA_PTRSIZE for addresses

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

[Arm64] emitIns_R_S imm size

[Arm64] emitIns_S_R imm size

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