[mcs] Accept and ignore command line args supported by csc that we don't

support:

/analyzer
/appconfig
/baseaddress
/deterministic
/errorendlocation
/errorlog
/features
/link
/moduleassemblyname
/pathmap
/pdb
/preferreduilang
/publicsign
/reportanalyzer
/ruleset
/sqmsessionguid

[runtime] Make the appdomain-unload.exe test more verbose.

[aot] Reenable the emission of line number info using GAS .file/.loc directives, recent versions of clang seems to support it.

[jit] Fix the dwarf info structure when there is no line number table.

[jit] Remove the 'appending' support from the dwarf writer, its no longer used.

[jit] Get rid of the create_delegate_trampoline_data () helper function which only has one caller.

Merge pull request #2768 from lambdageek/dev/monoerror-cominterop

[com] MonoError-ize cominterop.c

Merge pull request #2663 from esdrubal/islocal

[System] Fixes HttpListenerRequest.IsLocal.

Create xammac_net_4_5 variant of System.Runtime.Remoting

Trigger a rebuild of Mono.Security

[Mono.Security]: MonoTlsProvider.InvokeSystemCertificateValidator() now takes 'ref X509Chain'.

[System]: Add private 'X500DistinguishedName' API to store the canonical encoding and use it for fast comparison.

[System]: SystemCertificateValidator.CheckUsage(): lazily convert to X509Certificate2 if needed.

[System]: Make 'X509Certificate2' and 'X509Chain' lazily-initialized.

Add new internal 'X509Certificate2Impl' and 'X509ChainImpl' classes similar to
'X509CertificateImpl' in corlib and only initialize them when actually needed.

[corlib]: Cleanup the internal 'X509CertificateImpl' API.

[ppdb] Disable the metadata verifier for ppdb tables for now. Fixes #39631.

[man] Remove documentation for the nonexistent MONO_ENABLE_SHM environment variable.

Merge pull request #2784 from mhutch/msbuild-fixes

Fix regressions in Microsoft.Build.dll

[cil-stringreplacer] Use Cecil as reference dependency

[cil-stringreplacer] Update debug symbol

Merge pull request #2783 from alexanderkyte/strdup_memory_leaks

[runtime] Fix memory leaks related to short-lived strings

Merge pull request #2775 from lambdageek/dev/monoerror-reflection-icalls

[SRE] MonoError-ize some icalls

* Cleanup some memory that would leak on failure; improve a few error messages
* Add MonoError to `mono_reflection_call_is_assignable_to` and `fieldbuilder_to_mono_class_field`.
* Update several SRE icalls to follow the pattern:
  ```c
    void
    the_icall (args... )
    {
       MonoError error;
       (void) the_real_work_function (args..., &error);
       mono_error_set_pending_exception (&error);
    }

    static gboolean
    the_real_work_function (args..., MonoError *error)
    {
       ... /* return TRUE on success, FALSE and set error on failure */ ...
    }
  ```
  Do this to:
        1. `mono_reflection_register_with_runtime`
        2. `mono_reflection_setup_internal_class`
        3. `mono_reflection_create_internal_class`
        4. `mono_reflection_generic_class_initialize`
        5. `mono_reflection_event_builder_get_event_info`
        6. `mono_reflection_initialize_generic_parameter`
        7. `mono_reflection_create_dynamic_method`

Merge pull request #2782 from madewokherd/process-sessionid

[System.Diagnostics] Return dummy value from Process.SessionId.

[System.Diagnostics] Return dummy value from Process.SessionId.

[sgen] Switch the default GC bridge to the new mode.

[Microsoft.Build] Shut down single-use build manager

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

[Microsoft.Build] Set IsBackground on builder thread

It should not prevent the process from terminating. Fixes
issue where the thread is not shut down if ProjectInstances
are built directly instead of via the build manager.

[Microsoft.Build] Fix exception disposing BuildManager

[runtime] Remove string memory leak related to basename

[runtime] Fix reassignment over string-pointing var without freeing

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.

[SRE] Refactor a few icalls

The icalls follow the pattern:
    void
    the_icall (args... )
    {
       MonoError error;
       (void) the_real_work_function (args..., &error);
       mono_error_set_pending_exception (&error);
    }

    static gboolean
    the_real_work_function (args..., MonoError *error)
    {
       ... /* return TRUE on success, FALSE and set error on failure */ ...
    }

Do this to:
1. mono_reflection_register_with_runtime
2. mono_reflection_setup_internal_class
3. mono_reflection_create_internal_class
4. mono_reflection_generic_class_initialize
5. mono_reflection_event_builder_get_event_info
6. mono_reflection_initialize_generic_parameter
7. mono_reflection_create_dynamic_method

[SRE] Some cleanups in mono_method_body_get_object_checked

Don't leak method header on failure.

[SRE] MonoError for fieldbuilder_to_mono_class_field

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

[reflection] MonoError for mono_reflection_call_is_assignable_to

[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

[com] Push MonoError through cominterop_get_ccw

[com] Push MonoError through cominterop_get_idispatch_for_object

[com] Push MonoError through cominterop_get_interface

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

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

[com] Change cominterop_raise_hr_exception to cominterop_set_hr_error

Don't throw the exception, just store the instance in a MonoError.

[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)