[runtime] Don't make the domain id allocatable before we clear the domain (#4942)

Otherwise, while we are clearing the domain, another domain might be created that has the same id as the current one. This would lead to crashes due to releasing of data from the newly created domain (ex gchandles).

Merge pull request #4938 from kumpera/optimize_ref_queries

Add concurrent read version of MonoGHashTable and use it with MonoDomain::refobject

[win32] Add new files.

[mcs] Don't emit param-array attribute on method overrides, it's ignored by C# compiler. Fixes #53244

Bump bockbuild

[S.R.I.RuntimeInformation] Fix FrameworkDescription on Android (#4946)

* [S.R.I.RuntimeInformation] Fix FrameworkDescription on Android

It turns out using `DllImport ["__Internal"]` doesn't work on
XA (and other cases where Mono is embedded) because it searches
for the method in the host which embeds Mono and doesn't find it,
resulting in an EntryPointNotFoundException.

We need to switch to an icall instead. However there's already
an existing icall+BCL wrapper: Mono.Runtime.GetDisplayName().

Added IVT from corlib to S.R.I.RuntimeInformation so we can call it.

* Update API snapshot

[mcs] Reset catch state variable after it has been checked. Fixes #54322

[mcs] New property/indexer cannot have accessors implementing an interface in base type. Fixes #56627

Bugfix Ping.cs (#4936)

[interp] implement RELOC_LONG_BRANCH.  fixes test/imt_big_iface_test.exe

[runtime] Handle mono_method_body_get_object returning NULL on success.

[Facades] Update to match latest contracts (#4939)

* [Facades] Update to match latest contracts

* Update API snapshot

[CI] ignore appdomain-unload-asmload.exe on interp and full-aot

[android] add hardware specific workaround for Nexus9 in armv7 mode

we got a couple of bug reports, all with the same failure:

https://bugzilla.xamarin.com/show_bug.cgi?id=44907
https://bugzilla.xamarin.com/show_bug.cgi?id=46482
https://bugzilla.xamarin.com/show_bug.cgi?id=51791

The bugs are private, therefore here what I wrote:

```
Thank you Matthias, this was very helpful.

>>> 09-30 12:24:50.347: A/DEBUG(6211): pid: 6191, tid: 6209, name: Thread-4  >>> com.distech.x50.ui.droid <<<
>>> 09-30 12:24:50.347: A/DEBUG(6211): signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x8
>>> 09-30 12:24:50.347: A/DEBUG(6211):     r0 d6b58728  r1 00004001  r2 00000000  r3 d6b58738
>>> 09-30 12:24:50.347: A/DEBUG(6211):     r4 00000008  r5 ea97427c  r6 00001f3c  r7 00000015
>>> 09-30 12:24:50.347: A/DEBUG(6211):     r8 00001f38  r9 00000015  sl d6b58728  fp d67fed90
>>> 09-30 12:24:50.347: A/DEBUG(6211):     ip ea9742c0  sp d67fed60  lr ea7a4504  pc ea716a58  cpsr 800e0010

The crash happens on this assignment:
https://github.com/mono/mono/blob/de1865dad5c0350f391fedcaa08f02f610530d3f/mono/mini/mini-generic-sharing.c#L418
Here the according disassembly:
https://gist.github.com/lewurm/b1094749027c9e5ea19fdc4fac7905a7

The crash happens in the last loop iteration (`r9=i`, `r7=slot`).  Looking
at the machine code it just _cannot_ happen, which is confirmed by the C
code as well: `*oti` successfully happens in the if check, but after
returning from `alloc_oti()`, `oti` doesn't contain a valid address anymore.
I suspect some weird hardware issue that fails to restore all registers
properly from the stack.
```
[...]

```
Thanks again Matthias.  Unfortunately, I'm out of ideas, and I can't
blame anything but the hardware. The situation we see is too weird. We
segfault at offset `0xdda60: str sl, [r4]`, but really we should
already segfault at offset `0xdda2c: ldrge   sl, [r4, #8]!"`.  So I
suspect two things why this could happen:

(1) The instruction at `dda2c` fails to do the post-increment correctly
    for *whatever* reason.

(2) Something along the execution path corrupts the stackslot, where
    `r4` is saved, in such a way that it *exactly* masks it with `0xf`.
    Everything else on the stack looks fine, so this is sort of very
    unlikely to be honest.

This only happens on a very specific device: The Nexus 9 is the only
device that was ever shipped with the Tegra K1 T132.  I suspect an issue
in the binary translation layer of `armv7` instruction set to the internal
micro-ops of the CPU.  I tried to stress test the instruction in
question (see https://github.com/lewurm/ldrinsntest), however I was not
able to trigger a crash.  So either, I'm missing some context in order
to trigger the bug or I'm on a completely wrong track.

That said, even if we could proof that it is indeed a hardware issue,
the workaround is also non-trivial (it would then either need a fix in
gcc or require a microcode update by the chip vendor).
```

And then we saw:
https://github.com/golang/go/issues/19809#issuecomment-290804472

Another hint that this device is buggy.

Bump bockbuild

[runtime] Change MonoDomain::refobject_hash to use MonoConcGHashTable.

This change massively improves reflection query scalability.

Given this benchmark and my 4 cores laptop:

```
const int T_C = 4;
var list = new List<Thread> ();
bool done = false;
int count = 0;
for (int i = 0; i < T_C; ++i) {
var t = new Thread(() => {
while (!done) {
typeof (Driver)
.GetMethods (
BindingFlags.Public | BindingFlags.NonPublic |
BindingFlags.Instance | BindingFlags.Static);
Interlocked.Increment (ref count);
}
});
t.Start ();
list.Add (t);
}

Thread.Sleep (5000);
done = true;
foreach (var t in list)
t.Join ();
Console.WriteLine ("Got {0} lists done", count);
```

This change makes us go from 166.725 queries/s to 5.140.821 queries/s. Or 30x faster.

[runtime] Add concurrent read variant of MonoGHashTable name MonoConcGHashTable.

Use the same design as MonoConcHashTable.

Aaaaand confusing name will confuse.

[mcs] Improves imported destructor detection. Fixes #54159

Bump bockbuild

Bump bockbuild to fix wrench build

Merge pull request #4904 from kumpera/more_opts

[runtime] Change jit_info_table_num_elements to O(1).

Merge pull request #4928 from kumpera/ptr_to_struct_intrinsic

[mini] Intrinsify Marshal.PtrToStructure<T>

Interp sdb (#4911)

* [interpreter] Add beginnings of debugger support.

Handle debugger support similarly to the JIT:
- Add a MINT_SDB_SEQ_POINT instruction to the debugger IR. Replace it with a MINT_BREAKPOINT instruction when a breakpoint is placed.
- Store the sequence point information in the existing sequence point data structures.

* [interp] Add support for debugger stacktraces.

* [interp] Add support for debugger frame info.

* [interp] Connect seq points inside the same basic block so simple single stepping works.

* [interp] Link sequence points in different basic blocks the same way the JIT does it.

* [interp] Allocate some bitsets from a mempool.

* [interp] Rewrite the patching in the transform pass to make it a bit easier to understand.

* [interp] Add interruption support for sdb.

* [interp] Add support for mkrefany,refanytype and refanyval.

* [interp] Fix debugger failures.

* Add support for setting byref variables.
* Fix thread interruption when the top frame is an interpreter frame.
* Allow stack walks when a thread is suspended while transforming a method.
* Implement method entry/exit events.
* Fix ldstr+dynamic methods.

* [interp] Implement support for Debugger.Break.

* [interp] Change the offsets in MonoJitInfo/seq points into byte based from short based to reduce the amount of special casing needed in other parts of the code.

* [interp] Factor out the push/pop lmf code into a pair of helper functions.

* [mixed] Fix mixed mode EH.

* [interp] Implement support for MONO_VERBOSE_METHOD. Improve sequence point placement.

* [interp] Implement support for SetIP ().

Pass the target thread as an additional argument to mono_interp_set_resume_state (). Check the resume state after single
stepping/breakpoints in the interpreter.

* [interp] Disable the usage of the STLOC_NP opcodes when running under the debugger, it doesn't work if the ip is changed when the execution is stopped.

* [interp] Fix async stack walks in the debugger when the thread is in native code called from interpreter code.

* [interp] Fix the disable interpreter build.

* [interp] Fix the calculation of native offsets in the line number info.

* [interp] Fix the build.

* [interp] Make CMD_STACK_FRAME_GET_THIS behave the same when called on managed-to-native frames, with the interpreter, frame->has_ctx is set.

* [interp] Fix typos, remove some dead code, use g_print instead of printf in a few places, move some interpreter stubs to interp-stubs.c.

[runtime] Fix a race in mono_class_get_virtual_methods () which happens when klass->method is NULL initially, but gets initialized later. Use the lowest bit of the iterator pointer to distinguish between the two cases. Fixes #56824. (#4933)

[runtime] Change jit_info_table_num_elements to O(1).

On very large apps such as XS, this looking can take half a second of CPU time when opening a big solution.

[Mono.Posix] Adding .Net Core 2.0 support. (#4774)

Re-bump bockbuild for libtiff URL change

Revert "Bump bockbuild", as it breaks the build.

This reverts commit 30f9a22f0869d727c663cc4ba2c283ce75dc73f2.

Merge pull request #4931 from kumpera/com-interop-fix

Fix cominterop bug

[mini]Add test for PtrToStruct<T>

Merge pull request #4840 from kumpera/unaligned-access

Another round of cleanups plus fix how the JIT emit unaligned access.

Bump bockbuild

[mcs] Codegen for optimized new with valuetype load and awaited arguments. Fixes #56081

[arm] use r8 instead of r5 in icall trampoline, because r5 is used as IMT reg

[arm] respect iOS specific ABI for frame pointer in icall trampoline

[mini] Intrinsify Marshal.PtrToStructure<T>

[runtime] Add support for using COM types from corlib.

This can be reproduced by trying to load a type that implements a COM interface corlib ships.

[mini] Fix the llvmonlycheck test.

[aot] Avoid printing out AOT NOT FOUND messages for icall methods which have no body.

[arm] Do not use the ARM Run-time ABI in cross-compilers (#4914)

Merge pull request #4926 from BrzVlad/fix-tls-v5

[arm] Don't use fast tls on v5 arm

Fix mono build with disabled sgen (#4915)

Build of the Mono failed with the Boehm GC only (without SGEN)
if it configured like this: --with-gc=included --withsgen=no

sysctl takes 3 name components (#4883)

Adapted from:

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=212259

https://github.com/mono/mono/pull/3480

https://svnweb.freebsd.org/ports?view=revision&revision=423356

Move <sys/types.h> higher for integer types (#4884)

Fix build on FreeBSD 8 and maybe others
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=198404#c6

[arm] Don't use fast tls on v5 arm

The thread coprocessor registers appear only in v7 specification.

Merge pull request #4923 from lambdageek/bug-56694

[appdomain] Don't leak MonoReflectionAssemblyHandles firing events

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

Merge pull request #4872 from kumpera/add_test_for_eloop

[runtime] Fix runtime behavior when dealing with missing symlinks

Merge pull request #4845 from lambdageek/dev-coop-delegates

[marshal] Use coop handles for delegates icalls

Bug #56499: unreadable /etc/localtime should not cause an exception (#4881)

* Bug #56499: unreadable /etc/localtime should not cause an exception

* Fix build

[sgen] Scan the handle stack for dangling ptrs during domain unload

There should be no handles pointing to objects in the domain that is unloading
in any thread's handle stack once mono_gc_clear_domain () is called.

[coop handles] Fix wrong #ifdef

[appdomain] Don't leak MonoReflectionAssemblyHandles in callbacks to managed

[tests] Regression test for #56694

When an AssemblyLoad event fires in a domain, a System.Reflection.Assembly
object is created in that domain for the assembly being loaded.  The bug left
this object alive after the domain was unloaded, which lead to a GC crash.

[runtime] Implement interior handles as a special case to simplify and speed up the more common handle code. (#4921)

[corlib] Fixes warning

[mcs] Parsing of nested unbound types. Fixes #55400

[corlib] Remove unused field

Improve error messages to help users diagnose problems when using custom mode (#4922)

* Improve error messages to help users diagnose problems when using the custom mode

* Update mkbundle.cs

[runtime] Convert a g_warning to a type load failure. (#4920)

Fix make dist.

Fix the CpBlk and InitBlk intrinsics.

Update test for correct behavior.

[runtime] Fix multiple issues around symlink handling.

After fixing my windows setup, I managed to properly test MS's behavior, which is a lot saner and it's the following:

Opening fails with ERROR_CANT_RESOLVE_FILENAME (1921).
Testing and Deleting works.

Fixing File.Delete required removing the lstat we do before deleting, which no longer serves any purpose.

Fixing File.Exists requires handling ELOOP when stat fails and we retry with lstat.

[System]: Don't call OnCompleted(this) twice in SocketAsyncEventArgs.FinishWrapperConnectSuccess(). (#4894)

[bcl] Fix compiling xunit-based tests

We moved System.Runtime.InteropServices.RuntimeInformation to a facade.

[runtime] Add specialized versions of mono_icall_handle_new () for internal/non-internal pointers. (#4918)

Fix full aot (#4919)

* [runtime] Fix the generation of invalid IL in the return marshalling of stringbuilders.

* [runtime] Disable some failing fullaot tests.

[runtime] Use coop handles for System.Delegate.AllocDelegateLike_internal

[marshal] Use coop handles for System.Delegate.GetVirtualMethod_internal

[marshal] Use coop handles in mono_delegate_to_ftnptr

* Create mono_delegate_handle_to_ftnptr for the implementation
* Use coop handles for
  System.Runtime.InteropServices.Marshal.GetFunctionPointerForDelegateInternal

[runtime] Remove temporary fcn mono_delegate_ctor_with_method_fixme

[marshal] Change mono_delegate_ctor to use coop handles

[marshal] Use handles in mono_ftnptr_to_delegate

* Move most of the logic to mono_ftnptr_to_delegate_handle
* Use handles for
  System.Runtime.InteropServices.Marshal.GetDelegateForFunctionPointerInternal

[runtime] Use coop handles in mono_delegate_ctor_with_method

Use coop handles in icall System.Delegate.CreateDelegate_internal

TODO: change callsites fo mono_delegate_ctor_with_method_fixme to use handles

[marshal] Use coop handles for OffsetOf, SizeOf, Prelink, PrelinkAll

System.Runtime.InteropServices.Marshal methods
* OffsetOf
* Prelink
* PrelinkAll
* SizeOf

Remove outdated info and unused files from mcs/ (#4916)

Most of the removed files weren't touched in the last decade
and are unused so it's better to remove them to avoid confusion
about what they do.

I also went over and removed the most outdated info from readme
files, we probably want to do a more thorough overhaul in the future.

[mcs] When setting struct empty layout consider compiler generated fields. Fixes #55604

Bump bockbuild to fix build

[runtime] Add a fastpath to mono_object_isinst_icall (). (#4917)

Use nuget 4.3.0-preview1 to fix build

4.5.0 is not yet available.

Also use lowercase nuget.exe to match the URL on https://dist.nuget.org/

[mcs] Add expression bodied syntax for accessors. Fixes #54991

[interp] fix unaligned.

[MacSDK] Update Nuget to 4.5, switch to binary release instead of building

[bcl] Make monolite platform specific (#4910)

mscorlib.dll etc will be platform specific in the future. This also
means that we need to adapt monolite since it'll become platform
specific once we start the divergence.

monolite is essentially a repackaging of the "build" profile which
means we need to turn that profile into multi-platform profile
like net_4_x. We'll still have a symlink from build -> build-darwin
and the monolite which is downloaded or packaged in the tarball
is expanded into mcs/lib/monolite-<platform> depending on the platform
you're building on.

I had to move the get-monolite-latest target from the top-level
Makefile.am into mcs/class/Makefile so it has access to HOST_PLATFORM,
but it arguably should've been there anyway before.

[bcl] Exclude Mono.CSharp.dll from api diff

We don't guarantee a stable API there and it changes often
when mcs internals change.

[profiler] Fall back to CLOCK_MONOTONIC if CLOCK_PROCESS_CPUTIME_ID results in EINVAL. (#4913)

This is the case on WSL.

[mcs] Inflate referenced current types for imported type definitions. Fixes #56462

[mcs] Removes TypeManager calls from MoreSpecific

[ci] Add the collect-coverage tag for use on Jenkins. (#4912)

* [ci] Add the collect-coverage tag for use on Jenkins.

* Move collect-coverage up so we don't set different values for -O

[mcs] extend API for loading types and consuming source file from a stream (#4905)

[jit] Fix x86 failures and handle unaligned load with offset

[corlib] Address reviewer comments to new test.

Bump api-snapshot

Bump Roslyn to 2.3 preview

Bump bockbuild

Bump the btls submodule

Bring in the change that makes building the "tool" subdirectory
in BTLS off by default - we don't need it in Mono and it fails
to build on some platforms with older C++ compiler.

[mcs] Report more unsafe code errors in iterators. Fixes #56616

[build] Remove platforms from xbuild build

[amd64] Fix the allocation of the caller area in filter clauses. Fixes #56567. (#4903)

[bcl] Dispose Cecil AssemblyDefinition after usage (#4900)

Follow-up to https://github.com/mono/mono/pull/4899. I audited
all the places in BCL where we made use of Cecil's AssemblyDefinition
and made sure we're properly disposing them and not keeping files
unnecessarily open.

I used InMemory=true in cases where tracking the lifetime would
be complicated. There are also some tests in mcs/tests which I
didn't bother to fix.