Bump API snapshot submodule

Bump cecil and linker

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

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)

[sre] Add MonoDynamicImageTokCollision arg to mono_dynamic_image_register_token

The new argument controls what should happen when the given token is already
present.

In some situations it's expected that we either see the same object, or we
unconditionally want to replace it (for example if we previously registered a
TypeBuilder or a MethodBuilder, after the type or method is created, we will
re-register the RuntimeType or MonoMethod in its place).  In other cases (for
example when a module references a MonoMethod from a different assembly), we
may still call `mono_image_create_token()` multiple times but we expect to see
the same MonoMethod object.

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

This may occur if there are two AssemblyBuilders in progress and one of them
refers to a TypeBuilder from the other.

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

[test] Check that one AssemblyBuilder can refer to a TypeBuilder from another.

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

[interp] Make the 'td' variable a pointer in generate () for consistency with the rest of the code.

[interp] Rename some types to better reflect their function.

MonoInvocation -> InterpFrame
RuntimeMethod -> InterpMethod
runtime_method -> imethod

[mcs] Another workaround for enum builder SRE crash (this time on Mono)

Updates codeowners

[runtime] Move object.c to 'common_sources', as it no longer has compile time gc dependencies.

Sre not gc dependent (#5264)

* [runtime] Move the sre source files to 'common_sources', they no longer have compile time gc dependencies after the Boehm GC root changes. sre.c still depends on the gc dependent caching code in reflection-cache.h.

* [runtime] Remove compile time gc dependencies from the reflection caching code.

* [runtime] Move reflection.c/sre.c to 'common_sources', they no longer have compile time gc dependencies.

[corlib] Update Assembly class xml descriptor

[msvc] Update csproj files

[sre] Add some tests for saving assemblies. (#5265)

[Mono.CSharp] Updates csproj

[mcs] Adds workaround for repl enum declaration on .net SRE

[corlib] ModuleBuilder pseudo-token lookup needs to use references insteads of logical equality comparer.

Fixes #58291

[acceptance-tests] Another attempt fix a StringBuilder race in the profiler stress runner.

[runtime] Remove compile time gc dependencies from object.c. (#5266)

[runtime] Remove compile-time gc dependencies from domain.c/appdomain.c. (#5263)

* [runtime] Remove compile-time gc dependencies from domain.c/appdomain.c.

* [runtime] Move domain.c/appdomain.c to 'common_sources' since they no longer have compile time gc dependencies.

Merge branch 'vladimir-kazakov-xml-dsig-transforms-from-corefx'

Merge branch 'xml-dsig-transforms-from-corefx' of git://github.com/vladimir-kazakov/mono into vladimir-kazakov-xml-dsig-transforms-from-corefx

[msvc] Update csproj files

[interp] Fix a copy/paste error in ea4e4a9ef6fc42570a23026adbe826cf7248290e.

Bump roslyn (to include more files)

Bump API snapshot submodule

Bump CoreFX

[runtime] Remove handler block trampoline code

[runtime] Fix handler block

Instead of hijacking the return address when suspending a thread in a finally block for an abort, do an explicit check in managed code when we are returning from the handler in order to see if we need to abort. Hijacking the return address is arch specific and it doesn't work if we are suspended before it is initialized in handler start.

Fixes #5798

Update bockbuild. Fixes #57002

[runtime] Fix the size calculation for CustomAttributeType coded indexes, they can only point to the MethodDef/MemberRef tables even through 3 bits are used to encode them, only 2 values are used. Fixes #58210. (#5256)

[jit] Handle OP_VCALL_MEMBASE in get_simd_vreg (). Fixes #58344.

Bump bockbuild.

Bump roslyn to 2.3.1

XMLDSIG transforms from .NET Core.

Bump bockbuild.

Update CODEOWNERS

Merge pull request #5252 from alexrp/profiler-stress-fixes

[acceptance-tests] A few fixes for the profiler stress test suite.

Merge pull request #5253 from alexrp/profiler-lib

[Mono.Profiler.Log] Catch SecurityException for missing icalls.

[Mono.Profiler.Log] Catch SecurityException for missing icalls.

This is the exception that is thrown on .NET because icalls are not allowed in
non-GAC assemblies.

[acceptance-tests] Disable the ironjs-v8 test on arm32/arm64.

It times out even with a 24-hour timeout. Eventually we should investigate why,
but disable it for now so the lane can become green.

[acceptance-tests] Fix a crash in the profiler stress runner caused by racey modification of a StringBuilder.

[acceptance-tests] Set individual timeout for profiler stress tests back to 8 hours.