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

[mcs] Pending implementation of accessors cannot hide base implementation with different member type

[ci] (run-test-default) Run the corlib xunit tests

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

[custom_attrs] Add bounds checking to mono_reflection_create_custom_attr_data_args

and load_cattr_value_boxed

[custom_attrs] Add bounds checking to load_cattr_value

[custom_attrs] Add (some) bounds checking to create_custom_attr

In principle, mono_verifier_verify_cattr_content should prevent malformed custom
attribute blobs from being passed in here.

In practice:

1. The verifier is not on by default

2. System.Reflection.Emit allows an arbitrary byte[] to be passed in which
   means that code like this can cause mono to read past the end of the array.

   ```
   // "1 1-byte constructor argument and then 65280 named properties follow"
   assembly.SetCustomAttribute(constructor, new byte[] { 1, 0, 1, 0x00, 0xFF });
   var attributes = assembly.GetCustomAttributes(true);
   ```

Updates codeowners

[msbuild] Add option to nuget-hash-extractor to generate a list of the

.. denied assemblies usable by a msbuild task for filtering them.

Bump ikdasm.

[tests] Disable test on llvm

On llvm aborts can happen during finally blocks. This test was not failing before because the method in question had llvm disabled, which has recently changed.

Merge pull request #5299 from cherusker/cherusker-2017-08-01-next-generic-inst-id

[metadata] Fix racy next_generic_inst_id

Like suggested by @luhenry, I would like to propose this fix which was discussed in https://bugzilla.xamarin.com/show_bug.cgi?id=58423. As `next_generic_inst_id` is `int`, `InterlockedIncrement64 ()` seems to be the right choice.

Also, I cannot add any reviewers or assignees, which is why I would kindly ask @lambdageek to review this as well :)

[profiler] Fix MonoProfilerState::context_free_buffer type.

[mono-api-html] Fix bogus IgnoreParameterNameChanges behavior.

It never ignored parameter name changes. Make the entire code just simple.

Fix racy `next_generic_inst_id`
- as discussed in https://bugzilla.xamarin.com/show_bug.cgi?id=58423
- in addition, unify `int` and `guint` into `gint32` which can be used flawlessly with `InterlockedIncrement ()`

[bcl] Fix Delegate.GetHashCode () so it takes into account the target too. Fixes #58399. (#5296)

[runtime] Fix msvc build.

Update CODEOWNERS

Merge pull request #5298 from baulig/work-offsets-profiler

[offset-tool]: Add MonoProfilerCallContext to MonoAotOffsetsDumper.

[offset-tool]: Add MonoProfilerCallContext to MonoAotOffsetsDumper.

[mini] Remove the functions that were unexported in b05d4b93226c83489a65934f780beddab3c2ef32.

[runtime] Declare MonoProfilerCallContext::method offset.

[jit] Fix the DISABLE_JIT build.

Add myself to CODEOWNERS.

[System]: Fix memory leak in BTLS. (#5291)

Merge pull request #5288 from lambdageek/bug-58454

[sre] Pass declaring type to GetMethodFromHandle in GenericTypeParameterBuilder:InternalResolve (Fixes #58454)

Merge pull request #5292 from alexrp/profiler-symbol-fixes

[profiler] Move legacy profiler code to profiler.c.

Update CODEOWNERS.

[profiler] Move legacy profiler code to profiler.c.

These symbols weren't being properly exported in the final mono executable on
Mac because no code in the runtime called these functions. As long as they're
defined in an object file that contains used functions, they'll be exported.

Merge pull request #5274 from alexrp/profiler-call-contexts

[profiler] Implement call context introspection for enter/leave events.

[mini] Add a mono_jit_aot_compiling () function.

[mini] Remove some unstable exports that are no longer used by the profiler.

[profiler] Implement call context introspection for enter/leave events.

When this feature is enabled for a method, the enter/leave event receives an
additional argument, a so-called 'call context'. This call context contains
enough information about the stack frame of the instrumented method to allow
the enter/leave callback to inspect the 'this' reference, method arguments,
local variables, and the return value (for non-void methods).

This feature enables some interesting scenarios that were not possible with
the regular enter/leave events. For example, a profiler could instrument
well-known methods in the managed thread pool code to get an idea of how an
application is using the thread pool, or it could instrument network-related
methods to gather statistics or even log all network traffic.

This is implemented by storing a MonoProfilerCallContext on the stack, whose
MonoContext field is populated by executing an OP_FILL_PROF_CALL_CTX opcode
which stores the stack pointer, frame pointer, and all callee-saved registers
to it. For the epilogue, a pointer to the return value (for non-void methods)
is also stored in the MonoProfilerCallContext. An address to this context is
then passed to mono_profiler_raise_method_enter/leave. Based on debug info,
all arguments and locals can then be looked up in the instrumented method's
stack frame.

For the interpreter, we just store an InterpFrame pointer on the
MonoProfilerCallContext and look everything up from that. We don't need debug
info in this case.

This feature is currently not supported with LLVM (for regular LLVM mode, it
will fall back to Mono's JIT, while for LLVM-only mode, it's not available).

I also refactored the interpreter code so that enter/leave events are generated
not only when interpreter debugging is enabled. Also, the interpreter will only
call mono_profiler_get_call_instrumentation_flags () once per method now.
Finally, I made the interpreter also generate exception leave events.

[sre] Pass declaring type to GetMethodFromHandle in
GenericTypeParameterBuilder:InternalResolve (Fixes #58454)

When the parameter belongs to a method builder, we need to pass the declaring
type (appropriately resolved) to GetMethodFromHandle to avoid an
ArgumentException.

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

[ci] Compile the class lib tests in a separate step

Compiling in parallel via -j should also be faster.
We had this on Wrench but it didn't make it over to Jenkins.

[mono-symbolicate] Run tests with "run-test" target instead of "test"

The "test" target is used to precompile the tests but actual running
should happen with the "run-test" target.

[test] Check that GenericTypeParamBuilder:InternalResolve passes declaring type
to GetMethodFromHandle

Regression test for https://bugzilla.xamarin.com/show_bug.cgi?id=58454

Merge pull request #5234 from vkargov/vk-kiokunegi

[jit] Check method cache first in mono_get_method_constrained_checked.

[corlib]: Fix race condition in ExceptionDispatchInfo. (#5279)

[llvm] Fix the calling of fault clauses in llvm compiled code. Fixes #58446. (#5286)

[mcs] Implements C#7 expression bodied members enhancement

[mcs] Add parser sequence for case type pattern matching

Merge pull request #5260 from BrzVlad/fix-handler-block

[runtime] Fix handler block abort

Merge pull request #5272 from lambdageek/bug-58421

[sre] Handle typeref tokens in fixup_method (Fixes #58421)

[mcs] Implements cloning for yield break. Fixes #57796

Eglib integrate (#5267)

* [runtime] Integrate eglib's configure into mono's configure to simplify and speed up the configure process.

* [runtime] Delete some duplicate eglib configure code.

Merge pull request #5271 from lambdageek/bug-58361

[sre] Resolve EnumBuilder tokens in managed (Fixes #58361)

Bump corlib version

Due to changed semantics of ModuleBuilder:RegisterToken icall (it now does not
allow an EnumBuilder argument)

[sre] Resolve EnumBuilder tokens in managed. (Fixes #58361)

In particular, delegate to the TypeBuilder within the EnumBuilder.

Also do not register the EnumBuilder object with the runtime.  (Fixes
https://bugzilla.xamarin.com/show_bug.cgi?id=58361)

[test] EnumBuilder regression tests for #58361

Cause the creation of a token for an EnumBuilder object.  Test both typedef
tokens (EnumBuilder from same assembly) and typeref token (EnumBuilder in another assembly)