Merge pull request #1659 from alexanderkyte/stringbuilder-referencesource

Fixed StringBuilder marshaling capacity error, and then the tests broken by fixing it.

Merge pull request #1658 from esdrubal/spgenerics

Improved generics in exception's stack traces and mono-symbolicate.

[mono-symbolicate] Updated tool test.
Test now checks generic methods, generic classes, methods overloads, ref/out parameters.

[mono-symbolicate] make check now runs mono with -O=-inline.
Inlining otimization is disabled so the full call stack can be displayed.

[mono-symbolicate] Improved tool stack trace handling.
Generics are no longer a problem.

[corlib] Changed Exception.GetFullNameForStackTrace to .NET behavior.
.NET displays the method definition generic argument names instead of types.

[corlib] Exception stack traces now show method index if needed.
Method index will be set if the method metadata token is shared by multiple generic specializations.
For StackFrame.GetILOffsetFromFile to work with generics methods a method index is required.

[runtime] .msym now stores method index.
The method index is required to disambiguate between generic methods that share the same metadata token.

[runtime] Added function mono_aot_find_method_index.
Method index will be included in some aot stack traces, when method metadata token is not enough (e.g: generic methods).

Merge pull request #1660 from akoeplinger/fix-zone-url-with-at-sign

[corlib] Don't strip @ char in Mono.Security.Uri for absolute path

[runtime] Avoid a crash in mini_thread_cleanup () for attached threads.

[readme] Remove xamcore reference.

Bump referencesource to pick up 3176b17a2c636dfd5f885435f8449598ecd4197f.

[runtime] Initialize String.Empty from native code to allow removal of String:.cctor ().

[mcs] Report abstract member hiding only for members which one can override. Fixes #28213

[mkbundle] Use mingw assembler on windows.

[corlib] Don't strip @ char in Mono.Security.Uri for absolute path

When creating a Mono.Security.Uri with the string "file:///var/lib/jenkins/workspace/test-mono-mainline@2/..."
(happens sometimes in System.Security.Policy.Zone-related unit tests on Jenkins) the current implementation
strips everything up until the @ sign.

Zone thinks the file doesn't exist locally as the Uri.LocalPath doesn't refer to a valid file on disk
anymore and this results in an incorrect zone value, making the tests fail.

Fixing this by treating the @ char differently when it is in an absolute path (as is done for other chars).

[System]: Fix the AllowWriteStreamBuffering check.  Bugs #26640 and #26936.

Added new tests for this to the web-tests:
https://github.com/xamarin/web-tests/commit/e7b5ac42897e629e9e4107cf6f9ae9e0a9fef45a

and re-enabled this one, which regressed:
https://github.com/xamarin/web-tests/commit/046da02c631c8799c7ab5025cd3b685f4a3e66f6

Bump referencesource and add test for the problem.

Update failing test

[System.Runtime.Serialization.Formatters.Soap] A test fix to remove hashtable order dependency

[corlib] Fixed StringBuilder construction bugs in marshalling caused by changes to capacity setting.

Revert "[mcs] Special case flowanalysis of primitive structs only when they are imported. Fixes #28296"

This reverts commit b50e898ece06bf7dacbea6211406cca1af5ea5af.

[threading] Reorganize code to work with the coop backend.

This splits the native backend from core functionality it needs to provide. For example,on posix, we want to
use the pthread_create code regardless of the backend enabled.

[threading] Introduce a pair of states to handle native suspend.

Native code should not normally be in runnable state. That's reserved only for managed code. In theory.
In practice any code that can witness the managed heap must be in the runnable state.

This in practice means all runtime code (icalls) and embeders. Right now lets focus on the runtime side.

The solution for this is to wrap all calls that could lead to unbound suspension in code that coordinates
with the GC - most syscalls that can block. This is realized with a pair of additional thread states.

1)Blocking: This the state where the runtime is doing something that can potentially block. The suspend count
will be > 0 if the current thread is suspended. In practice it won't suspend, just wait on resume.

2)Blocking and suspended: When leaving the blocking state, if the thread has a pending suspend, we transition
to this state while waiting for resume.

[threading] Move jit_data from MonoThreadInternal to MonoThreadInfo. Cleanup jit attach code to support earlier attaching.

This move allows us to save the thread state much earlier on. In addition to the move, we now produce thread contexts without
the JIT TLS set. This might end up been an issue in some corner cases, just like with domain, so we'll address those when
they show up.

[threading] Surface mono_thread_info_current_unchecked.

[threading] Log the thread tid instead of the MonoThreadInfo pointer.

Other places are already logging tid so this makes correlation possible.

TID is a better pick than the MonoThreadInfo pointer as it's easier to find the thread on gdb.

[threading] Add configure option to enable cooperative suspend.

[mcs] Special case flowanalysis of primitive structs only when they are imported. Fixes #28296

[jit] Trucate the shift amount in the long shift emulation functions to better mirror the behavior of the processor shift instructions.

[corlib] Add missing file

Merge pull request #1644 from ztone/eval_class_attr

[eval] Allow class attributes

[mcs] Resize identifier buffer in all cases. Fixes #28292

[corlib] Fixed wrongly changing capacities in stringbuilder marshalling

[mcs] Fix parsing of interpolated strings inside conditional expression. Fixes #28293

Fix a typo and code indentation in readme.md

[corlib] String from reference sources, fixes mostly string.format issues

[runtime] Fixed the build on Android by relying on header checking for system headers.

[runtime] Fixed complex.h usage in MS's thread pool to work in Android.

Android does not provide complex.h in the API levels that we compile against so bundle our own header (based on the one from FreeBSD).

Turns out GCC already provides built-in handling for the complex functions used by the code so there's no need to provide our own implementations.

Merge pull request #1657 from alexrp/disable-perf-events

[profiler] Disable `perf` events by default.

[io-layer] Fix the freeing of MonoProcess structures. Fixes #28209.

[profiler] Disable `perf` events by default.

As it turns out, `perf` support crashes/hangs on a lot of Linux
kernels in the wild, and produces no output on some Android devices.
So, given that using a signal + helper thread is more reliable, use
it by default and make `perf` support optional.

The configure flag is now `--enable-perf-events`.

[profiler] Avoid a slow path in mono_jit_info_table_find ().

[profiler] Really fix looking up JIT info for AOT'd methods.

mono_aot_get_method () will not necessarily put methods into the
MonoDomain::jit_code_hash table. But it does return an IP that we
can use to look up the JIT info with mono_jit_info_table_find ().

[mcs] Allow built-in types to have partial declaration

[runtime] Revert eaf200b37bd8ba29d59fc7054f8409e9b76b5697 and all fixes to it since it still causes hangs.

[runtime] Avoid returning null from Marshal.AllocCoTaskMem () if size is 0. Fixes #28383.

[runtime] Fix list removal in eaf200b37bd8ba29d59fc7054f8409e9b76b5697.

Add submodule instructions

[MonoDataSqlite] Add test for Xamarin.iOS specific bug 27864

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

As of iOS 8.2 Apple updated Sqlite and broke queries that used to work
pre iOS 8.2 like the select command in this test, The pruppose of this
test is to know when apple fixes this. Expected test behaivour is as follows.

* If iOS 8.2+ Test will pass as long as apple does not fix the bug
* If iOS < 8.2 Test will pass as it does work as expected
* If iOS 8.2+ and apple fixed bug 27864 test will fail and we will need to
  remove the fail check on lines 105 and 106

[amd64] Fix OP_SHR_IMM/OP_SHL_IMM.

[runtime] Fix another regression caused by eaf200b37bd8ba29d59fc7054f8409e9b76b5697.

[arm] Handle OP_SEQ_POINT opcodes during peephole optimizations correctly.

Merge pull request #1655 from ghuntley/patch-1

corrected spelling mistakes.

corrected spelling mistakes.

[runtime] Fix an uninitialized memory error added by eaf200b37bd8ba29d59fc7054f8409e9b76b5697.

[io-layer] Fix the freeing of MonoProcess structures. Fixes #28209.

[tests] Tweak MemoryMappedFile test case that fails on iOS/ARM64 [#27667]

Filled with Apple. From bug report:

The problem appears to be caused by a bug in mmap on the arm64 ios.

In this test, we request a page and mmap returns successfully, but when trying
to access the mapped memory, the program crashes after the 4097th byte, meaning
that it only maps us 4kb(???). It should be mentioned that on arm64 ios the
page size is 16kb.

[corlib] ngen attributes from reference sources

[corlib] Remove unused files

[corlib] ExceptionServices from reference sources

[corlib] Debugger attributes from reference sources

Merge pull request #1634 from StephenMcConnel/bug-28025

Show thumbnail images in SWF.(Open,Save)FileDialog

[runtime] Remove unnecessary winbase.h header include on Windows

Merge pull request #1652 from debuglevel/bugfix-crosscompile

fixed case-sensitive file name for cross compilation

Merge pull request #1588 from BrzVlad/feature-aot-wbarrier

AOT write barrier method

Merge pull request #1638 from esdrubal/servicepoint

[System] Fixed RemoveConnectionGroup concurrent issue.

Merge pull request #1632 from alexanderkyte/bug24118

[corlib] Fixed change in 4b4ddb66b3 that prevented use of remoting fast path

[system] FrameworkName from reference sources

[mcs] Emit correct assignment to ref pointer

Fixes build

[system.windows.forms] Add AssemblyNamesTypeResolutionService to correctly fix 78b63f29d50d08ac2826f95164433a5ff3c843ae

Typo fix...

[arm] Also fix OP_ATOMIC_STORE_R4.

Man page update

Merge pull request #1596 from schani/feature-sgen-improvements

Small SGen fixes and improvements

[sgen] Fix slot size calculation.

The way we calculated slot sizes was very brittle when it came to
small slots.  Reducing the size of the block header could trigger it
skipping an important small size class, which would lead to increased
memory usage and slower execution times.

Now we always produce all possible small slot sizes, up to four times
the minimum object size.

[sgen] Remove stealing.

We only have one worker thread.  There is no point in stealing.

[sgen] Save more space in M&S block infos.

[sgen] Do pinning of M&S blocks in one pass.  Saves time and memory.

[sgen] Store worker's job names for debugging.

[sgen] Allow skipping of entries in `sgen-grep-binprot`.

[sgen] Some clarifying comments and rearrangements.

[sgen] Remove dead code.

[sgen] Binary protocol entry for pin-staging cemented objects.

[sgen] Only scan mod-union in LOS on marked objects.

[sgen] Free lists in blocks must be volatile.

[sgen] Rename a callback for clarity.

[sgen] Don't scan until jobs done in worker thread.

The jobs must always have priority because the concurrent collector
waits for them to finish before restarting the world.

[sgen] Fix names and remove superfluous parameters.

[sgen] Move debug code to `sgen-debug.c`.

[sgen] Rename `have_computed_minor_collection_allowance` to `free_swept_blocks`.

Because that's what it actually does.

[sgen] Remove superfluous function indirection.

[sgen] Replace `have_swept` pointer with callback.

We want to use a state variable instead of `have_swept`, so we need a callback
to abstract.

[sgen] Don't pass redundant `scan_mod_union` to `major_finish_collection()`.

[sgen] Disable cementing for concurrent M&S.

The code just worked because it never reset cementing.  It needs to be
completely rewritten.

With cementing enabled, at the start of a major concurrent
collection, we cannot reset it.  That's because if we have
cemented objects, some remsets to them will be missing, and the
major collector does not scan the whole heap immediately, like
the non-concurrent one does.

The proper way to do it seems to be to reset cementing before
finishing a concurrent collection.  All major objects that point
to formerly cemented objects must either be marked in the
mod-union table or the current card table.  When we scan the card
tables, we should be able to just evict objects.

Currently, the finishing collection pause still does some
scanning via the worker thread.  This might be a problem.

[sgen] Don't re-check reference after marking in concurrent mark.

If the reference was modified, it was done by the mutator, in which case it will
have marked the card if necessary.

[tests] Tweak DateTime.AddMillisecondsOutOfRangeException* tests to work on ARM [#28311]

Fix decimal comparison by using signed char comparison of signs; Fix size of jmp instruction sequence