Make it possible to Reflection.Emit default interface methods (#16257)

Fixes #15648.

Agruments -> Arguments (#16273)

Fixes GitHub Issue 16065
Bug in the CSE phase when a CSE use contains a CSE def

For the error cases we could unmark or discard a CSE def. This CSE def might be needed when
 performing a subsequent CSE replacement.  This could lead to an uninitialized value being
 used for the CSE use.

Changed optUnmarkCSE method to return a bool instead of a void.
Changed optValuenumCSE_UnmarkCSEs  to take a write-back mutable wbKeepList argument
Removed an unreachable code block in fgRemoveStmt and replaced it with a noway_assert
Added method headers for optUnmarkCSE and optUnmarkCSEs
Removed the code that allowed unmarking of a CSE def, instead we return false
Retyped the second arguments to optUnmarksCSEs and optValueNumCSE_UnmarkCSEs
 to support appending new nodes to keep using wbKeepList
Code modifications to PerformCSE() to support collecting both persistent side effects and
 any nested CSE defs found in CSE uses.  Move the location of the call to
optValueNumCSE_UnmarkCSEs so that it comes before the decision to create a GT_COMMA node
 for the cse use.

Check mismatched types for SIMD copy

If we have a block copy from an enregisterable struct (today, that's just SIMD) to a different type target, it needs to be makred as address-taken, because the destination type is what's used for the copy, and all non-enregisterable destination types expect their source in memory.

Fix #16254

Update BuildTools, CoreClr to preview2-02507-02, preview2-26207-11, respectively (#16256)

Removes Extra allocations for corner cases in CombineNochecks (#16255)

* Added String Overload

Initialize stack allocated data structures. (#16259)

[Arm64] Remove SIMD hacks to get*TypeForStruct

Fix various typos in the readme (#16251)

Add configuration to options to submission.py (#16127)

Fix publishing errors (#16253)

Fix two errors:
    * Remove unneeded signed symbol catalog injection
    * Build definition is not importing becuase of duplicated task name.

Archive CI perf results (#16230)

* Archive CI perf results

I need to diagnose a failure that only reproes in CI but the CI scripts don't archive the logs on failure. This change should fix that.

* Adding pipeline build per feedback

Fix Hebrew calendar month names list (#16247)

* Fix Hebrew calendar month names list

Disable crossgen for types containing intrinsic types (#16213)

Merge pull request #16241 from BruceForstall/FixJobNames

Fix name of OSX10.12 and Ubuntu innerloop jobs

Fix stacktrace formatting to make it work on CoreRT as well (#16245)

Add test case for contstructor devirtualization example (#16239)

See #16198 for discussion.

Fix GC reproting for by-ref-like structs (#16231)

Fix GC reproting for by-ref-like structs

Fixes https://github.com/dotnet/coreclr/issues/16044:
- Divide by pointer size was removed to produce the proper offset
- Fixed some assertion failures seen in https://github.com/dotnet/coreclr/issues/16229

Merge pull request #16200 from fiigii/loadstore

Implement AVX/AVX2/SSE3 Load* intrinsics

Implement AVX2 LoadAlignedVector256/128NonTemporal

Implement SSE3 LoadAndDuplicateToVector128 and LoadDquVector128

Implement AVX LoadAlignedVector256, LoadDquVector256, and LoadVector256

Merge pull request #16228 from fiigii/store

Implement AVX/SSE2 Store* intrinsics

Merge pull request #16235 from CarolEidt/FixPrefetchBuildBreak

Disable prefetch instructions for LEGACY_BACKEND

Fix name of OSX10.12 and Ubuntu innerloop jobs

Merge pull request #16006 from BruceForstall/AddWindowsArmCoreFxTesting

Enable Windows ARM32 corefx testing

Don't remove the first non-internal block that has profile weight. (#16227)

When using profile weights, fgComputeEdgeWeights expects the first non-internal block to have profile weight.
Make sure we don't break that invariant when removing an empty block.

This fixes VSO 546031.

Fix handling of return code for system() calls (#16225)

The value returned from system() is encoded bag of bits that gets
misinterpretted once it starts propagate through the system. Normalize
it to 0/1.

Enable Windows ARM32 corefx testing

We create a flow job for each arm32 corefx mode, e.g.
arm_cross_checked_windows_nt_corefx_jitstress1_flow_prtest. This
depends on a build job that is specific to the stress mode, e.g.
arm_cross_checked_windows_nt_corefx_jitstress1_bld_prtest, which
(on x64) builds CoreCLR, then clones and builds CoreFX using run-corefx-tests.py.
In particular, it only builds the CoreFX tests; it doesn't run them.
Note that because the CoreFX test build embeds the stress mode environment
variables in its generated RunTests.cmd scripts, we need a different
corefx build dependency for each corefx test run; we can't share the
builds. The build script then ZIPs up the CoreFX tests and generated
CoreFX runtime (which will include the coreclr runtime because we built
CoreFX with the `/p:CoreCLROverridePath` argument), and archives these.

The test job, which runs on an arm64 machine, then copies the ZIPed
tests and runtime, unzips them, and runs a batch script to run each
RunTests.cmd file. Note that we don't use any existing mechanism to
run each test (such as msbuild), and I believe the CoreFX msbuild
harness doesn't have any mechanism to just run a previously built set of tests.

There is a very simple test exclusion mechanism: an entire test assembly
can be excluded by putting its name (e.g., System.IO.Ports.Tests), in
a file (e.g., tests\arm\corefx_test_exclusions.txt).

Note that this corefx testing mechanism is only enabled for arm
(aka arm32), not armlb or arm64.

Replace comment inadvertently removed

Disable prefetch instructions for LEGACY_BACKEND

Move String.Manipulation.cs to shared (#16226)

Implement SSE2 Store* intrinsics

Implement AVX Store* intrinsics

Adding tests for the StoreFence/Prefetch* APIs and the new Sse scalar overloads.

Adding support for the StoreFence/Prefetch* APIs and the new Sse scalar overloads.

Reimplement string.Replace in managed code (#16214)

Removing the Sse2.LoadScalarVector128 overloads that are invalid.

Implementation of GetFullPath(string path, string basePath) (#15579)

GetFullPathAPI Overload

Moving a number of the Sse2 hwintrinsic tests to use the test template.

Adding tests for the SSE2 LoadVector128, LoadAlignedVector128, and LoadScalarVector128 intrinsics.

Adding support for the SSE2 LoadVector128, LoadAlignedVector128, and LoadScalarVector128 intrinsics.

Remove FEATURE_ACTIVITYSAMPLING and associated code. (#16185)

Merge pull request #16218 from briansull/revert-15910

Revert #15910 [R2R] add a virtualStubParamInfo as an argument

Revert #15910 [R2R] add a virtualStubParamInfo as an argument

fix the Incorrect EventPipe Assert, fixes #16158

Added a factory to FileStreamCompletionSource (#16190)

* Merged FileStreamCompletionSource and MemoryCompletionSource
- Change FileStreamCompletionSource to take a ReadOnlyMemory<byte> instead of byte[]
- Delegate to memory.Retain if the underlying ReadOnlyMemory<byte> is backed by an array that isn't the stream's internal buffer.

* Brought back MemoryFileStreamCompletionSource
- Added a factory method to FileStreamCompletionSource that
picks the an impl or FileStreamCompletionSource or MemoryFileStreamCompletionSource based on the buffer matching the underlying stream's internal buffer.

* PR feedback
- Cleanup the condition
- Flip to Debug.Assert in FileStreamCompletionSource

* Check for null before using the PreAllocatedOverlapped

Merge pull request #16103 from mikedn/xorps

Use xorps instead of xorpd

Update CoreClr to preview2-26205-05 (#16205)

Use stackalloc in string.Split (#15435)

* Use stackalloc in string.Split

* Added initial usage of ValueListBuilder

* Added usage of list builder to string separator Split overloads

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

[RyuJIT/arm32] Support GTF_IND_VOLATILE and GTF_BLK_VOLATILE

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*