[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.

Warn on failure to create default Reflection.Emit SymbolWriter instead of throwing and failing (#4748)

* Bug #53038: Warn on failure to create default Reflection.Emit SymbolWriter (on platforms like Android that don't ship one) instead of throwing and failing

[jit] Add unaligned tests.

[amd64] Remove the restriction on the locals stack size. Fixes #56452. (#4901)

[jit] Properly handle unaligned memory access.

[jit] Assorted small changes.

Remove 10000 size check in mini_emit_memory_copy_bytes, this is no longer needed now that the memcpy expansion logic in mini_emit_memcpy_internal was fixes.

Use a define for 10000.

Remove useless debug spew.

[mini] Remove assert and document reason it can't be used.

[interp] respect 64bit alignment on armv7

[pdb2mdb] Dispose Cecil AssemblyDefinition on pdb2mdb (#4899)

Context: https://bugzilla.xamarin.com/show_bug.cgi?id=56275

Recent Cecil 0.10 changed to not reading the assembly in memory but
instead reads directly from the underlying stream [0]. This however
means that the file handle is not freed until the AssemblyDefinition
is disposed.

Xamarin.Android is using the pdb2mdb source as part of
Xamarin.Android.Build.Tasks.dll and hit an issue where the file
would be locked inside of VS.

[0] http://cecil.pe/post/149243207656/mono-cecil-010-beta-1
(cherry picked from commit 5077205a44a7dc97edf6b67072bea53f043cf815)

[configure] Add stdalign.h checks for BTLS to ARM platforms too

We added it for x86/amd64 in 488b550ac0275c944e0a2c23be93173382084e5f
but missed adding that check when ARM was added recently.

[aot] Fix a memory leak related to constraint allocation/deallocation. (#4880)

* [jit] Make a copy of the constraint in mini_get_shared_gparam().

* [aot] Free the temporary gshared constraint in decode_class_ref().

Fixes for bug#54388 "InternalsVisibleTo is case sensitive" * Fix the JIT so that access checks are performed on constructors * Use ASCII case insensitivity when performing cross-assembly access checks for internals * Update mcs to use case-insensitive comparisons when evaluating InternalsVisibleTo attributes for access checks (matching the new runtime behavior, and the behavior of roslyn)

This issue is actually a pair of bugs:
1. We don't perform any access checks for constructors (which can give a false impression of our behavior for this issue). This means that you can also invoke private constructors from other assemblies.
2. InternalsVisibleTo checks are done using strcmp instead of a case-insensitive comparison.

[corlib] Bump DoubleWait test timeouts

Fix eglibc remapping and revive the test for its completeness. (#4896)

* [eglib] Add the missing eglib remappings.

* [tests] Revive the eglib remapping completeness test.

Adds System.Net.Http 4.3.2 to blacklist

[configure] Remove the --enable-dynamic-btls switch, it is the only option

[ci] Enable linker tests on PRs

Add linker tests