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

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

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

[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

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

[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

Add LinkerDescriptor

Bump linker

[runtime] Make RuntimeHelpers:SufficientExecutionStack () a no-op on android for now. Fixes #56240. (#4889)

[coop handles] Add mechanism to track handle leaks

When MONO_HANDLE_TRACK_SP is defined, each handle records the native C stack pointer
at the time of its nearest enclosing HANDLE_FUNCTION_ENTER.  If a subsequent
handle is allocated with a lower stack pointer (in memory; ie higher in the
call tree) then a HANDLE_FUNCTION_RETURN is missing and we're leaking handles.

[interp] assume EABI for passing arguments to arm trampoline

[interp] use helper function to check if generic inst is a value type

[arm] respect thumb mode in mixed mode unwinding

[interp] properly pass 64bit integers to native code on 32bit arches

[interp] disable some tests that fail on CI only

can't reproduce the crash on my dev device, will investigate later.

[interp] make mixed mode work on arm

[interp] generics/vt fix regarding elem size for ldelem.vt

[interp] intrinsify System.Array.UnsafeLoad

[interp] add a conversion that is only required for 32bit

[interp] icall trampoline for arm

[interp] make mixed mode work on arm64

[interp] fix gcc warnings

[interp] fix conv.u4.r8 on arm64

[interp/tramp] use MONO_STRUCT_OFFSET infrastructure and extract constants

[interp] implement icall_trampoline for arm64

Rename Microsoft.NuGet.Build.Tasks.dll.sources to exclude profile

The earlier name sent gensources.sh into an infinite loop,
not sure why yet. The directory is only built for xbuild_14
anyway so this should be safe.

Fix tarball build

The itanium backend was removed.

w32handle: double the available slots (#4856)

Mono crashes if the system has more than ~128G of RAM.  Doubling the
number of slots fixes the SIGSEGV error on FreeBSD.

[MacSDK] Remove some outdated info from ReadMe.rtf

uninstallMono.sh is no longer installed.

The website has instructions for removing Mono:
http://www.mono-project.com/docs/about-mono/supported-platforms/osx/#uninstalling-mono-on-mac-os-x

[btls] Add "desktop" ARM Linux configurations (for IoT, mostly) (#4875)

* [btls] Add "desktop" ARM Linux configurations (for IoT, mostly)

* [btls] Disable optimized asm on soft-float ARM

It depends on a NEON FPU, which we don't target on soft float

[btls] Add MonoPInvokeCallback attribute on all full aot profiles (#4879)

It was #ifdef'ed to only be available on monotouch before,
but the testing_aot_full profile in CI runs into this test failure:

```
MESSAGE:
System.ExecutionEngineException : Attempting to JIT compile method '(wrapper native-to-managed) Mono.Btls.MonoBtlsSslCtx:NativeVerifyCallback (intptr,int,intptr)' while running in aot-only mode. See https://developer.xamarin.com/guides/ios/advanced_topics/limitations/ for more information.

+++++++++++++++++++
STACK TRACE:
  at (wrapper managed-to-native) System.Object:__icall_wrapper_mono_thread_interruption_checkpoint ()
  at (wrapper managed-to-native) System.Runtime.InteropServices.Marshal:GetFunctionPointerForDelegateInternal (System.Delegate)
  at System.Runtime.InteropServices.Marshal.GetFunctionPointerForDelegate[TDelegate] (TDelegate d) [0x00013] in <fa55a8c31ba0490383239e5850d81686>:0
  at Mono.Btls.MonoBtlsSslCtx..ctor (Mono.Btls.MonoBtlsSslCtx+BoringSslCtxHandle handle) [0x00059] in <18810f39bffc41dcb0263eb2043cc10e>:0
  at Mono.Btls.MonoBtlsSslCtx..ctor () [0x0000b] in <18810f39bffc41dcb0263eb2043cc10e>:0
  at Mono.Btls.MonoBtlsContext.InitializeConnection () [0x00000] in <18810f39bffc41dcb0263eb2043cc10e>:0
  at Mono.Btls.MonoBtlsContext.StartHandshake () [0x00000] in <18810f39bffc41dcb0263eb2043cc10e>:0
  at Mono.Net.Security.MobileAuthenticatedStream.ProcessHandshake (Mono.Net.Security.AsyncProtocolRequest asyncRequest, Mono.Net.Security.AsyncOperationStatus status) [0x00004] in <18810f39bffc41dcb0263eb2043cc10e>:0
  at Mono.Net.Security.AsyncProtocolRequest.ProcessOperation (Mono.Net.Security.AsyncOperationStatus status) [0x0006b] in <18810f39bffc41dcb0263eb2043cc10e>:0
  at Mono.Net.Security.AsyncProtocolRequest.ProcessOperation () [0x0000d] in <18810f39bffc41dcb0263eb2043cc10e>:0
  at Mono.Net.Security.AsyncProtocolRequest.StartOperation () [0x0003c] in <18810f39bffc41dcb0263eb2043cc10e>:0
  at Mono.Net.Security.AsyncProtocolRequest.StartOperation (Mono.Net.Security.AsyncOperation operation) [0x00024] in <18810f39bffc41dcb0263eb2043cc10e>:0
  at Mono.Net.Security.MobileAuthenticatedStream.ProcessAuthentication (System.Net.LazyAsyncResult lazyResult) [0x00057] in <18810f39bffc41dcb0263eb2043cc10e>:0
```

The attribute itself is already conditional on MONOTOUCH and
FULL_AOT_RUNTIME so simply removing the ifdef should fix the issue.

Merge pull request #4877 from ntherning/fix-file-handle-leak-in-mono_mmap_open_file-on-windows

Fixes a file handle leak in mono_mmap_open_file() on Windows

Fixes a file handle leak in mono_mmap_open_file() on Windows. See
https://bugzilla.xamarin.com/show_bug.cgi?id=56493.

[xbuild] Add web targets for 15.0 (#4868)

[msvc] Remove order.xml (#4867)

It's a build artifact anyway since we check in the generated csproj's
so you don't really need the order.xml.

After the new platform-specific BCL changes went in it also started
containing stuff like `-r:./../../class/lib/net_4_x-linux/mscorlib.dll`
which makes it even less useful to have checked in as it'd get
changed when you build on OSX for example.

[threadpool] Fix alignment of ThreadPoolWorkerCounter (#4871)

It is accessed with a 64bits atomic operation, so it needs to be 64bits aligned, even on a 32bits platform. A CPU exception EXC_ARM_DA_ALIGN triggers on armv7 32bits.

Fix https://bugzilla.xamarin.com/show_bug.cgi?id=56202

Merge pull request #4752 from BrzVlad/feature-dynamic-nrs

[sgen] Dynamic nursery sizing

[corlib] Reduce the warning spew our testsuite produces.

[corlib] Add tests for the behavior of IO in the face of a symlink loop.

The behavior was empirically found on windows and dotnet.

[corlib] Remove the File.Exists check before opening the file.

It's both racy and not necessary.

Beyond that, it will produce the wrong exception if opening the file fails for a different reason that it's missing.

This is an issue in case of bad permissions or symlink loops.