Do not remap framework assembly if it's version is higher than the runtime version

Currently, assemblies like Microsoft.Build.Framework and
Microsoft.Build.Engine get remapped to the current runtime version, at
load time. This means that if an assembly references the above
assemblies with a version like `14.1.0.0`, then that would get remapped
to `4.0.0.0`. This is incorrect behavior and breaks msbuild. Instead,
now we do the remapping only if the requested version is lower than the
runtime version.

@akoplinger: This will actually impact xbuild too right now because
we're defaulting to xbuild 14.0 in Mono 4.4 now so we'd see messages
like Mono: The request to load the assembly Microsoft.Build.Framework
v14.0.0.0 was remapped to v4.0.0.0 in the assembly loading log, which
will break in very subtle ways.

Merge pull request #2707 from lambdageek/dev/monoerror-mono_image_load_module_dynamic

[sre] Propagate MonoError in mono_image_load_module_dynamic

[mcs] Fixes codegen for lifted null bitwise operators with constant left side of null value

Merge pull request #2708 from lambdageek/dev/monoerror-assorted-get_type

[reflection] MonoError for mono_reflection_{get_type_with_rootimage,type_from_name}

Merge pull request #2668 from lambdageek/dev/monoerror-mono_security

[runtime] MonoError cleanup in mono-security.c

Some easy MonoError cleanup in mono-security.c: use set_pending_exception, don't raise in helper functions, free memory on errors.

Merge pull request #2670 from lambdageek/dev/monoerror-mono_runtime_object_init

[runtime] Use MonoError in mono_runtime_object_init

Mark mono_runtime_object_init checked external only.  Runtime should use mono_runtime_object_init_checked.

Merge pull request #2687 from lambdageek/dev/monoerror-mono_param_get_objects_internal

[reflection] Use MonoError in mono_param_get_objects

[mcs] Null conditional operator is never suppose to work with await according to current C# standard. Fixes #39266

[reflection] Fix mono_reflection_type_get_handle when
DISABLE_REFLECTION_EMIT is set

Fixup of 20a111a215c2ad93ab77068d2e81e21c886b9846

[mcs] Remove test as it cannot reliably depend on process shutdown

Merge pull request #2727 from akoeplinger/fix-system-testports

[System] Fix more hardcoded test ports

A test failed on Jenkins today with "address already in use".
Opportunistically replaced a few more hardcoded test ports to avoid failures in the future.

@monojenkins merge

[System/ServiceModel] Fix more hardcoded test ports

A test failed on Jenkins today with "address already in use".
Opportunistically replaced a few more hardcoded test ports to avoid failures in the future.

[mcs] All returns with values from async block with finally need to be captured. Fixes #39153

[corlib] Enable the shim __ComObject for other profiles than mobile_static too

Otherwise they crash in a similar way during verification tests.

[System] Switch DNS tests to a more stable domain/IP

info.diku.dk stopped resolving today. Use Google's public DNS server instead which should be much more stable.

[runtime] Compile arch before utils when INSTALL_MONOTOUCH is enabled

We'd run into an error before:

```
In file included from ../../mono/utils/mono-context.h:234:0,
                 from ../../mono/utils/mono-stack-unwinding.h:10,
                 from ../../mono/utils/mono-threads.h:14,
                 from mono-mmap.c:36:
../../mono/arch/arm/arm-codegen.h:936:41: fatal error: mono/arch/arm/arm_dpimacros.h: No such file or directory
```

This was done for the other cases in 637ee7cbaa03a1f576230eb8601c0c4fa7f93b36, but not when the monotouch
special case was added in cf754cdb634f2bdd5bc959cec66145372c301fba.

Merge pull request #2714 from lambdageek/dev/monoerror-mono_reflection_type_get_handle

[reflection] MonoError for mono_reflection_type_get_handle

`mono_reflection_type_get_handle` is really a kind of normalization procedure that synthesizes real `MonoType`s from reflection's derived types such as `System.Reflection.Emit.ArrayType, ByRefType`, etc.  So we call it all over the place in System.Reflection.Emit runtime code.

That would all be fine, except that users are allowed to subclass `System.Type`, so we have to call the `UnderlyingSystemType()` method to get a real type.  Which means that an override can throw.  Which in coop mode means we have to thread around a `MonoError`.  So that's why this code is uglier.

But really this code is doing a lot of type inspection and high-level work that we should figure out how to migrate to managed.

[bcl] Ensure all .sources files have newline at the end

It causes an issue with gensources.sh when using #include on Linux otherwise.

[reflection] MonoError for mono_reflection_type_get_handle

Merge pull request #2704 from BrzVlad/fix-monitor-abort

[runtime] Fix monitor abort

Fix filename casing in mobile_System.Data.dll.sources

It broke on Linux.

Merge pull request #2722 from ludovic-henry/fix-39279-docker-cpu-count

[utils] Get processor count from scheduler affinity if available

In case the user has set a scheduler CPU affinity, we should use it as the number of available CPU for the runtime. This is particularly important on docket which uses cgroups that set the container a cpu affinity, and by not using that, the runtime report the total number of CPU the machine has, and not the number of CPU the container has.

Fixes bug https://bugzilla.xamarin.com/show_bug.cgi?id=39279

Merge pull request #2700 from akoeplinger/monotouch-mobile-static

Enable building of monotouch/xammac profiles without mono-extensions and expose mobile_static in configure.ac

Relies on https://github.com/xamarin/mono-extensions/pull/17

This enables us to compile the monotouch, xammac and mobile_static profiles on public Jenkins without mono-extensions.

Merge pull request #2723 from akoeplinger/fix-decimal-test-culture

[corlib] Fix three decimal tests to not rely on current culture for formatting

They'd fail on cultures where "." is not the decimal separator before.

I've also reenabled them on mobile cause the workaround for #27269 is no longer necessary since all bots are now on Mono 4.0+.

@monojenkins merge

Merge pull request #2719 from kumpera/aot_cctor_encoding_in_method_info

[runtime] Explicitly encode the need of running a cctor on method infos.

Add a shim __ComObject class to the mobile_static build

We need this because when the runtime is built without --enable-minimal=com the runtime crashes
when trying to AOT because it tries to find the __ComObject (even though nothing in mobile_static
actually uses COM). With this shim we can AOT the mobile_static assemblies even with a "normal"
runtime.

[corlib] Fix three decimal tests to not rely on current culture for formatting

They'd fail on cultures where "." is not the decimal separator before.

I've also reenabled them on mobile cause the workaround for #27269 is no longer necessary since all bots are now on Mono 4.0+.

[corlib] Fixes build to using correct build profile dependency

[utils] Get processor count from scheduler affinity if available

In case the user has set a scheduler CPU affinity, we should use it as the number of available CPU for the runtime. This is particularly important on docket which uses cgroups that set the container a cpu affinity, and by not using that, the runtime report the total number of CPU the machine has, and not the number of CPU the container has.

Fixes bug https://bugzilla.xamarin.com/show_bug.cgi?id=39279

[tools] Remove unused comment

[lc] Remove Option.cs local copy

[bcl] Redo referencesource resource string integration to not depend on mcs

[corlib] Remove duplicate file entry

[runtime] Explicitly encode the need of running a cctor on method infos.

This removes the case where decode_klass_ref returning NULL would be ok.

[aot] Rename init_llvm_method () to init_method () since it is used for all methods, not just llvm ones.

[acceptance-tests] Work around an issue that caused Roslyn tests to stop working

xplat MSBuild uses a wrong, to be removed regex that doesn't catch our recent profile names. Temporarily work around it for now.

Merge pull request #2716 from BrzVlad/fix-tramp-jinfo

[aot] Set null uw_info for tramps without unwind info

Merge pull request #2268 from kumpera/coreclr-testsuite-fixes-1

First batch of bug fixes found in the CoreClr test suite.

Merge pull request #2710 from akoeplinger/fix-typo

[jit] Fix duplicated left-hand side in comparison

[jit] Fix target_type_is_incompatible to type check byref enums correctly.

Merge pull request #2709 from lewurm/jit-phase-typo

[jit] remove 'jit_' prefix in two phases in stats output

@monojenkins merge

Merge pull request #2645 from alexrp/profiler-stability

Fix a crash in the MONO_LLS_FOREACH_SAFE macro

[jit] Fix duplicated left-hand side in comparison

The typo introduced by 1a784475c04a7b371e53346bee032590f4f94853 caused the if to always be false,
because it would compare like if (true == MONO_EXCEPTION_FIELD_ACCESS).

[jit] remove 'jit_' prefix in two phases in stats output

[sgen] Use regular, non-safe LLS foreach macro.

As it turns out, all these uses of FOREACH_THREAD_SAFE were inside regions
where the suspend lock is held. This effectively meant that no modifications
to the thread list could actually happen while the foreach was executing. Since
the guarantees provided by FOREACH_THREAD_SAFE are not needed in any of these
cases, switch to FOREACH_THREAD. The latter macro also happens to be much more
lightweight, so this should result in an overall (small) performance gain.

[utils/lls] Make the node free function support more flexible.

The caller of mono_lls_init () can now specify whether the free function can or
cannot lock, rather than the LLS assuming that it can't. The only reason this
limitation existed was because the LLS wasn't extended to allow specifying this
information in/after 56a6ae2bd0ac3b9ebf8ad9daec6aca484b7e487b.

In addition, all LLS functions are now callable from both safe and async
context; the caller must specify the context as an additional argument. If a
free function is in use and it may lock, we'll simply enqueue it for execution
at a later point if an LLS function is invoked in async context.

[utils/hp] Improve the mono_thread_hazardous_free_or_queue () API.

1. The function suffered from the "too many bools" problem. Pass enum values
   instead so that call sites are clearer.
2. Don't restrict the combination of arguments. This makes it easier to use
   the function as part of data structures where the arguments aren't known
   ahead of time (e.g. the LLS).

[utils/lls] Fix the safe foreach macro.

This fixes a bug that would manifest when running the profiler on a program
that rapidly spawns/joins threads.

The problem was this: Given T1 which is joining on T2, T2 which is returning
and removing itself from the thread list, and T3 which is currently in the
SIGPROF signal handler, it was possible for T3 to encounter a pointer to T2's
freed thread info structure in the thread list. This can happen because the
SIGPROF handler doesn't actually acquire the suspend lock (it can't) while
iterating the thread list.

The MONO_LLS_FOREACH_SAFE macro was supposed to protect against this case
specifically and, as a result, only provide a snapshot of the thread list.
However, it was missing some very important bits of the mono_lls_find ()
algorithm. The gory details can be found in the paper that the LLS is based on:

    http://www.research.ibm.com/people/m/michael/spaa-2002.pdf

In particular, refer to the "Using Other Memory Management Methods" section
on page 6. It describes why lines E1-2 and E3-4 are necessary in order to
safely iterate the list in an implementation using hazard pointers (such as
ours). Basically, the iteration of the list has to be restarted if any sort of
modification to the list is detected during iteration. Otherwise, it's possible
to read and follow removed/freed memory, leading to all sorts of Bad Things™
happening.

Now, obviously we can't just restart a foreach loop without further thought;
that would cause all sorts of breakage due to the loop body being executed on
the same elements multiple times. As it happens, the LLS is an ordered data
structure: Elements are arranged in ascending order by key. We can exploit
this fact when iterating the list and restarting the loop. By simply storing
the last key we successfully processed, when restarting the loop, we can just
skip all keys that are not greater than the last key we successfully processed.

In addition to this, the MONO_LLS_FOREACH_SAFE macro now assists in freeing
removed elements that it encounters, just like mono_lls_find ().

The end result is that MONO_LLS_FOREACH_SAFE is now truly safe to use at any
point. It does have the potential to be ever so slightly slower, however, since
it does do significantly more work than it used to, and might even have to
restart the loop multiple times if lots of threads are modifying the thread
list at the same time. This doesn't seem to be a problem in practice, though.

Finally, to reduce the maintenance burden, the non-filtered versions of the
macros are now implemented in terms of the filtered versions, using a static
inline filter function that always returns true. When using the non-filtered
macros, the compiler will just eliminate the check entirely through its usual
optimization passes.

Merge pull request #2145 from dlech/patch-2

Strip extra null characters in socketAddress

[reflection] Use MonoError in mono_reflection_type_from_name

Mark it external only.  Runtime should use mono_reflection_type_from_name_checked.

[reflection] Use MonoError in mono_reflection_get_type_with_rootimage

[sre] Propagate MonoError in mono_image_load_module_dynamic

Merge pull request #2699 from pruiz/integrate-syswebrouting-master

Completely replace System.Web.Routing with implementation from reference source

[ilasm] When generating method signatures for method-defs, make sure they contain the full call conv. Fixes b35784.

When generating varargs signatures for method defs, ilasm would discard the rest of the call conv. Things
like instance or explicit this would be discarded.

[mini] Weaken type check for byrefs to allow narrowing stores of primitive types. Fixes b59858.

CoreClr allows storing a wider primitive byref into a narrower one. int32& -> int16&, for example.

This is a safe weakening as there's no risk of overflows, undefined behavior or unaligned access.

[jit] Using a literal field with stdsfld/ldsfld should trigger a MissingFieldException. Fixes b13452.

[runtime] Fail Void arrays with Invalid Program exception instead of a TLE. Fixes b353858.

[mini] ckfinite throws OverflowException and not ArithmeticException. Fixes JIT/Regression/VS-ia64-JIT/M00/b108366.

[jit] Try blocks can end at the last instruction of the method. Fixes DevDiv2_11321.

Ass oddly as it sounds, it's valid to emit the catch handler before the try block
and have it be the last sequence of instructions in the method body.

This patch checks for that condition before looking the BB in cil_offset_to_bb.

Merge pull request #2684 from ludovic-henry/monoerror-exception

[runtime] Use MonoError for mono_get_exception_type_initialization, mono_get_exception_reflection_type_load and mono_get_exception_runtime_wrapped

Merge pull request #2696 from lambdageek/dev/unlock-mono_class_create_runtime_vtable

[runtime] Don't raise exception while holding locks

[PEAPI] Fixed encoded token sizes for HasCustomAttribute, CustomAttributeType and MemberRefParent.

This would cause some large il files to be miscompiled as the token size would not be the expected one.

This was found in b28158/test.il.

[runtime] Null check the pointer argument of Interlocked::Exchange<T>. Fixes exchangetneg.

[msvc] Update .def files to fix Windows build

[runtime] Use MonoError for mono_get_exception_type_initialization, mono_get_exception_reflection_type_load and mono_get_exception_runtime_wrapped

[runtime] Obey requests to abort when waiting on monitors

If we are clearing the interruption_requested flag, we also need to throw the corresponding exception, otherwise we miss it.

Fixes #39042

[runtime] Don't return from Monitor.Wait without holding the lock

Even if we are interrupted we must throw the exception only after we have regained the lock. This prevents throwing an SynchronizationLockException in the finally clause of common scenarios like :

lock (obj) {
Monitor.Wait (obj);
}

The downside of this is that, if another thread doesn't give up the lock, we could fail to abort the thread. But this is what MS does.

[runtime] Remove unreachable code

A monitor enter internal method returns -1 for interrupted, 0 for timed out, 1 for acquired. When doing a monitor wait, we regain the lock using an infinite wait (which can't time out) and retrying in the case of interruption.

[runtime] Fix LOCK_DEBUG build, for monitor debugging

[aot] Set null uw_info for tramps without unwind info

We don't register jit infos for tramps that don't have unwind info because we can't unwind from them. Explicitly set uw_info to NULL, instead of pointing to a zero length array in the aot image, to not confuse the runtime when registering the tramp.

Likely to fix #38267

[runtime] Move some mono_error_raise_exception () calls to icalls from utility functions. NFC.

[runtime] Rename some icalls to match the icall naming convention. NFC.

Merge pull request #2705 from kumpera/loader-error-cleanup7

[runtime] Remove the loader error from the remaining of the loader.

Replaced System.Web.Routing with referencesources own implementation.

[misc] Multiple small fixes and cleanups.

[utils] Add mono_error_move.

[runtime] Ban mono_method_get_header and promote mono_method_get_header_check to the API as the profiler needs it.

Merge pull request #2662 from mattleibow/pcl-mono.options

[Mono.Options] Changed the code a bit so that it works with PCL

Fix marshaling of file name in Mono.Unix.UnixEndPoint.Create()

In some cases, the null terminator in the SocketAddress may not be the last
character. This changes the algorithm to ignore the null terminator and
anything after it instead of assuming that the last character is the null
terminator.

Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=35004

Add unit tests for Mono.Unix.UnixEndpoint and Mono.Unix.UnixListener

Tests for regression of https://bugzilla.xamarin.com/show_bug.cgi?id=35004

[runtime] Remove all usages of mono_method_get_header from the runtime.

[runtime] Replace mono_method_get_header with mono_method_get_header_checked.

[runtime] Use mono_method_get_header_checked in debug-helpers.c

[gc] Use the new MonoMethodBuilder:init_locals field.

[runtime] Add init_locals field to MonoMethodBuilder to make it easy to get it set to false.

[runtime] Add mono_method_get_header_checked and implement mono_method_get_header using it.

[runtime] Ban mono_method_get_signature_full. It has no runtime usage left.

[runtime] Ban mono_method_get_signature. Nothing else uses it.

[runtime] Ban mono_class_inflate_generic_type from usage by the runtime. Move code to use mono_class_inflate_generic_type_checked.

[runtime] Remove mono_class_inflate_generic_class, replace usage with mono_class_inflate_generic_class_checked.

[runtime] Add mono error to inflate_generic_header. Fix buglet in mono_method_get_signature_checked.

[runtime] Ban mono_metadata_parse_mh from the runtime.

[runtime] Introduce MonoError in mono_metadata_parse_mh_full.

[runtime] Remove dead function mono_metadata_parse_type_full.

[runtime] Disallow usage of mono_metadata_parse_type, mono_metadata_parse_param and mono_metadata_parse_field_type.

[mono-error] mono_metadata_parse_array is not longer used by the runtime, ban it.

[mono-error] mono_class_from_name and mono_class_from_name_case now can cleanly return NULL on failure.

Merge pull request #2686 from ludovic-henry/monoerror-mono_remoting_invoke

[runtime] Use MonoError for mono_remoting_invoke

Automatically define MONO_DISABLE_GSHAREDVT variable when gsharedvt isn't enabled

Before when the variable wasn't set (= the default), the mobile_static profile would try to pass
the -O=gsharedvt flag even when gsharedvt is not enabled (e.g. when building without mono-extensions)
which causes AOTing to fail.

A better solution is to export the variable from configure.ac into the mcs build system depending
on whether gsharedvt is enabled at configure time.

Merge pull request #2701 from akoeplinger/fix-async-read-test

Fix unreliability in async_read.exe test