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.

[sgen] Add missing memory barrier

When doing the first phase of sweeping we tag the block pointer within the block list with a CAS, after which we own the block and we are free to update the state and do sweep computations on the block. When we finish this phase we directly write the untagged block pointer back in the list, making the block available to inspect by other threads. We need a write barrier before writting the untagged pointer back because, when another thread might acquire the block, it might encounter an invalid block state.

[runtime] Decrease the verbosity of the runtime test runner when not running under CI.

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

I noticed this while looking at why Mono.CSharp is registered as "crashed" in the the new
fullaot PR job on Jenkins. It turns out all tests were excluded in mobile_static which means
no NUnit xml was produced and that is exactly what the script then interprets as a crash.

I saw that ASTVisitorTest.cs was not included in the net_4_x tests as well, likely inadvertently.
It got bitrotten over the years so I had to fix it up a bit, now it compiles/runs on mobile_static as well.

[ji] Run the mini tests without the emitnunit.pl script when not running under CI.

[versions.mk] Make path to SUBMODULES.json based on srcdir (#3314)

Since acceptance-tests/Makefile (and llvm/Makefile) include scripts/submodules/versions.mk which evals versions.py which looks
for a SUBMODULES.json file in the current working directory we got confusing error messages if the build directory is not the src dir:

```
Traceback (most recent call last):
  File "../../mono/scripts/submodules/versions.py", line 22, in <module>
    submodules = json.load(open(CONFIG_FILE))
IOError: [Errno 2] No such file or directory: 'SUBMODULES.json'
```

Instead we now pass the path to SUBMODULES.json as a variable to versions.py and set it based on $(top_srcdir) so it works
even for out-of-tree builds.

[runtime] Make AOT'ed TestDriver.dll dependency for builds (#3303)

[mini] Fail methods that try to throw scalars. Fix hard crash from #42417. (#3312)

[mobile_static] Use clang++ for the linker (#3308)

In order for our exception handling to work on linux, we need the
unwind-dw2-fde-dip implementation in libgcc to work with our
.o file. This requires that `--eh-frame-hdr` be passed by the linker.

By using clang++ as the linker, we will get this and other flags
necessary for correct interop with c++ libraries.

Merge pull request #3305 from esdrubal/response_stream_close

[System] Fix RequestCanceled ex on ResponseStream.Close

Merge pull request #3306 from kumpera/fix_wait

[wapi] Fix timed wait code to not overflow. Fixes #42688

[runtime] Only add Mono.Dynamic.Interpreter to mobile_static (#3302)

Merge pull request #3307 from alexanderkyte/aot_path

[mobile_static] Fix temp path for mcs/class AOT

[mobile_static] Fix temp path for mcs/class AOT

[wapi] Fix timed wait code to not overflow. Fixes #42688

Wait math must always subtract ticks and get deltas. That way we avoid overflows.

[System] Fix RequestCanceled ex on ResponseStream.Close

Fixes issues where ResponseStream.Close would throw an exception when
peer had already closed the connection and ResponseStream.Close would
still try to communicating to peer that it was closing.

The issue was brought to our attention by failures on:
  - MonoTests.System.Net.WebRequestTest.TestReceiveCancelation
  - MonoTests.System.Net.HttpWebRequestTest.TestLargeDataReading

[amd64] Rewrite the handler block trampoline to use normal calls instead of jumps so the functions called from the calls execute with an aligned stack.

[tests] Disable some more tests in the mobile_static profile.

[runtime] Add a missing mono_error_init () in a DISABLE_APPDOMAINS codepath.

[amd64] Fix stack alignment in the handler block trampoline.

[tests] Remove double inclusion of source file

[tests] Fix flakiness on some tests that assume object finalization

A common pattern that we use across the test suite is to create objects on a new thread, join the thread, collect then expect certain objects to be dead/finalized since we assumed that thread to be no longer a source of pinning. This is not always the case since, after joining, the thread might still be alive and running unmanaged code related to the thread shutdown, which can reuse the stack that previously contained the managed references.

The workaround is to still allocate objects far down the stack, even if we are on a separate thread.

At the moment the helper can be used in tests which run under the TestDriver (since it's included in TestDriver.dll) and can also be included as a helper source for test bcl classes that need it.

Merge pull request #3304 from kumpera/unlimited-abort

Change suspend abort timeout to infinite and fix timeout env

Revert "[Facades] Add System.Runtime.InteropServices.PInvoke"

This reverts commit 6d13f594fa39a4d7dedeab278192d48c5b563fa9.

[Facades] Add System.Runtime.InteropServices.PInvoke

[mono-threads] Change suspend abort timeout to infinite and fix timeout env var parsing.

[utils] Introduce MONO_INFINITE_WAIT instead of checking for -1 all over the place.

Merge pull request #3300 from esdrubal/mono-sym-fixes

[mono-symbolicate] Fixes symbolicate.expected

[tests] Disable tests which are expected to fail in the mobile_static profile.