Merge pull request #2780 from alexanderkyte/seq_point_optimize

[runtime] Implement performant unbounded-depth sequence point search

[marshal] Ensure uses_handles is initialized.

[mono-api-html] Show assembly version differences

Merge pull request #3213 from henricm/fix-for-win-securestring-to-bstr

Fix for win securestring to bstr

[mcs] Report second level references mismatch only for really used references. Fixes #42702

Merge pull request #3387 from xmcclure/test-suite-bump

Update ms-test-suite to fix broken test

[WinForms] Fix a test to correctly check for Mono

typeof (int).GetType ().GetType ().Name is now RuntimeType since https://github.com/mono/mono/pull/3045.
We did not notice this since most of our bots don't run with en-US locale set which this test requires.

[mono] Use -size 0 instead of -empty as find argument

The latter is not supported by the "find" binary in Alpine. The former works everywhere.

[mini] Fix mktemp argument in test_op_il_seq_point.sh

mktemp on e.g. Alpine requires at least six 'X' characters in the template argument.

[mdoc] Remove --exclude=.svn from diff arguments

We don't use svn anymore since a very long time and some diff implementations (e.g. in Alpine) don't support --exclude.
Removing the argument is an easy fix.

[acceptance-tests] Run ms-test-suite with debug symbols

So we get line numbers in stack traces.

[runtime] Use a coop semaphore instead of the io-layer one in the monitor code. (#3382)

Update ms-test-suite to fix broken test

Merge pull request #3386 from alexanderkyte/nunit_lite_return_status

[runtime] Make nunitlite return failure exit code when tests fail

Merge pull request #3373 from marek-safar/net-4.6.2

[bcl] Update Reference Source to .NET Framework 4.6.2

[runtime] Return failure return status if nunit-lite test suite fails

[runtime] Bump nunit-lite dependency for runner change

[System] Remove one no longer needed uri workaround. Fixes #43032

[bcl] Update Reference Source to .NET Framework 4.6.2

Moving BSTR conv to native code in SecureStringToBSTR.

Previous implementation laid out the BSTR structure
in directly memory in C# code. This layout was not correct
for windows.
Furthermore memory was allocated using AllocCoTaskMem but later freed
using SysFreeString which on windows are different memory areas.
Third issue is that SecureString asumes big-endian when copying the chars
to its internal byte buffer. So on little-endian environments,
a swap of bytes is needed before the buffer is sent to native side
for BSTR conversion. This should probably be handled internally in
SecureString instead.

Handle NetBSD specific iconv(3) call prototype (#3376)

In the past there were wars between constified and unconstified
second parameter.. NetBSD went with constifying it. POSIX went without
const and most other OSes standarized. NetBSD is still having
incompatible function prototype.

LIBRARY
     Standard C Library (libc, -lc)

SYNOPSIS
     #include <iconv.h>

     size_t
     iconv(iconv_t cd, const char ** restrict src, size_t * restrict srcleft,
           char ** restrict dst, size_t * restrict dstleft);

Merge pull request #3353 from esdrubal/st-format

[corlib] Removed MVID and AOTID metadata.

[ci] Another Python 3 fix in babysitter

```
  File "./scripts/ci/babysitter", line 424, in run
    for xml in (xml_list + wrote_xml):
TypeError: can only concatenate list (not "map") to list
```

Define _NETBSD_SOURCE for mincore(2) (#3380)

Fix the definition of legacy_closer(), detected on NetBSD (#3377)

[ci] Python3 fix in babysitter script

iteritems() was removed in py3, we can just use items() instead since we're iterating
over the list anyway so we don't need the lazy evaluation behavior of iteritems().

Introduce mono-threads-netbsd.c (#3374)

Fix shell portability, detected with NetBSD/pkgsrc (#3375)

Merge pull request #3349 from lambdageek/handle-icalls-in-jit

[handles] Add coop handles in wrapper to icalls with HANDLES() declaration

[marshal] Assert if adding handles to an out or inout parameter of an icall.

Eventually should support this (System.RuntimeType.GetInterfaceMapData
could use it, for one) but for now just assert.

[marshal] Load payload from return handle

[marshal] If an icall is getting handles and MonoError added, it better
check for exceptions.

[marshal] Wrap icall managed obj input arguments with handles.

[runtime] Modify Zoltan's prototype to query icall-def.h handles tables

TODO: there's a gratuitous call to mono_lookup_internal_call_full().
Should really cache the uses_handle info along with the addr since the
lookup by name is rather involved.

[runtime] Add HANDLES() macro to icall-def.h

The HANDLES() macro can be wrapped around an ICALL() to signal that
the icall should use the coop handles machinery from handles.h

Add RuntimeStructs class.

Pass a MonoError argument to icalls.

Merge pull request #3372 from ntherning/fix-g_win32_getlocale

Explicitly use the ANSI version of GetLocaleInfo() in g_win32_getlocale()

Explicitly use the ANSI version of GetLocaleInfo() in g_win32_getlocale()

Since UNICODE is defined when Mono is compiled under Windows the
GetLocaleInfo() macro will expand to GetLocaleInfoW() which returns the
current locale identifier in g_win32_getlocale() in 16-bit Unicode.
g_win32_getlocale() however expects an 8-bit encoding. By using
GetLocaleInfoA() explicitly we'll get the identifier back as an ordinary 8-bit
C string.

This patch fixes the mono/tests/threadpool-exceptions5.exe test.

[corlib] Removed MVID and AOTID metadata.

MVID and AOTID metadata is no longer displayed at the end of the stack
trace.

Now, every stack frame contains a MVID and AOTID inlined.

An IL offset stackframe looks like:
  at StackTraceDumper.Main () [0x00000] in <a3ec873a838c42c29f4eba7cab7d1dfb>:0

A native offset stack frame looks like:
  at StackTraceDumper.<Main>m__0 () <0x100e15d50 + 0x00029> in <a3ec873a838c42c29f4eba7cab7d1dfb#b37a9d11247b811ffffdaff8090d8a59>:0

Merge pull request #2903 from krytarowski/netbsd-support-4

Implement mremap(2) usage on NetBSD

Fix the loaded llvm build.

Merge pull request #3370 from ludovic-henry/fix-finalizer-wait

[gc] Fix crash when doing WaitForPendingFinalizers

[gc] Fix crash when doing WaitForPendingFinalizers

The crash would come from a use-after-free of stack memory: in coop_cond_timedwait_alertable, we would allocate the struct we pass to the mono_thread_info_install_interrupt function on the stack. But there is no guarantee as to when break_coop_alertable_wait is run. The fix is to allocate the data passed to mono_thread_info_install_interrupt on the heap, and free it in the callback, or if it hasn't been interrupted.

[x86] Only pinvoke methods are stdcall. (#3354)

For xamarin bug 30146.

[runtime] Fix the loaded llvm build.

Merge pull request #3356 from marek-safar/cecil-update

Cecil update

[runtime] Store the original invoke method in WrapperInfo for delegate invokes, use that to fix the check in find_aot_method_in_amodule ().

[monosymbolicate] Update project file

[Mono.Cecil] Add special handling of legacy 0.9.5 version

[mono-symbolicate] Better test failure reporting

[mdoc] Update project file

[Mono.Contract] Less obsolete warnings

[bcl] Reflect cecil breaking changes

Update Cecil to the latest master

Merge pull request #3363 from akoeplinger/fix-42938

[xbuild] Use the xbuild ToolsVersion even if another one is specified in the csproj

[arm] Fix aot support in the sdb soft breakpoint trampolines.

[ci] Don't build mobile profiles in a monolite build

The mobile profiles rely on system resgen2 which is not present in a monolite build.

Merge pull request #3359 from ntherning/sgen-avoid-scanning-stack-guard-page-on-windows

Prevent SGen from scanning the guard page used by Windows to dynamically grow the stack

Merge pull request #3361 from xmcclure/babysitter-xml

Add mono/tests support to CI babysitter

Initial prototype for handling handle stack setup and argument/return value conversion in icall wrappers.

Merge pull request #3365 from ntherning/fix-gc-graystack-stress-on-win64

Fixes the mono/tests/gc-graystack-stress test on Windows x64

Merge pull request #3337 from akoeplinger/fix-alpine

Fix Alpine build

[ci] Add a CI_TAGS option for building with monolite

Also ensure that all the CI_TAGS check work with multiple values.

[ci] Run "make" in parallel in run-jenkins.sh

To speed up the build.

Merge pull request #3288 from ludovic-henry/iolayer-extract-threading

[io-layer] Extract threading code

Prevent SGen from scanning the guard page used by Windows to dynamically grow
the stack.

Windows uses a guard page before the committed stack memory pages to detect
when the stack needs to be grown. If we suspend a thread just after a function
prolog has decremented the stack pointer to point into the guard page but
before the thread has been able to read or write to that page, starting the
stack scan at the thread's ESP/RSP will raise a STATUS_GUARD_PAGE_VIOLATION and
the process will crash.

This patch uses VirtualQuery() to determine whether the thread's stack_start
value points into the guard page. If it does the stack scan will instead be
started at the following, non-guard page.

Bump binary-reference-assemblies

Fixes the mono/tests/gc-graystack-stress test on Windows x64

Windows x64 only has 1 MB of stack per thread which is less than other x86
64-bit OSes. Using 10000 for "depth" in this test causes a stack overflow on
Windows x64. This patch changes "depth" to 5000 only on Windows x64. 5000 will
fit within the default stack size.

Merge pull request #3355 from ludovic-henry/process-fix-exited

[process] Fix Process.Exited event triggered too soon

Remove stray comment

[arm] Fix the initialization of the variable holding the single step trampoline address when using soft breakpoints + aot.

Merge pull request #3360 from alexanderkyte/mini_fix_timeout

[runtime] Increase mini bitcode timeout and add parallelism

[mobile_static] Give bitcode mini tests parallelism and more time

[xbuild] Use the xbuild ToolsVersion even if another one is specified in the csproj

This is the recent behavior in .NET/MSBuild as well [1]:

> Starting in Visual Studio 2013, the MSBuild Toolset version is the same
> as the Visual Studio version number. MSBuild defaults to this Toolset
> within Visual Studio and on the command line, regardless of the
> toolset version specified in project file.

With this change xbuild will always use its own version instead
of the one specified in the project, unlessa ToolsVersion is forced
via the /tv switch on the command line.

Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=42938

[1] https://msdn.microsoft.com/en-us/library/bb383796.aspx#Anchor_1

Add NUnit XML support to test babysitter, also document the communication protocol better

For test harnesses which don't support the full babysitter protocol,
the test harness can now report back paths to NUnit-format XML files
containing test results, or the paths to those XML files can be given
by whoever invoked the babysitter script. Failures listed in the XMLs
are included in the babysitter log.

test-runner.cs has been modified to report its XMLs, so we should now
see mono/tests failures in the babysitter log.

Merge pull request #3357 from tritao/zip_delete_entries

[System.IO.Compression] Remove the entry from ZipArchive entries when its deleted

[process] Fix Process.Exited event triggered too early

The Process.Exited event would be triggered too early, leading to crash in code using this event to access the process exit code. This is because in Process.ExitCode, we first check that the process has really exited.

Merge pull request #3358 from ntherning/disable-sgen-red-zone-on-win64

Set SGen's REDZONE_SIZE=0 on Win64

Set SGen's REDZONE_SIZE=0 on Win64 since the ABI defines no red zone and the
ABI states that "All memory beyond the current address of RSP is considered
volatile".

On Windows the stack grows dynamically as the thread needs more stack space.
Windows places a guard page before the page at the top of stack and when this
guard page is accessed by the thread Windows will add a new page to the stack
and resume the thread.

Before this fix SGen on Win64 subtracted 128 from the stack pointer of a
suspended thread and started the stack scan from this address. In the case
when the stack pointer would be less than 128 from the start of the memory
page SGen would attempt to read from the previous page. If this happens to be
the guard page a STATUS_GUARD_PAGE_VIOLATION exception is raised and the
process crashes. The sgen-weakref-stress.exe triggered this bug on almost
every invocation.

[io-layer] Remove threads.h, thread-private.h and wthreads.c

[io-layer] Remove unused _WAPI_THREAD_CURRENT

[io-layer] Extract thread MonoW32HandleOps

[io-layer] Extract GetThreadPriority and SetThreadPriority

[io-layer] Extract wapi_create_thread_handle

[io-layer] Extract wapi_thread_own_mutex and wapi_thread_disown_mutex

[io-layer] Extract wapi_current_thread_desc

[io-layer] Extract wapi_ref_thread_handle

[io-layer] Remove _wapi_thread_cur_apc_pending

[io-layer] Extract wapi_get_current_thread_handle

[io-layer] Extract wapi_thread_handle_set_exited

[io-layer] Make _wapi_getpid, _wapi_thread_own_mutex, _wapi_thread_disown_mutex and _wapi_mutex_abandon functions public

[io-layer] Make _WapiHandle_thread struct public

[mono-threads] Split platform and suspend specific functions

We split mono-threads-{posix,windows,mach}.c functions in 2 categories:
 - platform: depends on the host the runtime is compiled on (possible values: posix or windows)
 - suspend: depends on the mono-threads suspend backend in use (possible values: posix, windows or mach)

[io-layer] Make wait.c not depend on _wapi_thread_cur_apc_pending

[System.IO.Compression] Remove the entry from ZipArchive entries when its deleted. Fixes #43022.

Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=43022.

[mcs] Better error reporting for missing types during conversion

[build] Update order.xml data

[arm64] Assert that the displacement is not out of range int he b/bl macros, instead of in arm_get_disp (), so the assert location is more meaningful.

Merge pull request #3352 from alexanderkyte/mini_bitcode_fix

[runtime] Provide Mono.Simd.dll to llvmonlycheck+fullaotcheck