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

Update CoreFx to preview2-26313-03 (master)

Revert "Revert "Fix handling of incorrect assemblies on Unix (#16747)" (#16790)" (#16917)

This reverts commit 47bef69b68a35eafa069d08187727684a5f47901.

Merge pull request #16872 from fiigii/avxpermute

Implement AVX Shuffle and Permute intrinsics

change bool to bool (#16914)

Fix shuffling thunk for Unix AMD64 (#16904)

The shufflign thunk was generated incorrectly for some edge cases when
a struct was passed in a register or a pair of registers in the
destination, but on stack in the source.
This change implements a new algorithm that ensures that argument slots
are never overwritten before their current value is moved out.

It also adds an extensive regression test that checks various
interesting combinations of arguments that were causing issues before.

Merge pull request #16888 from jashook/up_jit_stress_timeout

Up the jit stress timeout to 5 hours

Unblock official builds (#16913)

Removing arm64 linux cross build for now

Merge pull request #16390 from sdmaclea/PR-16176

[Arm64] Mark removed compare zero nodes unused

Merge pull request #16822 from sdmaclea/PR-ARM64-BASE-INTRINSICS

[Arm64] Base HW Intrinsics

Update CoreFx to preview2-26313-03

[local gc] refactor apis for threading and suspension to avoid redundant calls in to the runtime (#16765)

* [Local GC] Refactor calls involving thread modes, suspension, and alloc contexts to always operate on current thread

* BOOL -> bool for enable/disable preemptive routines, also remove an unused call to GetThread

* avoid one indirection by having GCToEEInterface::EnablePreemptiveGC return a bool indicating whether the mode was changed

* Callback on IGCHeap for the runtime to notify the GC when it is trapping threads

* use g_fSuspensionPending instead of GCToEEInterface::CatchAtSafePoint for allow_fgc

* Remove CatchAtSafePoint

* Remove GetThread from WaitLongerNoInstru

* code review feedback

* code review feedback

* change BOOL to bool

Update BuildTools, CoreFx to preview2-02612-03, preview2-26313-01, respectively (#16902)

Implement AVX Shuffle and Permute intrinsics

Fix short path expansion (#16901)

Also make methods internal so I can add better unit tests.

Missing Case In Remove Relative Segments (#16869)

Use a cached canceled task in ValueTask (dotnet/corefx#27967)

When an IValueTaskSource reports it completed due to cancellation by the time we AsTask it, we can use a cached canceled task if a) this is the netstandard build, or b) this is the netcoreapp build and it reported cancellation but threw an exception other than a cancellation exception.  We cache the non-generic one in a readonly static, since there's only one of them, but lazily cache the generic one.

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

Merge pull request #16884 from mikedn/lclvar-dead

Remove unused LclVarDsc members

Update BuildTools, CoreClr, CoreFx to preview2-02612-02, preview2-26310-01, preview2-26312-02, respectively

[Arm64/Ubuntu] Update docker tag to include binutils (#16896)

Run ILLInk on System.Private.CoreLib.dll to remove dead code. (#16876)

We already run ILLInk on corefx assemblies to remove dead code;
this change enables that for System.Private.CoreLib.dll.

The new ILLink.targets file is similar to the one in corefx.

All runtime dependencies were already computed and rooted in an xml file that
was embedded in System.Private.CoreLib.dll. This change adds ILLinkTrim.xml
with a few manual roots. They are also included in the embedded xml.

The dead code found by ILLink that could be removed from sources was already
removed in https://github.com/dotnet/coreclr/pull/16759 The remaining code removed by ILLink
falls into 4 categories:
1. Inlined constants.
2. Private default constructors when there are no other instance constructors.
3. System.Number+DoubleHelper.Sign that needs to stay in the sources for corert mirror.
4. EventListener._EventSourceCreated event methods (the even field is retained).

ILLink also clears initlocals on all methods so I removed the workaround
we had in jitinterface.cpp.

[Arm64] Mark removed compare zero nodes unused

Optimize MemoryExtensions SequenceEqual for more T's (dotnet/corefx#27859)

* Optimize MemoryExtensions SequenceEqual for more T's

https://github.com/dotnet/corefx/issues/27487

* Add testcoverage for SequenceEqual<long>

* Remove #if BIT64 dependency for netstandard

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

Merge pull request #16709 from sdmaclea/PR-ARM64-Bootstrapping

Revise init-tools.sh to support bootstrapping

Merge pull request #16877 from CarolEidt/Fix16578

Fix LSRA enregisterLocalVars

Merge pull request #16879 from tannergooding/hwintrin-encode

Ensure that `id->idReg2()` is properly encoded for 4-Byte SIMD instructions

Fix lldb-3.9-dev package name (#16885)

Up the jit stress timeout to 5 hours

Current jitx86hwintrinsicnoavx job runs around 4 hours. Which will cause timeouts if over.

Remove unused LclVarDsc members

Set IsGeneratingNgenPDB flag early (#16882)

Fixes #16864

Convert SSE SetVector128 implementation to managed

Add SSE2 SetVector128 HW intrinsic tests

Implement SSE2 SetVector128 HW intrinsics - managed

[Arm64] Add Base HW intrinsic tests

crossgen plumbing cleanup (#16883)

* Delete unused ForceFullTrust command line option for crossgen

* Delete ALLOW_LOCAL_WORKER define

* Delete Zapper::m_legacyMode

* Delete ngenparser.inl

Adding tests for the Extend, GetLowerHalf, and StaticCast intrinsics

Adding support for the Extend, GetLowerHalf, and StaticCast AVX intrinsics

fix r2r relative indirection for arm. (#16171)

Ensure that `id->idReg2()` is properly encoded for 4-Byte SIMD instructions

[Arm64/Linux] Enable arm64 nuget builds (#16707)

Fix LSRA enregisterLocalVars

We don't bother with the parts of LSRA that deal with lclVars if we have no tracked lclVars. However, we need to check that *after* liveness has run post-Lowering - otherwise we miss any lclVars created by optimizations or during Lowering.

Fix #16578

Use ValueStringBuilder instead of StringBuilderCache

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

Fix the RMW delay handling for hwintrinsic nodes with RMW semantics and more than 2 args

Update isRMWHWIntrinsic to explicitly cover intrinsics that are always RMW

Updating the hwintrinsic codegen to handle the case where `op2Reg == targetReg`

Log inner exceptions for Environment.Failfast  (#16622)

* Add inner exception log to failfast

* cleanup

* Address PR comments

* Address more GC hole issue

* address more PR comments

Fix build with Clang 6.0 and enable build with Clang 5.0 (#16855)

* Fix build with Clang 6.0

* Add support for clang 5.0 too

Merge pull request #16860 from briansull/valuenum-fix

Add implicit cast for assignments in ValueNumbering when necessary

Merge branch 'master' into valuenum-fix

fix typo

Fix value numbering of assignments to insert an implicit cast when necessary

Revert "Disable fragile NGen for System.Private.CoreLib.dll on ARM" (#16863)

This reverts commit bc3b01b5612b21fdb8f11ec6917e5ec4503f1dcc.

Revise init-tools.sh to support bootstrapping

Add $DotNetBootstrapCliTarPath
Disable ILAsm nuget fetch while bootstrapping

When value numbering an assignment we may need to insert an implicit cast operation

Test case opt/CSE/NarrowStore.cs added

Disable fragile NGen for System.Private.CoreLib.dll on ARM

Don't build superpmi as crosscomponent (#16729)

* Don't build superpmi as crosscomponent
* Replace install with _install in all superpmi CMakeLists.txt files

Update UsingYourBuild.md (#16842)

This topic is about DotnetCli instead of CoreRun -- CoreRun is next topic

Fix Compile error

Collapse AsSpan().Slice(...) into .AsSpan(...) (dotnet/corefx#27867)

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

[RyuJit] Fix DCE in liveness. (#16845)

* add repro

* fix

 Disable some GC tests from GCStress runs  (#16846)

RemoveRelativeSegments moved from path to pathinternal (#16829)

Pass generic context argument for shared instance methods on generic interfaces (#16690)

* Add checks and regression test

* Uncomment InvokeConsumer tests

* Revert JIT Interface Check

Merge pull request #16809 from sdmaclea/PR-Enable-crypto-test

[Arm64] Enable crypto test

Updating the x86 hardware-intrinsics tests to properly accept filter arguments

Fix inlining of IAsyncStateMachineBox (#16830)

Prior to .NET Core 2.1, if a task was awaited in a default context (or if ConfigureAwait(false) was used), and if the task was then completed on a non-default context (e.g. code running a non-default TaskScheduler calling SetResult on a TaskCompletionSource), its await continuations are not allowed to be inlined, due to concerns that we could be running an arbitrary amount of unknown code in a scheduling environment that's not amenable to it, like a UI thread. The optimizations added in 2.1 erroneously ended up bypassing that IsValidLocationForInlining check, leading to continuations getting inlined in places they weren't previously.

This commit fixes that.  Previously we'd made the IAsyncStateMachineBox interface inherit ITaskCompletionAction, with its Invoke interface method just delegating to MoveNext; then the box could be added to a task as a continuation.  But that ITaskCompletionAction logic isn't specific to async/await and doesn't invoke IsValidLocationForInlining.  We instead need to follow the Action logic that is meant to be used for async/await.  I've removed the ITaskCompletionAction from IAsyncStateMachineBox, replacing it just with a MoveNext interface method, and added a case for IAsyncStateMachineBox to Task's RunContinuations logic.  I then duplicated the AwaitTaskContinuation.RunOrScheduleAction logic that's there for Action and tweaked it for IAsyncStateMachineBox.  In the process I cleaned up a little code to use some newer C# features.

Delete incorrect assert in stackLevelSetter. (#16816)

* add repro
* delete the assert

Make member visibility first accross corlib (#16836)

Mechanical find&replace to match https://github.com/dotnet/corefx/blob/master/Documentation/coding-guidelines/coding-style.md

Set helix source correctly for prodcon build (#16412) (#16837)

* Set helix source correctly for prodcon build

Update CoreClr, CoreFx to preview2-26308-06, preview2-26308-06, respectively (#16835)

Turn perf jobs back on (#16824)

Updating isRMWRegOper, OperIsCommutative, and canBeContained to handle HWIntrinsic nodes

Move String to shared corelib partition (#16749)

Dial back `\\?\ ` insertion (dotnet/corefx#27827)

Only premptively adding for FileSystemInfo if we end in period or space.
Long path is taken care of later.

Clean up code a little and add test for DirectoryInfo.Move.

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

Fix ensuring primary stack size (#16698)

1. Make it happen for Alpine only (it was incorrectly being done for
   OSX too.
2. Do it only when the Initialize is called for CLR initialization

Add back method needed by CoreRT

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

Update BuildTools, CoreFx to preview2-02607-03, preview2-26308-02, respectively (#16821)

Workaround coreclr issue (#16828)

VSTS does not seem to be correctly honoring the working directory for batch script steps.  Explicitly qualify the setup_vs_env.cmd path

[RyuJIt] Clean and fix gen return (#16725)

* create genSimpleReturn for arm64

* create genFloatReturn for x86

* create genLongReturn for x86 and arm

* merge genReturn

Fix for issue 14144 (#16674)

an additional assert for fgValueNumberHelperCallFunc (#16789)

[Arm64] Add Base HW intrinsics

Delete AsReadOnly* APIs that are no longer part of the public surface (#16799)

Perf fix for month-day parsing ambiguity (#16782)

* Perf fix for month-day parsing ambiguity

* Use DateTime.TryCreate

Remove dead code found by ILLink. (#16759)

I'm preparing changes to enable running ILLink on SPC
(similarly to what we do on corefx assemblies).
There will be a separate PR for those changes.

This PR just removes the dead code flagged by ILLink from the sources.

Address issue 16800 (#16810)

Remove libuuid from scripts and docs (#16805)

[Arm64] Enable crypto test

Update CoreFx to preview2-26307-06 (#16798)

fix inconsistent AVX/AVX2 intrinsic names (#16772)

Update BuildTools, CoreClr, CoreFx to preview2-02606-05, preview2-26307-01, preview2-26307-03, respectively (#16791)

Merge pull request #16662 from jashook/remove_vs2015_dependencies

Remove vs2015 build dependencies

Add debugger usage note (#16788)

Updating the x86 hardware-intrin test templates to use named format arguments

CoreFX #24343 Vector Ctor using Span (#16733)

* CoreFX #24343 Vector using Span

dotnet/corefx#24343

* CoreFX #24343 Vector using Span

dotnet/corefx#24343

* CoreFX #24343 Vector using Span

dotnet/corefx#24343

Revert "Fix handling of incorrect assemblies on Unix (#16747)" (#16790)

This reverts commit 383736b96b643ba46ad290fc86601fc2d62a9436.

Remove vs2015 build dependencies

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

Update BuildTools, CoreClr, CoreFx to preview2-02606-04, preview2-26306-01, preview2-26306-04, respectively (master)

Update BuildTools, CoreClr, CoreFx to preview2-02606-04, preview2-26306-01, preview2-26306-04, respectively

Disable Perf Runs on Commit (#16781)

We need to disable these jobs on commit until dotnet/core-eng#2808 is fixed.

Fix hang on Ctrl+C when tiering is enabled (#16719)

Fix hang on Ctrl+C when tiering is enabled

- When the work thread count is nonzero at the time of Ctrl+C, by the time `TieredCompilationManager::Shutdown(BOOL)` runs (during `DllMain` for process detach), the background worker thread has already been torn down abruptly and it does not get a chance to exit gracefully and set the event, and Shutdown() hangs the process waiting for the event forever.
- It looks like there is no benefit to waiting for the work to complete, fixed by removing the wait and the event
- In the MusicStore benchmark it looks like the arch folder was missing in the 'publish' folder path. In case there is a difference in bin directory layout between the CI dotnet and current dotnet, changed to check for both paths.