Allow jit internal errors to propagate to user visible exception (#16320)

May help users find workarounds for cases where the jits hit some
kind of implementation limit, as in #12480 or #14762.

Update PgoData to master-20180210-0042 (#16319)

Implement certain simple SIMD intrinsics for AVX/AVX2

Add AVX/AVX2 templete-drien tests and enable containment tests

Add new test template

Add test cases for some AVX/AVX2 intrinsics

Update BuildTools, CoreClr, CoreFx, PgoData to preview2-02509-01, preview2-26209-08, preview2-26210-01, master-20180209-0035, respectively (#16269)

Updating emitxarch to check TakesVexPrefix(ins) when adding REX bits.

Add back TraceFormat.Normal to avoid additional newlines from being added

Address PR comments

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

exclude errorSource from call to Environment.FailFast on Unix

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

dependancy -> dependency (dotnet/corert#5350)

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

Fix for RETURNTRAP xarch codegen. (#16292)

This fixes a bug that caused an assert in GetSingleTempReg.
In this case we had an an interop call (resulting in RETURNTRAP in its epilog)
when there are live 256 bit values at the callsite.
The bug was that codegen for RETURNTRAP node requested a single temp register
and asserted that there was only one. In this case the set of temp registers
includes floating-point registers that may be needed for saving/restoring the upper
part of 256-bit registers. The fix was for codegen to request a single temp int register.
GetSingleTempReg will assert that there was only one int register in this case.

Fixing up fgValueNumberIntrinsic to use IsMathIntrinsic rather than checking individual cases.

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

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

* fix ifdef misplacement in pal cordebug.h

* add IID to cordebug_i.cpp

Merge pull request #16308 from sdmaclea/PR-JIT.SIMD.VectorUtil.CheckValue

Fix JIT.SIMD.VectorUtil.CheckValue<T>

Fix JIT.SIMD.VectorUtil.CheckValue<T>

Remove an old JIT64 thread suspend work around (#16304)

JIT64 isn't ever used for CoreCLR

Fix T[]->ArraySegment cast to return default for null (#16301)

Merge pull request #16248 from briansull/fix-16065

Fixes GitHub Issue 16065

Merge pull request #16217 from adamsitnik/eventPipeAssert

fix the Incorrect EventPipe Assert, fixes #16158

Merge pull request #16300 from jkotas/revert

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

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

This reverts commit 3d689d00843618105e735c5647e1cb64e721a333.

Revert "fix for issue #16289 (#16291)"

This reverts commit cd3f6e77757729d3d02dfbec4af21cdfe83f5435.

Fix GC reporting for slow tail calls with by-ref-like args (#16264)

Fix GC reporting for slow tail calls with by-ref-like args

Fixes https://github.com/dotnet/coreclr/issues/16229:
- Offsets recorded in the GC layout are intended to be offsets from the top of the stack, fixed by calculating the proper offset

Change compliement to complement

Fixing up the emitOutput handling for RR/RRR instructions

Enabling the named intrinsic support on SSE4.1 hardware for Math.Round, Math.Ceiling, and Math.Floor

Updating the emitter to more generally handle 4-Byte SSE4 instructions.

fix for issue #16289 (#16291)

Merge pull request #15595 from mikedn/rc-binop

Remove useless RangeCheck::Limit::keArray

Change corefx test process (#16263)

Instead of using the `/p:CoreCLROverridePath` switch when building
corefx, to bring in coreclr, build corefx normally. This builds
against the coreclr packages that corefx has already been validated
against in the CI, so the build should always work. Then, copy
over our built coreclr bits before testing. This method is expected
to have better behavior in most breaking change scenarios.

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)

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