Merge pull request #3269 from alexanderkyte/silence_zero_len

[runtime] Disable noisy warning

[test] Fix sporadic Mono.Debugger.Soft ThreadpoolIOsinglestep failure.

ReadAsync size arg should not be greater than array.Length - offset.

Fix the build.

[runtime] Implement WaitForPendingFinalizers used cond vars instead of io-layer events. (#3315)

Bump to include the F# bockbuild patch

[arm] Use soft breakpoints on watchos.

Merge pull request #3342 from ludovic-henry/gc-domain-finalizer-sem

[gc] Replace domain finalizer request cond+mutex by a semaphore

Merge pull request #3335 from lambdageek/dev/system-reflection-assembly

[reflection] Move AssemblyName to managed

[Facades] Emit symbol files for lib like facades

[man] Fix typo in environment variable name

The section was added in 410a200ecd07f5e53ffea858d204b3bbf88d311d but with the wrong name.

[gc] Fix timeout in mono_domain_finalize

[gc] Ensure we clean the DomainFinalizationReq

[gc] Replace domain finalizer request cond+mutex by a semaphore

This is possible because we only wait for it at one place, and only post it at one other place.

[arm] Implement support soft breakpoints + AOT.

[sdb] Fix an aot assert.

[corlib] Bring AssemblyName(string) to the happy C# world.

[corlib] Move Assembly::GetName to be implemented in managed.

This involves a few steps.

First define MonoAssemblyName in RuntimeStructs.cs

Then add some helped to decode blob arrays and handle hex digits.

Finally, move the actual implementation from native to managed.

Merge pull request #3320 from lambdageek/dev/reflection-in-managed

[reflection] Marshal name strings from managed

[reflection] Marshal name arg of RuntimeType.GetNestedTypes in managed code

Also do the name unmangling on the managed side.

[reflection] RuntimeType.GetNestedTypes in managed

[reflection] Marshal name arg to some RuntimeType search methods

Marshal the name argument of Get{Methods,Properties}ByName,Get{Fields,Events}
in managed code.

Cleanup the corresponding icalls.

[corlib] Add Mono.SafeStringMarshal struct

Allows safe allocation of native utf8 char* strings from managed
strings.

using (var h = new SafeStringMarshal (str)) {
   /* use h.Value IntPtr */
} // g_free'd by h.Dispose ()

[mcs] Recursive interface declaration check is needed for interfaces only. Fixes #42198

[mcs] Do exactly matching expression check for anonymous methods with successfully inferred return type only. Fixes #42611

Revert "[runtime] Add documentation to mono-os-{mutex,cond,sem} + Move to .c file"

This reverts commit 9ba062e5a183f3ace3a21e2e3d240adc3db0e5a4.

[gc] mono_os_cond_timedwait returns -1 on timeout, not ETIMEDOUT

[runtime] Add documentation to mono-os-{mutex,cond,sem} + Move to .c file

[reflection] Plug more leaks.

Fix a cut'n'paste error

Merge pull request #3338 from lambdageek/dev/plug-ptrarray-leak

[reflection] Don't leak GPtrArray internal array

Merge pull request #3334 from alexrp/arm-cpu-detect-fix

[arm] Also check the 'model name' line for the architecture version.

[corlib] change Mono.SafeGPtrArrayHandle to a struct

Don't GC allocate short-lived internal garbage.

[reflection] Don't leak GPtrArray internal array

`g_ptr_array_free (..., FALSE)` was the wrong thing here.  We always
want TRUE in our use cases.

[mcs] Add codegen for compound assigment of complex expressions with concatenated string using await inside params arguments. Fixes #42224

[arm] Make HAVE_ARMV{5,6,7} macros override hwcap detection.

This is necessary on weird devices where the hwcap code might fail to detect
the ARM version it's running on (usually caused by differences in /proc/cpuinfo
output).

Merge pull request #2961 from schani/fix-sgen-refactoring

SGen fixes and refactorings

Merge pull request #3328 from BrzVlad/finalizer-thread-exited2

[sgen] Avoid popping the entire finalizer queues if finalizers are su…

[System.Core] Add MonoTODOAttribute to workaround linker bug with IVT from mscorlib

[arm] Also check the 'model name' line for the architecture version.

Some Android devices for some reason have a 'model name' line instead of a
'Processor' line. On such devices, we would fail to detect the architecture
version at all, and end up thinking that it's ARMv4 (even if the runtime
was compiled for ARMv7).

[mcs] Detect constant switch statement fall-through. Fixes #42585

[Facades] Remove duplicate System.IO.Compression

[sdb] Explicitly encode in the JIT debug info whenever there is precise information about variable locations instead of checking whenever the method has seq points. Fixes #42750.

[sgen] Avoid popping the entire finalizer queues if finalizers are suspended

When we shutdown we set suspend_finalizers flag in metadata so we no longer run any finalizers. After recent changes, we avoid traversing gc hashes for finalizer enqueuing when this flag is set. The counterpart, popping the queues empty (even without running the finalizers) can still take some time, especially since we lock for each entry. Exit early from this loop if we no longer need to run finalizers.

Achieve this by setting a new flag inside sgen, instead of passing the address of suspend_finalizers all the time to the sgen api.

Merge pull request #3199 from lambdageek/dev/proxy-setter

[remoting] Convert mono_store_remote_field_new_checked to managed code

[mcs] Report interpolated strings feature. Fixes #42410

Merge pull request #3330 from madewokherd/compactframework

Remap the Compact Framework PublicKeyToken for assemblies it ships.

[System.Text.Encoding.CodePages] Small improvements

[mono-api-info] Add contract-api option

[mono-api-html] Ignore more non breaking virtual changes when enabled

[bcl] Update some of .net apis

[semaphore] Return error for mono_os_sem_timedwait

[semaphore] Make mono_os_semaphore_* functions return more meaningful values

By asserting on error, we ensure that we return from these functions only when successfull, or when it makes sense to return an error (for example, timedwait return -1 only on timeout or interrupt).

[mutex] Make mono_os_mutex_* functions return more meaningful value

By asserting on error, we ensure that we return from these functions only when successfull, or when it makes sense to return an error (for example, timedwait return -1 only on timeout).

Merge pull request #3327 from rolfbjarne/bug41745

[System*] Disable tests that require working networking stack on watchOS.

[threadpool-ms] Fix use-after-unloaded of domain

The following scenario might happen:
 - thread A: returns from tpdomain D1, then parks
 - thread B: returns from tpdomain D1, D1 is unloading so it's removed with `domain_remove`
 - thread A: unparks, calls `domain_get_next` with tpdomain D1
  - domain_get_next asserts as it can't find tpdomain D1 in the list of domains

To fix that, we simply set the previous_tpdomain to NULL when unparking.

[System*] Add category for tests that use API that require BSD sockets.

We don't have access to BSD sockets on watchOS, so that API
doesn't work.

With this category we'll be able to exclude these tests.

Merge pull request #3331 from lambdageek/dev/test-for-42625

[debugger-agent] Regression test for #42625

Merge pull request #3262 from lindenlab/add_continuations_test

Added Mono.Tasklets test

[runtime] Accomplish BITCODE build symbol sharing with only make (#3329)

Added Mono.Tasklets test

[debugger-agent] Regression test for #42625

Check that we can suspend I/O blocked threadpool workers in coop GC mode.

[ci] Don't cd in run-test-acceptance-tests.sh

The other scripts like TESTCMD rely on paths being based off of the repo root.

[ci] Fix typo in run-jenkins.sh

Remap the Compact Framework PublicKeyToken for assemblies it ships.

Merge pull request #3324 from alexanderkyte/threads_unhandled_exc

[runtime] Call unhandled exception handler explicitly to fix for bitcode

[acceptance-tests] Remove folders from Roslyn script that we now ship after 8f130a6e17c372

[ci] Move acceptance-tests script from Jenkins into the repo

Yay, less state on Jenkins!

Bump Mono version to 4.7.0

[sgen] Fix and simplify some debugging stuff.

[sgen] Document sgen-copy-object.h

[sgen] Fix major collection time accounting.

This was from when we did a nursery collection as part of the
finishing collection pause.  Now that we don't do that anymore, we
underaccount major collection times.

[sgen] Only do fence when concurrent collection is running.

The store/store fence incurs quite a high cost, so we only want it to
run when absolutely necessary.  The most efficient way is to
separately compile nursery object operations for when the concurrent
collector is running, so there has to be no dynamic check.

In the future we might want to try batching updates of references to
moved objects so we have to do only one fence per batch.

[sgen] Missing store/store fence.

[sgen] Fix whole heap check.

[sgen] Make unnecessary stack check into assertion.

[sgen] Don't crash in binary protocol flush when forcing.

[sgen] Flush binary protocol buffers when missing remsets.

The missing remset check uses the binary protocol to log missing
remsets it found.  Debugging is easier if we don't have to manually
flush the binary protocol.

[sgen] Remove stupid typedef.

[sgen] Get rid of the global gray queue.

[sgen] Pass GC thread gray queue into jobs.

[sgen] Don't expose the worker's gray queue.

[sgen] Give remset consistency check a proper name, and do it more often.

[sgen] Don't be too fussy about missing remsets to pinned objects.

If a thread is interrupted after a reference write but before the corresponding
write barrier, the remset will be missing, but the object will be pinned.

[runtime] Call unhandled exception handler explicitly to fix for bitcode

Merge pull request #3325 from alexanderkyte/fix_bitcode_mini

[mobile_static] Pass correct MONO_LLVMONLY for bitcode script

[bcl] Implement System.Text.Encoding.CodePages.CodepagesEncodingProvider in the facade

Fix Windows build broken in 2c061c18852

[runtime] Implement the GC domain finalization event using conditional variables instead of an io-layer event. (#3316)

[mobile_static] Pass correct MONO_LLVMONLY for bitcode

[runtime] Abort instead of stopping background threads on shutdown. Hopefully fixes #41644. (#3323)

[System]: Fix SystemCertificateValidator.EvaluateSystem() on Android.  Fixes #41290.

After the latest code changes, we may land here with `errors == SslPolicyErrors.None`,
so we need to make sure that we actually set it to `RemoteCertificateChainErrors` if
the system validator returns an error.

(cherry picked from commit b4304354b149334e5ddd11c4c143aac64f46fe03)

Merge pull request #3318 from akoeplinger/cleaner-mini-nunit

[mini] Explicitly turn nunit output on or off based on variable

Merge pull request #3319 from akoeplinger/fix-mono.csharp

[Mono.CSharp] Fix test .sources to include ASTVisitorTest.cs

Merge pull request #3322 from grendello/offsets-tool-xa

Modify the offsets dumper to work with Xamarin.Android

[sgen] Fix build

Modify the offsets dumper to work with Xamarin.Android

This is a handful of quick and dirty changes to make the utility
work with Xamarin.Android which uses a different directory layout
than monodroid.

Merge pull request #3289 from BrzVlad/fix-critical-finalizer

[sgen] Fix critical/normal finalization order at domain unload

Merge pull request #3321 from BrzVlad/fix-block-state-membar

[sgen] Add missing memory barrier

Merge pull request #3290 from BrzVlad/fix-finalizer-tests

[tests] Fix finalizer tests

[mini] Explicitly turn nunit output on or off based on variable

Alternative implementation to 8f092147bb604485c78c80b9781536ec995d51fe.

In CI we explicitly pass EMIT_NUNIT=1 and check-seq-points to generate the nunit xml and run the tests,
locally those are omitted. This avoids needing to sniff for Jenkins/Wrench variables.

Same for a9eec21d62ecd1da3929cdbfea872bfc4d22134b, but we already pass V=1 in Jenkins.