Merge pull request #2182 from lewurm/fix-arm-build

mini-llvm: fix wrong usage

mini-llvm: fix wrong usage

[tests] Adjust MathTest double epsilon value

The double.Epsilon value represent the smallest value that double is
able to represent. And that is diferent from the machine epsilon, which
is the smallest difference between two double values.

The second is equal to 2 ^ (-52), because there are 52bits to
represent the mantissa in a IEEE 754 double.

[llvm] Split sig_to_llvm_sig () into two versions, one which takes an LLVMCallInfo argument, and one which doesn't.

Comment-out [HostProtection] attribute in SslStream.cs, it seems to be causing problems.

[runtime] Disable the -Wreturn-stack-address clang warning.

[threads] Fix warning on Linux

s390x-codegen.h - Initial SIMD support
cpu-s390x.md - Add gc-safe-point instruction
mini-ops.h - Add TARGET_S390X for gc-safe-point
mini-s390x.c - Initial SIMD support; Fix floating point conversion
mini-s390x.h - Initial SIMD support

[llvm] Clean up the naming convention in the backend. 'module' refers to a MonoLLVMModule while 'lmodule' refers to a LLVMModuleRef. Some other small cleanups.

[llvm] Merge LLVMCallInfo and LLVMSigInfo.

[System]: Replace the 'SslStream' implementation with a proxy that's using the 'MonoTlsProvider' layer.

This allows consumers of the 'SslStream' class to take full advantage of
the new TLS switcher code.

[Mono.Security]: It is legal for a TlsClientCertificate message to contain no certificates.

Fix the build, the stub for mono_llvm_cpp_throw_exception () was inside a DISABLE_JIT ifdef block.

Fix my last commit.

[Mono.Security]: Rename 'MonoTlsSettings.ServerCertificateValidationCallback' into 'RemoteCertificateValidationCallback'.

This callback is also used to validate client-certificates when using SslStream in server-mode.

Don't pass a 'null' hostname to the system validator as it would crash.

Kill dead code.

[runtime] Handle the conditional compilation of llvm-runtime.cpp at the automake level since including a c++ file into a c project seems to cause linking problems on some platforms.

[runtime] Add a configure check for unwind.h

[profiler] Avoid a crash at shutdown by installing the assembly unload hook for the 'start_unload' event instead of the 'end_unload' event, the latter receives a mostly freed assembly.

[threads] Use runtime flag to enable cooperative suspend

To enable cooperative suspend, simply set the MONO_ENABLE_COOP
environment variable.

Merge pull request #2179 from BogdanovKirill/enummodulesfix

[Processes] Memory leak fix in EnumProcessModules function

Fix the build.

Use 'using SD = System.Diagnostics' to avoid ambiguous references.

[test-runner] Improve output

[Process] Catch the closing of handle

[Process] Do not count the output/error wait in the timeout of WaitForExit

[runtime] Fixed the non-LLVM build.

Fix the windows build.

[threads] Fix race between async suspend and compensation

The race only happens on Posix with unified suspend.

The chain of event is the following:
- T1 suspend the world (calls `sgen_unified_suspend_stop_world`)
- T2 is async suspended
- T2 execute suspend_signal_handler:
 - T2 finish async suspend
 - T2->suspend_can_continue is set to FALSE as the thread is detaching
 - T2 notify T1 via `mono_threads_notify_initiator_of_suspend`
 - T2 is yielded (via the OS scheduler, or forced via a `sleep`)
- T1 returns from `mono_threads_wait_pending_operations`
- T1 does its stuff
- T1 restart the world
- T1 (or any another thread, except for T2) suspend the world (it does not
  necessarily uses `sgen_unified_suspend_stop_world`)
- T2 is async suspended: incrementation of T2->suspend_count to 2, but it is
    not signalled because it is already in ASYNC_SUSPENDED state
- T2 wakes up (for any reason) and execute compensation <- ERROR: suspend_count != 1

This race can happen because when async suspending, we notify the
suspendor before executing the compensation, in this case for a thread
which is detaching. By simply post-ponning the notification, just after
the compensation, we eliminate this race.

[acceptance-tests] Write list of CoreCLR tests into file and pass it to test-runner

This shaved about 30 seconds off the time until the test-runner started on my machine compared to
passing the tests in the command line directly (we have *lots* of CoreCLR tests).

[test-runner] Add a new --input-file option for reading the test list from a file

[test-runner] Fix bug in commandline option parsing

i+i doesn't make sense, it should be i+1 (the next argument after the current one).

[tests] Add coreclr GC stress tests

[tests] Try to fix finalizer-wait

[tests] Fix Math.Pow precision bug on Android

[Processes] Memory leak fix in EnumProcessModules function

The main point is that get_process_name_from_proc's result string is never released.

This change is released under the MIT license.

[profiler] Disable 'time went backwards' debug message.

While this would have indicated a bug in the past, it's perfectly normal now
due to how we synthesize TYPE_JIT events to ensure sane event ordering.

[runtime] Use Unwind_Backtrace only on platforms which support it.

[llvmonly] Add support for direct calls to icalls.

[System/TLS]: Don't crash when we're called without user-supplied certificate validator.

[acceptance-tests] Move CoreCLR submodule reference to Mono fork

Oops, there is no README.md

Oops, forgot to add to the top-level Makefile for mobile.

[test-runner] Filter out invalid XML chars before writing to XML

The test-runner crashed sometimes when running the CoreCLR testsuite with the following exception:
"System.ArgumentException: '', hexadecimal value 0x0F, is an invalid character." when trying to
emit the NUnit xml.

The stacktrace of one of the failing tests contained the character 0x0F which is invalid in XML,
we now filter out those invalid chars.

[Mono.Security.Providers.OldTls]: New assembly.

This provides a 'MonoTlsProvider' for Mono's existing TLS implementation.

It's simply a public frontend for the internal 'LegacySslStream' class.

[System]: Make internals visible to Mono.Security.Providers.OldTls.

[sgen] Fix marking of cards on 64bit

[sgen] Fix call to `InterlockedCompareExchange()`.

This fixes random crashes on `sgen-weakref-stress.exe`.

[linker] match type pattern with nested types

 - this way the following xml will match all the types in the assembly
   MyAssembly

    <linker>
        <assembly fullname="MyAssembly">
            <namespace type="*" />
        </assembly>
    </linker>

   before this change it will only match the types defined in main
   module and none of the nested types

[acceptance-tests] Append a suffix to the CoreCLR IL tests

Some of the tests have both a test.il and test.cs in the same folder, which means we would
generate an .exe with the same name and overwriting one of them.

This is avoided by appending a _il suffix to the IL tests.

Merge pull request #2163 from xmcclure/image-audit

Mempool reference-auditing feature in checked builds

This introduces three macros, to be used in place of a simple pointer assignment when working with images or imagesets:

CHECKED_METADATA_WRITE_PTR - Use this to assign a pointer when it points from memory hosted by an image or imageset to another.
CHECKED_METADATA_WRITE_PTR_LOCAL - Use this to assign a pointer when it points from memory noted by an image or imageset to itself.
CHECKED_METADATA_WRITE_PTR_EXEMPT - Use this to assign a pointer in memory hosted by an image or imageset, but which the audit feature cannot correctly check at this time (this is the case when an image must contain a pointer to something that is not located in a mempool, such as a constant string). This is so that at least we know where these exceptions to the audits are.

In non-checked mode, each of these is a simple assignment. In checked mode, they walk the graph of references between images to make sure invariants related to ref_count are not violated.

[runtime] Normalize include style.

Merge pull request #2175 from xmcclure/pre-image-audit

Mempool reference-auditing feature in checked builds (prep work only)

Simplify function interface for rework of image.c hashes, based on PR comments

[runtime] Fixed cross-compile LLVM build.

[test-runner] Make sure to close resources hold by Process

[Process] Use output/error stream instead of naked handle

This is to make sure the handle is closed, whenever we close the streams

[test-runner] Make the --testsuite-name option required

Defaulting to a value doesn't make sense since the NUnit xml would be overwritten if multiple tests used the same default.

[runtime] Added --with-cross-offsets configure option for including an explicit AOT cross offsets file.

This option will define MONO_OFFSETS_FILE which will be used by the rest of the code to include the right file when looking for cross offsets.

[mcs] Declared fields cannot have unmanaged check done before all types are defined. Fixes #35269

[llvmonly] Fix EH when llvm is disabled.

[llvmonly] Fix an assertion for methods with many arguments.

[llvmonly] Fix the computation of the generic context used to initialize shared methods, previously we would be using the context of the class of the receiver, but that wouldn't work when its a non-generic class inheriting from a generic class. Use the context of the method which was used to lookup the shared method in the first place instead.

[sgen] Fix binary protocol limit default.

`size_t` is unsigned, so using `-1` by default will always impose a
limit, and make binary protocol file names numbered, which is not
usually what you want or expect.

[Process] Close process in/out/err handle on Process dispose

This would lead to leaked handle, and that would be visible when running
the test suite on Jenkins, with a "Too many open files" exception.

That also reverts the behavior to before 789808b507fe16941a31acc43ffc6bc8e0cdae75

Rework “image hashes” in image.c to segregate by-path and by-name lookups into two hashes

This is part of the continued cleanup work for the mempool reference auditing feature. Separating the hashes prevents redundant work when iterating over hashes, something the reference auditor does a lot. External interface to image.c still behaves the same.

Continued cleanup in preparation for mempool reference auditing feature

- Fixes to comments and the mempool rework, based on pull request comments
- Additional comments overall

[llvm] Fix aot support in non-llvmonly mode.

[llvmonly] Avoid disabling llvm for methods with pinvoke signatures in llvmonly mode.

[llvm] Register the llvmonly icalls even if llvm is not enabled.

Fix the non-llvm build.

[tests] Add timeout before failing

[tests] Increase timeout before failing

[mcs] Support auto-property with private setter with member definition from external assembly. Fixes #35236

[tests] Increase timeout before failing

[tests] Add --testsuite-name to avoid overwritting test result XML files

[tests] Use test-runner to run unhandled-exception tests

Add another missing file.

Add a missing file.

[llvm] Move the EH support code into mini-exceptions.c, and the c++ functions used by it into a new llvm-runtime.cpp file so llvmonly code works in a runtime which doesn't have llvm enabled.

[llvm] Fix support for asmonly+llvmonly by emitting the bitcode into the file given by the llvm-outfile aot option.

Merge pull request #2166 from ermshiperete/FontHeight

[WinForms] Check for null font Height

[profiler] Create a static version of libmono-profiler-log which doesn't link against eglib, to prevent duplicate symbols errors when its linked with the runtime. Fixes part of #29745.

[arm] Add a comment for a hack.

[arm64] Implement mono_threads_core_begin_async_resume () for arm64.

Merge pull request #2171 from lambdageek/dev/fix-marshal

[coop] One more pair of coop gc marshal fixes.

[sgen] Documentation about future-proofing the binary protocol.

We want to use the binary protocol to measure GC pause times in the
benchmarking suite.  To make that easier and be able to go back and
(re-)benchmark older versions, we'd like the binary protocol format to
be backwards compatible.

Add a new Moono_GetStackTraces method to Thread which can be used to obtain stack traces for all running threads.

[coop] One more pair of coop gc marshal fixes.

Missed a pair of
`mono_threads_reset_blocking_start`/`mono_threads_reset_blocking_end`
calls in 991e3ef3e47f0ccf9981a791f7da680b4ee9c4c6

Remove a few more 'using System.Reflection's.

[Mono.Security]: Use the new 'NoReflectionHelper' instead of reflection in 'MonoTlsProviderFactory'.

[sgen] Add some bridge documentation.

Merge pull request #2170 from alexanderkyte/mini_llvm_archfix

[runtime] Fix support for amd64 in mini-llvm-cpp.cpp

[runtime] Fix support for amd64 in mini-llvm-cpp.cpp

[System]: Add internal 'Mono.Net.Security.NoReflectionHelper' class to avoid reflection use.

Remove 'using System.Reflection'.

[Mono.Security]: Access System.dll's internals directly instead of using reflection.

[System]: Make internals visible to Mono.Security.