Merge pull request #3145 from erozenfeld/PreserveCastVN

Preserve value numbers when discarding a cast.

Merge pull request #3159 from mikem8361/security

Change the transport pipe and runtime startup semaphores permissions

Merge pull request #3153 from briansull/issue_2816_v2

Fix issues with passing 16-byte structs on Arm64

Change the transport pipe and dbgshim runtime startup semaphores permissions to user only

Since the mechanism used to enumerate process modules (/proc/xxxx/maps on Linux, running
vmvmap on OSX) is user only and the default group new users get on OSX is the same ("staff")
restricting the transport pipes and runtime startup named semaphores to user only is the
right way to go.

If we find some reason to give groups permissions, we can always change that later though
the process module enumeration would have to be figured out.

Fix issues with passing 16-byte structs on Arm64

Properly track the GC pointers that are being loaded/stored or copied from these pass by value structs.
Implements passing 16-byte value types in the outgoing stack area, previously not implemented.
Added utility method getJitGCtype for converting the VM's CorInfoGCType to the JIT's var_types enum
Added utility method getStructGcPtrsFromOp to examine a GT_LDOPB or GT_LCL_VAR
 that represents a 16-byte struct and get the GC layout information
Allow LSRA to pick two consecutive registers to pass 16-byte structs on the stack
Added utility methods checkAndAssignInterval and checkAndClearInterval to handle the
 second register used to pass arguments on ARM64.

Merge pull request #3121 from sejongoh/add_combo_stress_options

Add JitStress x JitStressRegs combo stress tests

Merge pull request #3158 from krytarowski/netbsd-support-44

Fix build on NetBSD: Allow to overload definition of wchar_t

Merge pull request #3088 from RussKeldorph/testwarn

Fix warnings in JIT test build

Fix build on NetBSD: Allow to overload definition of wchar_t

This allows to build 100% of the sources on NetBSD.

In file included from /tmp/pkgsrc-tmp/wip/coreclr-git/work/coreclr/src/pal/src/safecrt/safecrt_output_l.c:70:
/usr/pkg/bin/../lib/clang/3.9.0/include/stddef.h:90:24: error: typedef redefinition with different types ('int' vs 'unsigned short')
typedef __WCHAR_TYPE__ wchar_t;
                       ^
/tmp/pkgsrc-tmp/wip/coreclr-git/work/coreclr/src/pal/inc/pal_char16.h:32:17: note: expanded from macro 'wchar_t'
                ^
/tmp/pkgsrc-tmp/wip/coreclr-git/work/coreclr/src/pal/inc/pal_char16.h:44:24: note: previous definition is here
typedef unsigned short wchar_t;
                       ^
/tmp/pkgsrc-tmp/wip/coreclr-git/work/coreclr/src/pal/inc/pal_char16.h:32:17: note: expanded from macro 'wchar_t'
                ^
1 error generated.

Closes wchar_t redefinition broken on NetBSD #2937

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

Preserve value numbers when discarding a cast.

This is a fix for an assert during CSE. After processing a CSE definition
the tree got morphed. During morphing a cast was discarded by narrowing
the operand of the cast. The value number of the operand was cleared. Later,
when processing a use of the CSE, the code was attempting to process the side effects
in the tree of the use and was encountering a node with no value number resulting
in an assert. The fix is to transfer the value number from the cast node to the narrowed
operand.

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.

Add jitstress x jitstressregs

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).

Fix warnings in test build

```
warning : Layout specified for auto-layout class
    C:\git\coreclr\tests\src\JIT\Directed\PREFIX\unaligned\1\add.ilproj
    C:\git\coreclr\tests\src\JIT\Directed\PREFIX\volatile\1\add.ilproj

warning : Non-nested class has nested visibility (0x00000102), changed to non-nested (0x00000101)
    C:\git\coreclr\tests\src\JIT\jit64\regress\vsw\153682\test.ilproj

warning : Reference to undeclared extern assembly '<assembly>'. Attempting autodetect
    C:\git\coreclr\tests\src\JIT\Methodical\explicit\misc\_il_dbgrefarg_box_f8.ilproj (System.Console)
    C:\git\coreclr\tests\src\JIT\Methodical\explicit\misc\_il_relrefarg_box_f8.ilproj (System.Console)
    C:\git\coreclr\tests\src\JIT\Methodical\tailcall_v4\hijacking.ilproj (mscorlib, System.Threading, System.Threading.Thread)

warning : Undefined implementation in ExportedType '<type>' -- ExportType not emitted
    C:\git\coreclr\tests\src\JIT\Regression\CLR-x86-JIT\V1-M09.5-PDC\b32879\b32879.ilproj (HiDad, _vbProject)
    C:\git\coreclr\tests\src\JIT\Regression\CLR-x86-JIT\V1-M12-Beta2\b65423\b65423.ilproj (b1904)

warning C4311: 'type cast': pointer truncation from 'void *' to 'unsigned int' (+redundant C4302)
    C:\git\coreclr\bin\Native\src\JIT\Directed\pinning\object-pin\mirror.vcxproj
```

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