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

[runtime] Make the code more c++ clean by renaming this->this_obj.

[jit] Make the jit code more c++ clean by renaming class->klass and this->this_arg/this_ins.

Merge pull request #1950 from akoeplinger/fix-responsefile-win

[build] library.make: Avoid collisions in response file name for facades on Windows

Merge pull request #1943 from alexrp/profiler-improvements

Follow-up to #1926

[build] library.make: Avoid collisions in response file name for facades on Windows

When additional facades were added in 04114a5e9d141a4f424564fea95ac4480cb0a16a, the Windows Cygwin build broke.
Turns out, when building the System.IO.Comporession facade assembly it wanted to use the SharpCompress sources
of the real assembly and couldn't find it (which made no sense since since the facade doesn't even list them in
its .sources file).

After inspecting library.make what happens is this: the response file is defined as $(PROFILE)_$(LIBRARY).response,
but this results in the same value for both the real assembly and the facade assembly.
This means that since the real assembly was compiled first, the response file contained the source list of the real
assembly - which was then used to compile the facade (make happily reused the response file since it already existed).

The fix is to put $(LIBRARY_SUBDIR) into the filename so the facade assembly gets a different response file.
Same for the makefrag.

[runtime] Add docs for new domain and context API functions.

[runtime] Make some headers c++ clean.

[mcs] Fix obsolete checks on resolved types when type is inflated generic type with concrete type arguments.

Merge pull request #1923 from lukaszunity/start-managed-process-fix

Process.Start fix for managed program

Merge pull request #1948 from akoeplinger/fix-httplistener-test-race

[System] Fix race condition in HttpListenerTest.ConnectionReuse test

[System] Fix race condition in HttpListenerTest.ConnectionReuse test

There was a race so that sometimes ipEndPoint wasn't yet assigned when the method returned,
which resulted in the test failing like this:

 MESSAGE:  reuse1
  Expected: <127.0.0.1:54370>
  But was: null

To fix this the race, we now wait until ipEndPoint is assigned before returning.

The test was actually disabled on Android because of this (see 7cb4cc9c0e61d921c75ab654641a154d12cc8085),
we can reenable it again as it should now work fine.

[mcs] Reset doc reader state after expression bodied property. Fixes #32456

Merge pull request #1909 from esdrubal/reflection

[SRE] Implemented missing SRE features.

Merge pull request #1947 from directhex/fix-kfreebsd

mono-threads-linux.c also applies to other OSes with GNU userland

[build] Remove generated files

mono-threads-linux.c also applies to other OSes with GNU userland

Fixes a build failure for GNU/kFreeBSD (which doesn't use mono-threads-freebsd.c, since GNU/kFreeBSD doesn't have pthread_np.h)

Merge pull request #1941 from kasthack/system.web-fixes

[System.Web] Abstractions from reference source

Merge pull request #1946 from markbeaton/markbeaton-tzaustralia

GetSystemTimeZones() not returning Australia zones

GetSystemTimeZones() not returning Australia zones

A call to TimeZoneInfo.GetSystemTimeZones() is not returning any of the timezones for Australia due to the "Australia" string being missing from the array of continents used to look up the zoneinfo data at /usr/share/zoneinfo. This patch adds the "Australia" value to the array.

[profiler] Describe buffer size arithmetic with comments.

Merge pull request #1944 from akoeplinger/remove-net40-ifdefs

[bcl] Remove NET_4_0 defines from class libs

[bcl] Remove NET_4_0 defines from class libs

NET_4_0 is always defined now, so the ifdefs are redundant.
I verified the libs are exactly the same after this change, so this is effectively a no-op.

Merge pull request #1942 from akoeplinger/net4x

[build] Fix a few Makefiles to correctly use net_4_x instead of net_4_5

[RabbitMQ.Client] Add generated net_4_x-api-*.cs specs files and remove outdated ones

[build] Fix a few Makefiles to correctly use net_4_x instead of net_4_5

[Microsoft.Build] Fix name of framework directory in xbuild-testing RedistList

Fixes the BuildCommonResolveAssemblyReferences test that failed after the net_4_5 -> net_4_x switch.

Revert "[jit] Allow NEW_AOTCONST () to work in JIT mode too, use this to simplify some code."

This reverts commit bf736f6c5938641a77da1941f28c96513c42df38.

Revert this as it seems to break the build.

[System.Web] Abstractions from reference source

[jit] Allow NEW_AOTCONST () to work in JIT mode too, use this to simplify some code.

Fix installing resgen.exe.

Merge pull request #1940 from lewurm/master

[mono-config] update path to libMonoPosixHelper in runtime/ as well

[mono-config] update path to libMonoPosixHelper in runtime/ as well

Merge pull request #1939 from madewokherd/monoconddestroy

Define mono_cond_destroy on non-Windows.

Define mono_cond_destroy on non-Windows.

This commit is licensed as MIT/X11.

[runtime] Rewrite MonoContext code to work via libunwind.

The latest Xcode 7 beta 4 SDK disallows usage of getcontext and friends.

[runtime] Compile fixes for WatchOS.

[sgen] Adapt Mach OS support code to work with cooperative GC.