Merge pull request #1912 from ludovic-henry/threadpool-managed-asyncresult

[runtime] Move AsyncResult.Invoke to managed

[runtime] Move AsyncResult.Invoke to managed

[System] Increase timeout of a test in SslStreamTest

The AuthenticateClientAndServer_ClientSendsNoData test sometimes failed on Jenkins,
but the 2s timeout is pretty short (it takes about 0.5s on my fast machine as well).

Increasing it to 5s should allow the test to work on the slower Jenkins machines as well.

[runtime] Refactor AsyncResult creation with MonoAsyncCall

[runtime] Refactor AsyncResult.Invoke internal call

Instead of catching the exception and rethrowin it right away, we do not catch it. This simplifies the code, and will give us more insight into the underlying exception.

Also merge mono_async_result_invoke into ves_icall_System_Runtime_Remoting_Messaging_AsyncResult_Invoke as it is the only place we use it.

Merge pull request #1959 from alexrp/master

Fix atomic_add_i4 on PPC

Merge pull request #1958 from rolfbjarne/aot-error-reporting

[aot] use g_error instead of fprintf + exit(1).

[ppc] Fix atomic_add_i4 support for 32-bit PPC.

[ppc] Instruction length of atomic_add_i4 is 28.

[aot] use g_error instead of fprintf + exit(1).

printfs don't always show up in the iOS device log, and exit(1) does
not create a crash report, making these conditions hard to debug
(all you get is "app X exited with code 1", and then you need to
attach a native debugger and break on 'exit' to figure out more).

g_error will end up producing output using NSLog (which always
shows up in the device log), and a crash report as well, making
debugging a lot easier.

[llvm] Reenable the 'handler without invokes' llvm restriction, it causes llvm to eliminate the handler, causing finally_block_ending_in_dead_bb.exe to fail on amd64.

Revert "[runtime] Disable finally_block_ending_in_dead_bb.exe on wrench, it fails with llvm."

This reverts commit 7b7692a988f82e71c575e2c75b9293d8bef9ffe7.

Revert this as its not needed.

[runtime] Disable finally_block_ending_in_dead_bb.exe on wrench, it fails with llvm.

runtime] Add semicolons after the calls to the coop gc macros to avoid confusing editors.

[runtime] Clean up the mini makefile a bit.

[runtime] Clean up configure.ac a bit.

[arm] Remove usunsed arch/arm/tramp.c file.

[arm] Avoid running out of thunk space if the same instruction is patched multiple times. Fixes #32179.

[aot] Avoid passing empty quoted arguments to the linker.

Fix a warning.

[llvm] Fix support for returning empty structs from pinvoke methods on amd64.

[jit] Enable direct calls to the monitor icalls.

[runtime] Use mono_set_pending_exception () in the monitor icalls.

Merge pull request #1955 from LogosBible/servicepoint_nre

Fix NRE in CheckAvailableForRecycling. Fixes #32685

Merge pull request #1956 from akoeplinger/fix-processtest-race

[System] Fix a race in two ProcessTest tests

Merge pull request #1957 from akoeplinger/msbuild-fix

Revert "[Microsoft.Build.Engine] Fix bug with escaped semicolon and spaces"

Revert "[Microsoft.Build.Engine] Fix bug with escaped semicolon and spaces"

This reverts commit 51297ed7ab06480df84520c758639b6cef0790d9.

It caused a regression for escaped quotes (%22) in msbuild properties. See https://github.com/mono/mono/commit/51297ed7ab06480df84520c758639b6cef0790d9#commitcomment-11827605.

Disable test that now fails again after the revert. Add new test that verifies the behavior originally broken by the change.

[System] Fix a race in two ProcessTest tests

The tests sometimes failed in CI, due to a race condition between
the Exited event and our assertion of whether it was called.

Fix NRE in CheckAvailableForRecycling. Fixes #32685

[mkbundle] Add `mkbundle --dos2unix` parameter.

Fixes: https://bugzilla.xamarin.com/show_bug.cgi?id=25086
Fixes: https://bugzilla.xamarin.com/show_bug.cgi?id=31875
Fixes: https://bugzilla.xamarin.com/show_bug.cgi?id=32171

Reverts and reworks commit bcfec743.

The problem is that when Xamarin.Android is running mkbundle, dos2unix
should NEVER be used, because the Android NDK NEVER wants Unix-like
paths, which is what dos2unix creates.

Commit bcfec743 attempted to fix this by removing dos2unix invocation
from the Xamarin.Android codepath.

The problem is that the logic was backwards: it removed dos2unix
invocation when XAMARIN_ANDROID was NOT defined, and PRESERVED
dos2unix invocation when XAMARIN_ANDROID WAS defined.

Meaning commit bcfec743 broke normal/default mkbundle.exe use on
Windows -- it would no longer probe for and use dos2unix if present --
and Xamarin.Android use of mkbundle continued to check for and use
dos2unix if it was present, continuing the buggy behavior.

Doh!

Thus, a reworking: Add a `mkbundle --dos2unix` parameter.
If `mkbundle --dos2unix=false` is specified, dos2unix will NOT be
probed for, and will NOT be used. If `mkbundle --dos2unix` or
`mkbundle --dos2unix=true` is used, then dos2unix WILL be used.

This allows the caller to consistently, ahead of time, control
mkbundle's behavior regarding dos2unix, which will e.g. allow
Xamarin.Android to *always* specify --dos2unix=false, allowing things
to build as desired there.

[System.Runtime.Remoting] Use NotWorking category for test instead of Ignore attribute

The TestFixtureSetUp would sometimes fail in CI, but none of the tests in this class are
actually run since they are all either [Category("NotWorking")] or [Ignore].

Moving the one test from Ignore to NotWorking category allows NUnit to recognize there
are no tests to run and skips the unnecessary TestFixtureSetUp completely.

[mcs] Try to report better error message for file excedding PE/COFF limit. Fixes #32487

[runtime] Add 2.0 build assemblies to remapping table. Fixes #32561

[reflection] Filter out private accessors when queried from derived types. Fixes #32609

Parallelize the generation of aot images for make fullaotcheck.

[aot] Fix the caching of runtime-invoke wrappers after the wrapper cache changes, two methods with the same signature might have different runtime invoke wrappers now, and the caching would break their lookup.

[runtime] Support quoted assembly names. Fixes #32591

Merge pull request #1840 from ludovic-henry/iolayer-thread-interrupt

[runtime] Implement wait interrupt in thread info, instead of in the io-layer

[System]: Fix WebConnectionStream.SetHeadersAsync() logic for #31830.

This fixes commits 8ccfe0f8ff71d18151f8aad90d0855dfd2d2a723 and
c109f9ca03d38e608cbd85cb2fdf8cfaf55bb97c.

Added new test:
https://github.com/xamarin/web-tests/commit/d60bc3f637ca2bf4aa1379b839d878e224d344c8

(cherry picked from commit c133880bc407baed5d46222e20677d1427b32b4d)

Add missing {

[amd64] Allocate stack space for saving argument registers in the sdb trampoline code on winx64.

[amd64] Allocate stack space for saving argument registers in the generic trampoline code on winx64.

[jit] Avoid emitting the DW_CFA_mono_advance_loc mono extension opcode into the .debug_frame section.

Merge pull request #1929 from BrzVlad/feature-unwind4yall

Feature trampoline unwind info

[arm] Provide unwind info for tls thunks

We don't try to unwind from the fallback thunks since they call external code anyway.

[arm] Simplify declaration of global symbols in asm

[arm] Fix unwind info for managed methods

Add unwind information for the epilog.

[aot] Provide unwind info for plt trampolines

[arm] Add unwind info for trampolines

[x86] Add unwind info for registered trampolines.

[amd64] Add unwind info for registered trampolines.

Remove an obsolete comment.

Revert "Report XDG Documents directory for SpecialFolder.MyDocuments"

This reverts commit d6553bc988dd9f2a3c535c6075868f233ecd3302.

Revert this as it breaks the build.

[runtime] Register tramp jit info to the corresponding domain

[x86] Register throw trampolines

[runtime] Register unbox trampolines

[arm] Register and add unwind information for page trampolines.

[runtime] Register imt trampolines

[runtime] Register get gsharedvt arg trampolines

[runtime] Register static rgctx trampolines

[runtime] Register tramp infos for delegate invoke trampolines

Also refactor MonoTrampInfo creation, similar to the other trampolines.

[x86] Fix usage of virtual delegate invoke tramps for aot

We need to fallback if the vtable slot exceedes the limit.

[runtime] Fix lookup of aot symbol for imt delegate invoke tramps

Offset for calls through imt is negative

[runtime] Add missing delegate invoke trampolines

[amd64] Fix failure to emit imt delegate invoke tramps

The problem was due to implicit casting of the dividend to unsigned (which overflows) when divided by unsigned.

[aot] Register tramp infos when loading trampolines in aot.

[runtime] Expose trampoline jit infos that have unwind info to the runtime

Some trampolines cannot have typical unwind information because they don't get the return address from the stack. Others are called from native code that doesn't have a managed-to-native transition so unwind information would be pointless at this point. We don't include these in the jit info tables for now, not to confuse the runtime further more.

[runtime] We already checked that the IP is within the clause

[jit] Add different frame type for trampoline frames.

[jit] Include the unwind_info into the tramp JitInfos

[llvm] Avoid emitting a dummy personality function.

Merge pull request #1951 from akoeplinger/fix-monop

[monop] Set MONO_PATH when running tests so class libs are found

Merge pull request #1953 from brpocock/master

Report XDG Documents directory for SpecialFolder.MyDocuments

Merge pull request #1954 from kumpera/fix_aot_compilation

Fix OSX aot compilation when there's a space in a file path.

Fix aot compilation when the path has spaces.

[tests] iOS9 (beta) have fixed bug #27864, adjust test

Report XDG Documents directory for SpecialFolder.MyDocuments

[runtime] Use MONO_SIZEOF_TYPE instead of sizeof (MonoType) and compiler generated copying.

On amd64 MONO_SIZEOF_TYPE is 12 and sizeof(MonoType) is 16. This leads to memory corruption
if one uses sizeof instead of the specific size.

This is one of those oddball behaviors of the runtime that makes easy things very hard. :(

[runtime] Introduce mono_method_get_wrapper_cache to fetch the right wrappers set for a given method.

This removes a ton of duplicated code around the runtime. There are still a few cases that could not be
unified due to the hashtables been initialized differently, which requires further reviewing.

[runtime] Use a single function to free the contents of MonoWrapperCaches. Remove duplicated code.

[corlib] Fix out of range FileStream position error message

[mcs] Report error for missing win32 icon.

[xbuild] Use correct property for Win32Icon. Fixes #32499

[llvm] Emit the file info struct initializer using the proper types.

[amd64] Avoid saving r11 outside the red zone in the generic trampoline code.

[monop] Set MONO_PATH when running tests so class libs are found

When the monop tests were added in 6c9ad41d85a668550d0d6077cd3f7d504fe6b6eb the MONO_PATH
variable wasn't included. This means that the tests actually ran against the system-provided
Mono class libs instead of the in-tree versions (or failed in case of CI with only monolite).

Revert "[System] Don't assume WebConnectionStream of unknown http methods require write stream. Fixes #31830"

This reverts commit 8ccfe0f8ff71d18151f8aad90d0855dfd2d2a723.

This commit breaks the Content-Type logic for existing HTTP verbs such as for instance
using DELETE with an empty body [1].  Added two new test cases [2] and [3] to the stable
branch of the web-tests (https://github.com/xamarin/web-tests/tree/stable) for the particular
scenario from bug #31830 as well as the default HttpClient.SendAsync() without a Content [4].

The rules are as follows:

1.) HttpWebRequest adds the Content-Length header if and only if you're calling
    GetRequestStream(); a length of 0 will be used if you don't actually write
    any content.

    This is true for POST, PUT, DELETE and "obscute" verbs.

2.) HttpClient.SendAsync() always adds a Content-Length header.

Tests:

[1] https://github.com/xamarin/web-tests/blob/31b8fed0ec1283b8c0693bb14d1ac266792c1020/Xamarin.WebTests/Xamarin.WebTests.Tests/TestPost.cs#L84

[2] https://github.com/xamarin/web-tests/blob/31b8fed0ec1283b8c0693bb14d1ac266792c1020/Xamarin.WebTests/Xamarin.WebTests.Tests/TestPost.cs#L296

[3] https://github.com/xamarin/web-tests/blob/31b8fed0ec1283b8c0693bb14d1ac266792c1020/Xamarin.WebTests/Xamarin.WebTests.Tests/TestHttpClient.cs#L134

[4] https://github.com/xamarin/web-tests/blob/31b8fed0ec1283b8c0693bb14d1ac266792c1020/Xamarin.WebTests/Xamarin.WebTests.Tests/TestHttpClient.cs#L127

(cherry picked from commit 3f1ae72f1b385c3c5e4b5ac2b93d13418c020f02)

[runtime] Remove some dead code.

[runtime] Print an error message instead of asserting when encountering an old corlib.

[mini] Fixed void function returning warning.

[msvc] Update Mono DLL export definitions.

[msvc] Update Visual Studio project files.

[mcs] Do member declaration obsolete types references check after all members are defined. Fixes #32451

[runtime] Remove mono_marshal_free_inflated_wrappers because its only call site frees the caches anyways.

Since we now only remove methods from caches in the image set it belong we no longer needs to do it since
they will be gone together with the method itself.

[runtime] Move caches present in both MonoImage and MonoImageSet into an embedded struct.

[runtime] Duplicate some more of the image caches in MonoImageSet

Changes by Rodrigo Kumpera:

Added missing freeing code.
Fixed insertion into runtime_invoke_direct_cache.
Fixed some of the cleanup in mono_marshal_free_inflated_wrappers.

[runtime] Removed confusing/cross-cutting function mono_method_inflated_lookup and inlined at only call site.

[runtime] Move caches from Images to ImageSets if their keys contain generic types

We currently have a partial ordering of dependencies between ImageSets and images.
If you free an image that an ImageSet depends on, you must free the
ImageSet. The inverse is not true. Freeing the ImageSet does not free
any images.

We had some caches in each Image which used method signatures as keys.
When an image signature refers to a type stored in an ImageSet, and that
ImageSet has been unloaded, the signature will remain in the cache for
the Image.

When we have another object hash to the same slot as this signature, we
must check if the two signatures are actually equal, that is if they are
equal keys or a hash collision. In this check we will refer to the
MonoType in the freed ImageSet.

By moving the caches containing inflated type references into the ImageSets
holding these inflated types, we ensure that the cache items are always safe
to dereference.

Why not simply remove the signatures from all caches they contain on
freeing the signature? This is much more work. A signature with a dozen
type parameters in a dozen images may have been used as a key in any of these
images. We would have to check all caches in all images the signature
refers to. As these caches use open addressing, in the worst time this
can take

```
(# images in signature) x (# caches per image) x (# of slots traversed on average)
```

steps. By placing it on the ImageSet, this takes only the steps required
to free a hash table. It is also easier to reason about.

Changes by Rodrigo Kumpera:

Added locking comments to the new fields in MonoImageSet.

Fixed the selected cache in mono_marshal_get_delegate_begin_invoke, mono_marshal_get_delegate_end_invoke,
mono_marshal_get_delegate_invoke_internal and mono_marshal_get_synchronized_wrapper.

The original code was using the image set of the class instead of the IS of the inflated method. This would
pick the wrong image if the method belonged to a different IS.

[marsha] Use TRUE instead of 1 for setting a boolean field.

[runtime] Copy signature return type when copying MonoMethodSignature

This commit also removed an extra signature duplicator living in
marshal.c, so that this fix is used consistently