Merge pull request #16281 from alpencolt/clr-extra-args-4-cmake

[Unix] Passing extra CMake args

Merge pull request #16278 from BredPet/fix_cfi_unwind_info

[RyuJIT/ARM32] Fix crash while generating CFI unwind info

Adjust expected file size if ZapDisable is true (#16240)

Merge pull request #15673 from tannergooding/generatelayout-switch

Updating build.cmd to support 'generatelayout' when 'skiptests' is specified.

Debugger api to set a breakpoint on offset 0 of all methods (#15819)

* Debugger api to set a breakpoint on offset 0 of all methods

* code review feedback

* more code review feedback

* respect IsIl in CorDbFunctionBreakoint()

* change SIMPLIFYING_ASSERT to SIMPLIFYING_ASSUMPTION_SUCCEEDED for hrs, it outputs a much better diagnostic message

Updating runtime version for tests (#16271)

Change 93ec62347a4e97b920513296eefb27bdacbecfe9 removed Span.NonPortableCast from System.Private.CoreLib.dll requiring tests
to use an updated version of System.Memory.dll that no longer refers to it.

Explictly initialize EventData.Reserved for System.Private.CoreLib. (#16283)

Extra semicolons left at the end of Combine function taking 4 paths. (#16284)

Add tests for Sse2 memory fence instructions

Implement Sse2 memory fence instructions

Merge pull request #16268 from CarolEidt/Fix16254

Check mismatched types for SIMD copy

Merge pull request #16260 from sdmaclea/PR-ARM64-Remove-SIMD-hacks

[Arm64] Remove SIMD hacks to get*TypeForStruct

[Unix] Passing extra CMake args

Added support for compiler and linker additional arguments in CMake.
By using CLR_ADDITIONAL_LINKER_FLAGS and CLR_ADDITIONAL_COMPILER_OPTIONS
it's possible to pass toolchain options without changing Makefile.

Merge pull request #16279 from BruceForstall/FixArmCorefxAbsolutePaths

Make paths used by ARM corefx test script absolute

Make paths used by ARM corefx test script absolute

Cleanup redundant stale COR header definition (#16261)

[RyuJIT/ARM32] Fix crash while generating CFI unwind info

Signed-off-by: Petr Bred <bredpetr@gmail.com>

Throwing IO Related exception only on FileStream.Dispose(disposing:true) (#16250)

* Making Unix and Windows FileStream Dispose behavior more consistent and throwing IO related exception only on Dispose(true)

Fixes: #26734

* Commenting out SecurityException, also providing reason

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

Fixes #15648.

Agruments -> Arguments (#16273)

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

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

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>