Merge pull request #2006 from richardlford/gettypetypeofmatrix

Respond to review of gettypetypeofmatrix test.

Respond to review of gettypetypeofmatrix test.

The automatic formatter missed putting "public" and
"static" in a canonical order, so do that manually.

Also remove a case of Hungarian notation.

Merge pull request #2000 from richardlford/gettypetypeofmatrix

Add gettypetypeofmatrix test.

Merge pull request #1970 from mmitche/enbable-by-default

Enable coreclr tests by default

Enable coreclr tests by default

Merge pull request #1999 from adityamandaleeka/remove_suspension

More PAL thread cleanup (last one)

Add gettypetypeofmatrix test.

Ported the JIT/Directed/gettypetypeof/gettypetypeofmatrix.cs
test from the desktop clr suite to coreclr.

Remove InternalRand.

Remove thread parameter from several more PAL functions.

Merge pull request #1998 from pgavlin/AddNinjaSupport

Add support for ninja to the CoreCLR build.

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

Merge changes from TFS

Add support for ninja to the CoreCLR build.

Just what it says on the tin.

Merge pull request #1990 from stephentoub/fix_eventsource_typo

Fix a typo in EventSource_EventParametersMismatch error string

Merge pull request #1988 from stephentoub/fix_eventsource_emptyarray

Fix exception when writing null/empty arrays to EventSource

Expose Environment.GetCommandLineArgs in mscorlib

[tfs-changeset: 1546251]

Merge pull request #1925 from akoeplinger/remove-env-helper

Remove Env helper class in tests

Merge pull request #1926 from akoeplinger/fix-hardcoded-windows

Fix hardcoded Utilities.IsWindows property

Merge pull request #1987 from bbowyersmyth/StringCodeComments

Improved and corrected some String code comments

Fix a typo in EventSource_EventParametersMismatch error string

Fix exception when writing null/empty array to EventSource

EventSource has two WriteEvent overloads that take a byte[].  If the
byte[] provided is null, it uses an empty array.  It then tries to get the
address of the 0th element of the array, resulting in an
IndexOutOfRangeException.  We could instead just use ```byte* blob =
arg1``` instead of ```byte* blob = &arg1[0]```, but doing so with an empty
array has undefined behavior in C#.  Instead, we just special-case null
and empty arrays.

Improved and corrected some String code comments

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

Merge changes from TFS

Merge pull request #1984 from caslan/removeCSExports

Remove unneeded EnterCriticalSection / LeaveCriticalSection exports

Remove EnterCriticalSection / LeaveCriticalSection exports that are not needed

Fix build break when NO_HOST_CPP_EH_ONLY is defined

[tfs-changeset: 1546051]

Merge pull request #1959 from adityamandaleeka/remove_suspension

Remove thread parameter from a few more places in PAL

Merge pull request #1978 from Djuffin/debug-alloc-fix

Selective VirtualProtect in debugger heap for Realloc

Merge pull request #1980 from pgavlin/StructABITests

Add some missing files.

Add some missing files.

The lack of these files causes the struct ABI tests to fail to build.

Merge pull request #1905 from naamunds/XplatDoc

Add libcurl and libssl packages to instructions

Merge pull request #1841 from caslan/supportforBSTR

Add support for BSTRs on Unix

Merge pull request #1972 from pgavlin/StructABITests

Add some targeted by-value struct passing tests.

Selective VirtualProtect in debugger heap for Realloc

Merge pull request #1963 from Djuffin/debug-alloc-fix

Restrict allocation of executable memory by DebuggerHeap

Merge pull request #1971 from AndyAyersMS/SimplfiyTest

Reduce number of threads used by the LocallocLarge test

Merge pull request #1976 from mmitche/unzip-params

Fix unzip arg order

Fix unzip arg order

Add some targeted by-value struct passing tests.

These tests target the SysV AMD64 ABI in particular, which relies on
field types to decide how to pass by-value structs.

Reduce number of threads used by the LocallocLarge test

This test used to create ~3000 threads. Reduce that to ~300 while we sort out what is behind #1966.

Merge pull request #1968 from mmitche/reenable-coreclr-linux

Reenable coreclr tests by default on Linux

Update trigger phrases to be regex

Quieter zip and ignoring warnings while unzipping

Merge pull request #1967 from mmitche/use-unzip

Switch to using zip/unzip

Switch to using zip/unzip to avoid path length issues in the packer/unpacker

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

Merge changes from TFS

Merge pull request #1958 from stephentoub/delete_consolehandler

Remove SetConsoleCtrlHandler from Unix PAL

Restrict allocation of executable memory by DebuggerHeap

DebuggerHeap is a common class that is used for both executable and
non-executable memory allocations by debugging infrustructure.
On Windows, OS supports concept of executable heap and CLR doesn't need
to do anything extra for each executable allocation. On Linux/OSX
this is not true. That's why this changes preserves fExecutable flag for each heap
and makes sure that we mark memory as executable only when it is necessary.

Remove SetConsoleCtrlHandler from Unix PAL

Deleted SIGINT and SIGQUIT-related signal handling code, which means we no longer need to spin up a worker thread for all coreclr apps.

Make the debugger's SetConsoleCtrlHandler usage Windows-only (from offline conversation it sounded like we might choose to delete it entirely subsequently, but for now I've left it).

Removed all of the code related to SetConsoleCtrlHandler from the PAL.

Deleted all tests related to SetConsoleCtrlHandler from the palsuite.

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

Merge changes from TFS

Merge pull request #1949 from LLITCHEV/SystemVStructsCleanup

Some code cleanup for the SystemVStructs work.

Fix build break from CL#1544863 Part II

[tfs-changeset: 1545169]

Remove thread parameter from shmfilelockmgr functions

Remove thread parameter from some string functions in PAL.

Remove thread parameter from virtual memory functions in PAL

Remove thread parameter from some PAL initialization functions.

Remove NativeVsnprintf and NativeVfprintf.

Fix build break from CL# 1544863 (git PR #1946 merge commit 810b1162fff85075feb6cd5dd47b30e9f668ad38)

[tfs-changeset: 1545107]

Merge pull request #1927 from kouvel/OacrFixes

Static analysis fixes

Merge pull request #1946 from mikem8361/unwindi

Fix sos clrstack unwinding for the special HelperMethodFrame.

Fix sos clrstack unwinding for the special HelperMethodFrame.

Add and implement new ICorDebugDataTarget4 unwind interface using lldb stack unwinder ABIs. The implementation
does a linear search of the native frames for the stack pointer provided. It doesn't happen often so the
performance is fine.

Stub out the DBI's ICorDebugDataTarget4 (in ShimDataTarget::VirtualUnwind) for now. Since PAL_VirtualUnwindOutOfProc
is disabled it makes sense to just return E_NOTIMPL.

Fix bpmd command so it doesn't stop in lldb when it isn't a CLRN exception (continues).

Merge pull request #1939 from JohnChen0/platformID

Adjust platform IDs for Unix NIs

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

Merge changes from TFS

Some code cleanup for the SystemVStructs work.

Merge pull request #1940 from adityamandaleeka/remove_suspension

Clean up PAL File functions

Remove some unnecessary file-related functions from PAL

Merge pull request #1929 from bryanar/pr1-test-fix

Fixing Priority 1 Run-Only Linux tests that failed to run when built on a Linux box. The cause of the failure was that RunOnly tests were building scripts that were using absolute paths. The fix makes all pathing in the scripts relative.

Adjust platform IDs for Unix NIs

Previous change for native image platform IDs only works on x64 architecture.
Adjusted the code to work on all architectures.

Remove thread parameter from file-related functions

Merge pull request #1933 from LLITCHEV/Issue1831

Fix a bug in classification of structs with field holes.

Merge pull request #1936 from mmitche/disable-pr-test

Temporarily disable ubuntu coreclr release tests

Temporarily disable ubuntu coreclr release tests

Until file copy time issues are resolved (using unpacker) and tests fully pass

Fix assert / access violation in PEDeocder due to change from GitHub

Recent change on GitHub to modify platform ID of Unix platforms caused issues with desktop CLR.

[tfs-changeset: 1544631]

Fix a bug in classification of structs with field holes.

There is a bug in classifying the register passing structs where field
layout leaves holes in the struct layout.

Merge pull request #1931 from mmitche/use-packer

Use the packer/unpacker for coreclr tests to reduce copy time

Use the packer/unpacker for coreclr tests to reduce copy time

Merge pull request #1922 from janvorli/fix-multiple-exception-helpers

Fix handling of embedded scopes with multiple exception holders

Merge pull request #1930 from adityamandaleeka/remove_suspension

Remove thread parameter from InternalMalloc/Free/New/Delete

Fixing PR1-RunOnly Linux Tests

Remove thread parameter from InternalMalloc/Free/New/Delete

Static analysis fixes

Remove USE_SIGNALS_FOR_THREAD_SUSPENSION

Fix hardcoded Utilities.IsWindows property

This could probably use the new RuntimeInformation API in the future,
but since this would require touching project.json and bringing in a new
dependency I decided to go with the simple fix for now :)

Remove Env helper class in tests

It's not used in a lot of places and just makes life more complicated, because
Utilities.IsWindows is hardcoded to true right now (which should be fixed separately).

See https://github.com/dotnet/coreclr/pull/1907 for some details.

Merge pull request #1907 from wtgodbe/innerFinally

Fixed failing test 'InnerFinally'

Merge pull request #1921 from AndyAyersMS/InliningPlans

Initial draft for inlining plans

Initial draft for inlining plans

I am pushing these early drafts out to be transparent about the work we anticipate doing for improving inlining in both RyuJit and LLILC. Expect revisions and more details to emerge over time.

Fix handling of embedded scopes with multiple exception holders

During the exception handling pass 1 on Unix, we only find one holder for each
frame. But for the case where there are multiple scopes embedded in each other,
each of them having their own exception holder, this is not correct and we
need to call filters for all holders on such frame, starting from the inner-most
one.
This change fixes that.

In addition, it fixes one usage of the EX_CATCH_CPP_ONLY in the src/vm/threads.cpp
where the presence of the exception holder in the EX_TRY is not correct and causes
the exception handler pass 1 to consider the managed exception handled at that place.
The fix is to create a new EX_TRY_CPP_ONLY that doesn't contain the holder and use
it at that place.
In fact, the comment at EX_CATCH_IMPL_CPP_ONLY was suggesting that a separate EX_TRY
clone would be a better solution anyways, since it eliminates one try / catch level
in the EX_CATCH_IMPL_CPP_ONLY.

Merge pull request #1920 from jkotas/gcsample

Fix build breaks in GCSample

Fix build breaks in GCSample

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

Merge changes from TFS

Merge pull request #1914 from adityamandaleeka/jenkins_enable_unix

Enable Ubuntu x64 Release test runs in CI by default

Merge pull request #1908 from adityamandaleeka/fix_exc_lists

Fix Unix exclusion lists

Fix build break on Unix

[tfs-changeset: 1543814]

Fix Unix build breaks

[tfs-changeset: 1543811]

Fix Unix build break

[tfs-changeset: 1543757]

Fix Unix build break

[tfs-changeset: 1543746]

Change 'Linux' to 'Ubuntu' in CI run names and triggers

Merge pull request #1906 from LLITCHEV/Issue1831

Fix for GH issue #1884.

Enable Unix x64 Release runs in CI by default

Merge pull request #1883 from mono/gc-stress-tests

Make GC stress tests cross platform

Merge pull request #1903 from jashook/master

Remove redefinition of new and delete in alloc.h