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)

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

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.

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)

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.

Convert Byte Arrays to Hexidecimal Strings When Writing to LTTng via EventSource (#16762)

Fix potential buffer overrun in ClearWithReferences

Merge pull request #16700 from briansull/fix_dyn_blk

Fix morph issue with a fixed size block copy

Add tracepointprovider.cpp directly to PAL instead of to a separate tracepointprovider library. (#16771)

Fix whitespace

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

Fix whitespace

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

Improve performance of Span.Clear for referential T

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

Fix handling of incorrect assemblies on Unix (#16747)

* Return DPTR from PEDecoder::FindFirstSection()

Change type of the function's return value
to PTR_IMAGE_SECTION_HEADER instead of (IMAGE_SECTION_HEADER *)

* Fix handling of incorrect assemblies on Unix

This fixes the regression that was introduced by #10772 and is
caused by a missing check for validity of loaded assembly file.

Related issue: #15544

address PR feedback (#16761)

Add LoadHigh, LoadLow, and SetScalarVector128 SSE2 HW intrinsics tests

Implement LoadHigh, LoadLow, and SetScalarVector128 SSE2 HW intrinsics

Simplify SSE/SSE4.1 intrinsics

Update BuildTools, CoreClr to preview2-02606-01, preview2-26305-12, respectively (#16760)

Remove libuuid usage (#16643)

* Remove libuuid usage

This change removes dependency on the libuuid library that is used for GUID creation only.
It implements it using a random generator instead.

It also modifies return type of PAL_Random to VOID since it was always
returning TRUE and none of the existing callers were checking it.

1. Port the GUID creation to managed code.
2. Modify the PAL_Random to have 6 times better perf so that the perf of the
   CoCreateGuid that is used in the native runtime doesn't degrade that much
   w.r.t the previous state when the libuuid was used.
3. Use Interop.GetRandomBytes on Unix and fix Windows

SCEP package version update

Merge pull request #16727 from CarolEidt/NoIndirByte

Don't force byte regs for indir address

[ARM32] RyuJIT support for profiler ELT callbacks  (#16728)

* add assert that argReg was PreSpilled

* genProfilingEnterCallback for RyuJit arm

* fix genProfilingLeaveCallback for RyuJit arm

* call genProfilingLeaveCallback for arm from genJmpMethod

Update SCEP package version

Updating the CompareEqual{Ordered|Unordered}Scalar intrinsics to have slightly better codegen

Update CoreClr, CoreFx to preview2-26305-04, preview2-26305-02, respectively (#16755)

Scrub error messages for missing arguments(throw new) (#16753)

Change COMPLus -> COMPlus

Remove ConfigFile parameter which is not set on previous tasks

Merge pull request #16731 from tannergooding/hwintrin-avx

Implementing the ConvertToVector AVX hardware intrinsics

Delete dead code in PAL related to file locks (#16754)

Remove flock usage from InternalCreateFile in PAL (#16752)

This flock is causing issues on filesystems that don't support locking.
An issue with loading managed assemblies from such a filesystem due to
this issue was recently hit by someone.
Since the locking in the PAL is not really needed and due to the
advisory nature of the flock doesn't guarantee anything, this change is
removing the flock from that function.

Update CoreClr, CoreFx to preview2-26305-01, preview2-26305-01, respectively (#16740)

Fix encoding methods for Span.Empty (#16748)

* Fix encoding methods for Span.Empty

Encoding had a historic confusion about handling null pointers. Make sure that this confusion is not leaking into the new Span methods.

* Use GetNonNullPinnableReference helper method