[msvc] Update csproj files (#4151)

[build] Update list of aoted images in mcs mode

[System.Core] Add CryptoServiceProviders to mobile

[Facades] Move IncrementalHash to System.Security.Cryptography.Algorithms facade

It looks like the type won't be part of .net for anytime soon

[corlib] Fixes warning

[process] Fix process-unref-race.exe (#4144)

* [process] Rename MonoProcess to Process as it's not public anymore

* [process] Replace setting process->pid = 0 by process->signalled = TRUE

* [process] Remove processes_mutex lock when unrefing process->handle

We do not need to synchronize it here because we do not modify `processes`, and the only other place where we access process->handle is before adding it to `processes`.

* [process] Fix race condition

Without this memory barrier `process->next = processes` and `processes = process` could be reordered, and when accessing `processes` in mono_sigchld_signal_handler, we could observe a NULL processes->next. This would make us not find a pid in `processes`, and because of the posix API, we would never mark this process as exited.

This would trigger a hang in `process-unref-race.exe` test.

Thank you to @kumpera for the help figuring this out!

[s390x] Fix build

[aot] fix linker invocation for llvm mode on linux

clang++ should only be used for llvm_only (aka. bitcode) mode.

this is a follow up fix for:
https://github.com/mono/mono/commit/7d238ddd5edeeb42cec5e07b6d6527d16332f921
and
https://github.com/mono/mono/commit/fc5305d2af1b6891b05cb82342bf92d4659546e1

fixes https://bugzilla.xamarin.com/show_bug.cgi?id=46375

[tools] Add resx2sr internal tool

Merge pull request #4014 from BrzVlad/feature-tls-refactor

[runtime] thread local storage refactor

[arm64] Mark MONO_CONTEXT_GET_CURRENT () as clobbering x16.

Fix build break due to Cecil API change

https://github.com/jbevain/cecil/commit/42db79cc16f1cbe8dbab558904e188352dba2b41

[xbuild] Bump buildtools to get support for netstandard 1.6

https://github.com/mono/buildtools/commit/9b6ee8686be55a983d886938165b6206cda50772

[system] Remove port number from TLS Server Name Identification (SNI). Fixes #46549 (#4120)

The port number should not be included along the host name. Otherwise
the server will refuse the connection (and we'll throw). This is a
problem when servers are not using the default (443) SSL/TLS port.

ref: https://bugzilla.xamarin.com/show_bug.cgi?id=46549

The BTLS provider was fixed but the old MonoTLS (managed) provider had
the same issue.

Another PR will be made to fix AppleTLS in xamarin-macios repo [2]

[1] https://github.com/mono/mono/pull/3939
[2] https://bugzilla.xamarin.com/show_bug.cgi?id=45994

Bump cecil to get portable pdb fix

[arm] Resurrect inlined fast tls

Only on desktop linux, where we now inline aeabi_read_tp.

[arm64] Resurrect inlined fast tls

[x86] Resurrect inlined fast tls

[runtime] Global debug option for using fallback tls

[amd64] Resurrect inlined fast tls

By default all platforms will call into native getters/setters whenever they need to get access tls value. On certain platforms we can try to be faster than this and avoid the call. We call this fast tls and each platform defines its own way to achieve this. Fast tls should normally be inlined, otherwise there is little point to doing anything else in the first place (on linux, __thread access is 2-3 instructions, on mac pthread_getspecific is 2 instructions, other platforms also having decent implementations).  For this, a platform has to define MONO_ARCH_HAVE_FAST_TLS, and provide alternative getters/setters for a MonoTlsKey. In order to have fast getter/setters, the platform has to declare a way to fetch an internal offset (MONO_THREAD_VAR_OFFSET) which is stored in the tls module, and in the arch specific file probe the system to see if we can use the offset initialized here. If these run-time checks don't succeed we just use the fallbacks.

In case we would wish to provide fast inlined tls for aot code, we would need to be sure that, at run-time, these two platform checks would never fail otherwise the tls getter/setters that we emitted would not work. Normally, there is little incentive to support this since tls access is most common in wrappers and managed allocators, both of which are not aot-ed by default. So far, we never supported inlined fast tls on full-aot systems.

[runtime] Refactor and unify tls access

The purpose of this commit is to unify tls access across all platforms. We had tls information spread out throughout the entire code which was guarded with many defines/capabilities which made very hard to follow what was going on. Also this made switching between pthread and __thread very unstable since there were a lot of dependencies between different pieces of the tls infrastructure. Also it was not very clear what happens if one of the tls pieces failed.

Now we are moving all the tls variables to a single module that provides getters/setters and initialization API. Runtime code now knows that we can always access a tls variable (from managed code, we just do a nowrapper icall into the same global geters/setters). Even though we used to these type of calls all around before, it is worth mentioning that runtime should account for a thread to be in this type of unregistered code without m2n wrapper and no register jinfo.

On average this makes common tls intensive methods (m2n wrappers and managed allocator) 20% slower, with potential for more regression if we use tls getters/setters recklessly. As a consequence we will re-enable the most important pieces of fast tls code in the future.

[exception] do not wrap SIGILL into a managed exception

there is not much point in trying to recover from it and even worse,
wrapping it into a managed exception might hide it if the application is
sloppy with exception handling (cf. Pokémon Exception Handling).

[exception] rename mono_handle_native_sigsegv -> mono_handle_native_crash

[exception] pass signal as string already to mono_handle_native_sigsegv

Merge pull request #4139 from lambdageek/bug-49686

[sre] Implement DynamicMethod.GetCustomAttributes() and IsDefined() (Fixes #49686)

Merge pull request #4110 from BrzVlad/fix-sgen-win32

[sgen] Fix sgen win32

[jit] Add a workaround to avoid a crash in mini_above_abort_threshold () when using bitcode.

[process] Remove useless FIXME

[arm64] Save the fp regs in MONO_CONTEXT_GET_CURRENT (). (#4121)

[threadpool] Assert that we do not overflow ThreadPoolCounter starting and working fields (#4138)

[threadpool] Fix `MonoTests.runtime.unload-appdomain-on-shutdown.exe` (#4135)

* [threadpool] Increase and decrease refcount when removing domain

* [threadpool] Fix unload-appdomain-on-shutdown.exe

If the threadpool is not initialized, there is no need to initialize it, as no worker is going to start in the domain because mono_domain_is_unloading would return TRUE.

[sre] Implement DynamicMethod.GetCustomAttributes() and IsDefined ()

Per .NET documentation, DynamicMethod does not support custom attributes
exception the MethodImplAttribute pseudo-custom attribute.

Also change GetMethodImplementationFlags to include NoInlining as on .NET

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

[test] Test for System.Linq.Expressions.MethodCallExpression with a DynamicMethod

Regression test for https://bugzilla.xamarin.com/show_bug.cgi?id=49686

Revert "[ci] Move mcs switch for Win32 builds into the run-jenkins.sh script"

This reverts commit 2b8d9bdb0850d98ed65f025a4cdcb4ca3f1eaa19.

[sgen] Don't suspend thread in wow64 transition

The stack pointer provided by GetThreadContext is not reliable.

[test] Test for System.Reflection.Emit.DynamicMethod.GetCustomAttributes

[sgen] binary protocol on windows

[runtime] Fix prototype of functions

These functions are cast and invoked as a MonoThreadStart (which requires to receive on arg, return one arg and be WINAPI on windwos), which could lead to cconv problems if the prototype is not matching.

Fix a warning.

[amd64] Return from the altstack before handling native sigsegvs. This will make backtrace () actually work.

[jit] Pass '-nx' to gdb when printing native backtraces so it doesn't read .gdbinit, thus making it more deterministic.

Remove the debian-snapshot submodule

Jo said it's no longer used. We're using the following repos now:

https://github.com/mono/linux-packaging-mono
https://github.com/mono/linux-packaging-mono-snapshot

[msvc] Update csproj files (#4136)

Revert "Fix a warning."

This reverts commit b498facf7cebade5762d79f28cd102a5ed120bc5.

Fix a warning.

[threads] Destroy thread->suspended before removing the GC root to thread (#4133)

[build] Print CSC when compiling with roslyn

Reduces confusion about why the log contains lines starting with MCS
even when using roslyn.

[xbuild] publicsign and delaysign cannot be used at same time

[xbuild] Add a workaround for not yet implemented full /keyfile signing using csc

[unwind] reduce frame size of mono_unwind_frame ()

on arm32 it goes down from 5120 bytes to 676 bytes in terms of frame
size.

[Microsoft.CSharp] Tweaks 76f97e30731d6c98f21771b516b78399185c1c51 to not call new API from Microsoft.CSharp. Fixes #47353

This is a workaround for runtime assembly loading bug always preferring local assembly

[xbuild] Convert pbdonly to portable debug mode

[System] Always use mcs as CodeDOM backend compiler

[build] Add System.ValueTuple to bootstrap facades

Don't remove the roslyn binaries in "make dist" for now

Should help fix the package builds on Jenkins.

Add unimplemented Transaction.PromoteAndEnlistDurable (#4129)

Which was added to .NET 4.5.2.

[simd] Add intrinsics support for Vector<T>.CopyTo ().

[msvc] Update csproj files (#4128)

[Facades] Add System.ValueTuple facade (#4125)

[arm64] Handle fp regs in mono_sigctx_to_monoctx () on linux, recent gcc versions generate code which store object references in them, so sgen now requires this. (#4127)

[threadpool] Split domain and worker management (#4117)

* [threadpool] Remove `-ms` suffix

* [threadpool] Split domain and worker management

This will allow us to use native threadpool more easily: we will simply have to implement a threadpool worker with the specific API.

On windows, we will use the Win32 Threadpool, and on OSX we will explore using GCD (Grand Central Dispatch).

[threads] Inline thread_cleanup into mono_thread_detach_internal (#4119)

Expose mono_build_root to mcs build system

In an out-of-tree build the Makefile which contains get-monolite-latest
doesn't exist in the source directory (which $(topdir)/.. would point to)
but in the build directory.

In order to invoke get-monolite-latest we need to actually call the Makefile
in the correct directory which requires exposing the build directory to the
mcs build system.

[configure] fix out-of-tree-build with roslyn

[runtime] Add write barriers for FindFirstFile/FindNextFile icalls (#4114)

We were seeing crashes like [1] or [2] after https://github.com/mono/mono/pull/4042 was merged.
While debugging the issue I found out that when instead of using `out data.cFileName` for passing
the string through the the icall I use a temporary field and assign the result to `data` later on
the crash went away. It also worked if I made the `WIN32_FIND_DATA` class a struct instead.

I talked to @ludovic-henry and he was able to figure out what was going on (many thanks!):

> It seems to be a missing wbarrier
> and that would also explains why it would work with a struct, but not with a class
> so it would work with a struct, because `out data.filename` would be a `MonoString**` to the thread stack,
> and that would work because we would always scan the thread stack
> and it wouldn’t work with a class, because `out data.filename` would be a `MonoString**` to the heap
>
> and because we have a generational GC, if `data` is in the “old” heap (and because we are missing this write barrier),
> we would never scan `data`, thus freeing the `data.filename` string object, and as soon as you are trying to access it,
> well you access garbage
>
> with the write barrier, we will make sure to scan the `data` object, and not loose the reference to `data.filename`

[1] https://jenkins.mono-project.com/job/test-mono-mainline/label=osx-amd64/5422/parsed_console/log_content.html#WARNING1
[2] https://jenkins.mono-project.com/job/test-mono-mainline/label=osx-amd64/5419/parsed_console/log_content.html#WARNING1
[3] https://github.com/mono/mono/blob/dd88ff52e2f130ee188661a34794aa7c7b564fa8/mcs/class/referencesource/mscorlib/system/io/filesystemenumerable.cs#L280

[System] Add new test

[MacSDK] Include 'opt' as a PATH-available binary

[runtime] Use MonoBoolean instead of gboolean in monitor icalls. Add new internal variants for the ones which are exported. (#4109)

[jenkins] Rename runtime presets to match CI tags

[ci] Simplify run-jenkins.sh script variables (#4111)

Get rid of WORKSPACE which is set by Jenkins, this allows one to
more easily run the run-jenkins.sh script locally e.g. on cygwin.

The only thing that needs to be passed in is CI_TAGS and label.

[ci] Run aot-test on more platforms (#4108)

* [ci] Run aot-test on more platforms

Now that https://github.com/mono/mono/pull/4096 is merged, we should be able to run it on Linux too.

* [ci] Don't run aot-test on Windows

It's broken there.

[jenkins] Add `bitcode` CI tag and rename `bitcode_aot_only` runtime preset to `bitcode`

[jenkins] Add `aot_llvm` and `aot` CI tag as well as `aot` runtime preset

[jenkins] Add fullaot_llvm CI tag and rename aot_only to fullaot

[jenkins] Fix order of CI tags

[jenkins] Add hybridaot_llvm CI tag and rename aot_preferred to hybrid

[msvc] Update csproj files (#4115)

[build] Add System.Security.Cryptography.Primitives to bootstrap facades

Merge pull request #4102 from marek-safar/bootstrap

[build] Fixes win32 bootstrap in mcs mode

[ci] Move mcs switch for Win32 builds into the run-jenkins.sh script

So we can more easily switch it over to roslyn again in a PR.

[threads] Fix conflicting signal with boehm (#4096)

Boehm would use SIGPWR on linux 32/64bits for its suspend signal, but that would conflict with utils/threads suspend signal.

The fix consists in looking for an available realtime signal in utils/threads, and fallback to the default SIGXFSZ, SIGPWR and SIGXCPU

[llvm] Fix intrinsic Selector.GetHandle support. (#4103)

An OBJC_IMAGE_INFO section must be emitted when we emit the native equivalent
of @selector for Selector.GetHandle, otherwise the Objective-C runtime will
not process the selectors in this image at runtime (and we may end up with
duplicate selectors in memory).

The AOT compiler already does this here: https://github.com/mono/mono/blob/2eda626316fd8f70cd98947af2f89b7ca9b0ebe5/mono/mini/aot-compiler.c#L9657-L9661

[Mac SDK] Use MSBuild to build IronPython/IronRuby.

Xbuild currently fails with 'CSC: error CS0041: Unexpected error writing debug information -- 'Windows PDB writer is not available -- could not find Microsoft.DiaSymReader.Native.x86.dll'

Xbuild should be passing "/debug:Portable" to csc in order to work. There is a possibly broken attempt to this in 2d08353991213b17f58f64b6ef0fb4525eb4c659.

[System]: fix certificate validation on Wine/OSX

- the `is_macosx` static field should be false on Wine/OSX, as it is on Windows.

Revert "[runtime] Use MonoBoolean as the return value of monitor icalls."

This reverts commit 06088609b0ee50af41e59e7366100e25d55a9130.

It was breaking the build on i386 Linux:

```
Unhandled Exception:
 System.ExecutionEngineException: System.Threading.SynchronizationLockException: Object synchronization method was called from an unsynchronized block of code.
   at Microsoft.CodeAnalysis.CSharp.CodeGen.CodeGenerator.EmitExpression (Microsoft.CodeAnalysis.CSharp.BoundExpression expression, System.Boolean used) [0x00070] in <8d5a2b6695c6475bb1cb5b9a9f9c8dd7>:0
   at Microsoft.CodeAnalysis.CSharp.CodeGen.CodeGenerator.EmitStatement (Microsoft.CodeAnalysis.CSharp.BoundStatement statement) [0x000ad] in <8d5a2b6695c6475bb1cb5b9a9f9c8dd7>:0
   at Microsoft.CodeAnalysis.CSharp.CodeGen.CodeGenerator.EmitStatementAndCountInstructions (Microsoft.CodeAnalysis.CSharp.BoundStatement statement) [0x0000c] in <8d5a2b6695c6475bb1cb5b9a9f9c8dd7>:0
   at Microsoft.CodeAnalysis.CSharp.CodeGen.CodeGenerator.EmitSequencePointStatement (Microsoft.CodeAnalysis.CSharp.BoundSequencePoint node) [0x0002d] in <8d5a2b6695c6475bb1cb5b9a9f9c8dd7>:0
   at Microsoft.CodeAnalysis.CSharp.CodeGen.CodeGenerator.EmitStatement (Microsoft.CodeAnalysis.CSharp.BoundStatement statement) [0x00087] in <8d5a2b6695c6475bb1cb5b9a9f9c8dd7>:0
   at Microsoft.CodeAnalysis.CSharp.CodeGen.CodeGenerator.EmitBlock (Microsoft.CodeAnalysis.CSharp.BoundBlock block) [0x0009d] in <8d5a2b6695c6475bb1cb5b9a9f9c8dd7>:0
   at Microsoft.CodeAnalysis.CSharp.CodeGen.CodeGenerator.EmitStatement (Microsoft.CodeAnalysis.CSharp.BoundStatement statement) [0x0007a] in <8d5a2b6695c6475bb1cb5b9a9f9c8dd7>:0
   at Microsoft.CodeAnalysis.CSharp.CodeGen.CodeGenerator.EmitStatementList (Microsoft.CodeAnalysis.CSharp.BoundStatementList list) [0x00023] in <8d5a2b6695c6475bb1cb5b9a9f9c8dd7>:0
   at Microsoft.CodeAnalysis.CSharp.CodeGen.CodeGenerator.EmitStatement (Microsoft.CodeAnalysis.CSharp.BoundStatement statement) [0x000b4] in <8d5a2b6695c6475bb1cb5b9a9f9c8dd7>:0
...
```

[corlib] Put back Directory.GetDirectoryRoot() which was accidentally removed

The mistake happened in https://github.com/mono/mono/pull/4042.
Added a test for the method so we don't do it again.

Fix Microsoft.Build.Engine fail when built with mcs

Fixes the Makefile to not fail if the compiler hasn't generated a pdb file for
the TestTasks dll.

Revert "Merge pull request #4093 from ntherning/enable-building-bcl-with-mono-4.6-mcs"

This reverts commit d661684b9a7a49646720ff14dfaf63c62c8619ed, reversing
changes made to 556a4f45653f048c13a171cb03ba4b674f21afb1.

[build] Fixes win32 bootstrap in mcs mode

Uses installed mono instead on .NET to run same boostrap check as we
do on other platforms

[io-layer] Remove io-layer.h include from as many headers as possible (#4088)

[w32handle] Fix race in foreach and unref (#4089)

It could happen that we unref a handle that we are currently looping over. This could be the case if we are calling Process.GetProcessById (which loops over all handles) and mono_w32handle_unref on a MonoW32HandleProcess handle.

[runtime] Use MonoBoolean as the return value of monitor icalls.

Merge pull request #4093 from ntherning/enable-building-bcl-with-mono-4.6-mcs

Enable building the BCL with mcs from pre-4.8 mono installation

[threads] Do not switch to GC safe twice, it's already done inside mono_join_uninterrupted

[runtime] Use coop handles for the remaining System.Reflection.Assembly icalls (#4082)

* [reflection] Use handles for System.Reflection.Assembly.GetFilesInternal

* [reflection] Use handles for System.Reflection.Assembly.InternalGetType

* [reflection] Use handles for System.Reflection.Assembly.GetManifestResourceNames

* [reflection] Use handles for System.Reflection.Assembly.LoadPermissions

* [reflection] Add mono_method_get_object_handle

  Reimplement mono_method_get_object_checked in terms of the handles one.

* [reflection] Use handles for System.Reflection.Assembly.get_EntryPoint

* [reflection] Use handles for four more icalls in System.Refleciton.Assembly

  That's the lot of them.

    * get_ReflecitonOnly
    * get_code_base
    * get_fullname

* get_global_assembly_cache

* [security] Use handles for System.Security.Policy.Evidence.IsAuthenticodePresent

* [runtime] Use handles for System.Web.Util.Icalls.GetUnmanagedResourcesPtr

* [runtime] Use MONO_HANDLE_GETVAL instead of MONO_HANDLE_RAW in a few places

* [runtime] Use handles for System.AppDomain.ExecuteAssembly

Enable building the BCL with mcs from pre-4.8 mono installation

This skips passing -runtimemetadataversion when building the BCL with
--with-csc=mcs.

Merge pull request #4092 from ntherning/fix-w32process-win32.c-build-failure

Fix build failure in w32process-win32.c

Fix build failure in w32process-win32.c

Commit 07630fe8c5b73c5ed47038a8e63b098fbbea05ff broke the build on Windows.

Merge pull request #3997 from lateralusX/jlorenss/win-api-family-support-safearray

Add possibility to exclude COM SafeArray implementation on some none desktop Windows API families.

A few pretty print fixes for the GDB plugin. (#4001)