Merge pull request #4048 from kumpera/iface_casting_cleanup

[mini] Reorg casting code + optimize casting of magic array interfaces.

[w32handle] Rename `statuscode` parameter to `abandoned`

This parameter is only used to check if a mutex has been abandonned.

[corlib] Use Directory.GetFiles/GetDirectories implementation from referencesource (#4042)

We already had implementations of the Windows APIs like FindFirstFile/FindNextFile
in native code but we wrapped it with some convenience functions for our implementation.

The referencesource code works directly against those APIs so I added new icall functions
that are just used for passing data to/from native code.

Note that we have the same problem in the DirectoryInfo.GetFiles() implementation.
Right now that one still uses the old Mono implementation, switching it over to
the referencesource implementation will be done in a separate commit.

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

Merge pull request #4065 from akoeplinger/add-mcs-lane

[ci] Add job configuration for testing Mono with mcs as compiler

[io-layer] Remove DuplicateHandle

Fix typo in readme.md

Also format the options without braces like in the other examples.

Merge pull request #4045 from lambdageek/bug-47867

[sre] Ensure generic_context is created before instantiating a TypeBuilder

Merge pull request #4051 from Unity-Technologies/registry-bug-fixes

Fix a bug in registry API implementation where had wrong assumptions about possible registry size.

Fix code convention on changed code.

[ci] Add job configuration for testing Mono with mcs as compiler

To ensure it doesn't break now that we're using Roslyn by default.

[acceptance-tests] Use portable pdb for csc

Merge pull request #4033 from ntherning/no-stdcall-for-icalls-on-windows-32-bit

Use __cdecl rather than __stdcall for icalls on Windows 32-bit

[jit] Enable inlining inside gshared methods. (#4059)

Revert "Only allocate 1 empty string instance per AppDomain " (#4062)

[build] Don't use runtime flags on basic boostrap (they are too new)

Only allocate 1 empty string instance per AppDomain  (#4047)

* Reuse a single empty string instance

* Undo string empty change to mono_string_new_size_checked and add new embedding api for string empty instead

[ci] Bump timeout in ms-test-suite

Compiling the test exe's is taking longer now with roslyn.

[configure] Detect endianess and use mcs for big-endian systems.

Renames the flag to be more explicit on the intended purpose.  Also,
document the new configuration flag.

The default will probe the right compiler to use.

Bump bockbuild

[msvc] Update csproj files

[tests] Use CSC to compile gc-descriptors test (#4052)

Merge pull request #4050 from akoeplinger/profile-speedup

[ci] Only build the additional profiles on one arch instead of everywhere

[io-layer] Remove wait.h

[io-layer] Remove INFINITE

[io-layer] Extract WaitForSingleObjectEx, WaitForMultipleObjectsEx and SignalObjectAndWait

[reflection] Use mono_module_get_object_handle everywhere (#4010)

* [reflection] Refactor managed object caching

Split up all the mono_XXX_get_object_checked functions into a generic
cache check and a specific construction function that's called if
there's a cache miss.

* [reflection] Add mono_assembly_get_object_handle

* [reflection] Add mono_module_get_object_handle

* [reflection] Use mono_module_get_object_handle everywhere

instead of mono_module_get_object_checked.

Also implement mono_module_file_get_object_handle instead of mono_module_file_get_object_checked

Bump bockbuild

Bump corert submodule

[build] Add --with-mcs-compiler config option

[mcs] Fixes runtimemetadataversion option check

Fix a bug in registry API implementation where had wrong assumptions about possible registry size.

[build] Prepare roslyn 2.0 bootstrap dependencies

[build] Set monolite path also for Facades

Merge pull request #3748 from akoeplinger/fix-rng-monotouch

[corlib] Simplify RNGCryptoServiceProvider file name on MT and enable tests

[xbuild] Add workaround for pre-2.0 csc and convert full to portable debug mode

[ci] Only build the additional profiles on one arch instead of everywhere

The additional profiles like monotouch, monodroid etc. are normal
managed code anyway and are the same no matter where they're built
so we can save a lot of CI time by not building them e.g. on slower
ARM bots.

[build] Use custom nursery size and clean up csc runtime arguments

[referencesource] Fix a regression from 47a6f56439c9deab287be770c902d41e91480799

ResourceSets needs to be initialized on Mono.

[mini] Use the array fast path with array magic interfaces. Do the expensive check in the slow path.

[mini] Cleanup type casting a bit. Unify cast with with cache setup code.

[jit] Add counters for number of JITted method discarded as duplicates and the time spent JITting them. Remove opt set from -v output, it just adds noise.

Merge pull request #4041 from CyberShadow/pull-20161129-100901

ilasm: Add stub parsing for .data cil attribute

Add CoreRT submodule

[bcl] Clean up some warnings

[sre] Do we ever try to instantiate a generic instance? Assert that we don't.

[sre] Throw a TLE if a generic inst is used before the gtd is created.

This can happen if TypeBuilders are used to construct something like
  class S<T> {
    ...
  }
  class R {
    public S<int> SI;
  }

And then CreateType() is called for R and GetField("SI") is called on
  the result before CreateType() is called on the builder for S.

[sre] Make sure a generic definition TypeBuilder has its generic_container setup before we try to instantiate it.

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

[test] Generic typebuilder field in another typebuilder should work.

Even if the generic typebuilder hasn't had CreateType() called on it
yet.

Regression tests for https://bugzilla.xamarin.com/show_bug.cgi?id=47867

Revert "[build] Enable aot for Microsoft.CodeAnalysis.dll/Microsoft.CodeAnalysis.CSharp.dll, as the aot compiler generates smaller images now."

This reverts commit e186dcff2c2b6c6f32c72858425ba8b9d8b33efb.

[configure.ac] Rename `hybrid` runtime preset to `aot_preferred`

Merge pull request #4040 from ntherning/disable-symbolicate-tests-on-windows

Disable symbolicate tests on Windows

[System] Report test failure using Assert

[System] Update ClientWebSocket test to use Setup

ilasm: Add stub parsing for .data cil attribute

The "cil" attribute seems to indicate that the data should be placed
in the .text section, along with the code.

Like the tls attribute, the cil attribute is only implemented insofar
as to recognize it as a valid keyword that can occur in a .data
declaration.

Disable symbolicate tests on Windows

We will ignore the failing test-with-aot-msym test for now.

[bcl] Fix a regression introduced by 91c68de23f5067d000207194466813239cda7866. If a thread was added to (#4038)

blocking_threads twice, then removed once, the corresponding entry in threads_stacktraces was removed,
causing an exception in ReleaseHandle () later.

[runtime] Add error handling to mono_get_dbnull_object () it can fail due to a TAE. (#4039)

Merge pull request #4023 from lateralusX/jlorenss/test-driver-reporter

Adding support to get test results back from TestDriver.

[build] Enable aot for Microsoft.CodeAnalysis.dll/Microsoft.CodeAnalysis.CSharp.dll, as the aot compiler generates smaller images now.

[aot] Only emit instances when using full aot.

[runtime] Fix e659982b7b8f11ecf80919646353ad3c16240697, two callbacks were mixed up.

Prevent ThreadAbort from prematurely ending function calls in catch block (#1837)

* [runtime] Fix naming scheme for check if stack grows upward

It was very counter-intutitive to have a file-local preprocessor variable
with a name that sounds like it should have the same truthiness as a given global
preprocessor variable but was defined as the negation of the global variable.

* [runtime] Prevent ThreadAbort from terminating function calls in catch body

ThreadAbortExceptions are special in that if the thread has not
had it's abortion reset, when it reaches the end of a catch block it
must throw.

Consider the situation where a thread has been aborted and the
ThreadAbortException has been caught. If this catch block calls
a method that has a try/catch combination, the current implementation
will throw the ThreadAbortException at the end of the catch in that
function.

Usually you only care that Thread abortions are prompt, but this bug
could result in dramatic program behavior changes when this is combined
with lazy class initialization.

```
Example:
Console.WriteLine inside a thread abort with an uninitialized Console.
```

This fix records the stack pointer when a catch clause is entered, and
only rethrows the abortion if we're exiting a catch clause above or at the
level of the catch clause previously entered.

[aot] Bump aot file version after c690636c01df73badb1e24c146da8492dbeabade.

[jit] Call mono_jit_thread_attach () through a GOT slot instead of directly, so the call doesn't go through the trampoline code which wouldn't work if the thread was not attached. (#4037)

[llvm] Fix a regression introduced by d231b5134decb9f2303be441a119ca552b51d0e1.

add type-checking.c to the build.

[runtime] CEE_MONO_CISINST is dead. Removed the type-cast half.

[mini] Remove the need for isinst wrapper for MBRO.

[mini] Remove the need of the ininst wrapper for interfaces.

[mini] Remove CEE_MONO_CCASTCLASS. Could not remove from opcodes.def at it weirdly breaks mono/mini/arrays.cs.

[mini] CEE_MONO_CCASTCLASS is dead. Removed the type-cast half.

[mini] Avoid the cast wrapper for MBRO.

Remove dead code.

[mini] Avoid the cast wrapper for ifaces

[mini] Simplify casting of variant generic interfaces.

[mini] Extract type checking code into type-checking.c

Merge pull request #4009 from kumpera/lazy-array-ifaces

[runtime] Implement lazy interface implementation for arrays.

[runtime] Remove the eager array interfaces code. Good ridance.

[runtime] Clean mono_init_internal hacks. Moved to mono_class_setup_mono_type / mono_class_create_from_typedef.

[runtime] Fix bad packing in MonoClass.

[runtime] Implement lazy interface implementation for arrays.

We have the following set of problems:

Arrays implement interfaces that are invariant but behave like covariant.
Those are IList`1, ICollection`1, IEnumerable`1

Arrays of primitive types are type equivalent based on their underlying type.
This means int[], uint[] and IntEnum[] are all interchangeable. And so is casting among their interfaces.
Meaning this is valid: `(IList<IntEnum>)(object)new int[1]`.

Finally, the way we implement IEnumerator`1 for arrays forces it to behave just like the other 3 interfaces.

Implementation:

Introduce MonoClass::is_array_special_interface, set for all the above interfaces. This allow fast checking of the interfaces in question.

mono_class_is_assignable_from now includes the array casting rules.

Change mono_class_interface_offset_with_variance to take arrays into account when looking for a candidate iface.

The JIT implements casting of those interfaces using the cast with cache wrapper.

Finally, to handle the weird primitive rules, set the cast_class of sbyte, ushort, uint and ulong to their same-size buddies.
This makes it easier for the casting code to resolve this problem.

Problems:

Casting performance is terrible, anywhere from 60% to 1000% slower on a micro-benchmark.
Can be improved by extending our casting wrapper to handle those interfaces (probably would have to give up inlining as they would be huge)

The code in domain.c is icky, but not sure where to put it otherwise.

IEnumerator could be solved in Array.cs by not forcing it to be magic as well. Meaning the folowing

```
object array = new int[1];
var a = array as IList<int>;
var b = array as IList<uint>;

a.GetEnumerator().GetType () != b.GetEnumerator().GetType ();
```

The last line is false today but it's harmless to change it to avoid the casting penalty for such central type.

[build] Install also from mcs/packages

Revert "[CI] Disable winforms tests on all platforms."

This reverts commit 598183023b74fad649a43fd9fbeec0f962a55a8a.

[CI] Disable winforms tests on all platforms.

Convert MonoClassExt to class properties (#4022)

* [runtime] Store the class failure data as a property.

* [runtime] Store the list of nested classes as a class property.

* [runtime] Move the property relation information from MonoClassExt into a class property.

* [runtime] Move the event relation information from MonoClassExt into a class property.

* [runtime] Move the field_def_values field from MonoClassExt into a class property.

* [runtime] Move the declsec_flags field from MonoClassExt into a class property.

* [runtime] Remove unused MonoClassExt infrastructure.

* [runtime] Add some API docs.

Merge pull request #4015 from xmcclure/new-threads-collect-fix

Possibly fix hang in sgen-new-threads-collect.exe

Use __cdecl rather than __stdcall for icalls on Windows 32-bit

Mono assumes the same calling convention for icalls as the default P/Invoke
calling convention which is __stdcall on Windows. But none of the icalls are
marked with the required __stdcall attribute on Windows so there will be a
mismatch between the caller and callee.

This is the reason why the System.Web.HttpUtilityTest.JavaScriptStringEncode()
test crashes on Windows 32-bit. It generates a call to an icall
(char.ToString() inlines to new string(char,int)) which is assumed to be
__stdcall while it's actually __cdecl. The caller will decrement ESP as if the
callee was __stdcall resulting in ESP decreasing on each iteration. After a
couple of thousand iterations in the loop in that test the stack guard page is
hit and a stack overflow occurs.

Merge pull request #2774 from marek-safar/roslyn

Roslyn compiler switch

Merge pull request #3973 from mono/small-perf

[Perf] Replace some exhaustive O(n^3) or O(n^2) iterations with O(n)

Switch to cecil branch with better pdb handling

[build] Bump required bootraping mono version and download it automatically

[build] Roslyn switch

[marshal] Fix API documentation comment on mono_string_to_byvalwstr

[runtime] Fix mono_image_load_module_checked () so it actually stores the loaded netmodule in image->modules, not the parent image itself. Fixes #47762. (#4029)

[aot] Avoid a crash on class loading failures. Fixes #47152.

[runtime] Fix a warning.

[jit] Fix some SIMD problems on x86.

Merge pull request #4028 from monojenkins/update-csprojs

[msvc] Update csproj files

[msvc] Update csproj files

[Facades] Don't build System.Drawing.Primitives.dll by default for monotouch_watch profile

It needs special logic like the other monotouch profiles in the products
since we don't have a System.Drawing assembly/types for these profiles in Mono
and don't embed the types in the facade anymore after 1074fe91bd9b33a84040cc9d0d5da42079633d2a.