Merge pull request #4444 from lateralusX/jlorenss/windows-unwind-info

Improve unwind support on Windows x64.

Update Bockbuild

[arm64] Fix the saving/restoring of fp regs in exception filters, we were saving/restoring them from the incorrect position in MonoContext.fregs. (#4499)

[runtime] Add a test to verify that pedump works. (#4486)

Update Bockbuild

[MacSDK] Fix Ironlangs build with latest MSBuild

Merge pull request #4494 from kumpera/important-android-fixes

[runtime] Make sure we use lower numbered signals on Android.

[api] Remove dead mono_load_image API.

[socket] ret might not have been initialized (#4495)

[Mono.Options] Add missing test file to test sources

It was missed in c1b9fc566a365cf6923218eae9098d0fd0e632ae

[msvc] Update csproj files (#4479)

[mkbundle] Encode directory separator character on Windows (#4493)

When storing satellite assemblies, mkbundle prefixes their names with a
directory name derived from the language/locale of the assembly. It uses the
platform's default directory separator character which on Windows defaults to
'\' and that causes problems when building such a bundle on Windows since \ is
an escape sequence prefix inside strings and not escaping it with another \
leads to gcc errors when building the genrated source.

This commit fixes the problem by quoting the directory separator character
properly on Windows.

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

[ci] Remove ppc64el checks from run-test-default.sh

This was initially done so we didn't get orange builds
when ppc64el was in the same job as our other archs.

ppc64el was since moved into a separate job so we can
stop skipping the test suites.

Also fixed badges in README.md for ppc64el and s390x

Shorten function name length for static functions.

[mcs] Codegen for auto-property backing field initialization used as assignment. Fixes #53166

[Mono.Posix] UnixSIgnal.WaitOne() doesn't block for 0 timeout (#4489)

For timeouts of 0 milliseconds, WaitOne may produce incorrect results. In case of timeout, the result of WaitAny denotes the amount of milliseconds that have timed out. WaitOne relies on WaitAny and interprets the result falsely as the index of the completed handle. This change introduces a check on the timeout and returns `IsSet` when the handle is not allowed to block.

This is in accordance to Microsofts [documentation of WaitOne](https://msdn.microsoft.com/de-de/library/cc189983(v=vs.110).aspx) which states "If millisecondsTimeout is zero, the method does not block. It tests the state of the wait handles and returns immediately.".

[Here](https://github.com/StephenCleary/AsyncEx.Interop.WaitHandles/blob/master/src/Nito.AsyncEx.Interop.WaitHandles/WaitHandleInterop.cs#L51) is an example of some code relying on correct output of `WaitOne(0)` that is effectively broken by the current behaviour.

[runtime] Make sure we use lower numbered signals on Android.

Android on 32bits arm shipped broken headers that made sigset_t be 32 bits instead of 64.

This means that it's not possible to use signals whose value are >= 32. The default implementation
uses RT signals when available (and they are on Android). But RT signals don't work as expected
as their value is >= 32.

This fix hardcodes some safe signals on Android and add asserts about their values to avoid regressions.

See 5005c6f3fbc1da584c6a550281689cc23f59fe6d for more details.

[interp] remove g_error call

can hit this case in racy code (e.g. tests/thread5.cs)

[interp] fix another data.klass access.  due to generics it has to be resolved with a runtime helper

[interp] support stelem for g{,u}int64

[interp] pass single fp correctly to icall trampoline

add test to basic-math.cs that tests this specifically.

[interp] fix type init exception propagation for methods that trigger static constructors

[interp] support non-empty stack in switch branches

[interp] fix for ldflda unsafe corner case

[interp] propogate OOM exception on array instantiation

[exceptions.cs] enable one more test

[interp] respect lower_bound on array access and share behaviour with .Set, .Get and .Address

[interp] throw overflow exception on long to ulong cast where long is negative

[interp] throw overflow exception on ulong to long cast where ulong is too large to fit in long

[interp] throw overflow exception on (0/-1)

[interp] fix overflow corner cases for integer

[interp] fix a mem leak in transform

[interp] allocate one more slot for tracking stack types

due to the way basic block handling is done in the interpreter, it can
happen that we have one additional element on the stack (as opposed to
what metadata tells us)

[makefile] cleanup richeck target

[interp] support stelem for guint8

[interp] support reference case for unbox.any

[interp] remove STTHIS completly.

[interp] throw exceptions from icalls

[interp] proper overflowcheck for size passed to cee_newarr

[interp] throw NRE in mint_vcall insn if receiver is null

[interp] respect generic context for cee_sizeof

[interp] set proper type info for cee_ldtoken bytecode

[interp] refactor Array::Address, so that readonly. prefix is respected properly

[interp] cee_{stfld,ldfld{,a}} can access static fields too

[interp] fix cee_initobj for reference type

[interp] implement cee_cpobj

[interp] transform all kind of clauses

still need to handle filter clauses correctly during stack walking.

[interp] clear storage for sp, vt_sp and locals properly

[interp] implement cee_unbox and cee_ldobj correctly

unbox is supposed to push a managed pointer on the stack and ldobj does
the actual value copy.  unbox_any is basically unbox+ldobj according to
the spec.

fixes random crashes in test_0_regress_80622:iltests.il

[interp] fix issues around conv.ovf.* bytecodes

[interp] fix localloc instruction, storage must be cleared

[interp] clear locals storage also on pointer type

[interp] add iltests.exe to regression suite and mark crashing tests

[interp] add interpreter specific create_trampoline callback

[generics.cs] enable one more test for interpreter

[interp] implement constrained calls to interface methods correctly

compare with 9adc7023e66a04f162efa041ff2452222a85c977

[interp] support readonly. prefix for ldelema

[interp] respect variance in interface dispatch

[interp] inflate also on CEE_CALL etc.

also, remove weird hack for delegate.invoke ()

[offsets-tool] Remove dependencies on android directory layout by adding a --targetdir=dir argument argument which points to the mono build directory corresponding to the target abi. Only allow one --abi= argument on android. (#4487)

[System] Collections from CoreFX

Bump corefx

[ci] Check if X server is available before running WinForms tests (#4477)

This should hopefully "fix" the mass fails in the WinForms
tests we sometimes see in CI.

We're using the simplest WinForms code that invokes the XplatUI
and causes the X connection to be established, if this fails
then something is seriously wrong and we skip the WinForms tests.

[arm] Implement aot support for handler block trampolines.

[arm64] Implement aot support for handler block trampolines.

Merge pull request #4481 from migueldeicaza/master

[mkbundle] Fixes the embedding of dependency assemblies + test

[runtime] Fix pedump.

RFI - Support building System.Drawing for NETCOREAPP2.0 (#4483)

[mono/tests] Don't use -lib:<dir> use -r:<dir>/asm.dll for framework assemblies

Apparently Roslyn looks in -lib directories only if it doesn't find the needed
assembly in MONO_PATH first.  On the other hand, using a full path with -r will
take priority over the assemblies in system directories.

[mini] Run tests with --runtime=mobile for testing_aot_full profile

[runtime] Redirect mobile System.Net.Http to version 4.0.0.0

per mcs/class/System.Net.Http/Assembly/AssemblyInfo.cs
our System.Net.Http AssemblyVersion is 4.0.0.0 when using mobile.

[Mono.Options] Update documentation build

[Mono.Options] Add Mono.Options.Command, .CommandSet

Mono.Options.CommandSet allows easily having separate commands and
associated command options, allowing creation of a *suite* along the
lines of **git**(1), **svn**(1), etc.

CommandSet allows intermixing plain text strings for `--help` output,
Option values -- as supported by OptionSet -- and Command instances,
which have a name, optional help text, and an optional OptionSet.

var suite = new CommandSet ("suite-name") {
// Use strings and option values, as with OptionSet
"usage: suite-name COMMAND [OPTIONS]+",
{ "v:", "verbosity", (int? v) => Verbosity = v.HasValue ? v.Value : Verbosity+1 },
// Commands may also be specified
new Command ("command-name", "command help") {
Options = new OptionSet {/*...*/},
Run     = args => { /*...*/},
},
new MyCommandSubclass (),
};
suite.Run (new string[]{...});

CommandSet provides a `help` command, and forwards `help COMMAND`
to the registered Command instance by invoking Command.Invoke()
with `--help` as an option.

[mkbundle] Fixes the embedding of dependency assemblies + test

Fix the build.

[coop handles] Assert if interior ptr handle passed to mono_gchandle_from_handle

[coop handles] Use a bitmap per HandleChunk to track interior ptr handles

instead of using a flag byte in each HandleChunkElem.

[reflection] Use coop handles for MonoMethodInfo.get_method_info and MonoPropertyInfo.get_property_info

[marshal] Pin valuetype ref icall arguments.

There are now 4 kinds of arguments to icalls:
  1. Plain old values like 'int', 'bool', etc.  Passed from the wrapper to the
     icall as is.
  2. Objects in the managed heap.  Passed using coop handles.
  3. 'out' objects in the managed heap.  A NULL coop handle is passed in
     and its final value is written back out to the caller's out argument.
  4. Value types passed by 'ref' or 'out'.  An "interior" coop handle is created which has
     the effect of pinning the object into which the ref points (if it points
     to the stack, nothign happens).  The argument is passed to the icall as a
     raw reference (MonoFoo*, not MonoFooHandle).  The ref will be unpinned
     when the icall wrapper returns.

[coop handles] Add handles for interior pointers / valuetype handles

- Call mono_handle_stack_scan for both the precise and the imprecise gc scans.
- For now assert during scan that all handles are to the beginning of an
  object.
- Add TYPED_VALUE_HANDLE_DECL Macro to declare valuetype handles:
     TYPED_VALUE_HANDLE_DECL (MonoMethodInfo);
     /* Declare MonoMethodInfoValueHandle and MonoMethodInfoValueHandleOut */
- Add MONO_HANDLE_NEW_GET_VALPTR (h, type, field) macro to create a new handle that
  points to "field" (of type "type") in the object "h".

Make System.Drawing unit tests use Assert.Throws instead of [ExpectedException] (#4476)

* Use Assert.Throws<> intstead of ExpectedException

* System.Drawing.Tests: Use Assert.Throws instead of ExpectedException

[resx2sr] Handle duplicate keys

[sgen] use memmove as src and dst can overlap

valgrind said this on `mcs --version`:

```
==807== Thread 1:
==807== Source and destination overlap in memcpy(0x413f050, 0x413f050,
8)
==807==    at 0x4C32513: memcpy@@GLIBC_2.14 (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==807==    by 0x7FB6DD: sgen_qsort_rec (sgen-qsort.c:76)
==807==    by 0x7FB8CC: sgen_qsort (sgen-qsort.c:118)
==807==    by 0x7ED834: major_free_swept_blocks (sgen-marksweep.c:2136)
==807==    by 0x7F0590: sgen_memgov_calculate_minor_collection_allowance
(sgen-memory-governor.c:114)
==807==    by 0x7F0D2E: sgen_memgov_major_collection_end
(sgen-memory-governor.c:278)
==807==    by 0x7A6C71: major_finish_collection (sgen-gc.c:2116)
==807==    by 0x7A6E23: major_do_collection (sgen-gc.c:2158)
==807==    by 0x7A7464: sgen_perform_collection (sgen-gc.c:2355)
==807==    by 0x7A85AF: sgen_gc_collect (sgen-gc.c:2854)
==807==    by 0x77C022: mono_gc_collect (sgen-mono.c:2574)
==807==    by 0x726A18: mono_domain_finalize (gc.c:439)
==807==
==807== Source and destination overlap in memcpy(0x413f078, 0x413f078,
8)
==807==    at 0x4C32513: memcpy@@GLIBC_2.14 (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==807==    by 0x7FB6DD: sgen_qsort_rec (sgen-qsort.c:76)
==807==    by 0x7FB798: sgen_qsort_rec (sgen-qsort.c:92)
==807==    by 0x7FB8CC: sgen_qsort (sgen-qsort.c:118)
==807==    by 0x7ED834: major_free_swept_blocks (sgen-marksweep.c:2136)
==807==    by 0x7F0590: sgen_memgov_calculate_minor_collection_allowance
(sgen-memory-governor.c:114)
==807==    by 0x7F0D2E: sgen_memgov_major_collection_end
(sgen-memory-governor.c:278)
==807==    by 0x7A6C71: major_finish_collection (sgen-gc.c:2116)
==807==    by 0x7A6E23: major_do_collection (sgen-gc.c:2158)
==807==    by 0x7A7464: sgen_perform_collection (sgen-gc.c:2355)
==807==    by 0x7A85AF: sgen_gc_collect (sgen-gc.c:2854)
==807==    by 0x77C022: mono_gc_collect (sgen-mono.c:2574)
==807==
==807== Source and destination overlap in memcpy(0x413f088, 0x413f088,
8)
==807==    at 0x4C32513: memcpy@@GLIBC_2.14 (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==807==    by 0x7FB6DD: sgen_qsort_rec (sgen-qsort.c:76)
==807==    by 0x7FB798: sgen_qsort_rec (sgen-qsort.c:92)
==807==    by 0x7FB798: sgen_qsort_rec (sgen-qsort.c:92)
==807==    by 0x7FB8CC: sgen_qsort (sgen-qsort.c:118)
==807==    by 0x7ED834: major_free_swept_blocks (sgen-marksweep.c:2136)
==807==    by 0x7F0590: sgen_memgov_calculate_minor_collection_allowance
(sgen-memory-governor.c:114)
==807==    by 0x7F0D2E: sgen_memgov_major_collection_end
(sgen-memory-governor.c:278)
==807==    by 0x7A6C71: major_finish_collection (sgen-gc.c:2116)
==807==    by 0x7A6E23: major_do_collection (sgen-gc.c:2158)
==807==    by 0x7A7464: sgen_perform_collection (sgen-gc.c:2355)
==807==    by 0x7A85AF: sgen_gc_collect (sgen-gc.c:2854)
```

[cert-sync] Don't import into BTLS store if BTLS is not available (#4475)

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

Finalize the implementation of SignedXml.ComputeSignature(). (#4452)

* Finalize the implementation of SignedXml.ComputeSignature().

The implementation was taken from Reference Source, which makes the behaviour of this method the same as in .NET Framework, as well as makes it possible to use custom signature descriptions.

[jit] Fix an assert in mono_unwind_ops_encode_full () caused by an off-by-one error. Fixes #53066.

[arm64] Fix the passing of small arguments on the stack in the gsharedvt trampoline, the calculation of their stack offset was incorrect in two places. Fixes #52866.

[msvc] Update csproj files (#4474)

[metadata] Fix a possible crash when looking up a p/invoke function. (#4471)

It's possible for two threads to modify the same MonoMethodPInvoke.addr
in mono_lookup_pinvoke_call, with one thread setting it to NULL inside the
loop, and the other thread having completed the loop and expecting it to
be non-NULL.

Use an intermediate variable to prevent addr becoming non-NULL.

[corert] Import System.Threading.ThreadPoolBoundHandle

[corert] Import System.Diagnostics.Debug

[System.Core] Collections from corefx

[mono/tests] run the managed test runner using TOOLS_RUNTIME

The test runner itself should use the tools runtime, but the AOT step and the
tests themselves will be launched in the profile that's being tested.

[fullaot] configure.ac: pass --runtime=mobile to AOT_{BUILD,RUN}_FLAGS

For the profiles where we compile the BCL with -d:MOBILE, (where the
`[assembly: AssemblyVersion (Consts.FxVersion)]` attribute ends up pointing to
the mobile version) make sure we AOT and run the tests using the mobile
runtime.  This ensures that for the framework assemblies the version number in
the assembly matches the version number that the internal runtime remapping (
`mono/metadata/assembly.c` `framework_assemblies` array)

[threads] Remove ThreadState_StopRequested (#4462)

[jit] Remove some duplicate declarations.

[jit] Fix MethodAccessExceptions, the caller and callee were switched. (#4465)

[threads] Remove mono_threads_create_thread (#4411)

[runtime] Fix an infinite loop in the runtime class init code. (#4455)

The following can happen:
* Thread T1 starts initialization of C1
* Thread T2 starts initialization of C2
* T2 starts initialization of C1, it gets blocked on T1.
* T1 finishes initializing C1.
* T1 starts initialization of C3.
* T1 starts initialization of C2. It sees that T2 is
  still blocked on T1, but its not actually blocked, since
  C1 is finished. Still, we record that T1 is blocked.
* T3 starts initialization of C2, it gets into an infinite
  loop in the while loop, since the blocked_thread_hash
  now indicates that T1 and T2 are blocked on each other.

The solution is to not mark a thread as blocked, if its waiting
for a thread which is waiting on a finished lock. Fixes hangs
when running csc.

[threads] Remove dead field cached_culture_info (#4410)