Merge pull request #2764 from esdrubal/fileinfo_tostring

[corlib] Fix FileInfo.ToString after MoveTo

Merge pull request #2534 from mhutch/install-api-html

Install mono-api-html

We install its counterpart mono-api-info already

Install mono-api-html

We install its counterpart mono-api-info already

Merge pull request #2778 from xmcclure/checked-build-updates-2

Fix patch merge error in PR#2753, was breaking checked build

Fix patch merge error in PR#2753, was breaking checked build

Add note about acceptance-tests to top level README

Fix "make dist"

[gsharedvt] Add a test for #39528.

[runtime] Remove some dead test code added by c736aba5c94c20e6faad6b736f8e3a834e26d95f.

[runtime] Implement support for passing icall methods as delegates in pinvoke. Fixes #39347.

[runtime] Remove an mistaken assert. Fixes #39568.

[jit] Remove an mistaken assert. Fixes #39746.

Merge pull request #2777 from xmcclure/win-makefile-aotcheck

Make `make fullaotcheck` in mono/mini runnable on Windows

Run with
MONO_EXECUTABLE=/path/to/msvc/mono.exe LIST_SEPARATOR=";"
This assumes you are running the test harness in Cygwin and you built mono with MSVC. Also, the test does not currently succeed.
Being able to run the test is a zeroth step toward making the test pass.

[System.Net.Http] Make HttpClientTest.RequestUriAfterRedirect test more reliable

It was the only test in the class that required network access (for looking up xamarin.com in the redirect).
Replacing the redirect target with a second local HttpListener should be more resilient to network/DNS issues.

mono/mini make fullaotcheck can now be run on Windows

Test harness now uses the proper path separator and also now supports
MONO_EXECUTABLE if you built with MSVC.

Merge pull request #2753 from xmcclure/checked-build-updates

Changes to "checked build" feature

Here are some changes to checked builds I made while trying to debug some coop gc problems. The changes made are:

- Replace the checked build configure flags with a single --enable-checked-build= flag
- Runtime environment variable required for checked build asserts to run
- Report the thread transition history for more kinds of asserts
- For the `mono_threads_reset_blocking` functions, add an assert that guarantees proper nesting

In addition to this, I have documented the checked build feature as implemented here, see my pull request for the website: https://github.com/mono/website/pull/187

Overall my goals with this patch were

- Make it easier to do rapid-fire testing with the checked builds (runtime switch, default-off)
- Make the checked build implementation more internally consistent (more uniform names etc)

Notice I do make the change that instead of checked build sub-features being disabled with a blacklist, you now must enable them with a whitelist. I think this is an improvement because it makes it easier to just casually keep a checked build around  (for one thing I would have the option to ship debug builds to users with the checked build modes included but dormant) but I could be talked out of it if this change turns out to be controversial. The only thing I feel strongly about is that if the configure flag is changed back to a blacklist the environment variable should switch to a blacklist also.

If we accept this change the CI bots will need to adopt the new configure flag and environment variable.

Merge pull request #2746 from lambdageek/dev/monoerror-mono_reflection_create_runtime_class

[System.Reflection.Emit] MonoError in mono_reflection_create_runtime_class

Depends on #2745
Bubble MonoError up to `mono_reflection_create_runtime_class`.  Use `mono_error_set_pending_exception` instead of `mono_error_raise_exception` And try to release resources on the way out.

Add mono_threads_reset_blocking nesting checks to gc checked mode

Rename checked build assertion_fail to mono_fatal_with_history, use in more places

Merge pull request #2772 from WeiHeCN/master

Update mini-llvm.c

These header files are already included in mini-llvm-cpp.h.

Update mini-llvm.c

[SRE] MonoError for mono_reflection_create_runtime_class

1. Don't raise while holding locks.

2. Change remove_instantiations_of_and_ensure_contents (called via
  g_hash_table_foreach_remove) to populate a MonoError instead of
  throwing.  If there are multiple errors, we just stash the first one
  in user_data and clean up the rest.

3. Use mono_erro_set_pending_exception instead of
  mono_error_raise_exception since it's an icall.

Merge pull request #2745 from lambdageek/dev/monoerror-sre-internals

[System.Reflection.Emit] Propagate MonoError through some more static functions

Some externally visible changes include:
* Bubble MonoError up to one more SRE icall: `System.Reflection.Emit.CustomAttributeBuilder.GetBlob()`
* `fix_partial_generic_class` and `ensure_generic_class_runtime_vtable` now expect their callers to set the class failure state which they do under slightly more circumstances now than previously (now: whenever there's an error set from the above functions; before: only when we had trouble initializing the generic type definition.)
* Previous API for `mono_reflection_bind_generic_parameters` didn't make it clear whether NULL was a valid return value or not.  Now when we return NULL we always set the `error` out-arg. So non-NULL is success and NULL is failure.

[reflection] MonoError for mono_marshal_spec_from_builder

[reflection] Mark mono_reflection_get_custom_attrs_blob external only.

The runtime should use mono_reflection_get_custom_attrs_blob_checked.
The icall is now ves_icall_System_Reflection_Emit_CustomAttributeBuilder_GetBlob.

[reflection] MonoError for resolve_object

[reflection] MonoError for ensure_complete_type

[reflection] MonoError for mono_reflection_get_dynamic_overrides

[reflection] MonoError for mono_reflection_method_get_handle

[SRE] Don't set failiure flag in ensure_generic_class_runtime_vtable

Leave it up to the caller to set the flag.

[SRE] Don't set class failure in fix_partial_generic class

Leave it to callers to set it based on the MonoError.

Also return a boolean indicating whether we succeeded or failed.

[reflection] MonoError for fix_partial_generic_class

[SRE] Use MonoError for mono_reflection_bind_generic_parameters

This commit also changes the behavior of
mono_reflection_bind_generic_parameters slightly: if the passed in class
isn't generic, it sets error and returns NULL.  (It wasn't clearly
previously whether NULL was an error condition or not - with this commit
we clarify that a NULL result is an error).

[reflection] Use MonoError in encode_{cattr_value,named_val}

[reflection] MonoError in get_{prop,field}_name_and_type

[reflection] MonoError in parameters_to_signature

And a few other static fns

[gacutil] Add support for pdb file

[build] Fix more incomplete mcs references names

[mcs] Allow local variables and parameters on lhs of nameof expression. Fixes #39455

[mono] Remove al2/dmcs from TEST_SUPPORT_FILES too

[mono] Remove dmcs from temp install folder

[mono] Remove al2 from temp install folder

[acceptance-tests] Bump ms-test-suite to pick up 4a95604fdf2c0523e68eaad64a17d22eebb051b3:

[runtime] Cleanup the debug info module before the images module since ppdb debug info depends on images. Fixes #39715.

[ppdb] Avoid an assert if the ppdb image has an empty guid heap, it can happen if the assembly has no types thus no document guids. Fixes #39644.

[arm] Avoid storing fp arguments below the stack pointer if the param area has size 0. Fixes #38161.

Bump referencesource

Merge pull request #2766 from lambdageek/dev/monoerror-jit

[runtime] Propagate error from class loading when JIT-ing

[acceptance-test] Ensure the exit code of the main test script is used

Due to our usage of chained commands it'd use the exit code of the last command,
which is not what we want. We use chaining because e.g. in the ms-test-suite
case we still want to run the systemruntimebringup suite even if the conformance
suite had a failing test.

[runtime] Propagate error from class loading when JIT-ing

This is another approach to a341404ecdd3b5ca2ed0ab1e9a5bcb9b5ccd2566
which was backed out because it was
stale (2a899492c30740f6785883fae14bb1ddacbc8b77).

[bcl] Remove wrong IVT attributes

Merge pull request #2765 from akoeplinger/mcs-fix-test-windows-backslash

[compiler-tester] Fix incorrect failure with Windows paths in error messages

Revert "Merge pull request #2454 from tastywheattasteslikechicken/FixVtableAbort"

This reverts commit a341404ecdd3b5ca2ed0ab1e9a5bcb9b5ccd2566, reversing
changes made to 04a177b2be47ec1de995b3abb8cd378a15590647.

[build] Add missing explicit references in docs

[corlib] Fix FileInfo.ToString after MoveTo

Fixes #38796

[corlib] Test FileInfo.ToString after MoveTo

Covers #38796

Merge pull request #2454 from tastywheattasteslikechicken/FixVtableAbort

[runtime] Fixes an abort caused by a TypeLoadException in certain ins…

[corlib] RuntimeEnvironment from referencesource

[tests] Don't use AppDomain.AppendPrivateBinPath for assembly-load-remap.exe

It's not supposed to contain a full path to a directory, but rather a relative path
to a subdirectory of the ApplicationBase. However, this isn't enforced in all cases
by Mono so setting a full path worked when the path doesn't contain a '.' because then
the code in [1] doesn't clear the variable and the path is added to the domain->search_path.

This caused the test to fail on Wrench on the mono-4.4.0 lane because the path contains
a dot while confusingly it worked fine on master.

Fixing this by appending the xbuild_14 path via an alternative (albeit slightly hacky)
method related to the standard MONO_PATH env var. Because the variable is only read
once at app startup we need to call into the runtime to explicitly set the new path
before trying to load the MSBuild 14 assemblies. This is fine since we only
want to test that the correct assembly version is loaded and nothing else.

[1] https://github.com/mono/mono/blob/37b2b9fbc25a2199aba1d794117924d4828360a7/mono/metadata/appdomain.c#L1319-L1337

[runtime] Fixes an abort caused by a TypeLoadException in certain instances

[System]: Fix certificate validation on Linux.  Bug #39307.

[System]: Properly propagate exceptions in 'MonoTlsStream'.

(cherry picked from commit a7ab399d093f4aac3f24b862e6075ab9f4fdb9e1)

[System]: Copy 'ServicePoint.Certificate' and 'ClientCertificate' from the referencesource.

(cherry picked from commit 76e0f64862732c3fb4f5647d5a2687ad9557be45)

[System]: In 'WebConnection.EndWrite()': always call EndWrite() on the underlying socket.

(cherry picked from commit dd9d66a7c28ba77bf5ef7e4d7f6cb60ca3feb5a9)

[System.Net.Http] Make HttpClientTest.GetString_Many test more reliable

It's flaky on Jenkins, removing the dependency on an external server and instead using a local HttpListener
to answer the requests should hopefully make it more reliable.

[mcs] Partially revert c9d4cc55a175d1cfc1f6d8f13e92606ef09dada1

[mcs] Don't sign sqlmetal and linker tools

[compiler-tester] Expand members error reporting to avoid problems with members called {}

Merge pull request #2747 from lambdageek/dev/monoerror-mono_remote_class

[runtime] Add MonoError to mono_remote_class

[tools] Don't install cil-stringreplacer

It's a tool that is only used during the Mono class lib build.

[getline] Fix condition where Console.WriteLi<TAB><ENTER> completed to Console.WriteLinene

[repl] Further fine tuning;   Show newlines as actual newlines in strings, plus fix quoting of chars < 32

[repl] Do not escape space when pretty printing strings

[build] Use full filename for csc references

[compiler-tester] Remove moonlight build

[monodoc] Remove redundant codepage argument

[System.Xml.Linq] Simplify dependencies

[Mono.Security.Providers.NewSystemSource] Use LIB_REFS to track references

[system.runtime.serialization] Use correct IVT key for System.ServiceModel.Web

Merge pull request #2543 from ermshiperete/Xamarin-31021

Don't ignore drives with type "aufs" or "overlay" (Xamarin-31021)

Merge pull request #2757 from akoeplinger/fix-coreclr-test

Two coreclr test fixes

Fix the build.

[corlib] Fix Delegate.Combine() to throw on non-matching types

This was inadvertently removed in https://github.com/mono/mono/commit/611a43ee0f672adbac3e25dc77731843a3d10cf1 from MulticastDelegate.CombineImpl().

Add the type check back in the Delegate base class, we already do the same in Remove().

Add unit tests to cover the behavior to the corlib test suite. (reverted from commit f29878dc713f4e148fe0a1e703fae8929e6fb6e0)

Needed to update some monodoc tests to match that an exception is now thrown again.

Merge pull request #2755 from xmcclure/disable-omit-fp

"disable_omit_fp" option for MONO_DEBUG

Merge pull request #2750 from radical/tasklogger

[xbuild] TaskLoggingHelper: Implement support for correctly using tas…

[xbuild] TaskLoggingHelper: Implement support for correctly using task resources

- make task resources available to TaskLoggingHelper
- verify TaskLoggingHelper method args
- tests

[jit] Remove some loader error usage.

[corlib] Fix UIntPtr.ToString() 64bit implementation

Cast to ulong instead of uint, this is also what referencesource does:
https://github.com/Microsoft/referencesource/blob/d925d870f3cb3f6acdb14e71522ece7054e2233b/mscorlib/system/uintptr.cs#L114-L122

Also bump coreclr submodule to bring in another UIntPtr fix.

[mcs] Better check for automatically implemented property inside a type with an explicit StructLayout. Fixes #39574

[system.net.http] The backslash character may be used as a single-character quoting mechanism within quoted-string. Fixes #39569

Fix the build.

Merge pull request #2754 from markusbeth/fix-mono-config

fixed replacement of $mono_libdir in config file

strncpy does not NUL-terminate the destination string in this case so one cannot use strcat to append

This fixes the error "malloc: top chunk is corrupt" as reported on mono-devel-list back in 2015-10-20 (subject: "malloc error executing OBS-built mono") at least for me.

Merge pull request #2756 from vargaz/llvm-jit-3.9

[llvm] Finish JIT support for llvm 3.9.

[llvm] Finish JIT support for llvm 3.9.

[mono-config] fix copying of prefix during replacement

[acceptance-tests] Bump coreclr and ms-test-suite to bring in some fixes

Also disable a test that makes no sense.

Merge pull request #2752 from kumpera/the_loader_error_highlander

This removed the last source of loader error from the runtime

The last source was on the AOT compiler in load_image.

I hooked enough MonoError to bubble the issue to a reasonably higher layer.

@vargaz please review.

fixed more srtncpy bugs

Merge pull request #2740 from lambdageek/dev/monoerror-reflection-icalls1

[reflection] MonoError for a few icalls

Merge pull request #2739 from lambdageek/dev/monoerror-type_load_class

[MonoError] Use class name for mono_error_set_type_load_class() exceptions