[coop] Transition various public APIs into an external/internal form to avoid unnecessary GC-unsafe calls (mono/mono#14427)

Relevant issue: mono/mono#14376

There are potentially more functions this can apply to, but this covers the ones listed in the issue and an extra one I found in the process.

Functions were either broken off into an external-only and internal version, or such a pair already existed and the internal version was moved from the source file to a separate `-internals` header, similarly to in mono/mono@3c74df566776f468366d96dd164dd1a6b3d780cc.

This gets slightly complicated with `mono_domain_set` and `mono_domain_set_internal`, both of which are currently public (thanks to mono/mono@4beada28924879ede55f3bf8b3f5b798d0f4637e) and have different signatures. Additionally, the latter does not actually include the gc-unsafe transition despite being public.
I've opted to introduce `mono_domain_set_fast` as the 'internal' version of `mono_domain_set`, to add the proper gc-unsafe transition to `mono_domain_set_internal`, and then to migrate internal usage to `mono_domain_set_internal_with_options`. As part of that I've marked `mono_domain_set_internal` as external only, which is unfortunate naming-wise but I believe the best option here without introducing API breakage.

Commit migrated from https://github.com/mono/mono/commit/96b1fd837d6caf1be1b657c0726af3b8caae7060

[llvm] Use CoreCLR GC interop configuration (mono/mono#14598)

[llvm] Use CoreCLR GC interop configuration

Since they're identical right now, we can use the coreclr configuration
to make our IR more easily analyzed by tools built around LLVM.

```
class CoreCLRGC : public GCStrategy {
public:
  CoreCLRGC() {
    UseStatepoints = true;
    // These options are all gc.root specific, we specify them so that the
    // gc.root lowering code doesn't run.
    InitRoots = false;
    NeededSafePoints = 0;
    UsesMetadata = false;
    CustomRoots = false;
  }

  Optional<bool> isGCManagedPointer(const Type *Ty) const override {
    // Method is only valid on pointer typed values.
    const PointerType *PT = cast<PointerType>(Ty);
    // We pick addrspace(1) as our GC managed heap.
    return (1 == PT->getAddressSpace());
  }
};

/// A GCStrategy for the Mono Runtime.
class MonoGC : public GCStrategy {
public:
  MonoGC() {
    UseStatepoints = true;
    // These options are all gc.root specific, we specify them so that the
    // gc.root lowering code doesn't run.
    InitRoots = false;
    NeededSafePoints = 0;
    UsesMetadata = false;
    CustomRoots = false;
  }
};
```

Source: https://github.com/mono/llvm/blob/mono/mono@64c0343537016c153894dc60316bd7b316b812c8/lib/CodeGen/BuiltinGCs.cpp#L105-L137

Commit migrated from https://github.com/mono/mono/commit/ea4ff9ba8017ca1976a594d8d6c57a9052c4c043

Cleanup: load_function_full -- avoid a string alloc/copy and cast. (mono/mono#14625)

Cleanup: load_function_full -- avoid a string alloc/copy and cast.

Commit migrated from https://github.com/mono/mono/commit/f93d42dbf5122374f63a114fcf1ee6e824551c2e

Fix build warning for orig_method (mono/mono#14603)

Variable is only used here: https://github.com/mono/mono/blob/mono/mono@82a993124888d470d1057eae9546e5be76524dad/mono/mini/mini-trampolines.c#L743

Commit migrated from https://github.com/mono/mono/commit/76c45bc2cbbcbc3a6beafaeb477c7c42c1069af9

[llvm] Remove JIT support for older llvm versions. (mono/mono#14605)

Commit migrated from https://github.com/mono/mono/commit/3ff4282abebe9b0d4674cbef8b38e69960d20d5b

[merp] Don't leak g_getenv return value (mono/mono#14602)

[merp] Don't leak g_getenv return value

Coverity said:
```
     CID 1445100:  Resource leaks  (RESOURCE_LEAK)
     Failing to save or free storage allocated by "monoeg_g_getenv("MONO_CRASH_NOFILE")" leaks it.
```

Commit migrated from https://github.com/mono/mono/commit/94adfce7351093e0e726fea857c52f69062abb12

[interp] Fix warning with coverity (mono/mono#14613)

[interp] Fix warning with coverity

Commit migrated from https://github.com/mono/mono/commit/3d331568e164eaa10a7c139421939f421fb0cda6

[merp] Add assertions against initing dumper from signal handler (mono/mono#13851)

Commit migrated from https://github.com/mono/mono/commit/fb2233b337242271e3660fedb3678fc1a11a99c2

[loaded LLVM] Add several missing MONO_LLVM_INTERNAL (mono/mono#14589)

* [llvm] Remove unused default_mono_llvm_unhandled_exception.
It references mono_gchandle_get_target_internal which is not marked MONO_LLVM_INTERNAL
so attempts to run it will fail.

This is caught by linking wit -z now (https://github.com/mono/mono/pull/14562).

* [loaded LLVM] mono_debug_get_seq_points and mono_class_enum_basetype_internak MONO_LLVM_INTERNAL

* Mark mono_debug_method_lookup_location MONO_LLVM_INTERNAL.
Found by linking with -z now (https://github.com/mono/mono/pull/14562)
which we should probably do for this and other reasons.

* #include mono-compiler.h for MONO_LLVM_INTERNAL.

* [loaded LLVM] Mark mini_is_gsharedvt_variable_signature as MONO_LLVM_INTERNAL.
Found by https://github.com/mono/mono/pull/14562.

* Mark mono_print_ins MONO_LLVM_INTERNAL.

* mono_rgctx_info_type_to_str MONO_LLVM_INTERNAL

* [loaded LLVM] mono_compile_create_var MONO_LLVM_INTERNAL

* ifdef around use of MONO_LLVM_INTERNAL and hope

* remove include mono-compiler.h from public header to internal header

* MONO_API_NO_EXTERN_C mono_debug_method_lookup_location() instead of mono_debug_method_lookup_location() MONO_LLVM_INTERNAL to workaround possible lack of MONO_LLVM_INTERNAL.

Commit migrated from https://github.com/mono/mono/commit/0c7ed830ac8d7c4ae404020d43a167c7146d136c

Make function pointers readonly. (mono/mono#14470)

Commit migrated from https://github.com/mono/mono/commit/0d22880940f102f1592b644b137f56b553a775e9

[interp] set correct stack type on op_Implicit/op_Explicit intrinsics for native types (mono/mono#14575)

[interp] set correct stack type on op_Implicit/op_Explicit intrinsics for native types

Needs some additional conversions which happen to be included by accident previously.

This fixes those warnings on amd64:
```
Test run:
image=/Users/lewurm/work/mono-dim-call/mono/mini/builtin-types.exe
BuiltinTests.test_0_nuint_fieldload: Store local stack type mismatch 0 1
BuiltinTests.test_0_much_casting: Store local stack type mismatch 0 1
Results: total tests: 84, all pass
Elapsed time: 0.026546 secs (0.026546, 0.000000)
```

and those warnings on armv7:
```
Test run: image=/mono/mono/mini/builtin-types.exe
BuiltinTests.test_0_nint_inc: 0011 arith type mismatch add.i4 0 1
BuiltinTests.test_0_nint_dec: 0011 arith type mismatch sub.i4 0 1
BuiltinTests.test_0_nint_implicit_decimal: 0034 arith type mismatch
add.i4 0 1
BuiltinTests.test_0_nuint_inc: 0011 arith type mismatch add.i4 0 1
BuiltinTests.test_0_nuint_dec: 0011 arith type mismatch sub.i4 0 1
BuiltinTests.test_0_nuint_implicit_decimal: 0034 arith type mismatch
add.i4 0 1
Results: total tests: 84, all pass
Elapsed time: 0.039214 secs (0.039214, 0.000000)
```

Commit migrated from https://github.com/mono/mono/commit/b564c26cb52ca195970d088af46041d5a6db7615

[Android] Fix runtime loading of DSOs for 64-bit processes (mono/mono#14577)

[Android] Fix runtime loading of DSOs for 64-bit processes

Context: https://github.com/xamarin/xamarin-android/issues/2780

On Android the executable for the application is going to be
`/system/bin/app_process{32,64}` depending on the application's architecture.
However, libraries for the different architectures live in different
subdirectories of `/system`: `lib` for 32-bit apps and `lib64` for 64-bit ones.
Thus appending `/lib` below will fail to load the DSO for a 64-bit app, even if
it exists there, because it will have a different architecture. The quickest fix
is to use `lib64` explicitly for 64-bit Android apps.

<!--
Thank you for your Pull Request!

If you are new to contributing to Mono, please try to do your best at conforming to our coding guidelines http://www.mono-project.com/community/contributing/coding-guidelines/ but don't worry if you get something wrong. One of the project members will help you to get things landed.

Does your pull request fix any of the existing issues? Please use the following format: Fixes #issue-number
-->

Backport of mono/mono#13281.

/cc @lambdageek @grendello

Commit migrated from https://github.com/mono/mono/commit/72eb2ee84a4f397c5887159e288894778ea316a9

[netcore] add some intrinsics for interpreter (mono/mono#14547)

[netcore] add some intrinsics for interpreter

```console
$ MONO_ENV_OPTIONS='--interp' make -C netcore xtest-System.Runtime.Tests
Microsoft.DotNet.XUnitConsoleRunner v2.5.0 (64-bit .NET Core 3.0.0-preview5-27620-01)
  Discovering: System.Runtime.Tests (method display = ClassAndMethod, method display options = None)
  Discovered:  System.Runtime.Tests (found 5266 of 5402 test cases)
  Starting:    System.Runtime.Tests (parallel test collections = on, max threads = 16)
[...]
  Finished:    System.Runtime.Tests
=== TEST EXECUTION SUMMARY ===
   System.Runtime.Tests  Total: 32485, Errors: 0, Failed: 23, Skipped: 2, Time: 126.432s
```
See failures here: https://gist.github.com/lewurm/c8372b0c7ba6a78cf8b7b7b8c7090820

Commit migrated from https://github.com/mono/mono/commit/fa3b4812d7fd61deee1b36abdb313af2e036b65c

Merge pull request mono/mono#14549 from lateralusX/lateralusX/add-coop-config-windows

Add ability to use cooperative suspend in configure step.

Commit migrated from https://github.com/mono/mono/commit/d96fa3ab317ec0ebf5fa26d0efab24c3142b8e29

Fix decoding of WRAPPER_SUBTYPE_ICALL_WRAPPER and WRAPPER_SUBTYPE_INTERP_LMF. (mono/mono#14546)

Commit migrated from https://github.com/mono/mono/commit/8804dbac2752969105cf6e0a9939e0fd79531dec

Fix warning: (mono/mono#14529)

aot-compiler.c: In function 'arch_init':
aot-compiler.c:1092:11: warning: unused variable 'has_custom_args' [-Wunused-variable]
  gboolean has_custom_args = !!acfg->aot_opts.llvm_llc;

Commit migrated from https://github.com/mono/mono/commit/37d6df9d6748fbb02e1d29bc4c6b3f82000b8115

Plug leak of CustomAttribute initializer name (mono/mono#14544)

* Plug leak of CustomAttribute initializer name

All the CustomAttribute initializer names (fields, property) except the last were leaked. Ensure all names are freed.

Fixes mono/mono#14543

* Apply suggestions from code review

Co-Authored-By: Martin Baulig <mabaul@microsoft.com>
Commit migrated from https://github.com/mono/mono/commit/fe74f621408d9fc822ec3b334f1bdb0a2b488cf2

Merge pull request mono/mono#14552 from lewurm/interp-dimcall-fix

[interp] add missing enum check on constrained callvirt

Commit migrated from https://github.com/mono/mono/commit/84bc4bfb8bae865ef7fba42b174c7c490a524aaf

Close two race conditions. (mono/mono#14510)

Commit migrated from https://github.com/mono/mono/commit/ee1868e82fca99b7aa487f16343803bd443a3e30

[driver] add --version=number (mono/mono#14538)

* [driver] add --version-number

Useful for third parties consuming the version number of mono.

* s/--version-number/--version=number/g

* add man page entry

Commit migrated from https://github.com/mono/mono/commit/574d87ca9ff47c99456174884b4625a406e7590c

More information to PowerPC assertion failure. (mono/mono#14569)

Commit migrated from https://github.com/mono/mono/commit/5334e32de968993725896162e22c1760ad31418f

Add more information to a stackless faulting community line. (mono/mono#14566)

Commit migrated from https://github.com/mono/mono/commit/d583190ce76c3f0fae06f28c71dfe93a023b5cea

[PowerPC] Fix compile error. (mono/mono#14564)

Commit migrated from https://github.com/mono/mono/commit/74faf3997129465aad7b188903d95f7b33f2822c

[netcore] Add Microsoft.Private.CoreFx.OOB package to corefx-restore.csproj  (mono/mono#14559)

* Add Microsoft.Private.CoreFx.OOB package

* Add GC.GetTotalAllocatedBytes stub

* Add missing icalls

* Add more tests to ignore

* Update GC.cs

* Add more tests to ignore

* Update Monitor.cs

Commit migrated from https://github.com/mono/mono/commit/64fe6a054ca3702677bd001056388341c87bbd91

[netcore] Set System.Private.Corlib AssemblyVersion to 4.0.0.0 (mono/mono#14556)

* clean up

* cleanup

* bump assembly version

Commit migrated from https://github.com/mono/mono/commit/36193f53781399a299a7d7a4032f32dc532ea37e

[netcore] Fix PointerTests (mono/mono#14557)

* Fix PointerTests

* Update CoreFX.issues.rsp

Commit migrated from https://github.com/mono/mono/commit/7f0382432701620dce0b4796fd09011180f0b56b

[jit] Fix signed/unsigned comparisons in JitHelpers.EnumCompareTo () intrinsic. (mono/mono#14553)

Fixes https://github.com/mono/mono/issues/14551.

Commit migrated from https://github.com/mono/mono/commit/e9fbb4210c9b1dadc171201ace0edc4fb01c7cce

MONO_PATCH_INFO_JIT_ICALL cleanup (mono/mono#14545)

The most important point of the "temporary" aspect, esp. given
the complete conversion, is renaming MONO_PATCH_INFO_JIT_ICALL to MONO_PATCH_INFO_JIT_ICALL_ID.
Any accidental or merged MONO_PATCH_INFO_JIT_ICALL will fail to compile.

Commit migrated from https://github.com/mono/mono/commit/f0cf94a5d198ec1404ab260078b645f7dfd1198e

[crash] Don't waitpid() if fork() fails (mono/mono#14539)

Commit migrated from https://github.com/mono/mono/commit/eaaf77189d01fea26943f68cadc31130c3876530

[crash] Handle OOM/blocked allocation during crash dumper (mono/mono#14540)

* [crash] Remove typo from crash sub-header

* [crash] Use error path that returns nonzero status

Commit migrated from https://github.com/mono/mono/commit/8c4c9a06cd641bf57dae1692c81ba79907a3a862

[interp] add missing enum check on constrained callvirt

This detail got lost when the code was duplicated here: https://github.com/mono/mono/pull/13154/files#diff-f115ce23fba6528256a47e2b7e8b0a19R1611

Fixes `mono/tests/dim-constrainedcall.il` on the interpreter. This PR will enable it: https://github.com/mono/mono/pull/14166

Commit migrated from https://github.com/mono/mono/commit/539e3f8546b77fec2b5888d66b1e05bfa6bcbde8

Initial icall-def.h / icall-def-netcore.h split. (mono/mono#14489)

* Copy icall-def.h to icall-def-core.h.

* Manually preprocess icall-def.h vs. icall-def-core.h.
The only common part in this construction is the JIT icalls at the end.
There much overlap, now duplicated.

* Remove JIT icalls fro icall-def-core.h.

* Merge icall-def-core.h with master.

* Merge icall-def.h with master.

* Rename icall-def-core.h to icall-def-netcore.h.

* Remove more unused icalls

Commit migrated from https://github.com/mono/mono/commit/c1d01670d343b5105e52678e71d5f8e310a01b8e

Add ability to use cooperative suspend in configure step.

Needed in order to setup CI lanes using --enable-cooperative-suspend.

Commit migrated from https://github.com/mono/mono/commit/70b26c9bc3f53fc503ba3d009a00c16f645d991f

Removing some remoting and multi-appdomain APIs on wasm and the AOT test profiles.

Build / profile changes:
------------------------

* `build/profiles/testing_aot_common.make`: define `DISABLE_REMOTING = yes` and
  `NO_MULTIPLE_APPDOMAINS = yes`.

* `mcs/class/corlib/Makefile`: check `DISABLE_REMOTING` when setting
  `REFERENCE_SOURCES_FLAGS` (and don't define `FEATURE_REMOTING`, `MONO_COM`
   and `FEATURE_COMINTEROP`).

* added new `testing_aot_common_corlib.dll.exclude.sources`.  Several types in
  the `System.Runtime.Remoting.Messaging` namespace are used by the runtime in
  non-remoting scenarios, so those need to be preserved.
  For all other remoting namespaces, we only need a few stripped down stub versions
  of the some of the types; those have been added to a new `legacy` directory, so
  we can wildcard-exclude all the files here.

* added new `testing_aot_common_corlib.dll.sources`; this currently only contains
  `legacy/*.cs` to list above mentioned stub versions.

* added new `testing_aot_common_corlib_test.dll.exclude.sources` as a common test
  excludes file.

API Changes (conditional to `DISABLE_REMOTING`):
------------------------------------------------

* `mcs/class/corlib/legacy/`: add stub-versions for some of the remoting-related
  classes that are used by the runtime (so we can't completely remove those types):

  - `System.MarshalByRefObject`.
  - `System.Runtime.Remoting.IRemotingTypeInfo`.
  - `System.Runtime.Remoting.ObjectHandle`.
  - `System.Runtime.Remoting.RemotingServices`.
  - `System.Runtime.Remoting.Activation.ActivationServices`.
  - `System.Runtime.Remoting.Contexts.Context`.
  - `System.Runtime.Remoting.Messaging.ILogicalThreadAffinative`.
  - `System.Runtime.Remoting.Messaging.LogicalCallContext`.
  - `System.Runtime.Remoting.Proxies.TransparentProxy`.
  - `System.Runtime.Remoting.Proxies.RealProxy`.

  some of these have fields that need to be preserved; these are marked with
  appropriate `#region` pragmas.

All changes below are conditional to `DISABLE_REMOTING`.

* `System.Runtime.Remoting.Messaging.AsyncResult`: this class is used by some of
  the async code, so we can't completely remove it.  However, we can remove the
  `MonoMethodMessage call_message` field when `DISABLE_REMOTING`.

* `System.Runtime.Remoting.Messaging.MonoMethodMessage`: this class is also used
  by some runtime code and thus needs to be preserved. However, when `DISABLE_REMOTING`
  is defined, then we make some changes to it:
  - remove all iterfaces (`IMethodCallMessage`, `IMethodReturnMessage`, and `IInternalMessage`).
  - `Properties` throws `PlatformNotSupportedException`.

* `System.Runtime.InteropServices.Marshal`: disable some COM-related code.

* `System.Threading.Thread`: remove `CurrentContext` property.

* `System.Threading.ExecutionContext`: adjust conditional logic; don't define
  `FEATURE_REMOTING` if `DISABLE_REMOTING`.

* `System.Activator`: add `DISABLE_REMOTING` to the existing
   `#if FEATURE_REMOTING || MOBILE_LEGACY` conditional in this class.

* `System.AppDomain`: since we only have one domain, `DefaultDomain` will always
   return the root domain; comment out the call to `RemotingServices.GetDomainProxy`
   as well as some internal methods.

* `System.__ComObject`: stub out on `DISABLE_REMOTING` as well as `!FULL_AOT_RUNTIME`.

* `System.Runtime.Serialization.Formatters.Binary.BinaryFormatter`: adjust `IFormatter`
  versus `IRemotingFormatter` interface logic to use the former when `DISABLE_REMOTING`.

* `System.Runtime.Serialization.Formatters.Binary.BinaryObjectReader`: adjust existing
  `#if FEATURE_REMOTING || MOBILE_LEGACY` conditional to add `DISABLE_REMOTING` to it.

* `System.Diagnostics.CorrelationManager`: comment out some `CallContext.LogicalGetData()`
  and `CallContext.LogicalSetData`.

* `System.Diagnostics.TraceEventCache`: likewise.

Test changes:
-------------

* `mono/tests/Makefile.am`: conditionally disable some remoting tests.

* `corlib/Test/System.Runtime.Serialization/SerializationTest.cs`: conditionally
   disable all tests requiring remoting.

* `corlib/Test/System/ActivatorTest.cs`: same; trying to preserve as much as possible
  in this test.

* added some `DISABLE_REMOTING` and `DISABLE_SECURITY` conditionals to some of the tests.

Commit migrated from https://github.com/mono/mono/commit/1eb9161ca6b096e5f13f1a604e38557dde6e5c74

Replace string/hash-based MONO_PATCH_INFO_JIT_ICALL with enum-based MONO_PATCH_INFO_JIT_ICALL_ID. (mono/mono#14512)

* Replace string/hash-based MONO_PATCH_INFO_JIT_ICALL with enum-based MONO_PATCH_INFO_JIT_ICALL_ID.

* Reconvert mono-llvm.c after merge with master.

Commit migrated from https://github.com/mono/mono/commit/c03814f8dcb2b8b5f028ad86bbd9d3889144ffc7

Fix leak in mono_parse_options (mono/mono#14533)

Fix leak in mono_parse_options

When MONO_ENV_OPTIONS is null, we would leak the array and buffer

<!--
Thank you for your Pull Request!

If you are new to contributing to Mono, please try to do your best at conforming to our coding guidelines http://www.mono-project.com/community/contributing/coding-guidelines/ but don't worry if you get something wrong. One of the project members will help you to get things landed.

Does your pull request fix any of the existing issues? Please use the following format: Fixes #issue-number
-->

Commit migrated from https://github.com/mono/mono/commit/57a3b24f750b4d4509b5a2a7c8fcc25f72d732d8

[llvm] Fix patching of calls from LLVM JITted code. (mono/mono#14520)

* [runtime] Fix mono_icall_get_wrapper_full () to always compile the wrapper even if a trampoline was created previously.

* [jit] Add a mini_patch_llvm_jit_callees () function to patch callsite variables for LLVM JIT.

* [llvm] Patch LLVM JIT callees in trampolines as well. Patching after a method is compiled is not enough, since new callsites can be created after a method has been compiled.

* [llvm] Cleanup calls from JITted code. Compile referenced jit icall wrappers immediately, since the calls will not be patched.

* Add a missing MONO_LLVM_INTERNAL.

Commit migrated from https://github.com/mono/mono/commit/ccb505b6ecc478cfc4ca9d30af1164a41685c298

[arm] one more attempt to fix slotsize issue on llvmonly (mono/mono#14444)

* [arm] one more attempt to fix slotsize issue on llvmonly

Regression of https://github.com/mono/mono/pull/12992 & https://github.com/mono/mono/pull/14362

In the end, the actual fix boils down to:
```patch
--- a/mono/mini/mini-arm.c
+++ b/mono/mini/mini-arm.c
@@ -2374,7 +2374,7 @@ mono_arch_get_llvm_call_info (MonoCompile *cfg, MonoMethodSignature *sig)
                                lainfo->nslots = ALIGN_TO (ainfo->struct_size, 8) / 8;
                                lainfo->esize = 8;
                        } else {
-                               lainfo->nslots = ainfo->struct_size / sizeof (target_mgreg_t);
+                               lainfo->nslots = ALIGN_TO (ainfo->struct_size, sizeof (target_mgreg_t)) / sizeof (target_mgreg_t);
                                lainfo->esize = 4;
                        }
                        break;
```

Tested on `xamarin-macios/arm64_32-v3` branch:
* mscorlib
* mini
* dont link
* monotouch tests
* and a modified version of https://github.com/mono/mono/issues/8486#issuecomment-414365860 (added larger structs too)

with each device, Watch Series 3 (`armv7k`) and Watch Series 4 (`arm64_32`).

I hope that is the last iteration on this.

* remove arm64_32_abi

Commit migrated from https://github.com/mono/mono/commit/c2118341c634fd845d3ab15304bdf3fa84191841

Work toward JIT icall hash removal. (mono/mono#14508)

i.e. when CEE_MONO_JIT_ICALL_ADDR takes enum instead of func.

Commit migrated from https://github.com/mono/mono/commit/1ecffe9554c23d6def0b39dd9382ae71cd981ac0

[netcore] Merge all *.rsp files into CoreFX.issues.rsp (mono/mono#14513)

* merge rsp files into CoreFX.issues.rsp

* Update CoreFX.issues.rsp

* Update CoreFX.issues.rsp

* clean up

* Update CoreFX.issues.rsp

* download OS specific tests

* Update Makefile

* Add comments for build-local-corefx-test-% rule

Commit migrated from https://github.com/mono/mono/commit/616e06225a462e6749901f70a76ad4a3a845c009

[netcore] Fix Math.ILogB corner cases (mono/mono#14500)

* Fix ILogb corner cases

* remove redundant tabs

* Add RSP for S.R.Extensions

* remove autotools magic

* add more comments

Commit migrated from https://github.com/mono/mono/commit/bd7ec7236285f59120264f78b1171a1038a5151a

[netcore] Respect attributes defined on parameters in Delegates (mono/mono#14468)

* Respect attributes defined on Delegates

* remove 'var's

* Unwrap System.Reflection.Tests.PointerTests.*

Commit migrated from https://github.com/mono/mono/commit/29938ada1f2b28b4b79070d889e44dbfd3056d47

Make mono_jit_icall_info MONO_LLVM_INTERNAL. (mono/mono#14525)

Commit migrated from https://github.com/mono/mono/commit/7601045796a805bac4265577c15c0a90c1c7003e

Add missing dependency on $(srcdir)/genmdesc.py. (mono/mono#14515)

Commit migrated from https://github.com/mono/mono/commit/24c0f17b1ca2560886737651bbb5a3b202ca7ee5

Merge pull request mono/mono#14490 from vargaz/bump-emsdk

[wasm] Bump emscripten to 1.38.31.

Commit migrated from https://github.com/mono/mono/commit/73baf1b2912aff20ea09a29b693f6e4f9640cab6

[interp] disable inlining for built-in types (mono/mono#14501)

[interp] disable inlining for built-in types

Fixes https://github.com/mono/mono/issues/14246

Commit migrated from https://github.com/mono/mono/commit/45a1c98df895195dd86d7912aed857984484aef8

Reduce JIT icall hash use. (mono/mono#14507)

* Reduce JIT icall hash use.

* PR: Rename mono_jit_icall_info_at to mono_find_jit_icall_info.

* Use enum instead of int, and update comment to reality (the assert is arguably leniently off by one.)

* Cast integer to enum for C++.

* More int vs. enum, and use two variables, and const.

* Undo a little fishing for CI failure.

Commit migrated from https://github.com/mono/mono/commit/3142f7d54dcdffd46e56e27a9976279092dcadc9

Merge pull request mono/mono#14488 from jeromelaban/dev/jela/wasm-dlopen

[Wasm] Expand support for dynamic linking

Commit migrated from https://github.com/mono/mono/commit/63b16ae32f376fbd39997a893ba8028d9471f987

Merge pull request mono/mono#14498 from lateralusX/lateralusX/fix-llvm-build-error

Fix Windows Mono LLVM build error on nightly master build.

Commit migrated from https://github.com/mono/mono/commit/42bc7daeb86535478d0c5957440842fd5843ebc8

[wasm] Avoid sharing C icall functions between icalls, in wasm to avoid calling them with incorrect signatures. (mono/mono#14493)

Commit migrated from https://github.com/mono/mono/commit/ab90b1f15bf2e3d9ec741a241535dd6920468e7e

[netcore] Build/upload Windows runtime nupkg (mono/mono#14506)

* [netcore] Switch build properties from /p: to -p: due to MSYS2/dotnet bug

* [netcore] Switch from nuget pack to dotnet pack

* [netcore] Add Windows x64 build

Commit migrated from https://github.com/mono/mono/commit/2b35b6bab451c77fcfe7c2cec725ba135333af31

Work towards removing JIT icall hashing. (mono/mono#14474)

* Work towards removing JIT icall hashing.

* PR: Use enums instead of pointers.

* Fix int/enum mismatch.

Commit migrated from https://github.com/mono/mono/commit/da6cb6c685efd62e0b8e36dc932d72eff3425ddd

Fix Windows Mono LLVM build error on nightly master build.

Commit migrated from https://github.com/mono/mono/commit/142d1a9ca02d407b2fe385c78b08cbda897b694e

[netcore] Exclude some unused icalls (mono/mono#14475)

* [netcore] Exclude unused native locale icalls

* Fix builds with mono_feature_disable_sockets, currently not enabled by configure script

* [netcore] Don't compile in System.IO icall in netcore profile

Commit migrated from https://github.com/mono/mono/commit/ad698c987888cb356d308e14c1e08860add4dd5d

Make genmdesc output better typed and more readable. (mono/mono#14477)

* Make genmdesc output better typed and more readable -- almost hand-maintainable
but no clear route to actually hand-maintainable.

* Replace != with < and undo debug code.

Commit migrated from https://github.com/mono/mono/commit/ebd0cdad5ed56cc62f77b10cf0b3d5c00f9b9403

Add two asserts to mono_emit_jit_icall. (mono/mono#14459)

Commit migrated from https://github.com/mono/mono/commit/495a93305bdd7eeae00735d6e69990adfdb859fb

[merp] Add an env var to disable all mono_crash files (mono/mono#14407)

Commit migrated from https://github.com/mono/mono/commit/4b111f373ebf5adf4941b0f331d3dda20e86bbae

configure: check for c++ availability (mono/mono#14461)

Since commit d8af775, c++ compiler is required to compile.
This patch checks if the c++ compiler is available as soon as possible
and returns an error in case it is not found.

Fixes: https://github.com/mono/mono/issues/14195

Signed-off-by: Angelo Compagnucci <angelo@amarulasolutions.com>
Commit migrated from https://github.com/mono/mono/commit/f29eae3e354b6d94b69d60fa6b7b59c229bc893e

Convert rest of JIT icall registration to static info. (mono/mono#14455)

* Convert rest of JIT icall registration to static storage.

* PR: Expand macros manually (they can actually emcompass more, not less, another time).

Commit migrated from https://github.com/mono/mono/commit/a1c476c04da5558a00efb5ca748d8c769d5936e3

hide props

Commit migrated from https://github.com/mono/mono/commit/1049f06668e5190def796b47404ed03f163c0a86

[arm64] set MONO_ARCH_HAVE_UNWIND_BACKTRACE for watch4 (mono/mono#14440)

[arm64] set MONO_ARCH_HAVE_UNWIND_BACKTRACE for watch4

Fixes a couple of mscorlib test failures, failing with
> Stack walks are not supported on this platform

Commit migrated from https://github.com/mono/mono/commit/1d90dcfd357c2bc070affa56865ad8cfff2e4b2e

Remove some malloc from JIT icall registration. (mono/mono#14431)

Disallow duplicate JIT icall registration (by name, address, or new static info storage).

Commit migrated from https://github.com/mono/mono/commit/04252ff72451b0dd21ba4cdd3056ab0bd84151d0

Merge pull request mono/mono#14425 from lateralusX/lateralusX/fix-profiler-on-windows

Add log profiler support to Mono MSVC build.

Commit migrated from https://github.com/mono/mono/commit/991c7b6a61101b0d7eb955b4287d116ec99ba2b8

Merge pull request mono/mono#14332 from lateralusX/lateralusX/add-llvm-windows-msvc-build-packaging-support

Add LLVM Windows MSVC build/packaging/archive/download support.

Commit migrated from https://github.com/mono/mono/commit/5f41fb86be4cc1bcc3b65383d47d9d5ea605fcfe

[netcore] Enable MSVC builds of Mono w/ ENABLE_NETCORE (mono/mono#14418)

* Enable MSVC builds of Mono w/ ENABLE_NETCORE

* Fix non-netcore build

* Move the ENABLE_NETCORE definition to config.h where it belongs

* Replace .def with linker #pragmas

* TARGET_WIN32 -> HOST_WIN32 && HOST_X86

* Pick up MONO_ENABLE_NETCORE from cygconfig.h

* Make `./configure --with-core=only` MSVC builds working

* Include netcore sources unconditionally, move the condition into the .c file

* Fix MSVC C++ netcore build on Win32

Commit migrated from https://github.com/mono/mono/commit/643b2ebfc6ad8a7cb2c21d3cac1a89ca3ceb3c9f

Fix minor build warning (mono/mono#14426)

Fix minor build warning

Commit migrated from https://github.com/mono/mono/commit/0c70519707010ff63bfbc5f5eca6fc7e3c181d4f

[corlib] Change the RuntimeModule.GetGuid and RuntimeAssembly.GetAotId icalls to return a byte array. (mono/mono#14393)

* [corlib] Change the RuntimeModule.GetGuid and RuntimeAssembly.GetAotId icalls to return a byte array instead of a string, so it doesn't have to be parsed by the Guid ctor. Also allocate the arrays in the c# code and pass them to the icalls.

Commit migrated from https://github.com/mono/mono/commit/29fb43eb0905ec7a9968f3ea08325a2c61fba82c

Do not allocate handles in ReleaseMutex. (mono/mono#14403)

* Do not allocate handles in ReleaseMutex.
Take advantage of thread/internal_thread maturity/pinning.

There are arguments for and against this.

It is arguably better to use handles everywhere.
But it is dubious to require an allocation to release a lock.
Releasing a lock should be infallible.
And surviving low memory shouldn't be a lost cause, imho..

This diff could also be viewed as an opportunistic cycle/size optimization
and be placed under ifdef -- ifdef THREAD_ALWAYS_PINNED orsuch.

This is an alternative to https://github.com/mono/mono/pull/14397.

* PR: Move comments.

Commit migrated from https://github.com/mono/mono/commit/77ab9ee0b884854e877f50f131620ecd7191e4a2

Add log profiler support to Mono MSVC build.

Several fixes to the log profiler in order to work on Windows.

* Add new dynamic library project to build log profiler on Windows MSVC.
* Fix dependecy between log profiler and Mono.
* Rewrite command pipe logic on Windows since it's not supported to select
  on both sockets and pipes.
* Several additional socket fixes needed for log profiler to work on Windows.

Commit migrated from https://github.com/mono/mono/commit/2a703e72ff9001c9c872f1629ce0842e061a1c8a

[netcore] Fix HideDetectionHappensBeforeBindingFlagChecks (mono/mono#14336)

* Hide public property in Base if there is a private in Derived

* Update excludes-System.Runtime.Tests.rsp

* Update RuntimePropertyInfo.cs

Commit migrated from https://github.com/mono/mono/commit/735489b10bc6f839ddde7d8debea92c70d506b67

[metadata] use handle stack in mono_runtime_object_init_handle (mono/mono#14399)

Fixes this crash on watchOS with llvmonly:
```
(lldb) bt 35
* thread %1, name = 'tid_303', queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x4d55545c)
  * frame %0: 0x01ab81de mscorlibtests`sgen_vtable_get_descriptor(vtable=0x4d555458) at sgen-client-mono.h:29:33
    frame %1: 0x01a9e278 mscorlibtests`major_copy_or_mark_object_canonical at sgen-marksweep-drain-gray-stack.h:158:10
    frame %2: 0x01a9debc mscorlibtests`major_copy_or_mark_object_canonical(ptr=0x182f260c, queue=0x01dd4894) at sgen-marksweep.c:1363
    frame %3: 0x019d31f2 mscorlibtests`mono_handle_stack_scan(stack=0x17e200b0, func=(mscorlibtests`major_copy_or_mark_object_canonical + 1 at sgen-marksweep.c:1362), gc_data=0x01dd4894, precise=1, check=1) at handle.c:340:5
    frame %4: 0x01a45680 mscorlibtests`sgen_client_scan_thread_data(start_nursery=0x00000000, end_nursery=0xffffffff, precise=1, ctx=ScanCopyContext @ 0x01dd4580) at sgen-mono.c:2254:5
    frame %5: 0x01a93e80 mscorlibtests`job_scan_thread_data(worker_data_untyped=0x00000000, job=0x0511c004) at sgen-gc.c:1416:2
    frame %6: 0x01adef0a mscorlibtests`sgen_workers_enqueue_job(generation=1, job=0x0511c004, enqueue=0) at sgen-workers.c:184:3
    frame %7: 0x01a9334a mscorlibtests`enqueue_scan_from_roots_jobs(gc_thread_gray_queue=0x01dd4894, heap_start=0x00000000, heap_end="", ops=0x01c6aaac, enqueue=0) at sgen-gc.c:1661:2
    frame %8: 0x01a94d16 mscorlibtests`major_copy_or_mark_from_roots(gc_thread_gray_queue=0x01dd4894, old_next_pin_slot=0x01dd4864, mode=COPY_OR_MARK_FROM_ROOTS_SERIAL, object_ops_nopar=0x01c6aaac, object_ops_par=0x00000000) at sgen-gc.c:2070:2
    frame %9: 0x01a95658 mscorlibtests`major_start_collection(gc_thread_gray_queue=0x01dd4894, reason="LOS overflow", concurrent=0, old_next_pin_slot=0x01dd4864) at sgen-gc.c:2189:2
    frame %10: 0x01a92ccc mscorlibtests`major_do_collection(reason="LOS overflow", is_overflow=0, forced=1) at sgen-gc.c:2362:2
    frame %11: 0x01a8e9ce mscorlibtests`sgen_perform_collection_inner(requested_size=21616, generation_to_collect=1, reason="LOS overflow", forced_serial=1, stw=1) at sgen-gc.c:2563:14
    frame %12: 0x01a8e76c mscorlibtests`sgen_perform_collection(requested_size=21616, generation_to_collect=1, reason="LOS overflow", forced_serial=1, stw=1) at sgen-gc.c:2640:2
    frame %13: 0x01a8e73e mscorlibtests`sgen_ensure_free_space(size=21616, generation=1) at sgen-gc.c:2514:2
    frame %14: 0x01a99184 mscorlibtests`sgen_los_alloc_large_inner(vtable=0x185749d0, size=21616) at sgen-los.c:380:2
    frame %15: 0x01a81c0a mscorlibtests`sgen_alloc_obj_nolock(vtable=0x185749d0, size=21616) at sgen-alloc.c:172:16
    frame %16: 0x01a44940 mscorlibtests`mono_gc_alloc_vector(vtable=0x185749d0, size=21616, max_length=900) at sgen-mono.c:1317:20
    frame %17: 0x00a5b408 mscorlibtests`aot_wrapper_icall_mono_gc_alloc_vector + 102
    frame %18: 0x00a56818 mscorlibtests`mscorlib_wrapper_alloc_object_AllocVector_intptr_intptr + 324
    frame %19: 0x01092b26 mscorlibtests`mscorlibtests1_MonoTests_System_DecimalTest2__ctor + 7286
    frame %20: 0x00a78912 mscorlibtests`aot_wrapper_gsharedvt_out_sig_pinvoke_void_this_ + 28
    frame %21: 0x00a4a4d2 mscorlibtests`mscorlib_wrapper_runtime_invoke_object_runtime_invoke_sig_void_intptr_intptr_object_intptr_intptr_intptr + 312
```

The hint here was that the crash happened in `mono_handle_stack_scan ()` and inspecting the handle stack there. After enabling `MONO_HANDLE_TRACK_OWNER` and `MONO_HANDLE_TRACK_SP` in `handle.h`, I got this:
```
warning: Handle 0x17886a0c (object = 0x16e73fd0) (allocated from "../../../../../mono/metadata/object.c:3182") is leaking.

    frame %1: 0x019833ba mscorlibtests`mono_handle_new(obj=0x01f194a0, info=0x17882600, owner="../../../../../mono/metadata/gc.c:1337") at handle.c:183:2
    frame %2: 0x019823c0 mscorlibtests`mono_gc_alloc_handle_string(vtable=0x17888760, size=24, len=5) at gc.c:1337:9
    frame %3: 0x019de896 mscorlibtests`mono_string_new_size_handle(domain=0x16e73fd0, len=5, error=0x01dc804c) at object.c:6643:6
    frame %4: 0x019d6fc4 mscorlibtests`mono_string_new_size_checked(domain=0x16e73fd0, length=5, error=0x01dc804c) at object.c:6655:2
    frame %5: 0x019de64e mscorlibtests`mono_string_new_utf16_checked(domain=0x16e73fd0, text=0x1842ab10, len=5, error=0x01dc804c) at object.c:6536:6
    frame %6: 0x019debc2 mscorlibtests`mono_string_new_checked(domain=0x16e73fd0, text="ar-SA", error=0x01dc804c) at object.c:6778:7
    frame %7: 0x019b06a8 mscorlibtests`construct_culture(this_obj=0x01f19428, ci=0x01b2d878, error=0x01dc804c) at locales.c:357:2
    frame %8: 0x019b0e3a mscorlibtests`ves_icall_System_Globalization_CultureInfo_internal_get_cultures(neutral='\0', specific='\x01', installed='\0') at locales.c:714:9
    frame %9: 0x007061f0 mscorlibtests`aot_wrapper___System_dot_Globalization_System_dot_Globalization_dot_CultureInfo__internal_get_cultures_pinvoke_cl2a_System_2eGlobalization_2eCultureInfo_5b_5d__cl4_bool_cl4_bool_cl4_bool_cl2a_System_2eGlobalization_2eCultureInfo_5b_5d__cl4_bool_cl4_bool_cl4_bool_ + 100
    frame %10: 0x00704fb2 mscorlibtests`mscorlib_System_Globalization_CultureInfo_GetCultures_System_Globalization_CultureTypes + 192
    frame %11: 0x0100f422 mscorlibtests`mscorlibtests1_MonoTests_System_DateTimeTest_Parse_Bug53023b + 192
    frame %12: 0x00a28702 mscorlibtests`aot_wrapper_gsharedvt_out_sig_pinvoke_void_this_ + 28
    frame %13: 0x009fa2c2 mscorlibtests`mscorlib_wrapper_runtime_invoke_object_runtime_invoke_sig_void_intptr_intptr_object_intptr_intptr_intptr + 312
    frame %14: 0x019278f2 mscorlibtests`mono_llvmonly_runtime_invoke(method=0x17953820, info=0x1842a9a0, obj=0x053b7c40, params=0x00000000, exc=0x01dc8550, error=0x01dc88a4) at mini-runtime.c:3000:2
```

@lambdageek suggested this fix after showing this trace to him. As far as I understand we do _not_ know the root cause yet. The handle leak happens in this trace:
```
(lldb) mbt
* thread %1
  * frame %0: 0x0176192c mscorlibtests`do_debug_me_please_kkthx at object.c:128:2
    frame %1: 0x01761a62 mscorlibtests`mono_runtime_object_init_handle(this_obj=MonoObjectHandle @ 0x01b53fbc, error=0x01b5404c) at object.c:146:3
    frame %2: 0x0176191e mscorlibtests`mono_runtime_object_init_checked(this_obj_raw=0x01fde880, error=0x01b5404c) at object.c:176:2
    frame %3: 0x0173ce32 mscorlibtests`ves_icall_System_Globalization_CultureInfo_internal_get_cultures(neutral='\0', specific='\x01', installed='\0') at locales.c:712:4
    frame %4: 0x004931d8 mscorlibtests`aot_wrapper___System_dot_Globalization_System_dot_Globalization_dot_CultureInfo__internal_get_cultures_pinvoke_cl2a_System_2eGlobalization_2eCultureInfo_5b_5d__cl4_bool_cl4_bool_cl4_bool_cl2a_System_2eGlobalization_2eCultureInfo_5b_5d__cl4_bool_cl4_bool_cl4_bool_ + 100
    frame %5: 0x00491f9a mscorlibtests`mscorlib_System_Globalization_CultureInfo_GetCultures_System_Globalization_CultureTypes + 192
    frame %6: 0x00d9b8f4 mscorlibtests`mscorlibtests1_MonoTests_System_DateTimeTest_Parse_Bug53023b + 192
    frame %7: 0x007b56ea mscorlibtests`aot_wrapper_gsharedvt_out_sig_pinvoke_void_this_ + 28
```
`mono_runtime_object_init_checked ()` already should take care of the handle stack 😕

Commit migrated from https://github.com/mono/mono/commit/a093cbddbab1fc9dd6b1fde89e623ad8384e6392

Usually do not create a handle in mono_runtime_invoke_handle. (mono/mono#14404)

Use mono_runtime_invoke_handle_void instead.

Commit migrated from https://github.com/mono/mono/commit/2524d5613be83c6b6e3f4c1800b5a0f4eb2450aa

[netcore] Set of workarounds for ArrayTests (mono/mono#14369)

* Fix ushort to char conversion

* fix one more test

* Enable more Array tests

* fix comment

* Fix CreateInstance_NotSupportedType_ThrowsNotSupportedException

* Address feedback

* Update ArraySortHelper.cs

* fix compilation error?

* bump corefx-tests, fix one more array.copy issue

Commit migrated from https://github.com/mono/mono/commit/b4e565f21b5dda3ecd3513986eb2eb15d184559b

Mechanize strings in marshal.c register_icall and cleanup a little. (mono/mono#14357)

The less mechical parts:
 - spell out monoeg_g_free
 - register_dyn_icall not used, remove
 - register_icall_no_wrapper only used once, spell it out instead

This is nice cosmetically on its own and will help remove JIT icall hashing.

Commit migrated from https://github.com/mono/mono/commit/c7697e06438ebb66d74e2ec20dd90d08773b2f4b

Introduce enum and static data for JIT icalls, that are currently hashed (mono/mono#14359)

* Introduce enum and static data for JIT icalls, that are currently hashed
by string and address and use dynamic allocation.

These will be used shortly.

* PR: Move MONO_JIT_ICALLS content from class-internals.h to jit-icall-reg.h.
Not clear what is the best name but moving along.

Commit migrated from https://github.com/mono/mono/commit/997190c12c5826364874208a57b0e0334a02b59d

[gitignore] Add crash dump files, .dylib.dSYM directories, and test-path (mono/mono#14408)

[gitignore] Add crash dump files, .dylib.dSYM directories, and test-path

Wasn't sure where in the file to best put the .dylib.dSYM stuff, so suggestions are welcome

Commit migrated from https://github.com/mono/mono/commit/2894e57ed72387a17db9f9ee01f56c320d82a819

Merge pull request mono/mono#14203 from kjpou1/wasm-sdk-lower-version

[wasm][sdk] Fix warning NU1604, does not contain an inclusive lower b…

Commit migrated from https://github.com/mono/mono/commit/0dd432096be8585865a9b12fbe1616c0ae30ab13

[llvmonly] make local copy of trace_ips pointer from exception object (mono/mono#14400)

Fixes a race in llvmonly (watchOS):
```
Process 397 stopped
* thread %10, name = 'tid_8e2f', stop reason = EXC_BAD_ACCESS (code=1, address=0x28)
    frame %0: 0x0177f690 mscorlibtests`mono_llvm_load_exception at mini-exceptions.c:3886:23
   3883                 size_t upper = mono_array_length_internal (mono_ex->trace_ips);
   3884
   3885                 for (int i = 0; i < upper; i += TRACE_IP_ENTRY_SIZE) {
-> 3886                         gpointer curr_ip = mono_array_get_internal (mono_ex->trace_ips, gpointer, i);
   3887                         for (int j = 0; j < TRACE_IP_ENTRY_SIZE; ++j) {
   3888                                 gpointer p = mono_array_get_internal (mono_ex->trace_ips, gpointer, i + j);
   3889                                 trace_ips = g_list_append (trace_ips, p);
Target 0: (mscorlibtests) stopped.

(lldb) mbt
* thread %10
  * frame %0: 0x0177f690 mscorlibtests`mono_llvm_load_exception at mini-exceptions.c:3886:23
    frame %1: 0x00e23b7a mscorlibtests`mscorlibtests1_MonoTests_System_LazyTest__c__DisplayClass16_0__ConcurrentInitializationb__1 + 254
    frame %2: 0x0087af98 mscorlibtests`mscorlib_wrapper_delegate_invoke__Module_invoke_void + 610
    frame %3: 0x00368856 mscorlibtests`mscorlib_System_Threading_ThreadHelper_ThreadStart_Context_object + 448
  [...]
(lldb) p/x mono_ex
(MonoException *) $2 = 0x01d9e968
(lldb) p/x mono_ex->trace_ips
(MonoArray *) $3 = 0x00000000
(lldb) thread list
Process 397 stopped
  thread %1: tid = 0x38772, 0x018156aa mscorlibtests`jit_info_table_copy_and_split_chunk(table=0x17635a00, chunk=0x17b767f0) at jit-info.c:483:28, name = 'tid_303', queue = 'com.apple.main-thread'
  thread %3: tid = 0x387ad, 0x1c4f9764 libsystem_kernel.dylib`__psynch_cvwait + 24, name = 'SGen worker'
  thread %4: tid = 0x387b0, 0x1c4f1224 libsystem_kernel.dylib`semaphore_wait_trap + 8, name = 'Finalizer'
  thread %5: tid = 0x387b5, 0x1c4fa6cc libsystem_kernel.dylib`__workq_kernreturn + 8
  thread %6: tid = 0x387b6, 0x1c4f11d4 libsystem_kernel.dylib`mach_msg_trap + 20, name = 'com.apple.uikit.eventfetch-thread'
* thread %10: tid = 0x389a6, 0x0177f690 mscorlibtests`mono_llvm_load_exception at mini-exceptions.c:3886:23, name = 'tid_8e2f', stop reason = EXC_BAD_ACCESS (code=1, address=0x28)
  thread %11: tid = 0x38985, 0x1c4f9764 libsystem_kernel.dylib`__psynch_cvwait + 24, name = 'tid_8113'
  thread %12: tid = 0x38986, 0x1c4f123c libsystem_kernel.dylib`semaphore_timedwait_trap + 8, name = 'Thread Pool Worker'
  thread %13: tid = 0x38987, 0x1c4f123c libsystem_kernel.dylib`semaphore_timedwait_trap + 8, name = 'Thread Pool Worker'
  thread %14: tid = 0x389a3, 0x1c4f123c libsystem_kernel.dylib`semaphore_timedwait_trap + 8, name = 'Thread Pool Worker'
  thread %15: tid = 0x389a4, 0x1c4f123c libsystem_kernel.dylib`semaphore_timedwait_trap + 8, name = 'Thread Pool Worker'
  thread %16: tid = 0x389a5, 0x1c4f123c libsystem_kernel.dylib`semaphore_timedwait_trap + 8, name = 'Thread Pool Worker'
(lldb) thread select 1
* thread %1, name = 'tid_303', queue = 'com.apple.main-thread'
    frame %0: 0x018156aa mscorlibtests`jit_info_table_copy_and_split_chunk(table=0x17635a00, chunk=0x17b767f0) at jit-info.c:483:28
   480                          jit_info_table_split_chunk (chunk, &new_table->chunks [j], &new_table->chunks [j + 1]);
   481                          j += 2;
   482                  } else {
-> 483                          new_table->chunks [j] = table->chunks [i];
   484                          ++new_table->chunks [j]->refcount;
   485                          ++j;
   486                  }
(lldb) mbt
* thread %1
  * frame %0: 0x018156aa mscorlibtests`jit_info_table_copy_and_split_chunk(table=0x17635a00, chunk=0x17b767f0) at jit-info.c:483:28
    frame %1: 0x01815304 mscorlibtests`jit_info_table_chunk_overflow(table=0x17635a00, chunk=0x17b767f0) at jit-info.c:579:10
    frame %2: 0x018146ee mscorlibtests`jit_info_table_add(domain=0x16624cc0, table_ptr=0x16624d9c, ji=0x176f4428) at jit-info.c:614:33
    frame %3: 0x01814662 mscorlibtests`mono_jit_info_table_add(domain=0x16624cc0, ji=0x176f4428) at jit-info.c:674:2
    frame %4: 0x0175f5da mscorlibtests`mono_aot_find_jit_info(domain=0x16624cc0, image=0x16c07200, addr=0x00271ff1) at aot-runtime.c:3710:3
    frame %5: 0x0181443e mscorlibtests`mono_jit_info_table_find_internal(domain=0x16624cc0, addr=0x00271ff1, try_aot=1, allow_trampolines=0) at jit-info.c:306:9
    frame %6: 0x018145e0 mscorlibtests`mono_jit_info_table_find(domain=0x16624cc0, addr=0x00271ff1) at jit-info.c:335:9
    frame %7: 0x01780212 mscorlibtests`build_stack_trace(frame_ctx=0x01c2fb18, state=0x01c2ffb8) at mini-exceptions.c:392:31
    frame %8: 0x1c598aa2 libunwind.dylib`_Unwind_Backtrace + 232
    frame %9: 0x0177f4fc mscorlibtests`throw_exception(ex=0x01d9e968, rethrow=0) at mini-exceptions.c:3800:3
    frame %10: 0x0177f3c8 mscorlibtests`mono_llvm_throw_exception(ex=0x01d9e968) at mini-exceptions.c:3823:2
    frame %11: 0x00439ed0 mscorlibtests`mscorlib_System_Runtime_ExceptionServices_ExceptionDispatchInfo_Throw + 116
    frame %12: 0x00271ff0 mscorlibtests`mscorlib_System_LazyHelper_ThrowException + 98
    frame %13: 0x002730c2 mscorlibtests`mscorlib_System_Lazy_1_T_REF_CreateValue + 194
    frame %14: 0x00273390 mscorlibtests`mscorlib_System_Lazy_1_T_REF_get_Value + 128
    frame %15: 0x0027307a mscorlibtests`mscorlib_System_Lazy_1_T_REF_CreateValue + 122
    frame %16: 0x00273390 mscorlibtests`mscorlib_System_Lazy_1_T_REF_get_Value + 128
    frame %17: 0x00e22eb0 mscorlibtests`mscorlibtests1_MonoTests_System_LazyTest_ConcurrentInitialization + 1422
    frame %18: 0x008948e2 mscorlibtests`aot_wrapper_gsharedvt_out_sig_pinvoke_void_this_ + 28
    frame %19: 0x008664a2 mscorlibtests`mscorlib_wrapper_runtime_invoke_object_runtime_invoke_sig_void_intptr_intptr_object_intptr_intptr_intptr + 312
```

Commit migrated from https://github.com/mono/mono/commit/19f57ad0b82c73c2d56f98b56a32a07f84a0f6d0

[runtime] Use MAP_JIT automatically when running under a hardened runtime on osx. (mono/mono#14395)

* [runtime] Use MAP_JIT automatically when running under a hardened runtime on osx.

* Fix memory ordering.

Commit migrated from https://github.com/mono/mono/commit/f879e35e3ed7496d819bd766deb8be6992d068ed

Mechanize strings in mini-runtime.c register_jit_icall. (mono/mono#14374)

* Mechanism strings to register_dyn_icall in mini-runtime.c.
That is both, generate them kinda, and make them available for some later manipulation.

* Mechanize strings for JIT icall registration in mini-runtime.c.
Generate them slightly and make them amenable to crude preprocessor transforms later.

* Try to improve github diff presentation.

Commit migrated from https://github.com/mono/mono/commit/8928ed6fd89d03c26d3428cf390c1f5384e8686f

[interp] Add tracing support. Return values are not yet supported. (mono/mono#14389)

Commit migrated from https://github.com/mono/mono/commit/5ccb9854b8d73552d5cca531bd75494095032826

Don't apply MONO_ATTR_FORMAT_PRINTF to mono_error_set_argument (mono/mono#14385)

The constructor for mono_error_set_argument lacks a `const char *msg_format`
so clearly we shouldn't be applying formatting from an entirely
unrelated parameter.

This was flagged by `-Werror=format-security`

Commit migrated from https://github.com/mono/mono/commit/1ecbff9e934c8ffc27446d684b4b053e4eaad54d

Remapping should affect GAC search paths considered by the loader (mono/mono#14384)

Closes: mono/mono#14366

Fixes Mac package failures during IronRuby compilation

Commit migrated from https://github.com/mono/mono/commit/80f99fc37d49e9efc415eff87435e966dbae196d

Merge pull request mono/mono#14370 from lewing/mixed-sample

[wasm] Add a mixed target for sample

Commit migrated from https://github.com/mono/mono/commit/b0ec83c4d0ceba3c5fd5f692c656339af6addcaf

Merge pull request mono/mono#14375 from lateralusX/lateralusX/fix-mutex-leak-on-gc

Mutex memory leak on sgen GC (nursery, major).

Commit migrated from https://github.com/mono/mono/commit/5bcf1663754ccfe1434aa848e04c0114583a50d3

[arm] fix armv7k regression on struct passing (mono/mono#14362)

We can't use `slotsize=8` on `armv7k` as it leads to stack corruption.

Note that `mtouch` already calls the AOT compiler with `--aot=mtriple=arm64_32-ios`, so no changes are required there.

Regression introduced by https://github.com/mono/mono/pull/12992

Fixes https://github.com/mono/mono/issues/13454

Commit migrated from https://github.com/mono/mono/commit/8913a1a73e85c60a7d896832d0dac4cfea979e6a

Mutex memory leak on sgen GC (nursery, major).

During GC we allocate a gray queue protected by a mutex, sgen_gray_object_queue_init.
This mutex is never destroyed when clearing up the object containing it,
sgen_gray_object_queue_dispose, so it will leak. Overtime this could cause
OOM on some systems depending on how memory related to system primitives are
handled and partitioned.

Commit migrated from https://github.com/mono/mono/commit/c5d014397a675fc34e2c69d3505a3a35560bdab9

Mechanize strings in cominterop register_jit_icall. (mono/mono#14356)

This reduces the current source code and will be useful in eliminating JIT icall hashing.

Commit migrated from https://github.com/mono/mono/commit/b3ecd1710998baf1397db05e265fb659f81b8726

Mechanize strings in remoting.c register_icall. (mono/mono#14360)

This will help remove/reduce JIT icall hashing, as the
name will also be a field name or token paste to an enum value.

Commit migrated from https://github.com/mono/mono/commit/072ff8fcfb75eefb3744f8b88f1965db3b275960

A deadlock happens when during a process_suspend there is a pending invoke and then when it's calling the invoke there is another call to process_suspend in the same thread. (mono/mono#14368)

Commit migrated from https://github.com/mono/mono/commit/1294e4d0e5597d1174bddb19258b6cf47c905e19

Make mono_fmod extern "C and not static, to simplify later (mono/mono#14355)

* Make mono_fmod extern "C and not static, to simplify later
removal of JIT icall hashing. Making it more consistent with other icalls.
Costing maybe one instruction in some slow paths.

* Move mono_fmod from mini-runtime.c to jit-icalls.c.

* Fix declaratoin of mono_fmod.

Commit migrated from https://github.com/mono/mono/commit/a41be288719e56df153158faba0ad3c27af2e896

Remove special casing (mono/mono#14155)

* match the parameter

* Removed special handling.

Removed special handling of TypedReference and RuntimeArgumentHandle

* Remove the whole special casing block

* Revert part of last commit

Restored the check for IsByRefLikeAttribute

* Corrected the function name

Commit migrated from https://github.com/mono/mono/commit/f60a397ab5714751482d2823ad5eeee1fb03fa43

Merge pull request mono/mono#14259 from jeromelaban/dev/jela/wasm-dlopen

[Wasm] Add support for P/Invoke through dynamic linking

Commit migrated from https://github.com/mono/mono/commit/8d80ccc897c678d7bdae645ca8629b0c5cc0b667

Add execute permissions on build scripts.

Commit migrated from https://github.com/mono/mono/commit/edc7c34502b61e1941ddd0dc8bc3e8076ebd6eb9

Merge pull request mono/mono#14331 from BrzVlad/feature-interp-ldsflda-opt

[interp] Optimize ldsflda opcode

Commit migrated from https://github.com/mono/mono/commit/b2a9049de4835386f7ad4831986a0f8481536736

[netcore] Fix CreateInstance_PublicOnlyValueTypeWithPrivateDefaultConstructor_T… (mono/mono#14321)

* Fix CreateInstance_PublicOnlyValueTypeWithPrivateDefaultConstructor_ThrowsMissingMethodException

* Update excludes-System.Runtime.Tests.rsp

* Update RuntimeType.cs

Commit migrated from https://github.com/mono/mono/commit/534c6883b1fd155113867f9b684863dffae98e04

Merge pull request mono/mono#14340 from lewing/wasm-link-descriptor

[wasm] Add --link-descriptor arg to packager

Commit migrated from https://github.com/mono/mono/commit/887887a7fe3fb302cea267db760fed9dfb46209a

Regenerate m2n trampolines

Commit migrated from https://github.com/mono/mono/commit/ff4839d99e57a4bd53e92e005245c483ccfaeda0

Adjust indentation

Commit migrated from https://github.com/mono/mono/commit/40981f0cfdfff290ae5000498724e252053339da

Add vifffff and viffffff trampolines

Commit migrated from https://github.com/mono/mono/commit/21743bd58da762558e04752882b2760d0b6c30e9