Merge pull request #2397 from alexanderkyte/debugger_appdomain

[runtime] Read fields out of TransparentProxies correctly in debugger agent FIXES 36823

Merge pull request #2450 from grendello/io-layer-trace

Replace the old DEBUG messages with calls to mono_trace.

Merge pull request #2451 from akoeplinger/fix-osx

[Mono.Posix] Fix OSX build break by checking if accept4() is available

It's a Linux-specific extension and not available on OSX or Linux <2.6.28

/cc @steffen-kiess @jonpryor

@monojenkins merge

[Mono.Posix] Fix OSX build break by checking if accept4() is available

It's a Linux-specific extension and not available on OSX or Linux <2.6.28

Merge pull request #2447 from kumpera/amd64_gsharedvt

[amd64] First pass at fixing issues related to gsharedvt.

[llvmonly] Avoid inlining methods which call MethodBase:GetCurrentMethod ().

[llvmonly] Fix the passing of byref arguments in runtime invoke.

Replace DEBUG messages with calls to mono_trace

The messages contain useful debugging information which is unavailable
unless the mono runtime is recompiled. The debugging information is
particularly useful on mobile platforms and in test runs on bots.

The I/O layer trace messages can be disabled on compile time by
defining the DISABLE_IO_LAYER_TRACE preprocessor symbol.

Move mono_trace to header file

This commit moves mono_trace to a header file so that all the
calls are inlined, avoiding one function call at the small expense
of binary size.

[llvmonly] Fix the MethodBase.GetCurrentMethod () intrinsic so it works with generic methods.

Merge pull request #2006 from steffen-kiess/posix-sockets-2

[Mono.Posix] Add wrappers for struct sockaddr_*

Merge pull request #2 from xmcclure/rodrigo-fix-build

Additional stubs to fix standard build following [amd64] changes

Additional stubs to fix standard build following [amd64] changes

[llvm] Fix some type imcompatibility problems which happen when a vreg holding an objref is made global and stored into its location which has a different type (IntPtrType () vs ObjRefType ()).

Merge pull request #2435 from xmcclure/x-refresh

Improve support for Xamarin platform extensions

[llvmonly] Fix the catching of exceptions of open types in shared methods by passing mrgctx/this to the mono_llvm_match_exception () function.

[llvm] Add some missing opcodes.

[runtime] Fix native-to-managed calls to delegates which wrap a virtual method. Fixes #37695.

[linker] improve method dependencies dumping

[Mono.Posix] Add wrappers for struct sockaddr_*

Add wrappers for the sockaddr_* structures and add the syscalls using these
structures (bind(), accept(), ...).

[llvmonly] Add an instrinsics for MethodBase.GetCurrentMethod () so it works on platforms without stack walks.

[corlib] Improve CancellationTokenSource test

[msvc] Fix Windows build

handle.c/h were missing from the projects.

[llvmonly] Share more wrapper types. Avoid adding out wrappers for calls from gsharedvt code to gsharedvt code with a normal signature. Fix the handling of pointer types in runtime invoke. Fix the passing of the same fixed gsharedvt argument in multiple calls.

[llvmonly] Avoid a crash in ves_icall_GetCurrentMethod () if the stack walk fails.

[amd64] Fix setting of MonoCompile:ret:dreg in mono_arch_allocate_vars.

The code in mono_arch_allocate_vars would set cfg->ret->dreg to cfg->ret->inst_c0
regardless. This is ok in some cases, but bad in others.

For OP_REGVAR this would do the right thing, dreg should be register that holds
that argument.

For when vret_addr is used, it would modify a var that is not used, so harmless.

For OP_REGOFFSET, inst_c0 alias with inst_offset, which is set to the offset from
RBP to read the var from. So we'd have a dreg set to, sometimes, a negative number.

This was not a problem as after mono_arch_allocate_vars only mono_spill_global_vars
is called and it only checks the dreg in the case of gsharedvt.

The fix is pretty simple, move setting from happening in all cases to only for OP_REGVAR.

[amd64] Add arg_size field to ArgInfo and update it when storage is ArgOnStack

[amd64] Properly emit loads to gsharedvt stack args.

[amd64] Add is_gsharedvt_return_value to ArgInfo. Save to the stack ArgGSharedVtInReg args.

The usage of is_gsharedvt_return_value is easier than having a separate storage as supporting
it would require changing all usage sites and this is only needed by start_call.

All incoming arguments in registers must be saved on the stack as the rest of the JIT
expects to be able to load them from their well known places. Without this change, it would
load a random value instead.

[amd64] Add definition of GSharedVtCallInfo to mini-amd64.h

The GSharedVtCallInfo structure used is the same to the one used by
other targets.

[amd64] Add some of the missing bits of gsharedvt.

This was enough to get mscorlib to FullAOT with gsharedvt enabled.

Makefile.am.in:
Don't exclude GSHAREDVT from the tests.
Removed DYNCALL too, as it's supported too.

mini-amd64.c (mono_arch_init):
Register mono_amd64_start_gsharedvt_call jit icall.

mini-amd64.c:
Fix a typo in the ifdef guarding inclusion of the gsharedvt extension.

mini-amd64.c (mono_arch_emit_outarg_vt):
Implement decomposing of ArgGSharedVtOnStack to a simple store on the stack.

mini-amd64.h:
define MONO_ARCH_GSHAREDVT_SUPPORTED
Export mono_amd64_start_gsharedvt_call

tramp-amd64.c (mono_arch_create_sdb_trampoline):
Add missing mono_profiler_code_buffer_new call.
Add a couple of functions to !ENABLE_GSHAREDVT

aot-compiler.c (arch_emit_gsharedvt_arg_trampoline):
Implement it for amd64 as a pair of GOT loads (same for other archs).

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

Fix incorrect comment on mono_jit_init_version

Clarify when ucontext_t is available

mono-context currently uses a mix of MONO_SIGNAL_USE_SIGACTION and
UCONTEXT_REG_RAX #ifs to determine whether ucontext_t is available.
UCONTEXT_REG_RAX is unreliable, and MONO_SIGNAL_USE_SIGACTION is
misnamed because it is nowhere used to gate use of sigaction.
Standardize on a new MONO_SIGNAL_USE_UCONTEXT_T, replacing SIGACTION.

Bottleneck module loader platform macros

processes.c contains implementations for module loading for several
platforms. Rather than scattering PLATFORM_MACOSX etc through the
file, pick a loader at the top of the file and base #ifs on that. This
makes BSD code clearer and makes it easier to special-case OS variants
where the normal loader is not available.

Fix build on platforms where native thread IDs are not pointer sized

Certain of our debug prints assume thread ids can be cast directly to
a pointer. This does not work on platforms where thread IDs are ints.

[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.