Merge pull request #2732 from radical/asm_remap

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

…ntime 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.

Merge pull request #2741 from akoeplinger/acceptance-tests-split

A few improvements to acceptance-tests

Merge pull request #2743 from qmfrederik/fixes/complete-eventsource-3

System.Diagnostics.Tracing: Add EventListener and related classes

[xbuild] Remove explicit warning level set when csproj does not have it and leave the default to compiler

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.

Bump ikdasm

System.Diagnostics.Tracing: Add EventListener and related classes

Merge pull request #2742 from mono/revert-2738-fixes/complete-eventsource-2

Revert "Add EventListener and EventListenerEventArgs to Mono"

Revert "Add EventListener and EventListenerEventArgs to Mono"

[acceptance-tests] Better description in readme

Merge pull request #2738 from qmfrederik/fixes/complete-eventsource-2

Add EventListener and EventListenerEventArgs to Mono

[acceptance-tests] Stop symlinking System.Console.dll facade

It should no longer be required after 6e81edfa241bedbb3971ca1e7aece82f31de54c1.

[acceptance-tests] Split the test suites out of Makefile.am into own files

[mcs] nameof operator can use static member. Fixes #39455

[llvm] Add beginnings of JIT support for LLVM > 3.9.

[acceptance-tests] Move git "submodules" to acceptance-tests/external instead of next to Mono repo

A few people told me it's surprising that acceptance-tests checks out repos into <mono repo>/../
and it can cause unwanted changes when a repo with the same name already exists there.

While this structure is used in several internal repos I agree for Mono it's better to keep
everything inside the repo itself.

[aot] Fix a buffer overflow in the statistics code.

[runtime] Remove some unused code.

Remove #if NET_4_6

Remove conditional compilation statement as per PR feedback

Merge pull request #2736 from xmcclure/more-trampolines

Increase default number of aot-compile trampolines

The current defaults are not enough to successfully run
`make fullaotcheck` in mono/mini

Merge pull request #2735 from xmcclure/early-lookup-addr

Early ICALL_ADDR lookup on pinvoke to fix checked-build assert

The pinvoke implementation in the failure case was creating a managed
exception object while in blocking mode, a rules violation. This patch
puts a redundant lookup early so that any exceptions occur before
blocking mode is entered. (The "real" lookup is unchanged because the
current pattern is optimized well by the JIT.)

Merge pull request #2731 from Phyyl/master

Remove throw statement in IPAddress.TryParse

In Microsoft's implementation, no exceptions are thrown in this particular method. See here for details: http://referencesource.microsoft.com/#System/net/System/Net/IPAddress.cs,54a3a3f536d5a22c

Merge pull request #2674 from lambdageek/dev/monoerror-mono_field_get_value_object

Use MonoError for mono_field_get_value_object

Mark it external only.  Runtime should use mono_field_get_value_object_checked.

Increase default number of aot-compile trampolines

The current defaults are not enough to successfully run
`make fullaotcheck` in mono/mini

Merge pull request #2718 from lambdageek/dev/monoerror-sre

[System.Reflection.Emit] Use MonoError in static functions

Fix an assert in checked-build GC mode

The pinvoke implementation in the failure case was creating a managed
exception object while in blocking mode, a rules violation. This patch
puts a redundant lookup early so that any exceptions occur before
blocking mode is entered. (The "real" lookup is unchanged because the
current pattern is optimized well by the JIT.)

[runtime] Fix the support for coop in managed-native wrappers by finishing blocking inside the try clause before converting the result, and only unblocking in the finally clause if we didn't unblock in the try clause.

Added null check in IPAddress.Parse
This check was previously handled in the IPAddress.TryParse method but, but the latter never throws anymore

Merge branch 'master'

[SRE] MonoError in mono_image_emit_manifest

[SRE] MonoError in mono_image_fill_export_table

[SRE] MonoError in mono_image_get_array_token

[SRE] MonoError in mono_image_get_sighelper_token

[SRE] MonoError in mono_image_get_generic_field_token

[SRE] MonoError in add_custom_modifiers

[SRE] MonoError in create_generic_typespec

[SRE] MonoError in mono_image_get_field_on_inst_token

[SRE] MonoError in encode_constraints

[SRE] MonoError in mono_image_get_event_info

[SRE] MonoError in property_encode_signature

[SRE] MonoError in mono_image_get_field_info

[SRE] MonoError in encode_marshal_blob

[SRE] MonoError in field_encode_signature

[SRE] MonoError in method_encode_clauses

[SRE] MonoError in encode_locals

[SRE] MonoError in encode_reflection_type and encode_custom_modifiers

Merge pull request #2706 from ludovic-henry/fix-unhandledexception

[runtime] Set exit code on unhandled exception even if a managed handler is present

In case we would add a AppDomain.CurrentDomain.UnhandledException callback, we would not exit the VM in case of unhandled exception.
This goes against the expected behaviour, as we should indeed call these managed callbacks in all cases, but they should not stop the VM from exiting.

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

Merge pull request #2634 from ludovic-henry/coop-fix-managed-to-native

[coop] Fix managed-to-native wrapper

[runtime] Use MonoError in mono_field_get_value_object

Mark it external only.  Runtime should use mono_field_get_value_object_checked.

Embedded API. Fix method signature match failure for generic types with multiple type parameters.

Details are on the list.
http://mono.1490590.n4.nabble.com/Re-Embedded-API-ctor-method-signature-query-mono-td4667551.html

Robert Jordan said:

It looks like a bug in mono_method_desc_new ():

https://github.com/mono/mono/blob/master/mono/metadata/debug-helpers.c#L378

The function is treating

.ctor(System.Func`2<string, string>)

like a method with 2 arguments:

arg0 = System.Func`2<string
arg1 = string>

This is obviously wrong :)

The function is then storing the (wrong) argument count
for optimization purposes, and the comparison of methods
is starting to fail.

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

[sre] Propagate MonoError in mono_image_load_module_dynamic

Updated TryParse_IpString_Null for IPAddress tests

Now reflects the new behaviour in IPAddress class, not throwing any exceptions in the TryParse method

Remove throw statement in IPAddress.TryParse

In Microsoft's implementation, no exceptions are thrown in this particular method. See here for details: http://referencesource.microsoft.com/#System/net/System/Net/IPAddress.cs,54a3a3f536d5a22c

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

[tests] Add unhandled exception tests with AppDomain.UnhandledException managed handler

[runtime] Set exit code on unhandled exception even if a managed handler is present

In case we would add a AppDomain.CurrentDomain.UnhandledException callback, we would not exit the VM in case of unhandled exception.
This goes against the expected behaviour, as we should indeed call these managed callbacks in all cases, but they should not stop the VM from exiting.

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

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

[runtime] Wrap mono_prepare_blocking and mono_finish_blocking in try { ... } finally { ... } clause

This is to guarantee that even in the case an exception is thrown between both calls, we switch back and forth correctly from RUNNING to BLOCKING state.

[runtime] Fix the emission of CEE_MONO_VTADDR opcodes, they need to be emitted right after native calls.

[jit] Add a new ICALL_ADDR_CALL patch type which is only used on calls, since ICALL_ADDR might be used without a call.

[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