Merge pull request #2538 from ludovic-henry/fix-sgen-38012

[sgen] Fix register scanning on ARM

This bug would trigger a use after sweep in System.Threading.Tasks.Task.FinishContinuations:3621.

The issue would arise on ARM, as `MonoContext` and `ARCH_NUM_REGS` are defined as followed :

```
typedef struct {
  mgreg_t pc;
  mgreg_t regs [16];
  double fregs [16];
  mgreg_t cpsr;
} MonoContext;

\#define ARCH_NUM_REGS 14
```

As you can see, the MonoContext structure is bigger than 14 words, and it does not even covers the last 3 values of `regs`. By using pointer arithmetic, we ensure that we do not miss some parts of the `MonoContext` structure.

The observed behaviour in System.Threading.Tasks.Task.FinishContinuations would be that `continuationObject` would not be marked, and thus freed; `continuationObject` would still contains the pointer to the old location, meaning it wouldn't be null. We would then try to check its type via a call to the `as` operator. This would call the `isinst` IL opcode which would load `continuationObject->vtable->klass->supertypes [...]`, which would trigger a segfault, as `supertypes` would be null.

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

Merge pull request #2541 from ludovic-henry/fix-coop-hang

[coop] Fix hang in ves_icall_System_Net_Dns_GetHostByName_internal

[binary-reference-assemblies] Install 4.5 api facades

[coop] Fix hang in ves_icall_System_Net_Dns_GetHostByName_internal

[llvm] Make mini-llvm-loader.c simpler by putting all the function pointers into a structure.

[amd64] Add support for float arguments/return values to the dyncall code.

Merge pull request #2547 from ermshiperete/Xamarin-33183

[SWF] Fix behavior when scrollbar values change in TreeView

[llvm] Remove the goto based error handling code, use explicit error checks and returns instead.

[msvc] Update order.xml and csproj files

Merge pull request #2546 from ermshiperete/libtoolize

Check for libtoolize rather than libtool

[mcs] Stop after fields initializers resolve errors

[runtime] Disable the constant-logical-operand warning.

[runtime] Make the MONO_CURRENT_ABI_ALIGNOF macros c++ safe.

[jit] Remove unused MONO_PATCH_INFO_MONITOR_ENTER/EXIT.

Merge pull request #2544 from ermshiperete/Xamarin-24641

[corlib] Fix handling of ignorable chars when using ordinal comparison

Check for libtoolize rather than libtool

Current Debian packaging puts libtool in its own package (libtool-bin)
leaving only libtoolize in the main package. The mono build process
uses libtoolize not libtool, so during autogen check for libtoolize
not libtool.

[corlib] Fix handling of ignorable chars when using ordinal comparison

* recognizes ignorable characters when CompareOptions.Ordinal is
  specified
* adds unit tests
* fixes bug 24641

Change-Id: I5102e7392fa54b5671c7f2536cb8778f847e9040

[SWF] Fix behavior when scrollbar values change in TreeView

* only call EndEdit in scrollbar value changed handlers when currently
  editing a node
* fix for bug 33183

Change-Id: I54407e00314f4bfbdc175e883199d3d1ef057911

[llvmonly] Fix stack trace calculation.

[amd64] Exclude the GSHAREDVT tests as well since they don't work yet.

[jit] Disable the DYNCALL tests on amd64 again, not all cases are supported right now.

[jit] Use mono_method_get_full_name () to print out method names in a few places.

[llvmonly] Fix the fallback to no_gsharedvt_in_wrapper, which was broken by the error handling changes.

[llvmonly] Catch all exceptions in mono_jit_exec () to avoid terminating the runtime with an unhelpful libc++ assertion.

[mcs] Rework naming convention of lifted state machine fields to include scope index. Fixes #32315 (1st part)

[sgen] Fix register scanning on ARM

This bug would trigger a use after sweep in System.Threading.Tasks.Task.FinishContinuations:3621.

The issue would arise on ARM, as `MonoContext` and `ARCH_NUM_REGS` are defined as followed :

```
typedef struct {
  mgreg_t pc;
  mgreg_t regs [16];
  double fregs [16];
  mgreg_t cpsr;
} MonoContext;

\#define ARCH_NUM_REGS 14
```

As you can see, the MonoContext structure is bigger than 14 words, and it does not even covers the last 3 values of `regs`. By using pointer arithmetic, we ensure that we do not miss some parts of the `MonoContext` structure.

The observed behaviour in System.Threading.Tasks.Task.FinishContinuations would be that `continuationObject` would not be marked, and thus freed; `continuationObject` would still contains the pointer to the old location, meaning it wouldn't be null. We would then try to check its type via a call to the `as` operator. This would call the `isinst` IL opcode which would load `continuationObject->vtable->klass->supertypes [...]`, which would trigger a segfault, as `supertypes` would be null.

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

[sgen] Abort with a message on failure to initialize binary protocol

[corlib] Improve CancellationTokenSource test

Get rid of the Thread.Sleep calls that are prone to timing issues.

Merge pull request #2522 from ludovic-henry/test-socketresponder-task

[SocketResponder] Use Task instead of Thread + Use explicit delegate so it appears in stacktraces

[System.ServiceModel.Internals] Add a missing .sources file for watchOS. Fixes #38145.

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

Merge pull request #2448 from BrzVlad/feature-cprop-opt

[jit] Improve strength reduction handling within cprop

Merge pull request #2531 from esdrubal/systemweb

[System.Web] HeadersCollection read only exception

[llvmonly] Do a stack walk from mono_llvmonly_get_calling_assembly (), the runtime might support stack walks on some platforms.

[runtime] Avoid an assert if a runtime compiled without DISABLE_COM is used with an mscorlib from the mobile_static profile.

[llvmonly] Remove bind-to-runtime-version from the aot flag used to compile the assemblies, it makes testing harder.

[llvmonly] Allow any value for MONO_LLVMONLY not just TRUE.

Merge pull request #2533 from vargaz/gshared-errors

[jit] Add MonoError support for the generic sharing code.

Merge pull request #2529 from lambdageek/monoerror-location

[runtime] mono_error_assert_ok() include location

[jit] Add MonoError support for the generic sharing code.

[jit] Move the raise version of mono_jit_compile_method () to metadata/.

Merge pull request #2525 from kumpera/embedding-api

[api] Add API kill switch.

Merge pull request #2528 from vargaz/jit-errors2

Jit errors2

Merge pull request #2419 from esdrubal/xmlsubclassel

[System.Xml] Fix sub class serialization on lists

[System.Web] HeadersCollection read only exception

In reference source HttpHeaderCollection [1] when Add, Set or Remove is
called and the collection is readonly a PlatformNotSupportedException is
thrown.

Mono HeadersCollection was throwing NotSupportedException and
Microsoft.Owin.Host.SystemWeb.OwinCallContext.RemoveAcceptEncoding was
not catching the exception because it was expecting
PlatformNotSupportedException.

Fixes #33809

[1] http://referencesource.microsoft.com/#System.Web/HttpHeaderCollection.cs,73

[System.Web] Test HttpRequest.Headers Add/Set/Remove

In .NET while calling Add, Set or Remove of a HttpRequest read only
collection a PlatformNotSupportedException is thrown.

Covers #33809

[runtime] mono_error_assert_ok() include location

[metadata] Fix duplicate variable declaration in coree.c that breaks msvc

A new 'error' variable was added in d7908572783b1a8f3b6f4b4685385a7becf0bab3 that clashes with the existing one.

[mono-api-html] New interface members are only breaking if the interface existed.

[mono-api-html] Fix compiler warning.

[mono-api-html] Make it possible to hide/show non-breaking changes in the html output.

We now add data-is-[non-]breaking attributes to the generated
html elements, and then use some javascript logic to show/hide
non-breaking elements (and the javascript also looks in the
container elements show/hide containers that have only
non-breaking descendants).

Also use css classes/styles to colorize.

[mono-api-info] Add an is-override attribute to determine if a method is overriding another method.

[xbuild] Use -api folder for 2.0/3.5/4.0 profiles as well

Similar to e2a79ed88800493cb0444445a176d9c7febbfabe. Those profiles are only contract/reference assemblies since Mono 4.0.

Merge branch 'mono-api-info-improvements'

[mono-api-info] CreateAttributeMapping: don't create dictionaries unless there is something to put in them.

[mono-api-info] Rework AttributeData to use a static method instead of creating an instance every time.

[mono-api-info] Improve CleanupTypeName to not create 3 strings every time somethings needs to change.

[mono-api-info] First filter, then order, and use ordinal comparison.

[mono-api-info] Rework SkipAttribute to use ordinal string comparison, and no string comparison unless needed.

[mono-api-info] Use ordinal string comparison. It's much faster.

[mono-api-info] Remove unused method.

[mono-api-info] Use XmlWriter instead of XmlDocument to make this faster.

Generating all the API references (in parallel) for Xamarin.iOS goes from
5.5s to 4.5s.

[xbuild] Set .net 4.5 target to 4.5 restricted api

[system] Add DeliveryFormat to SmtpSection. Fixes #37891

[mono-api-info] Add support for writing output to a file specified using a command line argument.

[mcs] Implements null operator for dynamic expressions. Fixes #37801, #37824

[SocketResponder] Use Task instead of Thread

This allow us catching and rethrowing the exception on the main thread, as well as making sure we wait for the Listen thread to finish in a timely manner.

[SocketResponder] Start in constructor and Stop on Dispose

In conjuction with `using` we ensure that we Start and Stop the SocketResponder correctly, and everytime

[Tests] Use explicit delegate so it appears in stacktraces

In case of exception in this callback, the name of the test method will appear in the stacktrace, making debugging way easier.

Merge pull request #2497 from ludovic-henry/monoerror-mono_gsharedvt_constrained_call

[runtime] Use MonoError for mono_gsharedvt_constrained_call

[System] Move HttpListenerResponse.GetStatusDescription to ease linking

The use of HttpResponseMessage.ReasonPhrase means:

* marking the HttpListenerResponse.GetStatusDescription method;
* marking the HttpListenerResponse type;
* which implements IDisposable, so it's Dispose method is marked
* marking the Close method...

from there a metric ton of code gets recursively marked to the point
where the whole SSL/TLS server code gets inside the app.

To avoid this we move the GetStatusDescription to a different
type that won't bring the end of the civilized world so fast.

[runtime] Use MonoError in mono_jit_runtime_invoke () to return errors, the 'exc' parameter is still used to return the possible exception from the invoke call.

[System.Web] Fix another wrong test assembly name

Didn't catch that one in 180e745ccee4a80c9641a2e70235387e74b9eddb as it hides itself pretty well :)

Merge pull request #2527 from vargaz/jit-errors

[jit] Convert some of the error handling to use cfg->error instead of…

[jit] Use mono_error_set_pending_exception () to throw exceptions from the trampoline code. This only works if the generic trampoline code checks for exceptions after the call to the C function.

[jit] Use MonoError in mono_jit_compile_method () and the trampoline code.

[api] GCC is much better at enforcing MONO_RT_EXTERNAL_ONLY.

Merge pull request #2526 from alexanderkyte/llvmonly_mobile_static_fixups

[mobile_static] Requested NunitLite comment and llvmonly configuration changes

[jit] Pass a MonoError* instead of a MonoException** to mono_jit_compile_method_inner ().

[runtime] Initialize the error fields in mono_error_set_exception_instance ().

[bcl] Fix dll names and InternalsVisibleTo for test assemblies

The format of the test assembly name changed in 860334ff45a8d1c5886b8dbcfadaf28ac55b9393
from System.Web_test_net_4_x.dll to net_4_x_System.Web_test.dll.

This fixes all the places where the old name was used.

[jit] Reduce the usage of the loader error more by converting it to a MonoError where it was detected.

Merge pull request #2523 from akoeplinger/fix-profiler-test-reporting

[profiler] Fix test reporting success even though a test failed

[jit] Convert some of the error handling to use cfg->error instead of cfg->exception_type.

Use S390_SET to get addresses rather than the previous more convoluted method

[jit] Add a new jit icall mono_interruption_checkpoint_from_trampoline () and use that instead of mono_thread_force_interruption_checkpoint () so the latter can go away eventually.

[mobile_static] Don't use gsharedvt with llvmonly

[nunitlite] Documented console .config usage

Merge pull request #2432 from alexanderkyte/compile_corlib_with_aot

[aot/bcl] Compile the BCL with AOT

[aot/bcl] Add Makefile configuration to build the bcl with aot, use it in mobile_static.

Merge pull request #2431 from alexanderkyte/tests_with_excludes

[mcs] Fix .sources/.excludes for tests, exclude corlib monotouch-excluded tests on mobile_static

[api] Mark external Only: mono_get_method, mono_get_method_full and mono_get_method_constrained.

[build] Add MONO_RT_EXTERNAL_ONLY macro to the public API.

This macro should be used to decorate functions that are part of the public API but the runtime is
not supposed to call.

This is to allow us to keep them around but not go crazy with commit reintroducing their usage.

[profiler] Fix test reporting success even though a test failed

I noticed this in https://jenkins.mono-project.com/job/test-mono-pull-request-armhf/1715/ which reported success in the "profiler" test step,
even though the step actually reported errors:

```
MONO_PATH=/media/ssd/jenkins/workspace/test-mono-pull-request-armhf/mcs/class/lib/net_4_x perl ./ptestrunner.pl ../..
Checking test-alloc.exe with report ...
Checking test-busy.exe with report ...
Checking test-busy.exe with report,sample ...
Error: Missing thread Finalizer.
Error: Missing thread Main.
Error: Wrong loaded images 2.
Error: Not enough compiled method.
Error: Not enough compiled code.
Error: Missing thread BusyHelper.
Total errors: 6
Checking test-monitor.exe with report ...
Checking test-excleave.exe with report ...
Checking test-heapshot.exe with report,heapshot ...
Checking test-heapshot.exe with heapshot,output=-traces.mlpd ...
Checking test-traces.exe with output=-traces.mlpd ...
Checking test-traces.exe with nocalls,output=-traces.mlpd ...
make[1]: Leaving directory `/media/ssd/jenkins/workspace/test-mono-pull-request-armhf/mono/profiler'
make: Leaving directory `/media/ssd/jenkins/workspace/test-mono-pull-request-armhf/mono/profiler'
++ bc
+++ date +%s
++ echo 1453874768 - 1453874741
+ echo -e '*** end(27): profiler: \e[42mPassed\e[0m'
*** end(27): profiler: Passed
```

The culprit is that ptestrunner.pl checks whether $total_errors is non-zero to determin the process exit code.
This is wrong, as $total_errors is cleared before each test so unless the last test fails the exit code would always be zero.

Introduced a new $global_errors variable which is only incremented and never cleared to fix this.

Revert "[runtime] Use explicit loop checking to remove depth limitation of sequence point search."

This reverts commit 23ce21ede0d26d2815c670a11e6d870506b91dd0.

Revert this as it is non-linear and can take a lot of time for complicated cfg structures.

Merge pull request #2520 from kumpera/loader-error-cleanup5

[mono-error] Another round of loader-error

Merge pull request #2521 from akoeplinger/fix-sockettest-endconnect-error

[System] Fix error in new BeginConnectToIPV4EndPointUsingDualModelSocket test

It was added with 78c758494a04132c0965f8af82dd7370da5f4af1. The test fails on Jenkins:

```
Test Case Failures:
1) MonoTests.System.Net.Sockets.SocketTest.BeginConnectToIPV4EndPointUsingDualModelSocket : System.InvalidOperationException : EndConnect can only be called once per asynchronous operation
at System.Net.Sockets.Socket.ValidateEndIAsyncResult (IAsyncResult ares, System.String methodName, System.String argName) [0x0004a] in /var/lib/jenkins/workspace/test-mono-mainline/label/debian-amd64/mcs/class/System/System.Net.Sockets/Socket.cs:3412
at System.Net.Sockets.Socket.EndConnect (IAsyncResult result) [0x00006] in /var/lib/jenkins/workspace/test-mono-mainline/label/debian-amd64/mcs/class/System/System.Net.Sockets/Socket.cs:1584
at MonoTests.System.Net.Sockets.SocketTest.BeginConnectToIPV4EndPointUsingDualModelSocket () [0x0005d] in /var/lib/jenkins/workspace/test-mono-mainline/label/debian-amd64/mcs/class/System/Test/System.Net.Sockets/SocketTest.cs:4365
at (wrapper managed-to-native) System.Reflection.MonoMethod:InternalInvoke (System.Reflection.MonoMethod,object,object[],System.Exception&)
at System.Reflection.MonoMethod.Invoke (System.Object obj, BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00038] in /var/lib/jenkins/workspace/test-mono-mainline/label/debian-amd64/mcs/class/corlib/System.Reflection/MonoMethod.cs:295
```

The reason is that BCCallback already calls EndConnect() so we need to remove it and just wait for the callback in the test itself.

@monojenkins merge

[mcs] Reset null operator barrier in parenthesis expression

Fix serialization under XM 4.5 Target Framework

- System.Configuration.ConfigurationErrorsException: Failed to load configuration section for dataContractSerializer
- Fixes #36401 #38064

[System] Fix error in new BeginConnectToIPV4EndPointUsingDualModelSocket test

It was added with 78c758494a04132c0965f8af82dd7370da5f4af1. The test fails on Jenkins:

```
Test Case Failures:
1) MonoTests.System.Net.Sockets.SocketTest.BeginConnectToIPV4EndPointUsingDualModelSocket : System.InvalidOperationException : EndConnect can only be called once per asynchronous operation
at System.Net.Sockets.Socket.ValidateEndIAsyncResult (IAsyncResult ares, System.String methodName, System.String argName) [0x0004a] in /var/lib/jenkins/workspace/test-mono-mainline/label/debian-amd64/mcs/class/System/System.Net.Sockets/Socket.cs:3412
at System.Net.Sockets.Socket.EndConnect (IAsyncResult result) [0x00006] in /var/lib/jenkins/workspace/test-mono-mainline/label/debian-amd64/mcs/class/System/System.Net.Sockets/Socket.cs:1584
at MonoTests.System.Net.Sockets.SocketTest.BeginConnectToIPV4EndPointUsingDualModelSocket () [0x0005d] in /var/lib/jenkins/workspace/test-mono-mainline/label/debian-amd64/mcs/class/System/Test/System.Net.Sockets/SocketTest.cs:4365
at (wrapper managed-to-native) System.Reflection.MonoMethod:InternalInvoke (System.Reflection.MonoMethod,object,object[],System.Exception&)
at System.Reflection.MonoMethod.Invoke (System.Object obj, BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00038] in /var/lib/jenkins/workspace/test-mono-mainline/label/debian-amd64/mcs/class/corlib/System.Reflection/MonoMethod.cs:295
```

The reason is that BCCallback already calls EndConnect() so we need to remove it and just wait for the callback in the test itself.

[System.Runtime.Caching] Bump timeout in tests to make them more reliable

Failed on Jenkins on ARM today:

```
Tests run: 38, Failures: 1, Not run: 0, Time: 32.132 seconds

Test Case Failures:
1) MonoTests.System.Runtime.Caching.MemoryCacheTest.TimedExpiration :   #A4-4
  Expected: not null
  But was:  null

at MonoTests.System.Runtime.Caching.MemoryCacheTest.TimedExpiration () [0x002b6] in /media/ssd/jenkins/workspace/test-mono-mainline/label/debian-armel/mcs/class/System.Runtime.Caching/Test/System.Runtime.Caching/MemoryCacheTest.cs:902
at (wrapper managed-to-native) System.Reflection.MonoMethod:InternalInvoke (System.Reflection.MonoMethod,object,object[],System.Exception&)
at System.Reflection.MonoMethod.Invoke (System.Object obj, BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00038] in /media/ssd/jenkins/workspace/test-mono-mainline/label/debian-armel/mcs/class/corlib/System.Reflection/MonoMethod.cs:295
```

100ms could be too short, bumping the timeout should make it more reliable on the slower ARM machines.

Merge pull request #2420 from stefansedich/master

Handle automatically mapping IPv4 addresses to IPV6 for dual mode sockets, fixes #36192