Merge pull request #14954 from dotnet/revert-14722-arm64_windows_nt_rework

Revert "Rework Arm64 Windows jobs to flow jobs"

Revert "Rework Arm64 Windows jobs to flow jobs"

Merge pull request #14722 from jashook/arm64_windows_nt_rework

Rework Arm64 Windows jobs to flow jobs

Rework Arm64 Windows jobs to flow jobs

This drops our dependency on the internal Arm64CI

[RyuJIT/arm32] Fix GC hole in address mode generation requiring temps (#14949)

* [RyuJIT/arm32] Fix GC hole in address mode generation requiring temps

In cases requiring a temp register to compute a portion of an LEA,
where the partial computation involves a GCREF/BYREF base register,
the temp register must be marked as a BYREF, so it can get
updated during a GC.

This fixes a case like this:
```
add r2, r0, r1 LSL 2
ldr r0, [r2+24]
```

where `r2` needs to be marked BYREF.

Related to #14856.

* Formatting

Merge pull request #14906 from stephentoub/add_memory_string

Add string support to ReadOnlyMemory<char>

Merge pull request #14918 from BruceForstall/NoLegacyUnsafeCpBlk

Don't allow unsafe CpBlk for legacy backend

Merge pull request #14935 from sdmaclea/PR-ARM64-SIMD-genCodeForLclFld

[ARM64] SIMD genCodeForLclFld

Add foreach support to Span<T> and ReadOnlySpan<T> (#14922)

Adds a ref struct Enumerator, and a corresponding GetEnumerator method to each.

Merge pull request #14912 from briansull/add-b124443

Added missing JIT test V1-M15-SP2/b124443/b124443.cs

Adding placeholder Span debugger proxy (#14749)

* Adding placeholder Span debugger proxy.

* Remove unnecessary unsafe keyword.

Small number of bug fixes. (#14900)

- Added stability prefix to the scenario benchmark (JitBench)
- Specify output directory to the `run-xunit-perf.cmd` script and avoid the extra step to xcopy files to the archive folder.
- Added a command line parser class to the illink scenario, and changed its behavior where it used to fail when a new command line option passed to xUnit was not recognized.
- Save the output log of the tests into the sandbox-logs folder.
- Updating the label of the machine pool used by the illink scenario

Merge pull request #14942 from jcagme/master

Adding IsPublic to CreateAzureContainer

Adding IsPublic to CreateAzureContainer

Change Sym publish to use UnzipSymbolPackagesForPublish (#14941)

Add missing license header (#14936)

Increase retry attempts in publish (#14940)

Pass in property SkipCreateWindowsPdbsFromPortablePdbs to fix (#14939)

Remove RelativePath from Publish (#14938)

Added missing test JIT/Regression/CLR-x86-JIT/V1-M15-SP2/b124443/b124443.cs
Also added an IL version of this test for regression repro on CoreCLR

Merge pull request #14928 from sdmaclea/PR-ARM64-SIMD-genPutArgStk

[Arm64] SIMD genPutArgStk

Merge pull request #14937 from jcagme/master

Updating BuildTools version

Updating BuildTools version

[ARM64] SIMD genCodeForLclFld

Merge pull request #14861 from sdmaclea/PR-ARM64-SIMD-genFnPrologCalleeRegArgs

[Arm64] SIMD genFnPrologCalleeRegArgs

Remove ConfigurationGroup from Publish Symbols (#14931)

JIT: make suitably optimistic prejit inline assessments (#14850)

When prejitting, the jit assesses whether each root method is a potential
inline candidate for any possible caller. Methods deemed un-inlinable in any
caller are marked in the runtime as "noinline" to save the jit some work
later on when it sees calls to these methods.

This assessment was too conservative and led to prejit-ordering dependences
for inlines. It also meant that prejitting was missing some inlines that
would have happened had we not done the prejit root assessment.

This change removes some of the prejit observation blockers. These mostly
will enable more prejit methods to become candidates. We also now track when
a method argument reaches a test.

When we are assessing profitability for a prejit root, assume the call site
best possible case.

Also, update the inline XML to capture the prejit assessments.

This increases the number of inlines considered and performed when prejitting
and will also lead to slightly more inlining when jitting. However we do not
expect a large througput impact when jitting -- the main impact of this change
is that inlining when prejitting is now just as aggressive as inlining when
jitting, and the decisions no longer depend on the order in which methods are
prejitted.

Closes #14441.
Closes #3482.

[Arm64] SIMD genPutArgStk

Add MemoryMarshal.AsMemory

Add string support to ReadOnlyMemory<char>

fix symbol sync (#14925)

Merge pull request #14911 from briansull/fix-desktop-warn

Fix warning about signed/unsigned mismatch in new assert

Merge pull request #14913 from briansull/retail-fix

Fix desktop retail build break

Fix glob paths for nupkgs in publish (#14924)

Download pkg and symbolpkg flat (#14919)

Merge pull request #14896 from dotnet-maestro-bot/master-UpdateDependencies

Update BuildTools, CoreFx, PgoData to prerelease-02207-02, preview1-25907-04, master-20171107-0019, respectively (master)

Fixed trash register usage at PollGC helper (#14347)

* Fixed trash register usage at PollGC helper

Don't allow unsafe CpBlk for legacy backend

The arm32 legacy backend does not implement the code to disable/re-enable
GC interruptibility based on the gtBlkOpGcUnsafe flag. So, disable
that mode of OBJ to BLK morphing.

Fixes an arm32 legacy backend GCStress=C case in #14856.

Fix warning about signed/unsigned mismatch in complex assert

Fix desktop retail build break

error C2220: warning treated as error -
from jit\codegencommon.cpp(11228): warning C4702: unreachable code

Merge pull request #14908 from BruceForstall/AddNoSimdTesting

Add `COMPlus_FeatureSIMD=0` testing

Merge pull request #14899 from mikedn/rc-fix

Make sure that Limit::vn is initialized

Merge pull request #14883 from sdmaclea/PR-ARM64-SIMD-misc-varTypeIsStruct

[Arm64] SIMD miscellaneous  varTypeIsStruct

Delete dead code (#14901)

Update BuildTools, CoreFx, PgoData to prerelease-02207-02, preview1-25907-04, master-20171107-0019, respectively

Add `COMPlus_FeatureSIMD=0` testing

Merge pull request #14887 from sdmaclea/PR-ARM64-genSIMDIntrinsicNarrow-Widen

[Arm64] genSIMDIntrinsicNarrow/Widen

[Arm64] SIMD miscellaneous varTypeIsStruct

Merge pull request #14853 from sdmaclea/PR-ARM64-SIMD-applyCalleeSaveHeuristics

[Arm64] SIMD applyCalleeSaveHeuristics

Merge pull request #14904 from dotnet/MattGal-patch-1

Fix Conditional Task Expressions

Fix Conditional Task Expressions

Other conditions in the folder look OK.

Remove XplatEventLogger::IsEventLogging where possible (#14706)

Merge pull request #14881 from sdmaclea/PR-ARM64-SIMD-ContainCheckStoreLoc

[Arm64] SIMD ContainCheckStoreLoc

Merge pull request #14879 from sdmaclea/PR-ARM64-SIMD-ins_Move_Extend

[ARM64] SIMD ins_Move_Extend

Make sure that Limit::vn is initialized

There is code in RangeCheck::MergeEdgeAssertions that checks the limit's vn even if the limit type is keConstant.

Always attempt to create container to ensure it exists

Fix RelativePath and item metadata in publishing project

[Arm64] SIMD genFnPrologCalleeRegArgs

Merge pull request #14287 from tannergooding/benchmarks-game

Adding the new fastest mandelbrot implementation to benchmarks-game.

[Arm64] SIMD applyCalleeSaveHeuristics

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

[Arm64] genSIMDIntrinsicNarrow/Widen

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] SIMD ContainCheckStoreLoc

[ARM64] SIMD ins_Move_Extend

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