Merge pull request #2236 from akoeplinger/add-dataflow

[System.Threading.Tasks.Dataflow] Replace implementation with CoreFx version

[Mono.Security]: Add 'X509Chain' to ICertificateValidator.InvokeSystemValidator and MonoTlsProvider.InvokeSystemCertificateValidator.

On OS X and Mobile, the X509Chain will be initialized with all the
certificates from the X509CertificateCollection, but not actually
built.

ServicePointManager.ServerCertificateValidationCallback can use the 'chain' argument
to get all the certificates from the server.

(cherry picked from commit 73e37f0ecfd403780d60f36d8b0d68f27987b256)

[Mono.Security]: The default value of 'MonoTlsSettings.UseServicePointManagerCallback' is supposed to be true.

This fixes a regression where ServicePointManager.ServerCertificateValidationCallback
was not called when using WebClient.

(cherry picked from commit 5022db1956cd7dbde46928473800ff5b5e6b651e)

[threads] Fix build

[mcs] Set null operator barrier between invocation instance and its arguments. Fixes #33341

[arm] Avoid an assertion check in aot mode.

[runtime] Add MONO_ABI_SIZEOF macros similar to the existing MONO_ABI_ALIGNOF macros, use these to compute sizes of types.

Merge pull request #2240 from UCIS/patch-6

Fixed hang in SslStreamBase when stream is closed during I/O

[sgen] Remove sgen specific suspend code

Merge pull request #2241 from f00f/patch-1

Fixed typo

Fixed typo

Fixed hang in SslStreamBase when stream is closed during I/O

If the SslStreamBase is disposed while a read or write operation is in progress, the asynchronous I/O state is never marked as completed, resulting in synchronous Read/Write calls never returning. To fix this, the internal I/O callbacks should mark the asynchronous operations as completed, possibly with an error, even if the SslStreamBase is disposed.

Merge pull request #2201 from rolfbjarne/linker-better-xml-processing-errors

[linker] Provide better error message when an error occurs while processing xml descriptions

Remove link to Wrench from README.md

We're retiring public Wrench and replacing it with Jenkins, see http://lists.ximian.com/pipermail/mono-devel-list/2015-November/043399.html

Add monotouch_*_runtime profiles for watchOS and tvOS.

Ignore extensions-config.h

Fix the build.

Remove obsolete interpreter embedding sample.

[jit] Remove unused mono_aot_register_globals () function.

[jit] Remove the nursery_bits from the wrapper info for the gc write barrier methods, its no longer needed after 85581ff1bb8b28141ec1823b7371efa5a14551bb.

[jit] Add a header file for aot-compiler.c.

[jit] Fix the check for emulation of div instructions in 9f253965d02d20b7fd39549ba07b35e76a69e48f.

Merge pull request #2239 from xmcclure/interbals2

Minor: Style cleanup on internal headers (continued)

Minor: Style cleanup on internal headers (continued)

Previous patch missed some build files.

Merge pull request #2238 from xmcclure/interbals

Minor: Style cleanup on internal headers

Minor: Style cleanup on internal headers

For consistency, rename any -internal.h files to -internals.h. Also remove INTERBALS

Updated Assert.Equals to Assert.AreEqual

[System.Threading.Tasks.Dataflow] Replace implementation with CoreFx version

We were seeing some random test failures on Jenkins with our Dataflow implementation.
Replacing it with Microsoft's CoreFx version fixed those and makes us more compatible.

While we'd ideally not ship this assembly at all with Mono (it doesn't ship with .NET),
we shipped it in the past and as such people might rely on it so we can't remove it.

The CoreFx commit this version was taken is 905a1940bcda0afdca2f14ceb2b0161ebc4d1d02.

[arm] HACK: Disable fast TLS on ARM for now.

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

[linker] Always preserve Interlocked.Exchange<T> ().

The runtime expects to be able to find the generic variant of this method in
the metadata even when code is calling the `object` overload.

Merge pull request #2228 from lambdageek/dev/sgen-timeouts

[test] sgen tests should behave nicer on slower machines.

[arm] Simplify the seq point code used for soft breakpoints.

[Windows.Forms] add a warning to the Carbon driver when running on 64 bits

[runtime] Fix Thread.CurrentThread in non-root appdomains by setting the tls slot in start_wrapper, otherwise Thread.CurrentThread would create a new Thread object so there would be two. Fixes #35828.

Fix a warning.

[amd64] Make amd64_patch non-inline, it seems to be miscompiled by older clang versions (5.1).

[amd64] Enable div checks when running on tvos.

[jit] Emit div checks if MONO_ARCH_NEED_DIV_CHECK is enabled even when div instructions are not emulated.

Merge pull request #2233 from madewokherd/ucontextgregs

Do not define UCONTEXT_GREGS on win64.

Revert "[jit] Emit div checks if MONO_ARCH_NEED_DIV_CHECK is enabled even when div instructions are not emulated. Enable div checks for tvos."

This reverts commit 276824dd28c4b16ccd9ce9b71ec22cb59cce9bc3.

Revert this as it breaks on some android devices:
https://bugzilla.xamarin.com/show_bug.cgi?id=35844

[test] Give the sgen tests more time.

[test] Make sgen tests behave nicer on slower machines.

  * TestDriver.cs Add `TestTimeout` utility to check how much time CI
       allocated for the test.
  * sgen-domain-unload-2.cs print warning if running low on time.
  * sgen-new-threads-dont-join-stw.cs print progress dots and a warning
       when running low on time.

[Mono.Security]: Cleanup the internal `Mono.Security.Interface.ICertificateValidator'.

* ICertificateValidator.SelectClientCertificate(): return bool, indicating whether
  a certificate has been selected or the caller should use its default
  implementation (if it has any).

* ICertificateValidator.ValidateCertificate(): new common function for both client
  and server certificates; replaces ValidateChain() and ValidateClientCertificate().

* MonoTlsProvider.InvokeSystemCertificateValidator(): return bool, indicating whether
  the certificate has been validator or the default system validator should be used.

(cherry picked from commit 42421eec1e91cbcb09a531a77df296da9e5e9f5d)

Do not define UCONTEXT_GREGS on win64.

Merge pull request #2227 from esdrubal/tzparse

[corlib] Fixes TimeZoneInfo.ParseTZBuffer abbrevs.

[corlib] Fixes TimeZoneInfo.ParseTZBuffer abbrevs.

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

In a few android devices TimeZoneInfo.ParseTZBuffer was failing ito
parse Europe/Moscow with:
at System.ThrowHelper.ThrowKeyNotFoundException ()
at System.Collections.Generic.Dictionary`2[System.Int32,System.String].get_Item (Int32 key)mscorlib/system/collections/generic/dictionary.cs:176
at System.TimeZoneInfo.ParseTimesTypes (System.Byte[] buffer, Int32 index, Int32 count, System.Collections.Generic.Dictionary`2 abbreviations) [0x0002f] in mcs/class/corlib/System/TimeZoneInfo.cs:1293
at System.TimeZoneInfo.ParseTZBuffer (System.String id, System.Byte[] buffer, Int32 length)

It was failing while getting the third index [3] from] from abbrevs.

abbrevs added values were the following:
[0, MMT]
[1, MT]
[2, T]
[4, MST]
[5, ST]
[6, T]
[8, MDST]
[9, DST]
[10, ST]
[11, T]
[13, S]
[15, M]
[17, MSK]
[18, SK]
[19, K]
[21, EET]
[22, ET]
[23, T]
[25, MSD]
[26, SD]
[27, D]
[29, EEST]
[30, EST]
[31, ST]
[32, T]

Index 3 was not added because its values is an empty string, as indexes
with empty values are used in this case we are now adding them to the
abbrevs dictionary.

Fixes #31432

[corlib] Tests TimeZoneInfo.ParseTZBuffer

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

Tests TimeZoneInfo.ParseTZBuffer with Europe/Moscow data that thrown
at System.ThrowHelper.ThrowKeyNotFoundException ()
at System.Collections.Generic.Dictionary`2[System.Int32,System.String].get_Item (Int32 key)mscorlib/system/collections/generic/dictionary.cs:176
at System.TimeZoneInfo.ParseTimesTypes (System.Byte[] buffer, Int32 index, Int32 count, System.Collections.Generic.Dictionary`2 abbreviations) [0x0002f] in mcs/class/corlib/System/TimeZoneInfo.cs:1293
at System.TimeZoneInfo.ParseTZBuffer (System.String id, System.Byte[] buffer, Int32 length)

[IO] Dump handle when we run out of them

When running out of handle, it's hard to know exactly why. Dumping them
will help troubleshoot handle leaks.

Bump referencesource

[jit] Emit div checks if MONO_ARCH_NEED_DIV_CHECK is enabled even when div instructions are not emulated. Enable div checks for tvos.

[arm] Fix register assignments during hfa parameter passing. Fixes pinvoke_ppdcd.exe.

[runtime] Rename iltests.il.in back to iltests.il, architecture specific tests should be handled using Category attributes.

[linker] do not try to iterate thru null array

 - fixes the case when custom attribute instance constructor contains
   null value for System.Type[] array

Rewrite the test case from commit b503b87a to not use Assert.IsInstanceOfType().

[Facades] Fix tarball (for real this time)

Turns out EXTRA_DISTFILES only works with library.make or executable.make,
so we need to just use DISTFILES here...

[Facades] Fix tarball by adding a new file to EXTRA_DISTFILES

It was added in 29a02f2b12cee2e7b0cf10b3ae863b7ea1c588c6.

Merge pull request #2224 from Gankov/patch-1

Fixed a typo in Mono.Cairo

Fixed a typo

[runtime] Remove a bogus assertion.

[jit] Use emit_runtime_constant () in a few more places to reduce code duplication between the JIT and mono_resolve_patch_target ().

Merge pull request #2213 from kumpera/widen-unsafe-mov

[jit] Add support for widening of Array.UnsafeMov intrisic. Fixes #35310

[jit] Allow reference types with Array.UnsafeMov. Document restrictrions in Array.cs.

[runtime] Use 'this_obj' instead of 'this' in a few more places.

[jit] Add support for widening of Array.UnsafeMov intrisic. Fixes #35310

This enables usage of JitHelper.UnsafeEnumCast<> with enums smaller than int.

This changes enables the intrinsic for int conversions of same reg class.

[acceptance-tests] Split compiling/cleaning test sources and bump coreclr

The coreclr-compile-tests would throw a shell error "Argument list too long" on OSX due
to the huge amount of tests passed to make in one go. The same happened for make clean.

We now compile and clean the three sets of coreclr test sources separately to avoid this.

Additionally, bump the coreclr reference and include a new test that was added upstream.

Merge pull request #2222 from akoeplinger/dataflow-test-reliability

[Dataflow] Fix flaky tests by bumping timeouts

[Dataflow] Fix flaky tests by bumping timeouts

We were seeing random Dataflow tests fail on Jenkins from time to time.
I managed to reproduce the BroadcastBlockTest.CloningTest and WriteOnceBlockTest.CloningTest
fail on my machine by generating CPU load and running those test in a while(true).

Bumping the timeouts from 100ms -> 1s fixed the flakiness for me, so I went ahead and
opportunistically bumped all such small timeouts throughout the testsuite.

Merge pull request #2207 from lambdageek/dev/bug-35375

[reflection] Fix for  35375 - explicit interface implementation using a generic type instance

Merge pull request #2198 from BrzVlad/feature-concurrent-work

[sgen] Fix memory use of the concurrent collector

Merge pull request #2217 from rcruzs00/master

El Capitan fix : system integrity protection compatibility

[runtime] mono_class_init() - don't look for metadata if the dynamic image doesn't have it.

Crashing test in mono_class_init() from a MonoGenericClass.

[#35375](https://bugzilla.xamarin.com/show_bug.cgi?id=35375)

El Capitan fix : system integrity protection compatibility

Merge pull request #2216 from akoeplinger/fix-array-sort

[corlib] Fix Array.Sort throwing when fewer keys than items are provided

Merge pull request #2215 from kumpera/crash_in_mono_class_get_method_from_name_flags

[runtime] Fix bad assert in mono_class_get_method_from_name_flags.

[runtime] Fix a few more class->klass cases.

[runtime] Use 'klass' instead of 'class' to make the code more c++ friendly.

[corlib] Fix Array.Sort throwing when fewer keys than items are provided

The `Sort<TKey, TValue> (TKey [] keys, TValue [] items, IComparer<TKey> comparer)` overload in Array throwed when keys.Length != items.Length.
However, on .NET you can pass in a keys array with fewer elements than in items and it'd sort the elements in items up to that point.

Fixed our implementation by only throwing when keys.Length > items.Length.

This was revealed by the coreclr/tests/src/CoreMangLib/cti/system/array/arraysort12.exe test.
I decided to port the test into our testsuite so we catch regressions earlier.

[Mono.Security]: Add destructor to MonoSslStream.

(cherry picked from commit f158270d867bee65b8980479b84124f9ef1a0f61)

Merge pull request #2124 from mhutch/mono-api-info-forwarders

[mono-api-info] Allow following type forwarders

[runtime] Fixed typo.

[Mono.Security]: 'Mono.Security.Interface.MonoSslStream' needs to implement IDisposable.

(cherry picked from commit 4c1760785a60b01f0be41b312c791fae23699ba4)

[Mono.Security]: Make MonoTlsProviderFactory.CreateHttpListener() take 'X509Certificate' instead of 'X509Certificate2'.

(cherry picked from commit 3d81daf796c00e3a37bdb20aacbaaa3b5769466d)

[runtime] Fix bad assert in mono_class_get_method_from_name_flags.

Merge pull request #2209 from esdrubal/facades_subdirs

[Facades] Moved subdirs list to subdirs.make

[corlib] Disable FullNameGetTypeParseEscapeRoundtrip test on Android.

This requires the symbol writer which is not available on Android.

[acceptance-tests] Output Roslyn and ms-test-suite NUNit xml result to current dir

Roslyn used xunit-style xml by default, we hack the cibuild.sh script to output nunit-style
for now until we get an option upstream.

This allows us to catch those test results in Jenkins.

[mcs] Handle using aliases in evaluator. Fixes #32894

[mcs] Don't store invalid using static expression in evaluator queue. Fixes #35604

[System.Web] WebColorConverter from reference source

Merge pull request #2214 from kumpera/fix_enum_get_get_hashcode

[runtime] Fix Enum.GetHashCode when basetype is short or sbyte.

[Mono.Posix] Update incorrect test

[mcs] inherited interface constraints checks on defined interfaces only. Fixes #35535

[mcs] Restore local variables state after probing run. Fixes #35674

Merge pull request #2210 from ystr/FromHtml-xxx

ColorTranslator.FromHtml 3-digit codes support (#35664)

[bcl] Add versioning to the TestTasks.dll file name used by the Microsoft.Build.Engine tests since it needs to be compiled against the same xbuild version as the rest of the tests.

[runtime] Fix Enum.GetHashCode when basetype is short or sbyte.

In addition to fixing the runtime, fix the test suite to correctly detect this problem.

This popped up because, for some reason, those two types have hashcodes that
don't return identity on reference sources.

[sgen] We only use workers when starting the concurrent collection

[sgen] Optimize the finishing of the concurrent collection

If we force the finish of the concurrent collection, instead of waiting for the concurrent mark to finish followed by the finishing mark&sweep, we immediately stop the worker thread, change its object operations and drain the remaining gray stack, also following references to the nursery. This way we avoid the cardtable overhead in cases where there are a lot of references from heap to the nursery.

[jit] Refactor loading of runtime constants from IL