[msbuild] Bump to xplat-master branch HEAD

[acceptance-tests] Bump coreclr submodule

Merge pull request #5353 from jonpryor/jonp-mono-api-html-ignore-class-removal

[mono-api-html] Allow `-r` to ignore class removals

[MacSDK] Switch to HTTPS for downloading SQLite

[profiler] Fix code coverage support.

Update Bockbuild for a CI fix/update (shorter build paths for MacSDK)

Merge pull request #5352 from urisimchoni/fix-logprof-manpage

[docs] fixes to log profiler manpage

[bcl] Clean up FRAMEWORK_VERSION 4.0 and 2.0 checks (#5356)

We don't build those profiles anymore, we have a net_4_x configuration instead.

[docs] mprof-report manpage - remove "onlycoverage" log profiler option

This option does not exist in the code.

[docs] mprof-report manpage - document heapshot-on-shutdown

Document the heapshot-on-shutdown log profiler option.

[docs] mprof-report manpage - document '#' output option

Document ability to use the log profiler with an open file
descriptor.

[llvm] Avoid generating invalid llvm IR when a finally clause contains a loop which branches back to the beginning of the finally clause. (#5354)

Merge pull request #5323 from cherusker/cherusker-2017-08-07-fix-strerror

[eglib] Fix g_strerror / monoeg_g_strerror

[mono-api-html] Allow `-r` to ignore class removals

Allow `mono-api-html -r REGEX` to be used to silence messages
regarding class and namespace removals in addition to member removals.

[docs] mprof-report manpage -- fix desctiption of existing file behavior

Fix the description of what happens when the output file already exists.

[docs] remove non-existant option from log profiler manpage

The option of prepending a '-' to the log profiler output
filename is not supported by the code - remove it.

[docs] Capitalization / puctuation fixes in mprof-report manpage.

More codeowners updates

Update codeowners

Distinguish between the XSI-compliant and the GNU-specific version of `strerror_r ()`
- add USE_STRERROR_R_XSI to use the feature test macros conveniently later
- deal with the GNU-specific version of `strerror_r ()` separately (leave the code for the XSI-compliant version untouched)
- make `buff_len` size_t as this is ISO compliant (even though it should not really matter in this case)

Add rules to run TSan with run-jenkins.sh (#5254)

- export TSan flags that are used by the compiler and the compiled program
- introduce `make_timeout` to set individual timeouts for `make` based on the config (e.g. TSan will only get 30 minutes to complete)

Add implementation and tests for MatrixValueSerializer and MatrixConverter

Uncomment fixed SizeConveter tests

[monodis] Not all override methods are errors

Check whether mono_get_method_checked returned an error, if not, just decode
the override normally.

This partly reverts f100c2f5ff1a626fa4f467973e5cff7e5b2d3692

[TSan] Yet another idea about whitelisting data races (#5310)

* Introduce racing.h
- introduce the `RacingIncrement* ()` functions
- test the `RacingIncrement* ()` functions with races of `mono_stats` in class.c
- add racing.h to Makefile.am, libmonoutils.vcxproj and msvc/libmonoutils.vcxproj.filters

* [fixup!] Rename "racing" to "unlocked"
- rename racing.h to unlocked.h
- update new file name in Makefile.am and *.vcxproj* files
- rename the functions from Racing* to Unlocked*
- update the macro logic

[msbuild] Bump to xplat-master branch HEAD

[mono-symbolicate/mdoc] Don't fail tests in mcs mode (#5348)

The checked in test files which the test suites in mono-symbolicate
and mdoc verify their output against have a few roslyn-specific
things in them and would fail when running in mcs mode.

While in theory we could check in both mcs- and roslyn-specific
test files it seems not worth the effort and we should just skip
the tests in mcs mode so we get greener builds on platforms
where we still use mcs by default.

Revert "[runtime] Avoid building a separate static library, use the shared object files for the static library as well. (#5233)"

This reverts commit 56a1881bff83365978e7eb30d18da8809b56c1e2.

It caused a consistent test error on ARM64:
https://jenkins.mono-project.com/job/test-mono-mainline-linux/label=debian-8-arm64/2810/testReport/regression-tests/dummy/MonoTests_regressions_100percentsuccess/

Update CODEOWNERS

[interp] builtin types (#5339)

* [interp] support System.{nint,nuint,nfloat} (64bit only so far)

* [interp] 32bit support for System.{nint,nuint,nfloat}

* [fixup] explain mint_niy

[System]: Fix memory leak in AppleTls. (#5305)

* [System]: Fix memory leak in AppleTls.

* Apply Miguel's patch to use GCHandleType.Weak.

* Actually assign context.

Merge pull request #4379 from ermshiperete/master

[runtime] Fix bug when marshaling COM interfaces in arrays

Default powerpc to mcs instead of roslyn (#5326)

It is crashing still way more than it works and now that the mcs
configuration works again we should default to that to make our CI
more pleasant.

[runtime] Remove ves_icall_object_new_fast (), its only a bit faster than object_new_specific (), fast allocation is done by the managed allocators. Also remove mono_class_get_allocation_ftn () which is not needed anymore. (#5304)

[btls] Compile btls using ninja if available. (#5333)

[eglib] (goutput.c) Revert Android logging host-target change

Revert part of 590473d01b3a3e11f87b9f2a8af3023926efca00 - the Android-specific
logging in eglib/goutput.c should be used when the host is Android, not the
target.

Fix allocation of memory to match the way it gets freed

Since we use g_free() to free the memory we should use g_malloc()
to allocate it.

[runtime] Fix bug when marshaling COM interfaces in arrays

* use the correct COM interface when creating native array
* do not free CCWs when freeing the native array

This fixes Xamarin-47560.

[runtime] Unit test for Xamarin-47560

Fix xdebug (#5309)

* [mini] fix xdebug null pointer dereferencing

Some trampolines are nameless, avoid crashing when generating their
debug info.

This change is released under the MIT license.

* [mini]: image-writer - move definition of USE_BIN_WRITER to header file

USE_BIN_WRITER is being used by xdebug for conditional compilation.

This change is released under the MIT license.

* [mini]: xdebug - obtain definition of USE_BIN_WRITER from header file

This re-enables xdebug on supported platforms.

This change is released under the MIT license.

[runtime] Fix a case where klass->blittable was written without holding the loader locked. (#5331)

[btls] Enable ccache support for cmake. (#5332)

add #if MONO around change to make it easier for merging upstream referencesource changes for the xamaran folks

Call array sort function that takes comparison instead rather than allocating for a wrapper struct

Add implementation and tests for SizeValueSerializer

[mcs] Soften referenced assemblies unification rules. Fixes #58621

Merge pull request #5335 from mono/master-bump-fsharp

Bump F# to 4.1.25 and remove already merged patches

Bump F# to 4.1.25 and remove already merged patches

Bump API snapshot

Merge pull request #5317 from alexrp/master

[profiler] Some profiler API improvements.

[runtime] Avoid building a separate static library, use the shared object files for the static library as well. (#5233)

[jit] Remove unused mono_arch_print_tree () function. (#5258)

[runtime] Remove empty eglib/ dir.

[runtime] Move eglib into mono/eglib so it becomes a convenience library similar to utils/ etc. (#5297)

[profiler] Split method_leave callback into a method_tail_call callback.

With this, profilers can distinguish between the two, and also know what the
target method of a tail call is.

The call instrumentation flags have been renamed accordingly and a new flag has
been added for instrumenting tail calls. I've also added a flag for exceptional
leave events, and method_exception_leave is only raised if this flag is given
for a method.

Also refactored the relevant JIT code a bit so it's easier to follow.

[profiler] Mark generated raise functions as ICALL_DECL_EXPORT.

This is currently needed for method_enter, method_leave, and gc_allocation.

[runtime] Declare MonoProfilerCallContext::return_value offset.

[runtime] Add ICALL_DECL_EXPORT.

ICALL_EXPORT is only used on function definitions, not declarations.

[profiler] Properly clean up the resources used by the profiler API on shutdown.

Also introduce mono_profiler_set_cleanup_callback ().

[profiler] Some improvements to the code coverage API.

Code coverage must now be explicitly enabled. This simplifies initialization
of some coverage-related structures and also lets us enable debug info which
is needed to produce good data.

Also change the signature of mono_profiler_get_coverage_data so it returns a
value indicating whether coverage data was available.

[msvc] Update csproj files

[genproj] Simplify strong name key handling

It doesn't need to be in a separate PropertyGroup.

[mono-threads-posix] Fix a buffer overflow (rather an information leak / over-read) (#5285)

[mono-threads-posix] Fix a buffer overflow (rather an information leak / over-read)

While testing Clang's AddressSanitizer, I found and fixed a buffer overflow.

Up for discussion: it might be worth importing `strlcpy` (http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib/libc/string/strlcpy.c?rev=1.11) for similar purposes as it provides additional safety and also helps with the readability?

Merge pull request #5321 from urisimchoni/prof-lin

Profiler - inlining fixes

[profiler] Avoid instrumenting inlined methods

An inlined method does not require instrumentation (if it
did, it would not have been inlined). The flags that
control instrumentation options are valid for the caller
method and are not to be consulted.

This patch replaces an assertion with a check and early
return. A method that "wants to be instrumented" does not
get inlined, but a method that "doesn't want to be instrumented"
can get inlined. However, the flags that control instrumention
options have been calculated against the caller method, hence
they should not be consulted at all. The extra check renders
the assertion unnecessary.

[profile] set the "no profiling" bitmask equal to zero

This makes constructs such as "if (!mono_profiler_get_call_instrumentation_flags(methos))"
correct. See mono_method_check_inlining()

Merge pull request #5313 from lambdageek/corefx-sre-fixup

[sre] Fix some argument checks, enable xtests

[System.ServiceModel.Web] Simplify AssemblyInfo.cs

We don't need separate AssemblyInformationalVersion for mobile and
non-mobile.

[ci] Add CI_TAGS option for --disable-mcs-build

Useful if you don't want to build the class libraries e.g. for
some static analysis of the runtime.

Bump version to 5.7 (#5325)

[w32handle] Remove use of w32handle for File, Console, Pipe and Socket (#5319)

* [w32file] Remove io_ops indirection

* [w32file] Replace use of w32handle by a GHashTable for File, Console and Pipe

* [Socket] Avoid relying on MonoIO.DuplicateHandle

* [w32socket] Remove force closing of sockets on shutdown

* fixup! [Socket] Avoid relying on MonoIO.DuplicateHandle

* [w32socket] Replace use of w32handle by a GHashTable

* [w32handle] Remove dead function mono_w32handle_new_fd

* fixup! [w32file] Replace use of w32handle by a GHashTable for File, Console and Pipe

* [w32handle] Remove dead variable mono_w32handle_fd_reserve

* [fdhandle] Factor w32file and w32socket common fdhandle code

* fixup! [Socket] Avoid relying on MonoIO.DuplicateHandle

* [w32socket] Fix GC safe/unsafe transitions

* fixup! [w32handle] Remove dead variable mono_w32handle_fd_reserve

[eglib] Add back NEWS file, it's required for the tarball

Revert "Bump version to 5.7"

This reverts commit 82eafc02e4774ed567237d15f0f24b13a674da38.

Bump version to 5.7

[System.Drawing] Simplify mobile profile check

[bcl] Specify the name of the key file on the command line instead of using the AssemblyKeyFile attribute for consistency and to make it easier to do path name manipulation on the file name. (#5316)

Merge pull request #5320 from urisimchoni/prof-sample

[profiler] fixes to profiler sample

[profiler] fixes to profiler sample

- Fix the Linux build instructions.
- Remove eglib dependencies, as they complicate the build of an
  extenral module.
- Make instance static instead of heap-allocated.
- Make the call counter 64 bits.
- Adjustments to accommodate the new profiling API.

Revert "[w32handle] Remove use of w32handle for File, Console, Pipe and Socket (#5283)" (#5318)

This reverts commit 78735091b62b8d7f65da2c83d36ca314a23009f4.

[w32handle] Remove use of w32handle for File, Console, Pipe and Socket (#5283)

* [w32file] Remove io_ops indirection

* [w32file] Replace use of w32handle by a GHashTable for File, Console and Pipe

* [Socket] Avoid relying on MonoIO.DuplicateHandle

* [w32socket] Remove force closing of sockets on shutdown

* fixup! [Socket] Avoid relying on MonoIO.DuplicateHandle

* [w32socket] Replace use of w32handle by a GHashTable

* [w32handle] Remove dead function mono_w32handle_new_fd

* fixup! [w32file] Replace use of w32handle by a GHashTable for File, Console and Pipe

* [w32handle] Remove dead variable mono_w32handle_fd_reserve

* [fdhandle] Factor w32file and w32socket common fdhandle code

* fixup! [Socket] Avoid relying on MonoIO.DuplicateHandle

* [w32socket] Fix GC safe/unsafe transitions

* fixup! [w32handle] Remove dead variable mono_w32handle_fd_reserve

Bump boostrap mono dependency

Bump corefx

[ci] Disable corlib-xunit on Windows for now

It seems to fail.

[mcs] Adds handling for declared variables from while loop condition

Bump API snapshot submodule

Bump cecil and linker

[test] (corlib) Enable more xtest SRE tests

- MethodBuilderDefineParameter
- ConstructorBuilderDefineParameter
- FieldBuilderSetCustomAttribute
- GenericTypeParameterBuilderSetCustomAttribute
- FieldBuilderSetOffset
- ModuleBuilderDefineInitializedData
- ModuleBuilderDefineUninitializedData

[sre] ModuleBuilder.DefineUnitializedData argument checks

1. Fix off by one in size check.
  Largest usable size is 0x3effff, first unusable size is 0x3f0000
  (Also change order of ArgumentException arguments - param name is second.
   Set it to null because of misguided CoreFX SRE test that expects a null name)
2. Check for empty string field name.

[sre] negative FieldBuilder.SetOffset should throw ArgumentException

[sre] Add explicit null checks of CustomAttributeBuilder arguments

[sre] ConstructorBuilder.DefineParamter(0,...) shouldn't throw

.NET Framework says:

> If you specify 0 (zero) for iSequence, this method returns a ParameterBuilder
>  instead of throwing an exception. There is nothing useful that you can do
>  with this ParameterBuilder.

So let's do that.

[runtime] Install profiler-events.h as a public header.

[bcl] Grab free ports randomly in NetworkHelpers (#5312)

* [bcl] Grab free ports randomly in NetworkHelpers

We're frequently seeing "address already in use" errors on Jenkins.

The theory is that when we're running tests and grab the next free
port via our custom NetworkHelpers we're getting a port which will
also be returned to a simultaneously running test (e.g. another chroot)
because we're closing the TcpListener and thus releasing the port until
we start using it in actual test code. By that time the other test
might've already opened the port, causing our test to fail.

Instead we now try to use a random port in the range 10000-60000
and try if it's available. This doesn't completely fix the inherent
race but should hopefully make it way less likely.

[sre] Throw ArgumentOutOfRange exn in MethodBuilder.DefineParameter if called
before SetParameters.

Fixes xunit testpoint MethodBuilderDefineParameter:DefineParameter_NoParameters_NonZeroPosition_ThrowsArgumentOutOfRangeException

Merge pull request #5306 from lambdageek/corefx-sre-tests

[test] (corlib) Use CoreFX System.Reflection.Emit xunit tests

Merge pull request #5303 from lambdageek/cattr-bcheck

[custom_attrs] Add bounds checking to custom attribute parsing

[btls] Disable BUILD_SHARED_LIBS while building boringssl, so its symbols are not exported from the btls shared lib. (#5307)

[mcs] Fixes codegen for type pattern matching in probing expressions over hoisted local variables

[mcs] Flow analysis on reduced expressions need to run on original expression

[mcs] Warnings clean up