Merge pull request #4327 from vkargov/vk-abcremedy

[runtime] Abcrem fixes.

[Facades] Update with new typeforwarders/APIs (#4838)

[mcs] Add platform specific files (#4837)

* [mcs] Rework %-recursive target to explicitely list subdirectories

* [mcs] Rename PLATFORM to BUILD_PLATFORM

* [mcs] Add platform specific files

This is to support adding corefx platform specific files. To do that, we now support `mcs/class/corlib/linux_net_4_x_corlib.dll.sources` for example, on top of `mcs/class/corlib/net_4_x_corlib.dll.sources` and `mcs/class/corlib/corlib.dll.sources`.

The 3 platforms that are supported are: `linux`, `darwin` and `win32`.

* [mcs] Build cross-platform bcl in separate folder

[System]: HttpListener.CreateSslStream() should always use passed-in callback. (#4835)

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

This reverts commit 47365cd54b69a869c50e39223ff64dc8e4fb8889.

We didn't do this in the private bockbuild and putting
'opt' from Mono into the PATH could be problematic.

This makes us consistent with the private bockbuild whitelist.

Specify Encoding.UTF8 when marshaling native runtime string. If not, string constructor accesses Encoding.Default which causes infinite recursion on Windows for codepages supported via I18N. While trying to load I18N assemblies, assembly names are marshaled in managed code now hitting this code path. Fixes Xamarin bug 43988.

[jit] Keep unwind info in the domain's memory pool. (#4825)

Merge pull request #4796 from lambdageek/dev-handles-may04

[runtime] Another batch of coop handles icalls

[MacSDK] Port change to license.rtf from private bockbuild

https://github.com/xamarin/bockbuild/commit/97d660778097b391151776a611494a545becd62c

Merge pull request #4816 from BrzVlad/fix-remoting-exception

[remoting] Serialize exceptions between domains inside try/catch

Merge pull request #4818 from kumpera/unaligned-cleanup-2

Last bucket of cleanups before I land the actual code for unaligned.

[System]: Add new 'common-networking.sources' and move stuff from 'common.sources'. (#4815)

* [System]: Add new 'common-networking.sources' and move stuff from 'common.sources'.

This moves all the networking-related sources which aren't supported on monotouch_watch
into a separate file, so we don't have to maintain a huge "Thing That Should Not Exist".

* Put back System.Net/NtlmClient.cs - we actually do need it.

* Add HttpListenerTimeoutManager.platformnotsupported.cs.

* Make it build.

[process] Disable tests that do not work on mobile (#4598)

[msvc] Update csproj files (#4822)

[security] Use coop handles for ves_icall_System_Security_Principal_WindowsIdentity_GetUserToken

[security] Use coop handles for ves_icall_System_Security_Principal_WindowsIdentity_GetTokenName

[security] Use coop handles for ves_icall_System_Security_Principal_WindowsIdentity_GetCurrentToken

Refactor it into mono_security_principal_windows_identity_get_current_token ()
and then call that from ves_icall_System_Security_Principal_WindowsIdentity_GetCurrentToken ()

[runtime] Use coop handles for System.RuntimeMethodHandle.GetFunctionPointer

[sre] Use coop handles for ModuleBuilder.set_wrappers_type

[sre] Use coop handles for ModuleBuilder.GetRegisteredToken

- Fix typo in REFLECTION_EMIT_INIT/REFLECTION_EMIT_SAVE in
  mono_dynamic_image_register_token
- Add mono_dynamic_image_get_registered_token ()
- N.B. locking change - ves_icall_ModuleBuilder_GetRegisteredToken used to take
  the loader lock.  Now it calls mono_dynamic_image_get_registered_token which
  just takes the image lock.

[runtime] Use coop handles for System.Environment.internalBroadcastSettingChange

[locales] Use coop handles for System.Text.Normalization.load_normalization_resource

[sre] Use coop handles for System.Reflection.Emit.EnumBuilder.setup_enum_type

[sre] Use coop handles for System.Reflection.Emit.SignatureHelper.get_signature_local

[sre] Use coop handles for System.Reflection.Emit.SignatureHelper.get_signature_field

[loader] resolve symlinks when looking for an image in the loaded_images cache

Consider the old code (which just used mono_path_canonicalize)

Suppose foo.dll is a symlink to bar.dll

Suppose we've try to load foo.dll the first time:
do_mono_image_open is passed "foo.dll" it then sets image->name to "bar.dll"
which is then used by register_image to do a lookup and then inserts the
mapping from "bar.dll" to the new image (call it Image1).

Suppose we try to load foo.dll a second time:
we check for "foo.dll" in the loaded_images cache and get NULL.
So we call do_mono_image_open ("foo.dll", ...) again, and again image->name is
set to "bar.dll".  (Call this Image2)
We call register_image again which looks up bar.dll and gets Image1.  So it
increments the refcount for Image1 and unloads Image2 and so
mono_image_open_a_lot returns Image1.

With the new code:

The first time we try to load foo.dll everything is the same.  The second time
we try to load foo.dll, we resolve the symlink from "foo.dll" to "bar.dll" in
mono_image_open_a_lot and return Image1 directly from loaded_images.

[System] Use 'ObjCRuntimeInternal' as the namespace instead of 'ObjCRuntime'. (#4820)

Use 'ObjCRuntimeInternal' as the namespace instead of 'ObjCRuntime' to avoid
these compiler-warnings when building xamarin-macios assemblies (which defines
the same types in the ObjCRuntime namespace, and to whom System.dll has an
InternalsVisibleTo attribute):

> xamarin-macios/src/build/watch/watch/Foundation/NSExtensionRequestHandling.g.cs(28,49): warning CS0436: The type `ObjCRuntime.INativeObject' conflicts with the imported type of same name'. Ignoring the imported type definition
> xamarin-macios/src/ObjCRuntime/INativeObject.cs(6,19): (Location of the symbol related to previous warning)
> xamarin-macios/_ios-build/Library/Frameworks/Xamarin.iOS.framework/Versions/git/lib/mono/Xamarin.WatchOS/repl/System.dll (Location of the symbol related to previous warning)

[mini] Cleanup usage of mini_emit_memcpy and bad values for alignment - align to 4 on 64bits is *bad*.

[jit] CPBLK and INITBLK now use the funtions in memory-access.c

[jit] Add mini_emit_memory_copy_bytes and mini_emit_memory_init_bytes to be used by cpblk and iniblk.

Merge pull request #4781 from kumpera/unaligned-cleanup-1

First pass of cleanups to introduce a clear layer that deals with emiting load/stores.

[remoting] Serialize exceptions between domains inside try/catch

If a remote invoke in another domain throws an exception, the xdomain-dispatch wrapper will serialize the exception and pass it over to the calling domain so that it can be rethrown there. Serializing the exception object can itself throw an exception which we didn't properly catch, leading to unwinding to the caller domain without changing the domain state back and wreaking havoc.

If the serialization of the original exception throws a new exception, this new exception takes its place, being passed instead to the caller domain.

Bump corefx

[runtime] Update msvc build scripts.

[jit] Allow inlining of methods with the AggressiveInlining flag even if they make calls. (#4811)

[jit] Mark bblocks making calls to the corlib ThrowHelper class as out of line. (#4812)

[System.Data] Update cas test category

[mcs] Conditional check on attribute type needs to check its base types too

[msvc] Update csproj files (#4813)

[System]: Remove 'SECURITY_DEP' conditional from HttpListener and related classes. (#4810)

[tools] Add resx2sr to "build" profile (#4807)

Similar to 9635b9cea0515432fead4c370b0533db91861bfe we
can now break the dependency on WinForms from resx2sr.

This allows us to add it to the "build" profile
and clean up the Makefile.

[jit] Add mini_emit_memory_store and use it for STOBJ.

[jit] Fix identation.

[jit] Use mini_emit_memory_load for ldind, ldobj and ldfld

[jit] Add mini_emit_memory_load that is to be used to lower CIL memory loads.

[System]: Add private '_internalLock' field to HttpListener. (#4809)

After switching to the CoreFx version, HttpListener.Mono.cs will remain,
so it needs to use the proper locking object.

[bcl] Add API diff for monotouch/monodroid profiles (#4805)

This runs the API diff against the monotouch and monodroid
profiles in addition to net_4_x to help us catch mistakes that
only affect the mobile profiles.

To make updating the API snapshot in case of an intentional change
easier without needing to build those profiles locally I added
a way to download the diff as a .patch file. You can just download
the file from the API diff Jenkins generates and apply it locally.

[bcl] Cleanup outdated csproj/sln files (#4806)

They were either replaced by the autogenerated csproj files
or just completely broken for years.

[sdb] Avoid printing out the ThreadAbortException used to abort the protocol receiver thread. (#4804)

[System]: Make HttpListener partial and move Mono-specific bits into HttpListener.Mono.cs. (#4808)

* [System]: Make HttpListener partial and move Mono-specific bits into HttpListener.Mono.cs.

* Remove ancient EMBEDDED_IN_1_0 conditional.

* Remove unused using statements.

* HttpListener.CreateSslStream() now uses RemoteCertificateValidationCallback.

[System]: Rename the internal class 'ChunkStream' into 'MonoChunkStream'. (#4803)

* [System]: Fully stub-out SslStream on platforms where it's not supported.

* Remove 'SECURITY_DEP' conditional from LocalCertificateSelectionCallback.

* Add the internal delegates and remove 'SECURITY_DEP' from SslStream.platformnotsupported.cs.

* In SslStream.cs, we now provide the full API when '!SECURITY_DEP' using the
PlatformNotSupportedException()-based implementation from SslStream.platformnotsupported.cs

* [System]: Rename the internal class 'ChunkStream' into 'MonoChunkStream'.

Our 'ChunkStream' is used by 'WebConnection', so it will stay after using
HttpListener from corefx.

[System]: Fully stub-out SslStream on platforms where it's not supported (#4802)

* [System]: Fully stub-out SslStream on platforms where it's not supported.

* Remove 'SECURITY_DEP' conditional from LocalCertificateSelectionCallback.

* Add the internal delegates and remove 'SECURITY_DEP' from SslStream.platformnotsupported.cs.

* In SslStream.cs, we now provide the full API when '!SECURITY_DEP' using the
PlatformNotSupportedException()-based implementation from SslStream.platformnotsupported.cs

* Fix typo.

Merge pull request #4799 from xmcclure/mingw-has-secure-api

Fix mingw/mxe build

[mono-error] Always call TLE(string,string) constructor

if assembly name is NULL, mono_exception_from_name_two_strings_checked looks
for a TLE(string) constructor - which takes a message - which we immediately
overwrite.

So pass the empty string instead, which goes to the (Mono-specific)
TLE(string,string) constructor which intializes ClassName and AssemblyName
members of a TLE.

[runtime] Fail classes that have a field with a bad underlying type

If a class T has a field e of enum type E, and E is either
incomplete (EnumBuilder that hasn't had CreateType called) or a broken class,
then mark T as failed.

[test] Don't crash if incomplete enum type is a static field in a class

Fix mingw/mxe build

wcscat_s is not available on mingw unless a special symbol is defined. Add this symbol to exceptions-amd64.c

[jit] Fix 4c0df6de333fe18abb97fbc7ea8f6e77d92f5451, use __GNUC__ instead of GNUC.

[interp] do widening for more arith ops

[configure.ac] Fix inadvertently invoking X server (#4792)

I noticed this when I cleaned my Mono repo and reran autogen.sh.
I suddenly got a popup from XQuartz about "Unrecognized option: =".

Turns out this was introduced in 9e967f00b3073671b340c1a4e3da7253d5213668,
the second half of the if is missing `test`, so it actually runs
`x = xyes` which starts XQuartz :D

Also cleaned up the USE_NLS checking since that ran immediately
before this and just printed "no" before which confused me.

Merge pull request #4782 from lambdageek/dev-mono_string_new-excise

[runtime] Mark mono_string_new external only

Merge pull request #4788 from xmcclure/if-winrt

Fix hybrid AOT build

[System]: Fix TLS Provider registration and use BTLS instead of Legacy. (#4793)

Require #if WIN_PLATFORM more strictly around some WinRT timezone functions

This was breaking the hybrid AOT build.

[runtime] Mark mono_string_new external only

Runtime should use mono_string_new_checked

[interp] Use mono_string_new_checked instead of mono_string_new

[mini] Use mono_string_new_checked instead of mono_string_new

(mono-security-windows.c) (aot-compiler.c) (debugger-agent.c) (driver.c) (mini-exceptions.c)

[runtime] Use coop handles for some simple icalls

- System.Text.EncodingHelper.InternalCodePage
- System.Runtime.InteropServices.Marshal.PtrToStringAnsi(intptr)
- Use coop handles for ves_icall_System_Configuration_DefaultConfig_get_machine_config_path
- Use coop handles for ves_icall_System_Configuration_InternalConfigurationHost_get_bundled_app_config
- Use coop handles for System.Web.Util.ICalls.GetMachineInstallDirectory

[locales] Use coop handles for System.Globalization.CultureInfo.get_current_locale_name

[runtime] Use coop handles for System.Environment icalls

- get_UserName
- get_NewLine
- get_MachineName
- GetWindowsFolderPath
- GetOSVersionString

MonoError-ize mono_icall_get_environment_variable_names

[appdomain] Make mono_domain_set_config external only

Runtime should use mono_domain_config_checked ()

[locales] Use mono_string_new_checked instead of mono_string_new

[runtime] Use mono_string_new_checked instead of mono_string_new

(w32file.c) (filewatcher.c) (appdomain.c) (attach.c) (appdomain.c)
(gc.c) (marshal.c) (mono-perfcounters.c) (mono-security.c) (object.c)
(sre.c) (threadpool.c) (threads.c) (w32process.c) (w32socket.c)

[jit] Compile unwinder_unwind_frame without optimization on arm64 as old gcc versions seems to miscompile it.

[mono-route] Use _checked functions

And include config.h

[mono-error] Use mono_string_new_checked instead of mono_string_new

[runtime] Use mono_string_new_checked in exception.c

[pedump] Fix running with cooperative GC

This is the pedump analog of 9f5115a343213e7ce27b197c96274cc611a05284
which fixed monodis under coop.

[bcl] Add .NET 4.7 reference assemblies (#4791)

[System.Net.Http] Fix XI build

[jit] Fix memory leak in trampoline info registration. (#4775)

[MacSDK] Build msbuild for Release config

Merge pull request #4787 from BrzVlad/fix-degraded-mode

[sgen] Fix accidental triggering of degraded mode

Bump bockbuild

Merge pull request #4780 from xmcclure/assembly-loader-default

Document --assembly-loader and change default to 'legacy'

Document --assembly-loader, change default to 'legacy' and fix comment

Also set testing_gac tests to use --assembly-loader=strict, since it is no longer the default

[sgen] Fix accidental triggering of degraded mode

We trigger degraded mode when we are unable to properly build the nursery fragments due to excessive nursery pinning. This is done by initializing degraded_mode at the end of a minor. alloc_degraded is used to do allocation to major heap either as part of mature allocation or as part of degraded mode (when we bump the degraded_mode so we can detect when the next minor should be triggered).

When failing to allocate from the nursery after doing a minor (because the other threads might have already consumed the nursery) we need to do a mature allocation (as a fallback) instead of a degraded mode allocation so we don't uselessly enable the mode.

Mixed mode exception handling (#4777)

* [mixed] Add beginnings of unwinding support for interp->jit transitions by pushing an LMF frame on the stack when exiting interpreted code.

* [runtime] Add a mono_debug_lookup_source_location_by_il () helper function to lookup a source location using an IL offset.

* [mixed] Add support for mixed mode managed stack walks.

* [interp] Generate line number info for interpreted code, using the same MonoDebugMethodJitInfo structure used by the JIT.

* [interp] Pass the clause index to the MINT_ENDFINALLY opcode. Not yet used.

* [interp] Create a MonoJitInfo structure for each interpreted method. Not yet used.

* [interp] Add a mono_interp_set_resume_state () function which can be used to set the frame/ip the interpreter will resume execution from when execution returns to it.

* [mixed] Add support for mixed mode exception handling by extending the normal JIT exception handling code in mini-exceptions.c to handle interpreted frames as well.

* [jit] Add an Unwinder type to reduce the amount of code duplication when unwinding though interpreter frames.

* [interp] Fix the !ENABLE_INTERPRETER build.

* [interp] Disable an assert which is hit even when running without --interpreter.

* Fix a comment.

* [interp] Fix an assertion. Fix/add comments.

[loader] Add a few Framework Facades assemblies (#4784)

that were previously missing, or mislabeled as ordinary framework assemblies.

[MacSDK] Add 'csc' (Roslyn compiler) to whitelist

[jit] Move memcpy and memset emmiters to new file called memory-access.c

[mini] Rename & expose a few functions to mini.h.

This adds to mini.h & fixes naming of the following functions:
emit_get_gsharedvt_info_klass
mono_emit_calli
get_memcpy_method
get_memset_method
emit_memory_barrier
emit_runtime_constant
mini_class_check_context_used
mono_emit_wb_aware_memcpy
emit_write_barrier

Those functions will be used later by the memory access functions to be extracted out of method-to-ir.c

[ci] Output nunit xml if "verify" test step fails

Allows us to better catch this on Jenkins.

[ci] Don't fail when setting GitHub status doesn't work

[msvc] Update csproj files (#4776)

Merge pull request #4767 from lambdageek/bug-55681

[sre-save] Handle ConstructorBuilder custom attribute constructors

Merge pull request #4759 from lambdageek/flags-strict-strong-names

Put strict strong name checking behind a configure flag and runtime option

[interp] implement MONO_JIT_{AT,DE}TACH, so native2managed wrappers can be handled by the interpreter

[interp] implement CEE_MONO_{TLS,ATOMIC_STORE_I4}

so we can execute the alloc wrapper

[interp] remove ves_exec_method_with_context_with_ip, othwerwise we end up having to native frames for each managed frame in the debugger

and that's annoying.