Bump API snapshot submodule

Bump cecil and linker

[sdb] Fix support for async debugging in optimized mode, roslyn generates valuetype IAsyncStateMethod implementations. Fixes #58728. (#5476)

[bcl] Fix a few hardcoded socket ports in tests (#5471)

* [System] Fix a few hardcoded socket ports in tests

Those could've shown up as "address already in use" in CI.

* [System.Runtime.Remoting] Fix a few hardcoded socket ports in tests

Those could've shown up as "address already in use" in CI.

Additionally remove a copy of RemotingServicesTest.cs from corlib
which also exists in System.Runtime.Remoting in updated form.

* [System.ServiceModel] Fix a few hardcoded socket ports in tests

Those could've shown up as "address already in use" in CI.

* [System.Net.Http] Fix a few hardcoded socket ports in tests

Those could've shown up as "address already in use" in CI.

The additional "keep-alive" header in Send_Complete_NoContent() and
Send_Transfer_Encoding_Chunked() is because we were using the same
port for all tests before so other tests would clear the header.
This doesn't happen now that we use distinct test ports.

* [System.ServiceModel.Web] Fix a few hardcoded socket ports in tests

Those could've shown up as "address already in use" in CI.

Revert "Merge pull request #5330 from alexanderkyte/dedup_mkbundle"

This reverts commit 9a287c04126d095e7371afee32632febd0dafd93, reversing
changes made to 4a79280b3bef8d5f15da9ddd2a2af3a03e194b03.

It breaks tons of tests.

Merge pull request #5330 from alexanderkyte/dedup_mkbundle

[runtime] Add Dedup Support to MkBundle

[Mono.Security]: Add 'MonoTlsProvider.SupportsCleanShutdown' and 'MonoTlsSettings.SendCloseNotify'. (#5465)

Only send close_notify when using BTLS and explicitly requested via 'MonoTlsSettings.SendCloseNotify'.

[runtime] Document mkbundle AOT options in man pages

[runtime] Fix tracking of dedup module in mkbundle

[runtime] Add mkbundle support for dedup

[runtime] Clean up temp mkbundle aot directory

[runtime] Fix mkbundle internationalization test results, build all

[runtime] Provide config to mkbundle

[runtime] Don't re-aot when running BCL tests

[runtime] Skip not supported tests with mkbundle

[runtime] Build stripper and use with mkbundle

Preliminary linker support ran into dependency-finding bug
with the monolinker.exe binary. Fix postponed until later.

[runtime] Add target to mkbundle all tests before CI

[runtime] Move testing dll into profile

[runtime] Fix in-tree building for mkbundle

[runtime] Skip building System.Security, System.IdentityModel tests on testing_aot_full

[runtime] Use mkbundle in BCL tests

[runtime] Fix path resolving for mkbundle

[runtime] Add new dedup interface to mkbundle

[runtime] Fix mkbundle compilation on OSX

[runtime] Enable compiling aot with mkbundle

[runtime] Add aot arguments to mkbundle

Update Clang's ThreadSanitizer blacklist (#5467)

Bump api-snapshot

Call `g_utf8_to_utf16` with a proper argument (#5470)

* The sizes of `glong` and `gsize` differ on 64-bit Windows.
* In order to avoid uninitialized data in the high four bytes of the `bytes` argument to `mono_unicode_from_external`, call `g_utf8_to_utf16` with a `glong`, then copy the result.

Merge pull request #5469 from marek-safar/ilasm

[ilasm] Adds AggressiveInlining support

Revert "[runtime] Wait for the thread to really exit in Thread.Join (). (#5454)"

This reverts commit ec30afd9dcee60bea5ce4c221a9680a23449ab87.

It causes test failures on s390x.

Revert "[llvm] Add support for building with more recent versions of the mono llvm branch, in particular https://github.com/mono/llvm/tree/2017-07-06-5a1ffa96376e8a3ee895b782790591e5e7814ccf, which is based on llvm 5.0. (#5408)"

This reverts commit f01f85481834e3d9cae2d6c757f9b694851c22aa.

It broke a bunch of tests with llvm.

[PEAPI] Don't add mscorlib reference when no type needs it

[ilasm] Adds noautoinherit option

[ilasm] Adds AggressiveInlining support

Remove leftover targets from Makefile.am (#5466)

win32getdeps and win32setup don't apply anymore.

[llvm] Add support for building with more recent versions of the mono llvm branch, in particular https://github.com/mono/llvm/tree/2017-07-06-5a1ffa96376e8a3ee895b782790591e5e7814ccf, which is based on llvm 5.0. (#5408)

[runtime] Wait for the thread to really exit in Thread.Join (). (#5454)

[sgen] Move the mono_gchandle_ functions to sgen-mono.c for consistency, add sgen_gchandle () functions instead. (#5460)

[sdb] Make the IL in dtest-excfilter.il valid. (#5463)

[amd64] Make code patching smp safe for the case where we emit a mov + call_reg. Hopefully fixes #57959. (#5452)

[gc counters] avoid using Interlocked*64 functions as they are broken on some arches (#5451)

They are implemented via a global lock, which can lead to deadlooks in
some situations, e.g.

```
-0  0xb6c92e14 in sigsuspend () from /lib/arm-linux-gnueabi/libc.so.6
-1  0x0043f3b4 in suspend_signal_handler (_dummy=35, info=0xb4afe6d0, context=0xb4afe750) at mono-threads-posix-signals.c:177
-2  <signal handler called>
-3  0xb6de6bc4 in __pthread_mutex_unlock_usercnt () from /lib/arm-linux-gnueabi/libpthread.so.0
-4  0x00440f00 in InterlockedCompareExchange64 (dest=0x530700 <worker+64>, exch=0, comp=0) at atomic.c:578
-5  0x002f228c in InterlockedRead64 (src=0x530700 <worker+64>) at ../../mono/utils/atomic.h:398
-6  0x002f29a0 in COUNTER_READ () at threadpool-worker-default.c:194
-7  0x002f3320 in worker_park () at threadpool-worker-default.c:426
-8  0x002f3774 in worker_thread (unused=0x0) at threadpool-worker-default.c:488
-9  0x002e66b0 in start_wrapper_internal (start_info=0x0, stack_ptr=0xb4b00000) at threads.c:982
-10 0x002e6894 in start_wrapper (data=0x5a8f38) at threads.c:1044
-11 0xb6de31a8 in start_thread () from /lib/arm-linux-gnueabi/libpthread.so.0
-12 0xb6d3a1bc in ?? () from /lib/arm-linux-gnueabi/libc.so.6
Backtrace stopped: previous frame identical to this frame (corrupt stack?)

Thread 4 (Thread 0xb40ff440 (LWP 10198)):
-0  0xb6deb56c in __lll_lock_wait () from /lib/arm-linux-gnueabi/libpthread.so.0
-1  0xb6de584c in pthread_mutex_lock () from /lib/arm-linux-gnueabi/libpthread.so.0
-2  0x00440e6c in InterlockedCompareExchange64 (dest=0x559868 <gc_stats+8>, exch=2855648, comp=2844091) at atomic.c:570
-3  0x0039bdcc in InterlockedAdd64 (dest=0x559868 <gc_stats+8>, add=11557) at ../../mono/utils/atomic.h:392
-4  0x003a1b50 in collect_nursery (reason=0x5083f4 "Nursery full", is_overflow=0, unpin_queue=0x0) at sgen-gc.c:1865
-5  0x003a3794 in sgen_perform_collection (requested_size=4096, generation_to_collect=0, reason=0x5083f4 "Nursery full", wait_to_finish=0, stw=1) at sgen-gc.c:2539
-6  0x003a35f0 in sgen_ensure_free_space (size=4096, generation=0) at sgen-gc.c:2505
-7  0x00389db0 in sgen_alloc_obj_nolock (vtable=0xb4d174d0, size=64) at sgen-alloc.c:256
-8  0x0036e330 in mono_gc_alloc_vector (vtable=0xb4d174d0, size=64, max_length=12) at sgen-mono.c:1743
-9  0xb6c16a38 in ?? ()
Backtrace stopped: previous frame identical to this frame (corrupt stack?)
```

fixes https://bugzilla.xamarin.com/show_bug.cgi?id=59039

Merge pull request #5459 from HinTak/patch-5

More informative g_error message

Merge pull request #5458 from HinTak/patch-4

remove redundant duplicate word 'not'

More informative g_error message

Encountered this message while running the new mkbundle simple-cross functionality introduced recently. "Function not supported" is seen on both Mac OS X and Ubuntu. Will possibly file a separate bug since HAVE_MKDTEMP should be true... but it should be "Function mkdtemp not supported" in any case.

remove redundant duplicate word 'not'

Found a consecutive 'not' pair while searching the code for somethng else.

[runtime] Use the AC_FUNC_STRERROR_R() autoconf macro to distinguish between the two possible signatures for strerror_r (). (#5453)

Merge pull request #5455 from HinTak/patch-1

Fixing confusing typos

Fixing confusing typos

Fixing confusing typos - should be self-explanatory.

[jit] Assert that the trampoline code is called in gc unsafe mode, throw exceptions by returning the exception to the assembly trampoline code and throwing it from there. (#5409)

[amd64] silence compiler warning (#5435)

Silence a GCC warning about param_regs and return_regs not being
used.

[System] Disable SocketTest.SendAsyncFile for now (#5447)

It's one of the flaky tests that we have had for a very long time.

There's work in progress fixing it as part of
https://github.com/mono/mono/pull/5345 and https://bugzilla.xamarin.com/show_bug.cgi?id=43172
but I think we should disable it in the meantime for clean CI.

[configure.ac] Don't include the .build number in runtime-corlib version (#5449)

It's causing unnecessary complexity because we need to create a new
monolite when bumping the .build number in a major.minor.build
version like from 5.2.0 -> 5.2.1

corlib version bumps are very rare in those cases and even then
you can just increment the MONO_CORLIB_COUNTER variable.

[profiler] Correct the description of the 'legacy' option.

[System]: Cleanup debugging and use the same technique from the new web stack. (#5441)

* Cleanup debugging and use the same technique from the new web stack.

* The MARTIN_DEBUG conditional has been renamed into MONO_TLS_DEBUG.

* Moved the actual implementation into MNS.MonoTlsProviderFactory.

* We now use an environment variable called 'MONO_TLS_DEBUG' to
  enable debugging (so to get debugging, you need to both compile with
  -define:MONO_TLS_DEBUG and set that environment variable).

The new web stack has a conditional and environment variable called
'MARTIN_WEB_DEBUG'.

* Use [Conditional] instead of #if.

[msvc] Update csproj files (#5450)

[man] Cleanup mentions of monocov in mono.1 (#5446)

Coverage support was added to the log profiler in a5ab6c642e38cbb60cf0c4f63007652ebf567060
and the monocov profiler was removed with 16570265149730ec6a4760cc0fa34decc1a9d981.

Retrieve thread lazily and avoid special case for defer. We were trying to get thread when in defer mode even for EVENT_KIND_VM_DEATH and hitting assert since current thread was not available. (#5440)

Ensure exception out argument is initialized to NULL. (#5424)

Merge pull request #5420 from nealef/z13v5

Add support for z13 processor

[System]: Cleanup internal TLS APIs in preparation of adding renegotiation support. (#5426)

* [btls]: Bind new native APIs to support renegotiation.

* mono_btls_ssl_set_renegotiate_mode(), mono_btls_ssl_renegotiate_pending():
  New native methods.

* MonoBtlsSsl: Added SetRenegotiateMode() and RenegotiatePending().

* MonoBtlsSslRenegotiateMode: New internal enum.

* [System]: Cleanup internal Mono.Net.Security.MobileTlsContext APIs.

Use tuple `(int ret, bool wantMore)' return value instead of out variables.

* [System]: Check for pending exceptions in MobileAuthenticatedStream.

This should fix some issues where we're hanging instead of throwing
an exception if something went wrong.

Merge pull request #5438 from cherusker/cherusker-2017-08-24-complete-prototype

[libtest] Complete the function prototype

Merge pull request #5439 from alexrp/master

[profiler] Fix some stack imbalance issues due to missing method leave events.

[profiler] Add a missing method_leave event for the CEE_MONO_RETOBJ opcode.

[profiler] Add a missing method_tail_call event when the tailc optimization is enabled.

Complete the function prototype

[mcs] Update logic when checking for tuple element names when overriding inherited member

Revert "[runtime] Fix a warning."

This reverts commit f99617537ba31a046ede2753733d20c20a30e714.

Revert this as the original code was right, there is a problem with the defines.

Merge pull request #5415 from kumpera/wasm-support

WebAssembly port

[wasm] Don't define MONO_CROSS_COMPILE in the wasm configure.ac

[utils] Fix mono_valloc and mono_valloc_aligned malloc-based implementation.

[wasm] Add wasm stubs for object-offsets and sgen-archdep.

[wasm] Add wasm strings to mono-config.c

[wasm] Add comments explaining the magic strings in mono-dl-wasm.c

[readme] Document --enable-wasm

[wasm] Add threading, dl and hwcap support for wasm.

Remove unnecessary ifdefed code

Skip load if same register

Add support for z13 processor

The z13 supports some useful instructions that can be exploited
by the JIT. Most notably the compare-and-branch instructions can
reduce a two instruction sequence to one. The load-and-add command
eliminates the more complex compare-and-swap sequence.

The z13 also supports three operand instructions dst,src1,src2
which eliminate checks and loads for instructions like add and subtract.

The unaligned.cs test has been changed to cater for big-endian platforms.
For s390x only, the --trace option will result in a file per thread
being created rather than the traces being written to stdout and
sometimes intermixing incomplete trace entries.

[TSan] Unlocking MonoStats, GCStats and JITGCStats (#5381)

[TSan] Unlocking MonoStats, GCStats and JITGCStats

As the `Unlocked* ()` functions have been approved in https://github.com/mono/mono/pull/5310, I used them to blacklist some of the most obvious counting variables whose data races should be ignored: `MonoStats`, `GCStats` and `JITGCStats` (more to follow if approved).

Please find details about all changes within the commit messages.

[threadpool] Reduce logging verbosity level. Fixes #58829

Added compile target option as library

Added -t:library since mcd does not interpret output filename for dll suffix.
Without -t:library option, it fails to compile.
{code}
mono$ mcs -debug+ -r:System.Core Options.cs -o:Mono.Options.dll
error CS5001: Program `Options.exe' does not contain a static `Main' method suitable for an entry point
Compilation failed: 1 error(s), 0 warnings
{code}

Merge pull request #5423 from vargaz/workaround-58901

[runtime] Disable a test which asserts because of an abi break in rec…

Revert "Disable pinvoke2.exe test on OSX i386 for now (#5411)"

This reverts commit 4d89d5f17bb72fb2e192736653923d8d90741a01.

Revert this as the problematic test is disabled in libtest.c.

[runtime] Disable a test which asserts because of an abi break in recent xcode/clang versions. Works around #58901.

[Mono.Security] Bugfix of bug 23153. Certificate loop variable not used to find certificate

[msi] Fix typos in f9bdab792800cd8f95bb71f02f403f2d414cce9b

[Syndication]Fix Runtime crash when parsing pubTime in RFC822 format (#5416)

[bcl] Remove unused left over .args files

[MacSDK] Whitelist the csi script

enable some tests for System.Text.UnicodeEncoding (#5414)

Remove attribute Category ("NotWorking") from tests GetString_Odd_Count_0
and GetString_Odd_Count_ff as they are now working.

[utils] Fix atomic fallout implementation function naming.

[wasm] Add wasm stubs for MonoContext and membar ops.

[WASM] Hack configure.ac to support wasm. Use --enable-wasm instead of --host / --target

Merge pull request #5406 from kumpera/fix_12157

[verifier] Base constructors can only be called on uninit'd this pointers. Fix #12157.

Merge pull request #5407 from kumpera/fix_31225

[pedump] Have the assembly search hook look at the same directory of the assembly being verified.

Fix method-access.exe test (#5412)

The exception message of a MethodAccessException was changed in 7b8811573784a244de3ce559ad4512e913ab31aa.
This caused the method-access.exe test to fail since it checked for the old format.

Disable pinvoke2.exe test on OSX i386 for now (#5411)

Reenable once https://bugzilla.xamarin.com/show_bug.cgi?id=58901 is fixed.

[msi] Pull in MSI changes from Mono 5.2 branch

(cherry picked from commit 929575bb0eba1139b786690e719844c4ef52b7db)

Bump libgdiplus (#5393)