Merge pull request #2446 from akoeplinger/remove-leftover-file

[corlib] Remove left-over files from long ago (System.PAL/Linux)

They are no longer in use and cause confusion as they show up in search results.

@monojenkins merge

[corlib] Remove left-over files from long ago (System.PAL/Linux)

They are no longer in use and cause confusion as they show up in search results.

Merge pull request #2444 from xmcclure/randfixes

Bugfixes in mono-rand.c

2 commits:

* Make mono-rand consistent (fixes threadpool on some UNIX systems)

  mono-rand.c defines a function interface and provides three implementations (Windows, modern UNIX, base C standard). The clients of this file (RNGCryptoServiceProvider.cs, threadpool-ms-io.c) appear to be consistent in how they use this interface, but the interface is not documented, and the C-only implementation deviates from it (the return value of init is different). The result is that threadpool-ms-io fails an assert when run on a system using the C-only implementation. I documented the interface and fixed the C-only quirk.

* Fix nonuniform distribution of mono_rand_try_get_uint32

  mono_rand_try_get_uint32 returns a value in a range min..max inclusive. Due to a math error the ‘max’ value will be returned only one out of every 2^32-1 calls to the function. This is currently harmless because the only current callers of this function (in threadpool-ms-io) will run fine even if their max value is never returned, but it could become a problem later.

Merge pull request #2445 from nealef/master

Fix certmgr when a unicode encoded null is used as password

Fix certmgr when a unicode encoded null is used as password

Fix nonuniform distribution of mono_rand_try_get_uint32

mono_rand_try_get_uint32 returns a value in a range min..max
inclusive. Due to a math error the ‘max’ value will be returned only
one out of every 2^32-1 calls to the function. This is currently
harmless because the only current callers of this function (in
threadpool-ms-io) will run fine even if their max value is never
returned, but it could become a problem later.

Make mono-rand consistent (fixes threadpool on some UNIX systems)

mono-rand.c defines a function interface and provides three
implementations (Windows, modern UNIX, base C standard). The clients
of this file (RNGCryptoServiceProvider.cs, threadpool-ms-io.c) appear
to be consistent in how they use this interface, but the interface is
not documented, and the C-only implementation deviates from it (the
return value of init is different). The result is that
threadpool-ms-io fails an assert when run on a system using the C-only
implementation. I documented the interface and fixed the C-only quirk.

[getline] Improve the embedded sample

[getline] Fix the logic for advancing the top item

[Process] Fix monodroid build

[llvm] Split long bblocks instead of disabling llvm for methods with them.

[llvm] Fix a type conversion problem when calling the same method with two different signatures.

Merge pull request #2362 from ludovic-henry/fix-process-missedoutput

[Process] Use ReferenceSource process async output/error reader

Bump reference source for CancellationTokenSource fixes

[mcs] Update project file

[mcs] Use probing like aproach for member access completion

Merge pull request #2403 from BrzVlad/feature-emulation-delay

[jit] Delay opcode emulation to allow optimization passes to run on these opcodes

[llvmonly] Fix mono_patch_info_equal () for MONO_PATCH_INFO_GSHAREDVT_IN_WRAPPER.

[llvmonly] Fix the addition of gsharedvt in wrappers around unbox wrappers.

[llvmonly] Fix the gsharedvt parameter passing code so it emits valid llvm il.

[llvmonly] Add unwind info for unbox wrappers, not the get_unbox_tramp () generated function.

[llvmonly] Fix initialization of static gshared methods, they receive a vtable as the rgctx argument, not a MonoMethodRuntimeGenericContext. Emit unwind info for unbox wrappers. Fix calls to gsharedvt methods from non-shared delegate invoke wrappers.

Merge pull request #2438 from lambdageek/cherry-pick-coop-handle

Cherry pick handles work

Merge pull request #2439 from akoeplinger/fix-seqpoint-test

[mono] test_op_il_seq_point test should use a temp file for NUnit results

[mono] test_op_il_seq_point test should use a temp file for NUnit results

We were seeing red builds on Jenkins because the NUnit xml was malformed when something bad happened
during the test_op_il_seq_point tests and Jenkins couldn't parse it.

To fix this, write the results into a temp file and only rename to .xml at the end when everything worked.

Additionally, fixed a small typo in cleaning the .so files.

[runtime] Fixed ARM cross compiler builds.

[jit] Delay emulation to allow optimizations to happen on these ops

Emulation used to happen at emitting time (method_to_ir). Now it is done after the copy propagation pass, where we propagate immediates and can do strength reduction on some of these ops. Because the call to the native function might imply an inlined wrapper which compiles more IL instructions, we need to run the decomposition pass again.

[jit] Support decomposition of long imm ops to simple long ops

[jit] Don't crash in mono_op_imm_to_op if opcode is not handled

Mimic the behaviour of its counterpart, mono_op_to_op_imm, since crashing limits the usage of the function and it is the responsability of the upper layer.

[jit] Don't decompose into unsupported instructions

If we emulate these ops, it means we don't support the *_2 ops and we shouldn't decompose into them.

[jit] Don't hardcode the long vreg components

[handle] rename mono_handle_arena_(initialize|deinitialize)

to mono_handle_arena_init and mono_handle_arena_cleanup, respectively.

[handle] Make MONO_HANDLE_ARENA_POP a function-like macro

[sgen] Remove DISABLE_CRITICAL_REGION as it's never defined

[checked-build] Add in GC critical region assertion

[checked] Nest MONO_{PREPARE,FINISH}_GC_CRITICAL_REGION

[coop] store MonoHandleArena pointer in MonoThreadInfo

[handle] Add _SETREF_NULL macros for MonoObjectHandle and MonoArrayHandle

[handle] Rename MonoHandleStorage->obj to MonoHandleStorage->__private_obj

[handle] Add mono_handle_obj_is_null utility function

[llvmonly] Implement Assembly.GetCallingAssembly () in llvmonly mode by saving the current assembly in the calling method. Enable it for Activator.CreateInstance () for now.

[Process] Use ReferenceSource process async output/error reader

By using the referencesource one, we remove a buggy part of the Process class. It will now simply use Stream.{Begin/End}Read, and not a custom implementation that introduces its own bugs.

[man/mono.1] Fix typo from 9115bf0a

[runtime] Make sure ptr-to-structure and structure-and-ptr wrappers are unique, this is needed by full aot. Fixes #20186.

[mono] Update predefined Hebrew negative sign. Fixes #37681

[corlib] Bump reference source for #37583

Merge pull request #2427 from alexrp/socket-responder-dedup

[bcl] Deduplicate the SocketResponder test helper class.

A number of fixes were made to the version in System but these fixes were never applied to the System.Web.Services version. Merge the two and place the class in the test-helpers directory to avoid this sort of maintenance problem.

Merge pull request #2434 from knocte/bitmapattributes

[System.Drawing] Include new .NET4.5 attributes (OWIN)

[System.Drawing] Include new .NET4.5 attributes (OWIN)

These stubs should fix this OWIN issue brought up in StackOverflow:
http://stackoverflow.com/questions/34751771/system-drawing-bitmapsuffixinsameassemblyattribute-missing-in-mono

Docs:
* https://msdn.microsoft.com/en-us/library/system.drawing.bitmapsuffixinsameassemblyattribute%28v=vs.110%29.aspx
* https://msdn.microsoft.com/en-us/library/system.drawing.bitmapsuffixinsatelliteassemblyattribute%28v=vs.110%29.aspx

Patch contributed under the terms of the MIT/X11 licence.

[bcl] Deduplicate the SocketResponder test helper class.

A number of fixes were made to the version in System but these fixes were never
applied to the System.Web.Services version. Merge the two and place the class
in the test-helpers directory to avoid this sort of maintenance problem.

Merge pull request #2426 from alexrp/http-test-remove-abort

Revert "[bcl] Fix System.Net.HttpWebRequestTest.GetRequestStream hang"

Merge pull request #2424 from alexrp/task-unobserved-exceptions

[TPL] Add support for the <ThrowUnobservedTaskExceptions /> config element.

[llvm] Add a missing r4 opcodes.

[llvmonly] Create empty stacktraces if backtraces are not supported by the platform.

[runtime] Disable a test which fails with bitcode.

Merge pull request #2428 from akoeplinger/fix-ipv6-test

[System] Ignore ReceiveIPv6() UdpClient test when IPv6 is not available

[aot] Avoid overwriting entries in domain->seq_points if decode_exception_debug_info () is called more than once. Fixes #37582.

[System] Ignore ReceiveIPv6() UdpClient test when IPv6 is not available

The test hangs on Jenkins otherwise.

[System] xamarin_start_wwan is not available for tvOS and watchOS.

[handle] Add macro to push/pop arena

[handle] Make handle.h use exclusively public headers and MONO_API functions

[handle] Remove SGen dependency

[handle] Fix typo

[handle] Implement mono_handle_(new|elevate) functions

[handle] Inline handle-private.h in handle.h

MonoHandleArena functions need to be as public as MonoHandle functions, as we cannot have one without the other.

Merge pull request #2274 from esdrubal/udpclientreceive

Fixes UdpClient.Receive with IPv6 endpoint

Merge pull request #2423 from radical/mcs-args

[mcs] Accept and ignore command line args supported by csc ..

Revert "[System] Tweak tests to work with Thread.Interrupt as well."

This reverts commit bb50c3f3c943ea85d49df160d45f8edea0005d10.

Revert "[bcl] Fix System.Net.HttpWebRequestTest.GetRequestStream hang"

This reverts commit b3fa096e5eceea1c3b289b3655244a42c2884274.

The Abort () call is problematic on many platforms and is no longer necessary
as we properly close the listening socket since 1103abf3830131ba2a1deabc133e8d27402302c7.

Conflicts:

mcs/class/System.Web.Services/Test/System.Web.Services.Protocols/SocketResponder.cs
mcs/class/System/Test/System.Net/SocketResponder.cs

[TPL] Add support for the <ThrowUnobservedTaskExceptions /> config element.

Improves the situation wrt: https://bugzilla.xamarin.com/show_bug.cgi?id=36560

Merge pull request #2422 from akoeplinger/environment-is64bitoperatingsystem

[corlib] Properly implement Environment.Is64BitOperatingSystem

[corlib] Properly implement Environment.Is64BitOperatingSystem

Before we only checked if IntPtr.Size == 8, but this returns false for a 32bit Mono running on a 64bit OS.
Added an icall that gets the correct value from the runtime instead.

Note: this currently only whitelists x86_64, aarch64 and ppc64 as 64-bit platforms.

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

Merge pull request #2261 from BrzVlad/fix-arm-fast-tls-master

[arm] Fix arm fast tls

[mcs] Accept and ignore command line args supported by csc ..

.. that we don't support:

/utf8output
/subsystemversion
/highentropyva
/highentropyva+
/highentropyva-

[llvmonly] Avoid disabling llvm for the Selector.GetHandle () intrinsics.

[llvmonly] Fix support for variant interface methods, they work similarly to generic virtual methods.

Revert "Apple has introduced the System Integrety Protection in OS X 10.11. This prevents writes to /usr amongst other paths. /usr/local is still writable though."

This reverts commit b723c89628187ab138334c108e16d62bc9530d6e.

@migueldeicaza thinks this is the wrong approach, reverting until we figure out the right way to go about this.

[llvm] Ignore OP_IL_SEQ_POINT.

Merge pull request #2418 from lambdageek/dev/bug-12205

[reflection] Correct ReflectedType for generic methods

[aot] Add support for name mangling for more types.

[reflection] Test MethodInfo.ReflectedType property

Test cases for [#12205](https://bugzilla.xamarin.com/show_bug.cgi?id=12205)

[mscorlib] Add support for intercepting threadpool work, and use it for Xamarin.iOS/Mac.

This is used to Xamarin.iOS/Mac to wrap threadpool work in an NSAutoreleasePool.

Also open up corlib's internals to Xamarin.iOS/Mac so that the linker
doesn't have to be involved when accessing internal API.

[reflection] remove override get ReflectedType for MonoGenericMethod and MonoGenricCMethod

Should return the type that was queried, not the type where the method
is declared.

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

[runtime] Bump version to 4.3.3.

Merge pull request #2382 from akoeplinger/system-net-mime-referencesource

[System] Port System.Net.Mime namespace from referencesources

[mcs] Fixes inconsistent accessibility check for partial types. Fixes #37232

[llvm] Allow support for emitting some methods using 'linkonce' linkage. This allows the linker to get rid of duplicate copies. Enable it for gsharedvt wrappers.

Merge pull request #2415 from razzfazz/fix_timespec

fix struct timespec member name

Merge pull request #2414 from akoeplinger/fix-threadpool-test-hang

[corlib] Don't throw exception for null locale to fix threadpool-exceptions5.exe hang

fix struct timespec member name

The second member of struct timespec is actually called tv_nsec
(nanoseconds), not tv_usec (microseconds); this was likely
confused with struct timeval, where the second member is in fact
called tv_usec.

[corlib] Don't throw exception for null locale to fix threadpool-exceptions5.exe hang

We were seeing persistent hangs in the threadpool-exceptions5.exe test on the Jenkins ARM machines.
Tracing the execution showed the following exceptions:

```
[0x7f65e3284700:] EXCEPTION handling: System.Threading.ThreadAbortException:
[0x7f65e3284700:] EXCEPTION handling: System.Exception: From OnCBFinished
[0x7f65e3284700:] EXCEPTION handling: System.ArgumentNullException: Value cannot be null.  <-- from CultureInfo.CreateSpecificCulture()
Parameter name: name
[0x7f65e3284700:] EXCEPTION handling: System.Threading.ThreadAbortException:
exception inside UnhandledException handler:

... program hangs here ...
```

It turns out that the machines don't have the LANG/LC_ALL env vars set, resulting in a null string returned
from get_current_locale_name(). The code then tries to construct a locale from that and throws an ANE which is
normally catched and turned into the InvariantCulture.

However, in the threadpool-exceptions5 test the first time a CultureInfo is created is in the OnUnhandledException
event handler when calling ToString(). This means that an exception is thrown while we are in the unhandled exception
handler and the runtime apparently just stops there (despite the exception actually being catched, probably because of
the thread abort) so we never get to signaling the monitor object -> we hang in Main().

The fix is to not rely on catching the ArgumentNullException when we get a null locale, which is a better approach anyway.

To easily reproduce the issue before the fix, use the following (on Linux only, we don't use LANG for locale on OSX):

```
~/dev/mono/mono/tests$ LANG=C MONO_PATH=../../mcs/class/lib/net_4_x/ ../mini/mono-sgen --trace=N:nothing threadpool-exceptions5.exe
```

Note: If you pass an invalid LANG=foo, the hang is still there as we throw a CultureNotFoundException and hit the same issue.
As the behavior looks like a runtime bug to me I filed https://bugzilla.xamarin.com/show_bug.cgi?id=37547

[llvmonly] Avoid creating an empty output file for the JITted code.

[llvmonly] Avoid emitting the 'globals' aot table, its not used in llvmonly mode.

Merge pull request #2412 from akoeplinger/disable-thread-test

[corlib] Disable Thread test that started failing after d5768a7f141e2a579cbca26f76c791c215f4aabf

[corlib] Disable Thread test that started failing after d5768a7f141e2a579cbca26f76c791c215f4aabf

Setting the priority before starting a Thread doesn't work right now as there's no handle to the native
thread at that point (the current implementation doesn't store the priority, it just tries setting it
on the native thread) so the test would remember the default 'Normal' priority instead of 'BelowNormal',
causing the test to fail later on.

Added an assert to verify that the thread priority can be set before starting and disable the test for now.

Note: variable names 'before' and 'after' were switched in the test, fixed those as well.

[llvm] Error out when using --aot=llvmonly if the runtime is not compiled with --enable-gsharedvt.

[csharp/getline] Add support for completion popup windows

This adds supprot to the getline.cs library to provide popup windows with
completion results.   There is a new public string property surfaced,
HeuristicsMode, that if set to "csharp" will trigger the completion window
independently of the "tab" character, based on the user input.

This mode triggers completion after the "." character is entered, and will
continue to trigger it as long as you type, or you backspace and the completion
succeeds.   The heuristic also ignores scenarios like "1.2", so there is no
compeltion provided after entering numbers by default.

Perhaps we can later introduce a "OnKeyEntered" command that users could
use to provide their own heuristics to control whether to popup the window
or not.

[linker] do not use System.IO.Path on assembly resource names

  - fixes #37305

  - resource name can be general string and using it as path
    might throw

       System.ArgumentException: Illegal characters in path.

  - it would be possible to check for the path's invalid characters in
    the resource name. that would cause crossplatform issues
    though.

    various platforms have different invalid charactes sets for path
    names. for example using assembly built on mac with embedded
    linker blacklisting xml file, which's name contains characters in
    its filename which are invalid on windows. so the windows built
    app using that assembly would fail mysteriously, because the
    important parts wouldn't be preserved.

    thus we use just string comparison

[io-layer] Revert 31f13881a483201933b16e75d47a6d103d959a64 and fix it in a simpler way

We don't need the configure.ac check as we can just directly check if the macro is defined.

[runtime] Bump AOT file format version.