Enable linux corefx testing

Merge pull request #3183 from janvorli/fix-unwound-funclet-stack-reporting

Fix unwound funclet GC stack reporting on Unix

Merge pull request #3157 from benaadams/QueueUserWorkItemCallback

Slimmer QueueUserWorkItemCallback

Fix unwound funclet GC stack reporting on Unix

This change fixes a problem with reporting object on stack for GC when
GC scan is performed right after a funclet is unwound during exception handling.
In such case, the parent of the funclet should report live object references,
but it was not doing so on Unix.
On Unix, the unwound funclet is detected in a different way than on Windows due
to the fact that the stack frames of the funclet are already reclaimed, while on
Windows, they are still there.
When we detect the unwound funclet on Unix, we set state of the stack walker so
that it behaves in the same way as on Windows. But we were missing one state
variable that makes the parent report the references instead of the funclet,
the m_fDidFuncletReportGCReferences.

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

Prefer BSD uuid over libuuid

Merge pull request #3149 from krytarowski/netbsd-support-41

Fix build issue on NetBSD: Include <time.h> for time(3) in gc.cpp

Merge pull request #3171 from krytarowski/netbsd-support-45

Add more CMake sugar for NetBSD

Merge pull request #3160 from josteink/master

Make the coreclr build-process support rtags

Merge pull request #3175 from krytarowski/netbsd-support-46

Fix build issue on NetBSD: Remove unneeded -nostdinc++ flag

Slimmer QueueUserWorkItemCallback

Prefer BSD uuid over libuuid

This fixes recent NetBSDbuild breakage:
../libcoreclrpal.a(miscpalapi.cpp.o): In function `CoCreateGuid':
/tmp/pkgsrc-tmp/wip/coreclr-git/work/coreclr/src/pal/src/misc/miscpalapi.cpp:356: undefined reference to `uuid_generate_random'
clang-3.9: error: linker command failed with exit code 1 (use -v to see invocation)

No need to link against libuuid.

Add detection of BSD-styl <uuid.h>.
Few Linux distros ship with custom <uuid.h> and they aren't compatibile.

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

Merge changes from TFS

Fix build issue on NetBSD: Remove unneeded -nostdinc++ flag

This flag isn't recognized by clang on NetBSD:
clang-3.9: warning: argument unused during compilation: '-nostdinc++'

$ pkg_info |grep -E 'lldb|llvm|clang'
llvm-3.9.0nb20160213 Low Level Virtual Machine compiler infrastructure
clang-3.9.0nb20160213 C language family frontend for LLVM
lldb-3.9.0nb20160213 next generation, high-performance debugger

$ uname -a
NetBSD chieftec 7.99.26 NetBSD 7.99.26 (GENERIC) #0: Wed Feb 10 21:58:18 UTC 2016
root@chieftec:/tmp/netbsd-tmp/sys/arch/amd64/compile/GENERIC amd64

This flag was disabled also for Darwin in #125 by @jkotas

Fix build issue on NetBSD: Add time(3) to PAL

Otherwise it triggers linker warnings.

/tmp/pkgsrc-tmp/wip/coreclr-git/work/coreclr/src/gc/gc.cpp:33869:
warning: warning: reference to compatibility time(); include <time.h> for correct reference

$ 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 #3172 from RussKeldorph/redisable

Re-disable tests failing due to #2728

Re-disable tests failing due to #2728

Add more CMake sugar for NetBSD

This resolves 'gen-dactable-rva.sh does not print to dactablerva.h' #3168

This fixes issue with g_dacTable unavailable in libcoreclr.so.

This patch assumes GNU ld(1) on NetBSD

Thanks to Bruce Mitchener @waywardmonkeys for pointing out ld(1) nits.

Make the build-process support rtags

Rtags depends on cmake to export its build commands in order to index
the codebase.

Adding this flag makes cmake do that.

Merge pull request #3161 from richardlford/restoredllnames0212

Restore original dll names, use Define constants.

Merge pull request #3145 from erozenfeld/PreserveCastVN

Preserve value numbers when discarding a cast.

Fix hfa_interop to omit ".dll" from DLLName

For interoperability cross platform, the DLL name given to the
DllImport attribute should not include the suffix since
dynamic link libraries use a different suffix on different
systems.

Merge pull request #3159 from mikem8361/security

Change the transport pipe and runtime startup semaphores permissions

Fix build issue #788907: crossgen /createpdb not working

[tfs-changeset: 1575079]

Restore original dll names, use Define constants.

This changes set has no functional change, rather it cleans
up some aspects of a previous port.

When the hfa dlls were originally ported it was not known that
C# #defines could be used in the tests, so source files that
use #if were expanded to multiple source files with the preprocessing
being done manually. This changes goes back to using the original
files with appropriate define constants in the project files.

In addition, when these dlls were ported abbreviated names were
used. This change restores the dll names to the originals.

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.