Merge pull request #16638 from jcagme/master

Install and uninstall SCEP

final set of comments been addressed

Remove variable

Remove creation of Auth'd nuget.config

Fix netstandard build references to UnsafeTask

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

Yet one more reverted change in PR

One more reverted change in PR

Merge branch 'master' of https://github.com/jcagme/coreclr

Addressing reverted change in PR

Delete scep-ops.cmd

Changes with NuGet package approach

Merge branch 'master' of https://github.com/dotnet/coreclr

Merge pull request #16715 from fiigii/fixmovemask

Fix a bad marge

[RyuJit] fix clang switch warning (#16693)

* fix clang switch warning

* Try to set throw blocks only for nodes that generates exceptions themselves.
Update comments.

fix a bad marge

Delete SingleGetHashCode and DoubleGetHashCode tests for arm/arm64 (#16688)

* Delete SingleGetHashCode and DoubleGetHashCode tests for arm/arm64

The tests themselves were deleted with #16567.

* Delete arm/arm64 tests deleted in #16571

StringCompare1, StringCompare2, StringCompare15

* Delete more references to these tests

Merge pull request #16703 from BruceForstall/GeneralizeRuntestExclusionFiles

Generalize runtest.sh test exclusion files

Fix !threads -special spurious error message (#16701)

Fix !threads -special spurious error message

Generate portable PDB for SOS.NETCore.

Fix issue #16665. Incorrect command names in messages on Linux.

Update CoreClr to preview2-26302-06 (#16706)

Merge pull request #16646 from fiigii/insertextract

Implement SSE4.1 Insert/Extract and simplify SSE/SSE2 intrinsic

Work around MCG bug around `ref char` marshalling (dotnet/corert#5481)

The change in the P/invoke signature that came from CoreCLR triggers a bug in MCG on the TFS side.

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

Merge pull request #16617 from echesakovMSFT/UseCodeTTypedefArm

[RyuJIT/ARM32]Use emitter::code_t typedef for ARM

Merge pull request #16669 from tannergooding/hwintrin-sse2-maskmove

Implement the SSE2 MaskMove intrinsic

Fixing GetFullPath (#16670)

Update CoreClr, CoreFx to preview2-26302-01, preview2-26228-08, respectively (#16702)

Generalize runtest.sh test exclusion files

Allow leading '#' as a comment character for all testsUnsupported and
testsFailing files.

Generalize testsFailingOnArm64.txt to testsFailing.$ARCH.txt

Generalize testsUnsupportedOnARM32.txt to testsUnsupported.$ARCH.txt

Vectorize String.IndexOf(char) and String.LastIndexOf(char) (#16392)

* Vectorize String.IndexOf(char) using the same algorithm as SpanHelpers
IndexOf(byte).

* Respond to feedback.

* Vectorize String.LastIndexOf

Clean up IndexOf vectorization.

Fix Adar month name in Hebrew calendar (#16697)

The returned data from ICU has 6th month name as 'Adar I' and 7th month name as 'Adar'
We need to adjust that in the list used with non-leap year to have 6th month as 'Adar' and 7th month as 'Adar II'
note that when formatting non-leap year dates, 7th month shouldn't get used at all.

Merge pull request #16668 from briansull/simd-ins-opt

Refactor the signature of genGetSimdInsOpt to use emitAttr

1-base Dictionary buckets to reduce initalization

Use EqualityComparer<TKey>.Default Intrinsic

Improve Dictionary TryInsert CQ

Improve Dictionary FindEntry CQ

Change several ValueTask properties to methods (#16691)

These do unsafe casts on an object, and we don't want a debugger's automatic evaluation accidentally invoking them.  That means either making them methods or adding [DebuggerBrowsable(DebuggerBrowsableState.Never)], and the former seems cleaner.

Implement SSE4.1 insert and extract

 Move MemoryExtensions.TryGetString to MemoryMarshal and remove TryGetArray (#16692)

* Remove Span.NonGenerics and update leftover AsRoS -> AsSpan changes

* Move MemoryExtensions.TryGetString to MemoryMarshal and remove TryGetArray

* Move TryGetString to common MemoryMarshal.cs

* Remove the `this` keyword, not an extension method

Remove Span.NonGenerics and update leftover AsRoS -> AsSpan changes (#16689)

Update 64-bit intrinsics and simplify SSE/SSE2 intrinsic

Merge pull request #16660 from echesakovMSFT/UseTarget64BitMacro

Use _TARGET_64BIT_ macro

Add tests and templates for insert/extract

Define emitter::code_t for _TARGET_ARM_ and use this type rather than ssize_t or size_t to hold encoded instructions in emitter

Adding tests for additional AVX hardware intrinsics

Adding support for additional AVX hardware intrinsics

Adjust System.Threading.Tasks.Extensions for IValueTaskSource

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

Improve MemoryMarshal.Cast (#16659)

Avoid unnecessary checked and signed arithmetic. Handle special cases such as cast between same size types and from byte sized types, the JIT is unable to optimize these cases currently.

Add back argument check for TypeForwardedFromAttribute (#16680)

Fixes dotnet/corefx#27554

Fix build break in System.Memory due to incorrect XML comments

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

Merge pull request #16618 from stephentoub/valuetaskextensibility

Implement ValueTask extensibility

Remove span helpers, use mirrored PathInternal helper (#27585)

* Remove span helpers, use mirrored PathInternal helper

Remove other unnecessary helpers (notably with Path.Join available)

* Fix Unix

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

Fix GetPathRoot issues (dotnet/corefx#27572)

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

Edits to fix conflicts in generated Vector.cs file (#16666)

Fixed minor space changes and one syntax error that causes conflict between checked in Vector.cs file and the freshly auto-generated Vector.cs file.

Adding tests for the SSE2 MaskMove intrinsic

Implementing the SSE2 MaskMove intrinsic

Release memory correctly during ComCallWrapperTemplate::Cleanup (#16671)

Fixes #15632

Remove ConvertILOnlyPE32ToPE64 and ConvertILOnlyPE32ToPE64Worker (#16676)

Move IValueTaskSource and friends to System.Threading.Tasks.Sources

Group these more advanced types into a subnamespace.

Add token to IValueTaskSource

Implement ValueTask extensibility

This commit adds support for extending `ValueTask<T>` with arbitrary backing sources.  Prior to this change, `ValueTask<T>` could wrap a `T` or a `Task<T>`; now it can also wrap an `IValueTaskSource<T>`, which can be implemented by arbitrary objects to be represented by `ValueTask<T>`.  These objects can then be pooled and reused to minimize allocation.  The commit also adds a non-generic `ValueTask` that can represent void-returning operations, including a `default` synchronous success, `Task`, and `IValueTaskSource`.  For the non-generic `ValueTask`, the commit also includes awaiters and async method builders, so it can be both awaited and used as the return type of an async method.

The rest of the changes fall into a few buckets all related to enabling this support:
- Modifying `AsyncTaskMethodBuilder<TResult>.AwaitUnsafeOnCompleted` to specially recognize any `ValueTask` and utilize either the `Task` or `IValueTaskSource` that backs it to avoid allocating an Action MoveNext method.  If every object awaited in an async method is either a `Task`/`Task<T>` or `ValueTask`/`ValueTask<T>`, regardless of what the `ValueTask`/`ValueTask<T>` wraps, we'll be able to avoid allocating the delegate and only allocate the single state machine object that also serves as the returned object.
- Changing `Stream.WriteAsync` to return `ValueTask` instead of `Task`.  This enables interested overriding stream types to use a reusable/pooled object to avoid `WriteAsync` allocations.
- Modifying Stream.CopyToAsync to use the new `Memory`-based overloads of `ReadAsync` and `WriteAsync`.  This enables the default `CopyToAsync` implementation to take advantage of any pooling done by derived streams, but even without pooling to take advantage of synchronously completing `ReadAsync`s returning `ValueTask<int>`s that contained an `int` rather than an allocated object. (While I was modifying this, I also removed some unnecessary array clearing that we'd added before later deciding it wasn't needed in general.)
- Modifying StreamReader/Writer to use the new `ReadAsync`/`WriteAsync` overloads.

Minor documentation improvements (#16677)

Create test for rundown events with EventPipe (#16667)

Replace sizeof(LPVOID)==4 with #ifdef _TARGET_64BIT_ in fieldmarshaler.cpp

Replace ambiguous _WIN64 and BIT64 with _TARGET_64BIT_ macro

Delete unnecesary StackCrawlMarks (#16648)

CAS leftovers

Update BuildTools, CoreClr, CoreFx, PgoData to preview2-02528-07, preview2-26228-08, preview2-26228-03, master-20180228-0037, respectively (#16661)

Refactor the signature of genGetSimdInsOpt to use emitAttr
Eliminates the use of bool is16Byte from the code base
Added my comments from the code review of PR #16565 to emitarm64.cpp

Update EventSource* Tests for EventPipe (#16645)

Merge pull request #16657 from debayang/crypto_arm64_intrinsics

Crypto arm64 intrinsics

Set isInternalRegDelayFree for several of the x86 hwintrinsics

Merge pull request #16538 from sdmaclea/PR-ARM64-R2R_FAIL

[Arm64/Windows] Make r2r green in CI

Add the array/arraysegment->Span/Memory overloads to CoreCLR. (#16658)

Part of https://github.com/dotnet/corefx/issues/26894

Revert: Simplify test dependencies for benchmarks (#16647)

This reverts commit 0598b6b8af0fb560837ce0bb8f3c8236c05bbdc9.

Updates to xunit console runner break things in Helix. So reverting
until we can get that part sorted.

Also undoes version updates from #16597; we can't move these forward
yet as we're back to netstandard 1.4 for the time being.

Merge pull request #16531 from echesakovMSFT/MakeTarget64BitMacroGlobal

Make _TARGET_64BIT_ macro global to coreclr

ARM64 Crypto intrinsics tests for AES/SHA1/SHA256

ARM64 SHA256 Crypto intrinsics implementation

ARM64 SHA1 Crypto intrinsics implementation

Mark MemoryMarshal.Cast methods for aggressive inlining (#16654)

Inlining doesn't streamline the cast logic any, but it facilitates
caller struct promotion which can substantially boost perf.

See discussion in dotnet/corefx#27485.

Add Thread.GetCurrentProcessorId() API (#16650)

Contributes to https://github.com/dotnet/corefx/issues/16767

Adding some asserts that we won't overwrite one of the hwintrinsic operand registers.

Adding tests for the implemented SSE41 hardware intrinsics

Adding partial support for the SSE41 hardware intrinsics

ARM64 Aes Crypto intrinsics implementation

Merge pull request #16599 from fiigii/fixstaticcast

Fix generic hardware intrinsics with NotSupportedException

Fix StaticCast with NotSupportedException

Add Path.Join() methods. (#16561)

* Add Path.Join() methods.

See #25536.

* Address feedback and fix a couple issues now that I've got tests running correctly.

* Update per final API approval

* Fix Unix, remove redundant helper.

* Merge and tweak join methods in GetFullPath(string, string)

* Tweak again

Fixing lambda capture unused errors. (#16555)

These errors were happening when compiling with clang 5.0.1.

Since the whole method is defined inside a #if DEBUG, it was
erroring when compiling in release mode because the lambda
variables were not being used.

Tests For GetPathRoot and Enabling NormalizeDirectorySeparatorTests (#27494)

Corrects its Implementation too

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

Use the same default value of MarshalAs.IidParamIndex for both Windows and Unix (#16639)

Merge pull request #16637 from dotnet-bot/from-tfs

Merge changes from TFS

Address comments

Actually adding SCEP script

Minimal Fix for the incorrect rangecheck elimination issue reported by Mike Danes

RangeCheck::IsBinOpMonotonicallyIncreasing has a bug that results in incorrect range check elimination – when the second operand is a lclvar it does not check if it’s a positive value, it only checks if it’s monotonically increasing and any constant is treated as such.

[tfs-changeset: 1689890]

Adding SCEP install/uninstall script and tasks

Bug in GetFullPath(basePath, Path) (#16598)

* GetFullPath and GetRootLength Corrected

* Removed getpathroot

* using span

* "\\" changed to @"\"

Merge pull request #16533 from briansull/default-fill

Implement JitDefaultFill

Change new ThreadPool.QueueUserWorkItem method to be generic (#16570)

Event pipe guid in metadata (#16316), fixes #16105

* some refactor to remove dumplicated code in tests and make them fit 1 screen

* write the failing test for no activity id

* use 17, the "free slot" of TypeCode enum to represent Guid in event pipe metadata

* remove the test which would not pass until new trace event is released and Event Pipe enabled for Windows

* added comment to TypeCode about using the value 17

* post code review fixes (thanks @justinvp)

* post code review fixes (thanks @brianrob)

* refactor the rest of event pipe tests to use NetPerfFile helper

Implement JitDefaultFill
- Changes and bugs fixes when changing the JitDefaultFill value from 0xff to 0xdd
- Implement randomized DefaultFill for stress
- Added comment header for compGetJitDefaultFill
- Changed C style comments to C++ comments

Fix P1 test build break (#16631)

add PHASE_COMPUTE_REACHABILITY (#16574)

Debugger FIFO pipes are created in TMPDIR if defined #16234 (#16452)

Debugger FIFO pipes are created in TMPDIR if defined #16234

Init name to null termination to ensure failure if GetTempPathA fails