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

Delete stale TryFormat and {Try}Parse overloads (#16746)

Clear executable bit on several .cs files (#16743)

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

fix legacy backend (#16741)

Clear executable bit on several .cs files

Update TraceLogging to use NULL-terminated strings instead of counted strings (#16672)

Fix use of EventPipeConfiguration after it has been freed on shutdown. (#16704)

Update CoreClr to preview2-26304-01 (#16739)

Fix MemoryDebugView and override Memory.ToString similar to Span (#16732)

* Fix MemoryDebugView and add Memory.ToString similar to Span

* Simplify implementation of ToString

Remove StringSpanHelpers and start using MemoryExtensions (#16718)

* Remove StringSpanHelpers and start using MemoryExtensions

* Address PR feedback.

Update CoreClr, CoreFx to preview2-26303-09, preview2-26303-04, respectively (master) (#16730)

* Update CoreClr, CoreFx to preview2-26303-09, preview2-26303-04, respectively

* Add System.Runtime.Intrinsics.Experimental to test dependencies

Implementing tests for more of the AVX hardware intrinsics

Implementing some more of the AVX intrinsics

Implement AVX/AVX BroadcastScalarToVector*

Merge pull request #15777 from dotnetrt/sse2part2

Implement Shuffle* Sse2 Intel hardware intrinsics

Use TARGET_POINTER_SIZE macro (#16640)

* Replace sizeof(void*) with TARGET_POINTER_SIZE in TYPEINFO in src/inc/cortypeinfo.h

* Replace sizeof(LPVOID) with TARGET_POINTER_SIZE in DEFINEELEMENTTYPEINFO in src/vm/siginfo.cpp

* Define TARGET_POINTER_SIZE as POINTERSIZE_BYTES in src/ToolBox/SOS/Strike/util.h

Don't force byte regs for indir address

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>

Add Shuffle* Sse2 hardware intrinsics tests

Implement Shuffle* SSE2 hardware intrinsics

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.