[mkbundle] Allow overriding linked Mono library path

Merge pull request #2342 from lewurm/mkbundlefix

[mkbundle] restore old behavior of -c flag

[test][System.Runtime.Serialization] Ensure test compile on XI (avoid more than one Main method)

Fix "make dist" WITHOUT accidentally un-bumping referencesources

Revert "Fix typo in dde9ee9e (Fixes `make dist`)"

This reverts commit 85cbe23b99807da23284740896a3966539c8000f.

Fix typo in dde9ee9e (Fixes `make dist`)

[jit] Fix the support for gshared types in mini_emit_initobj (), previously we were treating them as reference types, leading to stack overwrites. Fixes #37079.

[jit] Make it possible to enable gsharedvt in llvmonly mode even if MONO_ARCH_GSHAREDVT_SUPPORTED is not enabled.

[jit] Remove a superfluous assert.

[jit] Reorganize mono_jit_runtime_invoke () a bit so info->ret_box_class is set even if dyncall is not supported.

Add support for translating between normal and gsharedvt calling conventions in llvmonly mode.

This is implemented by generating two wrappers for each concrete signature found in the assembly:
* an 'in' wrapper translates from the normal to the gsharedvt signature
* an 'out' wrapper translates from the gsharedvt to the normal signature

The wrappers take an extra argument, which points to a function descriptor, which is an <addr, arg> pair. They
make a call to ADDR which ARG as an extra argument. ADDR/ARG is either a gsharedvt method with an rgctx
argument, or another gsharedvt wrapper with a function descriptor argument.

All calls made out of gsharedvt methods with a variable signature use the gsharedvt calling convention. If the
callee is also gsharedvt, no wrapper is needed. If the callee is not gsharedvt, the call goes thought
a gsharedvt out wrapper.
Calls made from normal methods to gsharedvt methods go though a gsharedvt in wrapper.

    * mini-generic-sharing.c (mini_get_gsharedvt_in_sig_wrapper): New function to generate gsharedvt in
    wrappers as IL code.
    (mini_get_gsharedvt_out_sig_wrapper): Ditto for out wrappers.

    * method-to-ir.c (emit_llvmonly_calli): New helper function to make a call to a function descriptor.
    (mono_method_to_ir): Make some indirect calls to MONO_RGCTX_INFO entries using function descriptors.

    * aot-compiler.c (compile_method): Generate gsharedvt wrappers for the signatures used by the method.

    * aot-runtime.c (decode_method_ref_with_target): Decode gsharedvt wrappers.

    * jit-icalls.c (resolve_vcall): Rewrite this so it can handle gsharedvt as well.
    (resolve_iface_call): Ditto.
    (mono_init_delegate): Ditto.
    (mono_init_delegate_virtual): Ditto.

    * mini-trampolines.c (mini_create_llvmonly_ftndesc): New helper function to create a function descriptor.
    (mini_add_method_wrappers_llvmonly): New helper function to add gsharedvt in/out wrappers, similar
    to mini_add_method_trampolines ().

    * mini-generic-sharing.c (mini_get_gsharedvt_wrapper): Return the newly added wrappers in llvmonly mode.
    (class_type_info): Insert gsharedvt wrappers if needed. Some MONO_RGCTX_INFO types will now resolve to
    function descriptors instead of function addresses.
    (instantiate_info): Ditto.

2015-12-16  Zoltan Varga  <vargaz@gmail.com>

Add mono_signature_full_name () helper function.

* debug-helpers.c (mono_signature_full_name): New helper function to stringify a signature, including
its return types as well.

2015-12-16  Zoltan Varga  <vargaz@gmail.com>

Add beginnings for support for gsharedvt in llvmonly mode. Gsharedvt methods use a calling convention
where everything is passed and returned by ref, even normal types. I.e.

public int foo<T> (T t, int i);

becomes:

void foo (int *vret, T* t_addr, int *i_addr);

Not enabled yet.

* mini-llvm.c: Implement support for gsharedvt.

* ssa.c (mono_ssa_remove_gsharedvt): Same as mono_ssa_remove (), but only remove the
OP_VPHI nodes referencing gsharedvt variables, since they cannot be handled by the
llvm backend.

* mini-llvm.c (mono_llvm_create_vars): New backend function, similar to mono_arch_create_vars ().

* decompose.c (mono_decompose_vtype_opts): Decompose more opcodes in llvm mode so
opcodes using gsharedvt variables don't reach the llvm backend. Some, like OP_LDADDR and
vcalls, still do.

* method-to-ir.c (mono_allocate_gsharedvt_vars): New function to allocate gsharedvt locals,
extracted from mono_spill_global_vars () so it can be used in llvm mode too.

[mono-api-info] Add `mono-api-info -L`, `mono-api-info -r`

**The Scenario**: `mono-api-info` and `mono-api-html`, together, are a
useful way to track API changes over time, specifically to check for
accidental API breakage:

# Have a "known good" assembly version;
# keep reference/assembly.xml in version control
$ mono-api-info assembly.dll > reference/assembly.xml

# ...then on every new build, check for breakage:
$ mono-api-info assembly.dll > new-assembly.xml
$ mono-api-html reference/assembly.xml new-assembly.xml > assembly-diff.html
$ if grep '<p>Removed' assembly-diff.html > /dev/null ; then \
echo "ABI BREAK IN assembly.dll" ; \
cat assembly-diff.html ; \
fi

This is a very nice workflow, and has been used in Xamarin.Android to
track accidental API breakage for some time.

**The problem** with the above workflow are *implicit* references:
`mono-api-info` will happily look for dependencies in the same
directory as `assembly`, but it will *also* look for assembly
references in various system-specific directories, which is fine...

...until it *isn't* fine, e.g. if you have your own version of
mscorlib.dll that you need to use, *not* the system one, as your own
version may have API differences from the system mscorlib.dll which
are "percolated" into `mono-api-info` output. For example, on
"desktop" profiles, System.Attribute implements
System.Runtime.InteropServices._Attribute, and thus *all* custom
attribute types will have _Attribute listed as an implemented
interface. The Xamarin.iOS and Xamarin.Android mscorlib.dll,
meanwhile, does *not* have System.Attribute implementing _Attribute.
Consequently, changing the mscorlib.dll which is used while processing
an assembly will change `mono-api-info` output, resulting in "API
breakage" which doens't actually exist, because the wrong mscorlib.dll
was used to generate the original API description in the first place.

We need a way to control which mscorlib.dll/etc. are used. This can be
done by allowing specification of the assembly search directories, so
that we can "override" the default system-specific directory location.

**The solution**: Add a `mono-api-info -L PATH` option which adds PATH
to the list of directories that `mono-api-info` will search when
resolving assembly references. This allows "overriding" the system
directory location, as PATH will be searched for mscorlib.dll before
the system directory location.

Additionally, add a `mono-api-info -r ASSEMBLY` option, which will
pre-process ASSEMBLY (and all dependencies) and add the directory
which contains ASSEMBLY to the assembly search paths.

These two options are conceptually similar to `mcs -L` and `mcs -r`.

Finally, while adding support for these options, add support for
`mono-api-info --help`, so that we don't need to use only the man page
to get option information.

Merge pull request #2324 from akoeplinger/caching-locking

[System.Runtime.Caching] Add missing locking when accessing cache entries

Merge pull request #2356 from ludovic-henry/fix-thread-suspend-deadlock

[threads] Fix potential deadlock on suspend

[threads] Fix potential deadlock on suspend

Because mono_thread_info_safe_suspend_and_run is locking the suspend lock, we cannot enter the blocking state in any of its callee, as that could lead to a deadlock.

Merge pull request #2353 from ludovic-henry/fix-servicemodel-15153

Add test and merge #1879

@monojenkins merge

Merge pull request #2351 from ludovic-henry/fix-socket-36941

[socket] Add test for SocketOptionName.MulticastTimeToLive

Merge pull request #2345 from akoeplinger/xbuild14

Bump xbuild script to 14.0 and a few other fixes

Add test for bug #15153

[socket] Add test for SocketOptionName.MulticastTimeToLive

Merge pull request #2354 from xmcclure/ugh-no-gdi

Fix Microsoft.Build.Tasks make check on OS X

Fix Microsoft.Build.Tasks make check on OS X

The mcs/class/Microsoft.Build.Tasks make check contains a test whose
goal is to verify that Microsoft.Build.Evaluation.Project can run a
MSBuild code fragment which has an assembly reference. The assembly
referenced in the test is System.Drawing. However on many systems
System.Drawing does not work without special setup. The test should
reference a different assembly (I picked System.Runtime.Serialization)
because ensuring System.Drawing is working isn't relevant to the test.

Merge pull request #2346 from xmcclure/proxy-load-fail

Let GetTransparentProxy fail clean on load error (bug 36848)

[scripts] Bump xbuild script to 14.0

This means we'll use xbuild 14.0 by default, which allows users to build projects with ToolsVersion=14.0

[xbuild] Add new TargetFrameworkVersion entries in Microsoft.Common.targets

Add the new 4.5.2, 4.6 and 4.6.1 versions to Microsoft.Common.targets so xbuild doesn't print "warning : TargetFrameworkVersion 'v4.6.1' not supported by this toolset".

Additionally I found a bug in the target where "and" was used instead of "or" which means the condition was never true, fixed as well.

Let GetTransparentProxy fail clean on load error (bug 36848)

icall for Remoting.RealProxy.GetTransparentProxy can sometimes attempt
to load an invalid class which is not previously loaded. Currently,
this leads to an assert failure in mono_class_proxy_vtable which does
not expect to be getting an invalid proxy class as input. Fix is for
the icall to load the class early and raise on failure.

A test is included.

Merge pull request #2338 from BogdanovKirill/httpwritefix3

[HttpConnection] Bug fix: HttpListener's "IgnoreWriteExceptions" property value is ignored when "Expect: 100-Continue" header set in request

That problem was discussed previously here: https://github.com/mono/mono/pull/2300
I want to offer alternative solution now.

Merge pull request #2344 from ludovic-henry/fix-servicemodel-flakiness

[tests] Fix more System.ServiceModel flakiness

@monojenkins: automerge

[tests] Fix more System.ServiceModel flakiness

Merge pull request #2343 from ludovic-henry/threads-cleanup

[automerge][threads] Various cleanups

None

[threads] Add mono_thread_info_usleep function

[threads] Bring mono_thread_info_safe_suspend_and_run code together

[threads] Remove dead code

Merge pull request #2340 from ludovic-henry/fix-servicemodel-flakiness

Merge pull request #2340 from ludovic-henry/fix-servicemodel-flakiness

[tests] Attempt at fixing System.ServiceModel.Dispatcher.Bug32886.Bug32886_Test flakiness

[mkbundle] restore old behavior of -c flag

[tests] Attempt at fixing System.ServiceModel.Dispatcher.Bug32886.Bug32886_Test flakiness

Merge pull request #2306 from migueldeicaza/master

[mono/driver] On OSX, increase the number of file handles available.

Merge pull request #2341 from ludovic-henry/fix-threadpool-max-worker

Merge pull request #2341 from ludovic-henry/fix-threadpool-max-worker

[automerge][threadpool] Fix various threadpool issues

Tests results:
 - ARM soft float Linux: pending
 - i386 Windows: pending
 - i386 Linux: success
 - ARM hard float Linux: pending
 - AMD64 Linux: success
 - i386 OSX (Proprietary): failure

[threadpool] Saner default for max number of worker threads on android and ios

[threadpool] Limit number of threads created per second

[threadpool] Fix overflow of max number of worker threads

In case all threads would wait on a Sleep or a Wait, the monitor thread would simply increase the number of max working thread, up to the point it would overflow the number of max worker threads.

Merge pull request #2218 from mono/assignProjectConfigurtionFix

AssignProjectConfigurtion task fix

Bump reference souces to include fix for #36786

[mcs] test update to work on 64bit mono

Merge pull request #2334 from iainx/handle-comma

[mcs] Handle commas inside quotes correctly

[mcs] Handle commas inside quotes correctly

OpenLiveWriter's build files break due to incorrectly parsing (".",",") as three arguments: a . and two " instead of two: a . and a ,

This is the error:

OpenLiveWriter/writer.build.targets: error : Error executing task WriteLinesToFile: Error converting Property named 'Lines' with value '#define FILE_VERSION $(BuildVersion.Replace(".",","));#define PRODUCT_VERSION $(BuildVersion)' to type Microsoft.Build.Framework.ITaskItem[]: Method 'Replace(., , )' arguments cannot be evaluated'

This patch ignores , as a separator character if it is inside ""

[HttpConnection] Bug fix: HttpListener's "IgnoreWriteExceptions" property value is ignored when "Expect: 100-Continue" header set in request

The problem is connected with "Listener" property of "context" object. It is null in "GetResponseStream" method, when it's called internally from "FinishInitialization" method of "HttpListenerRequest" class. This problem is possible only in situation when we have "Expect: 100-Continue" header from http client.

This change is released under the MIT license.

Merge pull request #2335 from iainx/implement-mir

Microsoft.Build.Engine] Implement ConvertToITaskItemArray

[acceptance-tests] Bump coreclr and ms-test-suite repos

Brings in similar fixes to those repos like in e6cf61f920b6ec1c6ecb67bcbe226256d6dc4db7

[corlib] Fix String.PadLeft/Right tests and add a new test for large string

We didn't catch the regression that is fixed in a780c52749695141c2037a5828808a82880292f6
because the tests didn't check the actual string or that an OOM is raised.

Add a new test for creating a large string (which works on 64bit Mono) to test a similar
codepath.

Microsoft.Build.Engine] Implement ConvertToITaskItemArray

Create a TaskItem containing the result of the invocation

Revert "[System]: Add _HelperAsyncResults.cs from the referencesource."

This reverts commit c6d239fa790ef48d932d1243d002253101648f7f.

(cherry picked from commit cfc81a28ab232777959d5abc43292b88d3359efa)

[mcs] Improve a780c52749695141c2037a5828808a82880292f6 to not emit extra conversion for constant values

Merge pull request #2311 from mlancione/master

[System.Windows.Forms] Locking Window Handle List

Merge pull request #1916 from ludovic-henry/iolayer-remove-sharedhandle

[io-layer] Remove shared handles cross process support

[threadpool] Fix GetAvailableThreads test

[mcs] Extend int to native int when doing pointer arithmetic on it. Fixes #36839

Typo error

Fixing a compiler error.

[io-layer] Remove shared handles cross process support

Merge pull request #2327 from akoeplinger/fix-msvc

[msvc] Fix casts that result in an error when building with VS

Fix endless recursion in XmlCompiledTransform on mobile

One of the XmlCompiledTransform overloads called itself recursively,
leading to either a stack overflow or segfault

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

[msvc] Fix casts that result in an error when building with VS

Those showed up after the recent C++ compliance commit.

[System]: Add _HelperAsyncResults.cs from the referencesource.

(cherry picked from commit 17e92c87759edcee72a06659e8a0790cf72e5190)

[jit] Better error message for a gsharing assert.

[configure] Bump mono version.D

Fixed Windows MinGW build and avoid leakage of io-layer types.

[corlib]: Cleanup X509Certificate and move the implementation-specific bits into a separate class.

(cherry picked from commit b82b0b8402297854bcceccc4329703024c5aad63)

[threadpool] Fix System.Web tests failures

Merge pull request #2310 from lambdageek/dev/bug-36305

[reflection] Fix MethodInfo.GetBaseDefinition for open constructed types (close #36305)

More code-style changes

Fixed a few other code style requirements.

[System.Runtime.Caching] Add missing locking when accessing cache entries

The "entries" collection in CacheEntryCollection can be modified while we're enumerating it,
which is not allowed. Add locking in FlushItems() to prevent this.

Code-style changes

Chaged code to match mono coding style.

Merge pull request #2305 from ludovic-henry/fix-threadpool-36414

[threadpool] Fix get min and available threads icalls

Merge pull request #2296 from xmcclure/webrequest-regex

Correctly escape proxy domains in WebRequest (bug #36356)

Merge pull request #2312 from ArsenShnurkov/bug36724

fix for https://bugzilla.xamarin.com/show_bug.cgi?id=36724

Fix the x86/ppc build.

[corlib]: X509Certificate needs to implement IDisposable.

Merge pull request #2288 from evincarofautumn/c++-compat

[runtime] Compile cleanly with -Wc++-compat.

[runtime] Compile cleanly with -Wc++-compat.

 * Add explicit casts for void pointers and enums.

 * Rename 'class' to 'klass' and 'this' to 'this_obj'.

 * Avoid other C++ keywords by appending an underscore.

 * Change MonoSeqPointInfo to avoid empty struct size warnings.

[sdb] Align the location of the MonoContext structure on the stack in the sdb trampolines on x86, to prevent a possible crash with memcpy.

[threadpool] Fix get min and available threads icalls

Fix bug #36414

[reflection] Fix MethodInfo.GetBaseDefinition() for open constructed base types.

When a generic derived class has an open constructed base type, we have
to apply the type arguments in order to find the correct base definition MethodInfo.

Merge pull request #2285 from knocte/filesystemwatcher

[FileSystemWatcher] Kill race condition (fixes BXC#10205)

Fixing Tab Spaces

Replaced spaces in previous commit with tabs, to follow format of
mono/master.

[mkbundle] Fix execution on OS X and Linux

fix for https://bugzilla.xamarin.com/show_bug.cgi?id=36724

[reflection] Test case for 36305.

When the base type of a generic type is an open constructed generic
type, GetBaseDefinition() must take the instantiation into account while
traversing the class hierarchy.

[mono/driver] On OSX, increase the number of file handles available.

We just reverted a patch that prevented the FileSystemWatcher from
opening too many file descriptors (200).  That was a band-aid at one
of the spots that opened too many file, as we would exhaust the number
of files, and later abort when some other operation needed files.

The problem is that OSX defaults to 256 file descriptors, and .NET
applications are file descriptor hungry, in fact, many applications
increase their file handled limit before starting.

This patch on OSX will attempt to increase the number of handles
available to 1024 if the number of handles avaialble is less than
that.

The commit that removed the limit was:

2af882232ce4961fdbe1ba0ae36246456bb1fbfb

And it was closed to fix:

#28693

Merge pull request #2297 from akoeplinger/msvc-autoupdate-config.h

[msvc] Automatically update (win-)config.h with the correct version number

Disable partial sharing again, it causes problems on ppc/s390x, and crashes in the roslyn benchmark:

https://performancebot.mono-project.com/builders/debian-armhf_utilite-desktop_auto-sgen/builds/743/steps/roslyn/logs/stdio

[llvm] Use a separate calling convention for gsharedvt methods in llvmonly mode, pass/return everything by ref. Not yet used.

Merge pull request #2295 from lambdageek/dev/test-conc-hashtable-coop-fix

(test-conc-hashtable.c) Make test work with cooperative suspend

Merge pull request #2302 from ludovic-henry/fix-delegate-36640

[delegate] Fix multicast Method property implementation

[runtime] remove debug statements introduced by a0e1dae32ceaf5331f5caf70b659de1392bbbc14

Merge pull request #2304 from BillSeurer/master

[ppc] Temporary fix for power to get builds working.

Merge pull request #2301 from ludovic-henry/fix-delegate-36646

[delegate] Fix multicast remove behaviour

[ppc] Temporary fix for power to get builds working.

[runtime] Initialize a dynamic_images_mutex.