Merge pull request #16199 from dotnet/revert-15944-CorefxTestUseCorrectVersion

Revert "For corefx testing, use a matching corefx repo version git hash"

Revert "For corefx testing, use a matching corefx repo version git hash"

Update CoreClr to preview2-26204-05 (#16191)

Merge pull request #16182 from CarolEidt/FixDD543057

Consider specialPutArgs for jitStressRegs

Fix failfast stacktrace (#15895)

* attempt to fix stacktrace getting printed twice

* Fix some default parameter issues, and wrong commit from last commit

* Fix build errors, switch call from Debug.Assert to new FailFast FCall

* Fix signature to allow more types of exception title

* cleanup

* Addressing comments from PR

* More PR comments

* remove useless using

* Address comments on GC hole and few naming changes

JitBench harness improvements (#15151)

* Improvements to JitBench perf test

1) More of the scenario setup is delegated to the scripts in the JitBench repo. The run now consists of (a) acquire source (b) RunBenchmark.ps1 -SetupOnly (c) run dotnet MusicStore.dll over and over to get measurements.
2) A specific last known good framework version is used instead of an open-ended latest version. This should make the test more reliable day-to-day at the cost that we have to bump the version when breaking changes occur.
3) Added an option to test with a local copy of JitBench instead of acquiring one from github. This is useful to for testing simultaneous changes in both repos.
4) Updated the output parsing to account for the changes I made in MusicStore's console spew.

* Code review feedback

* Don't use precompiled views by default

In order to match the previous benchmark results we should avoid
using precompiled views.

* Fixed JitBench local repo issue

In my earlier change I accidentally broke the normal scenario when you do not use a local copy of JitBench repo. This case was failing with an exception in SetupStatics because the null directory didn't exist.

Updating the templated x86 hwintrinsic tests to validate Load and LoadAligned

Updating the HWIntrinsic codegen to support marking LoadVector128 and LoadAlignedVector128 as contained.

Remove AsyncMethodBuilder from shared partition (#16149)

* Remove AsyncMethodBuilder from shared partition

* AsyncMethodBuilder => AsyncMethodBuilderCore

Changing Span/Memory to return default on null instead of throwing. (#16186)

* Additional changes based on feedback (bounds checks)

* Adding remaining bounds checks

Update BuildTools, CoreClr, CoreFx to preview2-02502-01, preview2-26203-01, preview2-26202-05, respectively (#16180)

Dictionary exposes internal type comparer (#16151)

Fixes dotnet/corefx#26033

Delete unused SSA_FEATURE_USEDEF ifdefs (#15654)

Fix GetILToNativeMapping3 to return mappings for the specified code start address (#16146)

Fix GetILToNativeMapping3 to return mappings for the specified code start address

Fix for https://github.com/dotnet/coreclr/issues/16145 in master:
- Previously, it was getting or creating the current entry point's corresponding DebugJitInfo and determining that the specified code address is not within the code range
- Fixed to get or create a DebugJitInfo corresponding to the specified code address and return that

Fix AV in crossgen when generating PDBs (#16184)

I missed updating one of the callers of `ReadyToRunInfo::GetEntryPoint`
in #15801. Fix by making the last arg explicit so this kind of type
confusion error is less likely, and updating the missed call site.

Closes #16177.

Fix mirroring build break

Span factoring (dotnet/corefx#26667)

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

Merge pull request #16133 from eerhardt/AddVectorToCoreLib

Add Vector<T> to CoreLib.

Consider specialPutArgs for jitStressRegs

When we have a lclVar that is being kept alive between its `PUTARG_REG` and the call, we need to take that into account in determining the minimum register requirement for a node.

Event Pipe File V3 (#16107)

* write missing information to the event pipe file (pointer size to make it work fo x86)

* define where the events start, not only where they end

* include process Id in the event pipe file, bump the version so old consumers get clear error message

* write the missing EndObject tag to close the header

* include expected CPU sampling rate in the event pipe header file

* include keywords in V3 of EventPipe metadata, fixes #11934

* remove forward references

* entry object comes after the header and ends after it's data, before the event block objects

* introduce event block

* fix the GC contracts

* generate metadata ids

* end the file with null reference tag

* getting it work

* 4 byte alignment of serialized event data

* Revert "include keywords in V3 of EventPipe metadata, fixes #11934"

This reverts commit 98ef2f588e271f928fd051e96da526dc1e0f017c.

* remove event Id and event version from metadata buffer (it was duplicated with native code)

* increase the block size to be the same as buffer size

* Write the last event block to the file after disabling the event pipe, right after last events

* include the sife in itself

* the native part was supposed to not duplicate the event id and version, not manged

* ensure 4 byte alignment

* build metadata when it's not provided, so payload is never empty (no need to serialize length)

* this todo is no longer valid

* don't align everything, just the content of event block as suggested by @vancem

* improvements after code review

* update TraceEvent dependency, make the test verify new feature

* InterlockedIncrement(Int32) is not available for non-Windows OSes

* code improvements after Skype code review from @jorive

Add assert for failure to open a file in FastSerializer. (#16178)

Update CoreClr, CoreFx to preview2-26202-06, preview2-26202-01, respectively (master) (#16169)

* Update CoreClr, CoreFx to preview2-26202-06, preview2-26202-01, respectively

* Re-enable and fix the Sse hwintrinsic tests that had API name changes.

Merge pull request #16157 from dotnetrt/sse2fix

Fix validation code in Sse2.MultiplyLow tests

Ifdef out Compiler::lvaVarPref (#15329)

update cla link (#16170)

ajdust -> adjust (dotnet/corert#5327)

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

[R2R] add a virtualStubParamInfo as an argument (#15910)

* add a virtualStubParamInfo as an argument

* Revert legacy workaround from lower.

Add New EventPipe Tests (#16130)

Revert "Disable some test cases to workaround compiler bug" (#16161)

This reverts commit a0033795f551053cb199d85c77926f0840f5b583.

Fixes #12233

Fix detection of YMM registers presence (#16168)

It was found that we incorrectly try to restore YMM registers in RtlRestoreContext
when the processor supports xstate, but doesn't have YMM registers. This change
fixes that by testing the YMM presence flag too.

Merge pull request #16092 from CarolEidt/RefactorRefBuilding

Refactor RefPosition and Interval Building

Fix DevDiv_543045 (#16167)

Add lsrabuild.cpp to jit.settings.targets

Use ValueTask shared from corelib (dotnet/corefx#26636)

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

Memory footprint gc latency level (#15469)

Starting the work on latency levels
Current tuning is latency_level_balanced
Added the latency_level_memory_footprint level for optimizing for memory footprint

Fix Arm64 compile error

Refactor RefPosition and Interval Building

Move code for building `RefPosition`s and `Interval`s out of lsra.cpp into lsrabuild.cpp. Also, move common code from lsraarm*.cpp and lsraxarch.cpp to lsrabuild.cpp.

Maintain the `ListNodePool` on the `LinearScan` object to be used by all the building methods.

Rename `TreeNodeInfoInit` methods to `Build`, to more accurately reflect the next round of changes where they will build the `RefPosition`s directly.

Merge pull request #16155 from jashook/merge_jit_and_JIT

Remove lower case jit test dir

Fix validation code in Sse2.MultiplyLow tests

Fixes #16153

Remote lower case jit test dir

Update CoreClr, CoreFx to preview2-26201-05, preview2-26201-05, respectively (#16150)

Fixing some of the x86 HWIntrinsics to only use byteable registers, where required

Moving the Sse Compare*OrderedScalar and Compare*UnorderedScalar tests to use a template.

Moving the Sse max/min tests to use the template.

Moving the Sse comparison tests to use the template.

[Arm64] Add Lower Compare Zero comments

[Arm64] HWIntrinsic codegen function headers

[Arm64] Lower Unsigned Compare Zero

[Arm64] Implement Simd.SetAllVector*

[Arm64] Implement Simd.BitwiseSelect

[Arm64] Implement Simd.Extract

Update CoreClr, CoreFx to preview2-26201-01, preview2-26201-01, respectively (#16142)

Change the Span ToString semantics to return the contents for T=char (#16143)

Rename ConvertScalarToVector128UInt32Scalar to ConvertScalarToVector128UInt32

Updating IF_RWR_RRD_MRD_CNS and IF_RWR_RRD_SRD_CNS to encode reg2, rather than reg1, in 3456

Merge pull request #16136 from CarolEidt/Fix16135

Fix encoding of RCPPS and RSQRTPS

Adding TrimExcess APIs for Dictionary class

Add ETL flags for ReadyToRun (#15801)

Add flags to track the presence of ReadyToRun codegen in an assembly
and module.

Add jitting flags to indicate when a method is jitted because the
precompiled code was rejected, either by a profiler or by ReadyToRun
dependence tracking.

Together these can be used to distingish between
* methods jitted because their assemblies were not precompiled
* methods jitted because they were not precompiled in an otherwise
precompiled assembly

Delete GenTreePtr. (#16027)

* jit sources: Each local pointer variable must be declared on its own line.

Implement https://github.com/dotnet/coreclr/blob/master/Documentation/coding-guidelines/clr-jit-coding-conventions.md#101-pointer-declarations
Each local pointer variable must be declared on its own line.

* add constGenTreePtr

* delete GenTreePtr

* delete constGenTreePtr

* fix arm

Merge pull request #16116 from fiigii/fixgeneric

Fix table-driven framework for one-type-arg generic HW intrinsics

Merge pull request #16114 from tannergooding/hwintrin-flags

Fixing up some hwintrinsic flags for the x86 intrinsics.

[local gc] Enable eventing (#16120)

* move GC etw enums to gcinterface.ee.h

* add GetActiveSyncBlockCount

* refactor reference to ETW::GCLog::ShouldTrackMovementForEtw()

* mov g_dwHandles to gc side

* enable FEATURE_EVENT_TRACE for gc

Fixing up some hwintrinsic flags for the x86 intrinsics.

Respond to PR feedback.

- Move IsIntrinicType check earlier, and use it during CheckIfSIMDAndUpdateSize.
- Check all types for [Intrinsic] attribute.

Span overload added for path apis (#15608)

Span overload path apis

Fix encoding of RCPPS and RSQRTPS

These instructions do not permit a 3rd register, and the vvvv field must be all ones (same as xmm0 encoding, which is why this can go undetected)

Fix #16135

Fix table-driven framework for one-type-arg generic intrinsics

JIT: handle GT_SUB in gtClone (#16128)

For uniformity's sake, handle GT_SUB just like GT_ADD in gtClone.
Otherwise for a span s, the jit codegen for s[i+1] and s[i-1] can
end up substantially different.

See notes in #13097.

JIT: add a bit more importer folding (#16111)

Eagerly fold expression trees for non-branch conditional operations.

Leads to elimination of boxes in some idiomatic uses. See notes and
examples in #14472.

Update BuildTools, CoreClr, CoreFx, PgoData to preview2-02431-05, preview2-26131-05, preview2-26131-05, master-20180131-0033, respectively (#16131)

Merge with latest code and PR feedback.

Add Vector<T> to CoreLib.

This requires the runtime to change to recognize the Vector classes in either System.Numerics.Vectors.dll or in System.Private.CoreLib.dll.  To do this, I added the [Intrinsic] attribute to Vector<T> struct and Vector static class.

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

Update BuildTools, CoreClr, CoreFx, PgoData to prerelease-02430-04, preview2-26130-05, preview2-26130-01, master-20180130-0046, respectively (master)

Disable MemoryMarshal.CreateSpan tests

Remove Span DangerousGetPinnableReference

Remove replaced Span/ReadOnlySpan methods.

Merge pull request #15585 from dotnetrt/sse2

Implement simple Sse2 hardware instrinsics

Update BuildTools, CoreClr, CoreFx, PgoData to prerelease-02430-04, preview2-26130-05, preview2-26130-01, master-20180130-0046, respectively

Replace JitIntrinsicAttribute with IntrinsicAttribute (#26700)

* Replace JitIntrinsicAttribute with IntrinsicAttribute

* Use s_ prefix for statics

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

Merge pull request #16097 from sdmaclea/PR-Create-hwintrinsic.cpp

Create hwintrinsic.cpp

Merge pull request #16102 from sdmaclea/PR-CORINFO_HELP_THROW_TYPE_NOT_SUPPORTED

CORINFO_HELP_THROW_TYPE_NOT_SUPPORTED

Reduce Execution Context Save+Restore (#15629)

* Reduce Save+Restore for ExecutionContext

* Use flag rather than comparison to static

* Skip null check for pre-checked EC.Run

* Feedback

* Add static helper lookup for default context for TP

* Add note for enregistering

* Return to Default context when no values

* Default + FlowSuppressed Context

* Move AsyncMethodBuilder.Start to static non-generic

* Feedback

Implement simple Sse2 hardware intrinsics

Move Vector<T> to the shared CoreLib partition. (dotnet/corefx#26656)

* Move Vector<T> to the shared CoreLib partition.

This is in support of eventually compiling it into System.Private.CoreLib. That way we can use it in other corelib algorithms.

* Update Test project to link to new CoreLib file path

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

Replacing Lock with Volatile.Read for SecureString Length (#16112)

* Replacing Lock on SecureString Length with Volatile Read

Fixes #26685

* Replacing Lock with Volatile Read/Write for IsReadOnly() and MakeReadOnly()

Add COMPlus variable to control rundown. (#16109)

Merge pull request #15771 from tannergooding/hwintrin-tests

Update x86 HWIntrinsic Tests

Merge pull request #16108 from briansull/fix-vm-contract

Fix contract violation in new method CEEInfo::getTypeForPrimitiveNumericClass

fix DevDiv_546017 (#16090)

* fix DevDiv_546017

* add repro

Building non-Windows builds using the CLI's MSBuild (#16101)

* Building non-Windows builds using the CLI's MSBuild

Merge pull request #16098 from stephentoub/valuetaskshared

Several updates to ValueTask

[Local GC] FEATURE_EVENT_TRACE 4/n: Event ports for all GC-keyword events (#16031)

* [Local GC] Event ports: GCStart and GCGenerationRange

* [Local GC] Event ports: GCEnd

* [Local GC] Event ports: GCHeapStats

* [Local GC] Event ports: GCCreateSegment

* [Local GC] Event ports: GCFreeSegment

* Event ports: GCCreateConcurrentThread and GCTerminateConcurrentThread

* [Local GC] Event ports: GCTriggered

* [Local GC] Event ports: GCMarkWithType

* [Local GC] Event ports: GCJoin_V2

* [Local GC] Event ports: GCGlobalHeapHistory_V2

* [Local GC] Event ports: GCAllocationTick_V3

* [Local GC] Event ports: GCAllocationTick_V1

* [Local GC] Event porting: PinObjectAtGCTime

* one last merge conflict

CORINFO_HELP_THROW_TYPE_NOT_SUPPORTED

Merge pull request #15923 from tannergooding/hwintrin

 Fixing some inconsistencies in the x86 HWIntrinsic APIs

Fix contract violation in new method CEEInfo::getTypeForPrimitiveNumericClass
Don't call the method CEEInfo::asCorInfoType THROWS TRIGGERS SO_INTOLERANT

Merge pull request #16087 from ianhays/span_strings

Update Span/ReadOnlySpan strings to match Portable Span

Move the MethodImpl iterator when checking default interface methods (#16099)

Fixes #16064.

Create hwintrinsic.cpp

Make ValueTask shareable with corefx

Update Span DebuggerDisplays