Add OS flag to calls to run-xunit-perf.py (#16100)

We were uploading windows results as win32 instead of Windows_NT.

Compiler::lookupHWIntrinsic: fix oob read (#16075)

Temporaily disable some HWIntrinsic tests due to the naming changes.

Replace TryCopyTo with CopyTo (#16078)

While doing various Span-related work involving copying, in a few places I previously used a pattern of calling TryCopyTo and then asserting its result, in places where I knew the destination was long enough.  This was because CopyTo was implemented as a wrapper around TryCopyTo and thus involved an extra unnecessary branch.  Now that that's no longer the case, I'm simplifying the call sites.

Remove ValueTuple usage from ConfiguredValueTaskAwaitable

It was superfluous and made sharing the code with corefx more challenging due to the netstandard1.0 build of System.Threading.Tasks.Extensions in corefx.

Make ValueTaskAwaiters readonly structs

The _value field wasn't previously readonly, because method calls were made on the ValueTask, resulting in a copy.  But now that ValueTask is a readonly struct, the field can be readonly, and these structs can then also be readonly.

Remove defunct CreateAsyncMethodBuilder from ValueTask

ThrowExceptionFromContextInternal, RtlCaptureContext: fix for asan (#16074)

- Save arguments on stack before calling __asan_handle_no_return in
  ThrowExceptionFromContextInternal

- Fix saving arguments on stack before calling __asan_handle_no_return
  in RtlCaptureContext for arm64

Cleanup + Small TimeZoneInfo optimization for Unix (dotnet/corert#5314)

* Delete non-shared TimeZoneInfo.cs
* Delete duplicate resource

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

Default interface methods test fixes (#15995)

* The diamondshape test should work now that #15979 and #15978 are merged.
* Create debug and retail version of diamondshape and sharedgenerics tests so that we have retail coverage. These tests hit issues around devirtualization (that is only active when RyuJIT is optimizing).
* Add license headers

Changing ConvertTo.*Scalar to ConvertScalarTo.*

Prevent compiler optimization that could cause local var values to change in multithreaded environments, in some places (#16089)

Add ActivityId Support to EventPipe (#16055)

Changing BroadcastElement to BroadcastScalar

Adding missing instruction mapping documentation and fixing some signatures.

EventPipe: Don't Generate and Emit Stacks on Rundown (#16083)

Adding the missing memory fencing and prefetch intrinsics for x86

Adding the Vector128 postfix on the Sse2.LoadScalar instructions

Merge pull request #16066 from tannergooding/sse-intrinsics

Adding basic codegen support for the SSE Store intrinsics.

Moving the listed instruction for the HWIntrinsics to its own line

Changing the hardware intrinsic APIs to use pointers rather than ref

Renaming Sse41.LoadAlignedNonTemporal to Sse41.LoadAlignedVector128NonTemporal

Adding the missing Sse2.MoveScalar intrinsics

Listing the instruction mappings for the AVX2 intrinsics.

Listing the instruction mappings for the FMA intrinsics.

Listing the instruction mappings for the AVX intrinsics.

Listing the instruction mappings for the SSE4.2 intrinsics.

Listing the instruction mappings for the SSE4.1 intrinsics.

Listing the instruction mappings for the SSSE3 intrinsics

Listing the instruction mappings for the SSE3 intrinsics.

Listing the instruction mappings for the SSE2 intrinsics.

Listing the instruction mappings for the SSE intrinsics.

Adding overloads for the x86 Scalar intrinsics to not take 'upper'.

Fix the naming of the arguments for LoadHigh and LoadLow

Fixing some inconsistencies in the Scalar intrinsics

Adding the missing Sse2.MoveMask(Vector128<byte>) intrinsic

Updating the auto-generated x86 HWIntrinsic test files to contain basic documentation.

Move TimeZoneInfo.Unix.cs to shared CoreLib partition (#16057)

- Refactored internal version of System.IO.File to be similar to full corefx version
- Cleaned up some Unix PAL APIs that are no longer used

Contributes to dotnet/corert#5248

EnsureCapacty(0) should return zero as capacity for a non initialized Dictionary rather than doing initialization. (#16076)

Issue is: dotnet/corefx#24445

Merge pull request #16077 from mikedn/smalltype

Use genTypeCanRepresentValue only with small types

Update Span/ReadOnlySpan strings

Use corelib-shared ValueStringBuilder

Use the shared version, delete the copy.

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

JIT: don't overlook strong nearby preference in LSRA (#16028)

When allocating an interval, LSRA will look at related intervals
for preference hints.

LSRA may look past a related interval I1 to its related interval
I2 if I1 looks like a simple copy. But if I1 has been previously
allocated then it seems we are often better served using I1's
preferences instead of using I2's.

Addresses some issues seen in #11390.

Perform PhysicalMemoryLimit check for workstation GC, refactor GetLargestOnDieCacheSize into GetCacheSizePerLogicalCpu (#15975)

* refactor: combine GetLargestOnDieCacheSize and GetLogicalCpuCount in GetCacheSizePerLogicalCpu

* Perform PhysicalMemoryLimit check also for workstation GC

Update CoreClr to preview2-26129-05 (#16073)

Added Grisu3 algorithm support for double.ToString(). (#14646)

* Added Grisu3 algorithm support for double.ToString().

- Implemented Grisu3 algorithm.
- When calling double.ToString(), try Grisu3 first, if it fails, fall back to Dragon4.

Fix #14478

* Added comments. Fixed a minor bug.

* Optimized the performance in DigitGen.

* Added shortcut for numbers without fraction.

* Updated according to the review comments.

Added more comments. Changed the value of D_1_LOG2_10

Merge pull request #15833 from sdmaclea/PR-ARM64-INITIAL-INTRINSIC-FRAMEWORK

[Arm64] Initial HW intrinsic framework

Disable broken EventPipe tests at build time (#16046)

Use EventSource guid ctor for ArrayPoolEventSource (#16054)

* Use EventSource guid ctor for ArrayPoolEventSource

* missing e

Use Guid ctor for TplEtwProvider (#16060)

Fix reflection to work with default interface methods (#16034)

Fixes #15645.
Fixes #15644.

Use genTypeCanRepresentValue only with small types

It's not needed and it's problematic to make it work for other types. In the JIT constant values are represented using `ssize_t` and you can't distinguish between `-1` and `UINT64_MAX`.

improve DllImport library name variation trying (#15912)

* dllimport: refactor libname variation code

* Change order of libname variations depending on suffix presence

* make MAX_LIBNAME_VARIATIONS static

* PR feedback

* Fix non PAL build

Update PgoData to master-20180129-0045 (#16070)

Fix ProjectN build break (dotnet/corert#5302)

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

Fix metadata format error checking (#16036)

NextStream_Verify can return NULL for invalid file. Report error for it instead of AVing.
Similar check is around the other calls to NextStream_Verify. It was missing here for some reason.

Update CoreClr to preview2-26129-01 (#16067)

Marking SSE HWIntrinsics as fully implemented

Adding tests for the SSE Store, StoreAligned, StoreAlignedNonTemporal, StoreHigh, StoreLow, and StoreScalar intrinsics

Adding support for the SSE Store, StoreAligned, StoreAlignedNonTemporal, StoreHigh, StoreLow, and StoreScalar intrinsics

Improve Interlocked.Exchange<T> (#16058)

Replace TypedReference with Unsafe.As, it generates far less code and, with the help of AggresiveInlining, it allows Exchange<T> to inline.

Update CoreClr, PgoData to preview2-26128-04, master-20180128-0035, respectively (#16059)

Update CoreClr to preview2-26128-01 (#16056)

Updating several of the Sse HardwareIntrinsic tests to be generated from a template.

Updating the Avx2.Add HardwareIntrinsic tests to be generated from a template.

Updating the Avx HardwareIntrinsic tests to be generated from a template.

Updating the Sse2 HardwareIntrinsic tests to be generated from a template.

Merge pull request #15961 from tannergooding/struct-packing

Updates `Vector64<T>`, `Vector128<T>`, and `Vector256<T>` to have the appropriate packing.

Moving the SetIsIntrinsic code earlier in the method table builder.

Updating the Vector64, Vector128, and Vector256 StructPacking tests.

Updating the VM to properly pack the SIMD hardware intrinsic types.

Updating the methodtablebuilder to handle the System V ABI packing requirements for i386

Adding some StructPacking tests to ensure existing types don't regress

Update BuildTools, CoreClr, CoreFx, PgoData to prerelease-02427-01, preview2-26127-04, preview2-26127-04, master-20180127-0046, respectively (#15992)

Optimize Span.Copy and Span.TryCopyTo (#15947)

* Introduce a ref-based version of Buffer.Memmove
* Remove pinning logic from Span.CopyTo
* Tweak flow graph of Span.CopyTo / TryCopyTo in order to encourage better codegen
* Push some uncommon logic (one-element buffers, perfectly overlapping buffers) down to Memmove

Fix some tests under GCStress (#16048)

- Disabled one test, it takes far too long under GCStress
- For another, added a second call to WaitForPendingFinalizers() to ensure correspondence with a GC.Collect() call

Fix side effect flags setting after expression cloning. (#16045)

Side effect flags need to be recomputed after cloning
an expression, since cloning may involve some simplifications
(e.g., replacing a local with a const).

I also did some minor refactoring and renaming of the side-effect-updating helpers
and made one of them more robust by checking whether a child is null before accessing
its flags.

This fixes VSO 543054 where this problem was encountered in a PMI ARM32 JitStress=2 run.

GCPrivate, GCHandle, and GChandlePrivate events for local GC (#16020)

* first event changed

* rest of the private events

* whitespace fixes

[Arm64] Initial HWIntrinsic implementation

Fix FreeBSD build and update build instructions (#15998)

Move test code include in global CMakeLists before PAL runtime includes to
avoid conflicts between PAL and system headers.
(ForeignThreadExceptionsNative.cpp -> thread -> functional -> memory ->
cassert -> pal rt "assert.h".)  Drop no longer needed remove_definitions
lines -- they're only set by the clrdefinitions.cmake include later in
the file.

Update build instructions to include needed components (ninja,
lttng-ust, python27).

Add missing ElfW() definition when libunwind does not define it.

FreeBSD alloca() is defined in stdlib.h, so include that instead of
alloca.h on FreeBSD.

Merge pull request #16035 from jashook/use_checked_centos

Use centOS Checked jobs

Use centOS Checked jobs

fix pointer in ILCodeVersionNode so it uses PTR_COR_ILMETHOD instead of COR_IL_METHOD *, which was causing a crash in the DAC (#16003)

Create the concept of EventPipeSession and refactor EventPipe to use it.  Also expose COMPlus configuration variables to support session configuration and output file path. (#16018)

Fix Windows-Specific EventPipe Bugs (#16025)

* Modify IsEnabled macros used on Windows.

* Fix arithmetic error due to order of operations.

[Local GC] FEATURE_EVENT_TRACE 3/n: Defining and Firing Dynamic Events (#16000)

Compute System.Private.CoreLib roots required by the runtime. (#15525)

Embed the result as a System.Private.CoreLib.dll xml resource
for consumption by ILLink.

Update mscorlib.h to make several dependencies explicit.

The actual msbuild task lives in the linker repo:
https://github.com/mono/linker/blob/master/corebuild/integration/ILLink.Tasks/CreateRuntimeRootDescriptorFile.cs

Fix asan false-positive errors: (#15563)

- Call __asan_handle_no_return in RtlRestoreContext if it doesn't return
  and in ThrowExceptionFromContextInternal function;

- Increase alternate signal stack size and use it also for asan.

Enable tiered jitting for R2R methods (#15967)

Enable tiered jitting for R2R methods

- Included R2R methods and generics over value types in CoreLib for tiered jitting. Tier 0 for R2R methods is the precompiled code if available, and tier 1 is selectively scheduled based on call counting.
- Added a delay before starting to count calls for tier 1 promotion. The delay is a short duration after frequent tier 0 jitting stops (current heuristic for identifying startup).
- Startup time and steady-state performance have improved on JitBench. There is a regression shortly following startup due to call counting and tier 1 jitting, for a short duration before steady-state performance stabilizes.
- Added two new config values, one for configuring the call count threshold for promoting to tier 1, and another for specifying the delay from the last tier 0 JIT invocation before starting to count calls

Revert "Add environment variable (COMPlus_EnableDiagnostics) to disable debugging and profiling. (#15878)"

This reverts commit 5bcfde404803f85451cf0ee9fd6406734cb878ff.

 Add GetLoadLibrary function for PAL & use in TrackErrorCode (#15831)

* Add GetLoadLibrary function for PAL & use in TrackErrorCode

* Finish changes to dllimport.cpp

* Fix unix build errors

* fix windows issues

* Address feedback

* Fix return type

* Address feedback

* Fix return type

* Fix usage of LPCWSTR

* Use shared library string

* Append message strings

* Fix append

* Append newline

* Fix spelling

* Modify error messages

* Resolve conflicts

* Add mac message & stop appending all exceptions

* Fix another error message

* Fix NoName Error

* Add newline to .h file

Resolve iterators and async methods (#14655)

Allows a developer to determine *correct* overload called for *async* and *iterators*, from a stacktrace.

Matches the compiler generated containing `Type` to the source method using:
`method.DeclaringType == StateMachineAttribute.StateMachineType` attribute to allow the stack trace to resolve the mangled method name to the correct overload.  Also appends the statemachine method name with `+` for iterators.

Resolves: dotnet/corefx#24627

Merge pull request #16017 from jashook/fix_inner_loop_centos_test

Correctly allow buildOnly jobs for release x64

Correctly allow buildOnly jobs for release x64

Merge pull request #16005 from CarolEidt/Fix15848

Disable FP to Unsigned Vector Conversions

Minor bug fix for Array.Sort: Replace keys.Length with length for FloorLog2PlusOne (#16002)

* replace keys.Length with length for FloorLog2PlusOne

* fix non-generic sort

[x86/Linux] Fix HelperMethodFrame::UpdateRegDisplay (#15993)

For DAC after initialization MachState using InsureInit method,
register pointers are NULL so we cannot use them to restore register
values.

Disable the ConvertToUInt intrinsics by removing them from simdintrinsiclist.h

Merge pull request #15813 from jashook/inner_loop_fixes

Add innerloop tag for pri0 jobs

Add innerloop tag for pri0 jobs

Everything that is not innerloop will run priority 1.