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

[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

[aot] Don't hardcode the nursery size in aot write barriers

It can be passed at runtime in the nursery-size option of MONO_GC_PARAMS environment variable.

Fixes #35414

[acceptance-tests] Ensure Roslyn is bootstrapping with Mono

It tried to use CoreCLR but that doesn't work on Jenkins since some of the dependencies are missing there.
We actually want to ensure it's always using Mono so we can catch errors on our side, so we need this
hack until we have an option in the script upstream.

Merge pull request #2200 from xmcclure/image-audit-oops

Fix --with-checked-build build on GCC/Linux

Fix to checked-build reference auditing

Remove an array idiom which both broke the GCC build (apparently Clang allows some things with variable-length arrays in "gnu99" mode which GCC does not) and wasn't right anyway (there was a typo which caused a read past end of buffer).

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

[Facades] Moved subdirs list to subdirs.make

The list of subdirs is useful for maccore build, it was moved to its own file so it can be included in other builds makefiles.

Revert "[bcl] Mark some System tests which depend on a working dns with InetAccess."

This reverts commit d644cb2ab300e02ba76165a13e7a51225bcb0e35.

Revert this, localhost needs to be resolvable.

[bcl] Mark some System tests which depend on a working dns with InetAccess.

[acceptance-tests] Fixes for Roslyn integration

We now use our own archive with the few additional pieces that MS added to their copy of
Mono used by Roslyn CI instead of downloading their archive.

Also bump coreclr to get a fix for the GC test that would fail on Jenkins in.

[runtime] Add a MonoAotMode entry for llvmonly.

[runtime] Add a dummy definition for mono_debug_personality (), aot compiled managed-to-native wrappers refer to it even if its not used.

Merge pull request #2102 from AdamBurgess/master

[System] Fix bug #31209 - HttpConnection infinite loop

Avoid setting mono_llvm_only when mono_aot_only is set.

[acceptance-tests] Bump coreclr to pick up a8cbbdddfe9923a7f99edf8870842ad373e66b5b

I also fixed the coreclr-gcstress target to check for the expected exit code of the testsuite.

[llvm] Use LLVMArgNormal for scalars passed by value instead of LLVMArgInIReg/LLVMArgInFPReg which is misleading since arguments can be passed on the stack too.

[llvm] Add a 'type' field to LLVMArgInfo, use it to simplify the code a bit.

[aot] Reduce the code duplication in the wrapper encoding/decoding code a bit.

[runtime] Make mono_marshal_get_wrapper_info ()/set_wrapper_info () use a WrapperInfo pointer instead of a generic one.

[runtime] Add WrapperInfos for a few wrappers which were missing it.

[runtime] Merge the 'AllocatorWrapperInfo' structure into the common WrapperInfo structure.

[Mono.Security]: Add API to let MonoTlsProvider provider a custom system certificate validator.

(cherry picked from commit 4364a6b752e7b6b3dbcfbeefebe8597c4c4fc347)

[Mono.Security]: Add ICertificateValidator.InvokeSystemValidator().

(cherry picked from commit 35f9501967d1d73a8f5e1f76d04353f01f464b04)

[System]: Cleanup SystemCertificateValidator.Evaluate() by passing it the settings instance.

(cherry picked from commit 6837c35527160304696aa3e046ab7c60322235c8)

[Mono.Security]: Add CertificateValidationHelper(MonoTlsProvider,MonoTlsSettings).

This is intended to be used by MonoTlsProvider implementations; the ChainValidationHelper
now knows about the provider (unless it's invoked directly from user code).

(cherry picked from commit 2dbf0a0ed101a4890690efd1a81f43496224075d)

[System/TLS]: Make 'SystemCertificateValidator' a static class with a single Evaluate() method.

(cherry picked from commit fbdb69cfffc8e167d86caee2ae0b25522f8d5fe8)

[System/TLS]: Use the official API in SslStream rather than hard-coding MonoDefaultTlsProvider.

(cherry picked from commit 0bcc31e32680d62c4d4b6a83fec30690a792419b)

[runtime] Save the nursery bits used to create wbarrier methods into the aot image, and check it at runtime.

Merge pull request #2197 from mono/feature-disable-specific-finalizers

[sgen] Allow disabling finalization of specific classes.

Merge pull request #2199 from mhutch/pcl46

[xbuild] Add PCL 4.6 targets

[acceptance-tests] Bump coreclr to pick up 65b122151adb6b5ef7c27d8e0c268ef479fbe8b4

+11 more passing tests. At this point I think there are no more false positives left and the rest are real bugs.

[runtime] Implement native-to-managed marshalling of byref delegate arguments. Fixes #35545.

[aot] Fix a crash introduced by 51c4e47f2 when the mscorlib aot image is missing.

Cosmetic; remove unused variable.

(cherry picked from commit a9afeddd80773abfe7400c77e0796acf6e5369c0)

Cleanup unused usings.

Mono.Net.Security/LegacySslStream.cs is the only file which should
reference the 'Mono.Security.Protocol.Tls' namespace.

(cherry picked from commit 43307ff08684e6889bc24976cb128216e82b7179)

[xbuild] Add PCL 4.6 targets

[sgen] Allow disabling finalization of specific classes.

[acceptance-tests] Sort source files and add a new MONO define when compiling the tests

This allows us to customize test behavior that is different on Mono on purpose.

Sorting the files provides a much nicer output in the test runner.

Bump CoreCLR to include changes from 20789373370fe8f72dde01ad7ffc0816f618197c

[Facades]: Remove 'System.IO.Compression'.

We do not need a 'System.IO.Compression' facade assembly because we
already have a regular assembly with the same name, which contains all
the classes from here - either as actual implementation or as type-forwarders.

(cherry picked from commit fff780579463094dc194b9a5fb0036f4afa7a3a2)

Merge pull request #2196 from alexischr/coop-gc_stackdata-problem

[coop] Make sure 'gc_stackdata' is aligned and properly sized.

[sdb] Add another fastpath to the invoke code to skip more of the single stepping code during invokes.