Merge pull request #5382 from kumpera/pedump_fix

Address pedump issues from https://bugzilla.xamarin.com/show_bug.cgi?id=58168

[msbuild] bump to xplat-master branch HEAD

[msbuild] Install a `deniedAssembliesList.txt` to support denied ... (#5358)

... assemblies.

msbuild tries to filter out the denied assemblies, which mono refuses to
load, during a build. It uses the `deniedAssembliesList.txt` installed
to `$mono_prefix/lib/mono/xbuild` to identify them.

This was generated with:

 `$ mono tools/nuget-hash-extractor/nuget-hash-extractor.exe nugets guids_for_msbuild > mcs/tools/xbuild/data/deniedAssembliesList.txt`

This file needs to be kept in sync with any changes in the assemblies
denied by the runtime.

msbuild PR that adds support for this -
https://github.com/mono/msbuild/pull/23 .

[msbuild] bump to xplat-master branch HEAD

[verifier] Add workaround for verifying corlib and valid-only output.

[verifier] Handle csc encoding of enums in cattr blobs. Fixes #58168.

[runtime] Add WINDOWS_RUNTIME type attribute that csc  generates

[verifier] Avoid double init of metadata when using pedump.

This is a workaround for the larger problem of it using a non-standard init sequence.

Tracked as https://bugzilla.xamarin.com/show_bug.cgi?id=58793

[verifier] Partially revert 6d37ffdf534eae04a5d06b2e097762bd64fc0591. It adds internal a header to a public one.

netstandard dependant facades update after 2.0 release (#5371)

mkbundle: avoid NREs and print useful errors

When scanning for references, emit an error when loading the assembly
into the universe fails or when loading a referenced assembly by
name fails.

[msbuild] bump to help debug wrench build failure

Add scripts/csi to .gitignore.

Cosmetic; remove unnecessary cast and fix compiler warning.

[System]: Fix minor leaks in AppleTls. (#5369)

These should be automatically collected by the GC, but explicitly disposing
will keep memory usage more stable over time and keep pressure off the GC.

[System]: Fix major AppleTls memory leak.  Bug #56814. (#5370)

[runtime] Use HOST_ defines instead of PLATFORM_ defines. (#5362)

Merge pull request #5355 from kumpera/fix_58379

[mini] Emit widen ops when storing to locals of size < 4. Fixes #58379.

[msbuild] bump to help debug wrench build failure

[MacSDK] Update bockbuild to work around build/staging issue
Commit message: Allow for non-fatal shell execution, and turn “install_name_tool” staging failures to warnings as we find a more permanent solution to AOT dylibs not always having enough field space for the new path (all other dylibs made in the build with the XCode toolchain typically do, as they are built with the “-headerpad_max_install_names” flag.

[mini] Add test for implicit widen on argument stores.

[remoting] transparent proxy GetType of an interface should return the
interface Type, not MarshalByRefObject. (Fixes #17325)

If a RealProxy instance is created (using the protected RealProxy(Type)
constructor) for an interface, mono internally sets up a MonoRemoteClass which
looks like a proxy for a MarshalByRefObject instance that happens to implement
the given interface.  However mono_class_proxy_vtable just does a memcpy() to
setup the MonoVTable:type field of the transparent proxy's vtable.  That type
field is hit by the JIT when System.Object.GetType() is called - and on .NET
Framework GetType for a transparent proxy of an interface returns the interface
not (as Mono used to) typeof(MarshalByRefObject).

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

[test] Add regression test for GetType() of a transparent proxy.

Check that calling GetType() on a transparent proxy of a RealProxy of an
interface returns that interface type.

[runtime] Fix a warning.

Bump nuget to latest 4.3.0 release (#5368)

Bump nuget-buildtasks

Includes https://github.com/NuGet/NuGet.BuildTasks/pull/35

Fix a warning.

Merge pull request #5360 from urisimchoni/fix-mon

[profiler] Fix log profiler SetMonitorEvents icall

[profiler] Fix log profiler SetMonitorEvents icall

Set the right flag when enabling/disabling monitor
event profiling

Use eglib g_* allocators rather than malloc/calloc (#5359)

[acceptance-tests] Increase the profiler-stress timeout to 9 hours.

msbiology seems to take longer than 8 hours on 32-bit OS X.

Bump roslyn-binaries.

[jit] Add implicit widening of stores to args of types < 4 bytes.

This fix follows what was done for #58379 but for arguments.

[ci] Add CI_TAGS for apidiff and ms-test-suite

Easier to maintain than the "label" which could change depending on the bots.

Install Roslyn binaries, including msbuild tasks to msbuild/Roslyn (#5251)

.. directory, which is `$mono_prefix/lib/mono/msbuild/15.0/bin/Roslyn/`

Mono installs `csc.exe` but the corresponding
`Microsoft.Build.Tasks.CodeAnalysis.dll` required for supporting msbuild C#
projects is installed by msbuild. But these two can get out of sync,
like they did and had to be worked around in
https://github.com/mono/msbuild/pull/19 .

Instead, we now install all the relevant roslyn bits (C# and VB) from the same
source. Only the msbuild specific files are copied to the Roslyn folder.
VBCSCompiler, csi and their dependencies are symlinked from `lib/mono/4.5`.

Also:

- csi.exe is included since the tool is surfaced as an msbuild task,
  and that requires the *Scripting.dll assemblies.
- VBCSCompiler* - shared compiler is installed
- vbc.exe doesn't work on mono currently, so skipping that
- And `Microsoft.DiaSymReader.Native.{amd64,x86}.dll` are dropped.

Address reviewer feedback.

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

[mini] Emit widen ops when storing to locals of size < 4. Fixes #58379.

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?