Merge pull request #3150 from krytarowski/netbsd-support-42

Fix build issue on NetBSD: Include <stdlib.h> for alloca(3)

Merge pull request #3146 from mikem8361/vso190301

Fix VSO bug #190301 contract problems in DACRaiseException.

Fix build issue on NetBSD: Include <stdlib.h> for alloca(3)

Header <alloca.h> isn't needed on other supported systems either.

/tmp/pkgsrc-tmp/wip/coreclr-git/work/coreclr/src/pal/src/file/directory.cpp:605:
warning: Warning: reference to the libc supplied alloca(3); this most likely will
not work. Please use the compiler provided version of alloca(3), by supplying the
appropriate compiler flags (e.g. not -std=c89).

$ uname -a
NetBSD chieftec 7.99.25 NetBSD 7.99.25 (GENERIC) #0: Fri Dec 25 20:51:06 UTC 2015  root@chieftec:/tmp/netbsd-tmp/sys/arch/amd64/compile/GENERIC amd64

Merge pull request #3076 from krytarowski/netbsd-support-37

Permit to inject additional CMake arguments via environment variables

Fix VSO bug #190301 contract problems in DACRaiseException.

Added the proper contracts to the DAC notification functions.

Merge pull request #3122 from krytarowski/netbsd-support-40

Fix build on NetBSD: Include <cstdarg> for va_list

Merge pull request #3136 from sivarv/simdfix2

Fix to issue #2970

Merge pull request #3082 from sejongoh/update_corefx_build_steps

Fixed corefx build failures

Merge pull request #3132 from stephentoub/taskscheduler_tracking

Track TaskSchedulers only when debugger attached

Merge pull request #3139 from stephentoub/task_id_perf

Fix access to Task.Id while finishing continuations

Updated build steps to fix corefx build issues

Fix access to Task.Id while finishing continuations

Task.Id is lazily-initialized with an interlocked operation and we strive to avoid accessing it anywhere in Task's internals on hot paths, but apparently when some ETW logging was added around continuations, calls were added that access Task.Id even if logging is disabled.  This is accounting for ~30% of the cycles consumed in a TCS Task's lifecycle!  This commit fixes it by making sure these accesses to Id are guarded.

Fix to issue #2970

The repro is a case of one of the method called is returning Vector3/4 type value.
On unix such a struct will gets returned in two registers and hence will
be assigned to a local by importer and marked as multiRegArgOrRet=1.
But it doesn't ensure that the local will not be register allocated.
For the repro case it gets register allocated.

IR: GT_LCL_FLD(tmp1) = GT_CALL returning SIMD12

Codgen routine genStoreRegisterReturnInLclVar() for stroing a struct returned in two registers to a lcl var or lcl_fld
is making the assumption that lcl var will be on stack not in a register.
This may not be true for SIMD structs returned in two registers.

For now to workaround Codegen limitation marking tmp1 as DoNoEnregister.

Fixes #2970

Merge pull request #3137 from wtgodbe/disable

Disable pri 1 tests failing in the CI

Disable pri 1 tests failing in the CI

Track TaskSchedulers only when debugger attached

Today the base TaskScheduler ctor registers every created scheduler in a ConditionalWeakTable.  This is then used in the internal TaskScheduler.GetTaskSchedulersForDebugger() to enable a debugger to find all of the schedulers that are active and then in turn find all tasks scheduled to them.  However, for scenarios that involve creating many, many schedulers, this registration adds up to a non-trivial cost, both in time and space, and is only relevant for debugging.  Plus, since this mechanism was introduced, Visual Studio has additional ways of tracking tasks for display in the Tasks window, e.g. ETW events.

This commit simply makes that registration more pay-for-play, only doing it if the debugger is attached at the time of the scheduler's creation.

Merge pull request #3119 from ellismg/add-rhel72

Add Red Hat Enterprise Linux 7.2 Runs

Merge pull request #3126 from jkotas/crossgen

Change crossgen defaults

Merge pull request #3120 from erozenfeld/MultToShiftVN1

Small correction for #3067.

Merge pull request #3127 from AndyAyersMS/FlakyTestDisable

Disable test causing Ubuntu failures

Disable test causing Ubuntu failures

Fails because of mismatched System.IO versions. See #3117 for some
discussion.

Change crossgen defaults

Change crossgen to generate R2R images by default

Merge pull request #3118 from jkotas/buffercopy

Add a fast path for byte[] to Buffer.BlockCopy

Add a fast path for byte[] to Buffer.BlockCopy

Fix build on NetBSD: Include <cstdarg> for va_list

Small correction for #3067.

The code to change multiplication to shift shouldn't be
under LEA_AVAILABLE since the code to process multiplication
by power of 2 isn't.

Add Red Had Enterprise Linux 7.2 Runs

Add new parameter with settable option: cmakeargs

This will allow to pass user-defined CMake arguments.

One of the users is pkgsrc, which defines it to following example string:
-DPYVERSSUFFIX:STRING=2.7 -DCMAKE_INSTALL_PREFIX:PATH=/usr/pkg \
-DCMAKE_MODULE_PATH:PATH=/tmp/pkgsrc-tmp/wip/coreclr-git/work/.buildlink/cmake-Modules \
-DCMAKE_SKIP_RPATH:BOOL=TRUE -DCMAKE_INSTALL_LIBDIR:PATH=lib \
-DCMAKE_INSTALL_MANDIR:PATH=man

Without these values unmodified build.sh won't integrate with pkgsrc as
expected.

Solution suggested by Jan Vorli (Microsoft)

While there, there was need to alter loop type to handle options passed to
arguments. As a bonus, new loop should be POSIX confirming.

Merge pull request #3112 from mikem8361/sosfix

Fix SOS managed breakpoints when coreclr symbols are stripped.

Fix SOS managed breakpoints when coreclr symbols are stripped.

Added a SOS DAC interface (ISOSDacInterface4::GetClrNotification) to get the exception
notification arguments instead of using the GetLastExceptionInformation function from the lldb
sosplugin that depends on coreclr symbols being present.

On the coreclr side, the clr notification arguments are saved in a global variable that is DAC
accessible. A critical section was added to protect this global variable while the special
exception is raised.

Setting the internal COMPlus_DebugBreakOnAssert environment variable causes 3 or 4 breaks in
the debugger with no reason.  It was breaking in the function that was determining whether it
should break.  I was using COMPlus_BreakOnEELoad=2 to break after coreclr was loaded and initialized
to set managed breakpoints and on a debug build it generates an assert (on release just a DebugBreak).

Merge pull request #3107 from wtgodbe/reenable

Re-enable previously disabled tests that are now passing on non-Windows platforms

Merge pull request #3095 from krytarowski/netbsd-support-38

Add support for Floating Point registers in ucontext_t on NetBSD

Merge pull request #3114 from mmitche/switch-to-auto

Switch to auto images

Switch to auto images

Merge pull request #3110 from jkotas/event-tracing

Make Unix event tracing conditionally enabled

Merge pull request #3108 from sivarv/gcstressfix

Fix to gc-stress issue #2921.

Make Unix event tracing conditionally enabled

Add condition around the expensive event tracing code to enable it conditionally only if the event tracing is enabled

Merge pull request #3106 from briansull/issue_3032

ARM64: For methods that have very large stack frames we need extra instructions

For methods that have very large stack frames we need an extra instruction
to load the offset when encoding the prolog/epilog instructions for funclets

Fixes Issue #3032
Incorporates code review feedback and provides a method called
genInstrWithConstant to handle the cases where we need extra instructions
Updated arm64\Tests.lst with additional tags:
MISSING_EXE, ISSUE_3104, ISSUE_3105 and NEED_TRIAGE

Fix to gc-stress issue #2921.

Merge pull request #3055 from wtgodbe/serverGC

Re-enable server GC & lengthen timeouts

Re-enable previously disabled tests that are now passing on non-Windows platforms

Re-enable server GC & lengthen timeouts

Merge pull request #3100 from AndyAyersMS/FixWarning

Fix const string literal warning

Merge pull request #3099 from CarolEidt/x86IntArgs

Constant and Popped Args for x86

Merge pull request #3102 from mmitche/add-llitchev-to-allowed

Add llitchev to allowed

fixup

Update arm triggers to be in line with others

Constant and Popped Args for x86

Add support for constant arguments,
and handle the case where the callee must pop the stack.

Fix const string literal warning

This came up during an FI from LLVM, testing LLILC, see microsoft/llvm#140 .

Merge pull request #3096 from gkhanna79/MonoCheck

Check for Mono presence until CoreCLR 3090 is fixed.

Check for Mono presence until CoreCLR 3090 is fixed.

Add support for Floating Point registers in ucontext_t on NetBSD

This code assumes x86_64 platform and modern fxsave style of FP regs.

i386 Floating Point registers ship with different flavors, what will be
handled once the 64-bit code will be finished.

Merge pull request #3086 from gkhanna79/CIMscorlib

Update CI to no longer request mscorlib build for osx/ubuntu on Windows.

Merge pull request #3093 from dotnet-bot/from-tfs

Merge changes from TFS

Merge pull request #3091 from jkotas/revert-workaround

Revert "Add un-prefixed signatures as temporary workaround"

Merge pull request #3084 from AndyAyersMS/InlineRefactor2b

Inline refactor: revise checks in fgFindJumpTargets

Merge pull request #2905 from AndyAyersMS/NewILTests

Some tests for caller/callee type mismatches

Merge pull request #3083 from AndyAyersMS/InlineRefactor2a

Inliner refactoring: inline impAbortInline

Merge pull request #3087 from sivarv/master

Fix to SquareRootUInt16 test failure under JitStress=1/2

Merge pull request #3067 from erozenfeld/MultToShiftVN

Preserve value numbers when morphing multiplication into shift.

Update CI to no longer request mscorlib build for osx/ubuntu on Windows.

Revert "Add un-prefixed signatures as temporary workaround"

This reverts commit fb80bad2ed19970472ddefe539520abef42a52d0.

Merge pull request #2981 from kouvel/Fixes

Miscellaneous fixes

Preserve value numbers when morphing multiplication into shift.

This change fixes a morpher transformation of multiplication to shift
to preserve the value number of the tree since the new shift expression
will compute the same value as the old multiplication expression.
Without that change we were getting  asserts in fgMoveOpsLeft,
which expects to see value numbers on trees it transforms.

Closes #2920.

Fix to SquareRootUInt16 test failure under JitStress=1/2

JitStress=1/2 is inducing GS check as a result of which the Vector argument
gets shadow copied.  But on the shadow copy lcl var SIMD specific state
is not set and as a result of which rationalization doesn't happen
properly leading to silent bad codegen.

Inline refactor: revise checks in fgFindJumpTargets

In this method, the inliner will check for unsupported opcodes and
similar things. Refactor these checks to directly bail out if an
issue is discovered, rather than going to a common handler snippet.

There are two functionality changes:

First, if an unsupported opcode is found (currently: switch, jmp,
calli, localloc, cpblk, mkrefany, rethrow) mark the inlinee as
never inlinable, rather than just failing the current attempt.
This should not change externally visible behavior but will improve
jit TP marginally if such cases are common.

Second, for NETCF quirk handling, make sure to guard some of the
observations with a check that we're reading the IL for inlining.

Merge pull request #2898 from adityamandaleeka/remove_sigprocmask

Use pthread_sigmask to change the mask of blocked signals.

Inliner refactoring: inline impAbortInline

Inline this helper method at its 3 call sites, and delete it.

Fix build breaks in the CTP JIT.

These breaks were due to unguarded usage of constants that are not present in the CTP JIT/EE interface.

[tfs-changeset: 1573367]

Merge pull request #3050 from sejongoh/add_more_stress_modes_to_corefx_windows_testing

Add more JIT stress modes to corefx testing

Merge pull request #3043 from gkhanna79/FixNugetPkg

 Generate Microsoft.NETCore.Runtime.CoreCLR nuget packages

Change to support Microsoft.NetCore.Runtime.CoreCLR nuget package generation.

Merge pull request #3073 from krytarowski/netbsd-support-36

Make missing libunwind fatal on NetBSD

Make missing libunwind fatal on NetBSD

Merge pull request #2973 from krytarowski/netbsd-support-25

Disable FEATURE_HIJACK in NetBSD as unimplemented

Merge pull request #3072 from krytarowski/netbsd-support-35

Fix build on NetBSD: Support different versions of "struct reg"

Fix build on NetBSD: Support different versions of "struct reg"

FreeBSD style: ".r_rax"
NetBSD x86_64 style: ".regs[_R_RAX]"

NetBSD i386 style: "._r_eax" will be defined once the amd64 version will be
finished.

Disable FEATURE_HIJACK in NetBSD as unimplemented

Solution suggested by Jan Kotas (Microsoft).

Merge pull request #3071 from dotnet/revert-3062-instrumentation

Revert "Temporary instrumentation in native image loading"

Merge pull request #3070 from jkotas/crossgen-fix

Fix loading of native images on Unix

Revert "Temporary instrumentation in native image loading"

Fix loading of native images on Unix

R2R images that were compiled from architecture-specific IL images failed to load with bad image format exception on Unix. The fix is to take IMAGE_FILE_MACHINE_NATIVE_OS_OVERRIDE into account for the platform check.

Merge pull request #3066 from gkhanna79/Fix3049

Fix for CoreCLR issue 3049

Fix for CoreCLR issue 3049

Merge pull request #3062 from jkotas/instrumentation

Temporary instrumentation in native image loading

Temporary instrumentation for PE file loading

Merge pull request #3059 from jkotas/intrinsics

Improve robustness of intrinsic fallback path

Merge pull request #3063 from dotnet/revert-3027-remove_security_checks

Revert "Remove useless security checks"

Merge pull request #3061 from stephentoub/update_index

Tweak Enum.TryParse whitespace skipping slightly

Revert "Remove useless security checks"

Merge pull request #3060 from dotnet-bot/from-tfs

Merge changes from TFS

Tweak Enum.TryParse whitespace skipping slightly

As a simple addendum to my previous Enum.TryParse change, if the string
begins with whitespace, we already know where the whitespace ends, so we
don't need to retraverse it.

Merge pull request #3048 from sejongoh/add_unit_test_for_issue_2580

Add unit test for issue 2580

Add corefx baseline

Add unit test for issue 2580

- The test checks if code generation for ushort is correct.

Delete mdil support from coreclr

[tfs-changeset: 1573006]

Improve robustness of intrinsic fallback path

- The fallback path for intrinsics nodes is assumed to be direct call. Verify this assumption using no_way assert instead of regular assert.
- Make VM to always tell the JIT to use direct calls for intrinsics, even in version resilient code

Change corefx jit stress scenario names due to git complaining too long
file name

Merge pull request #3056 from krytarowski/netbsd-support-34

Fix NetBSD build: Define virtual destructors when needed