[msvc] Update csproj files

Remove unused facade

[System.Diagnostics.Tracing] Don't generate empty debyg symbols

[System.Data.Common] Add IDbColumnSchemaGenerator

[Facades] Don't build debug symbols for System.Net.Sockets

Merge pull request #4720 from Unity-Technologies/config-checks-ipv6

Add autoconf checks for platforms without IPv6

Merge branch 'master' into config-checks-ipv6

[jit] Remove some debug spew.

Bump corefx

[runtime] Add explicit (float) casts to test_0_float_precision () to make the test work when compiled with csc /O.

Merge pull request #4710 from Unity-Technologies/additional-config-checks

Add #ifdef checks for code that might not exist

[build] Fixes setup for libraries without sources

[msvc] Update csproj files (#4740)

[cil-stripper] Mark stripped methods as noinline to prevent the JIT from trying to inline them. Fixes #55041. (#4732)

Merge pull request #4727 from BrzVlad/fix-xamarin-studio

[metadata] Fix special static field access

[Facades] Use Open.snk for netstandard.dll

We were using the wrong key since dotnet/standard switched to using the new Open key.

[System] UriBuilder from corefx

Bump Roslyn to 2.1

[corlib] Use AppContextSwitches implementation from RS. Fixes #54448 (#4682)

Make the loading of llvm EH info async safe. (#4725)

* [aot] Emit class references in EH info in such as way that they can be skipped when reading them in async context.

* [aot] Rewrite the LLVM EH info loading code by moving allocations to callers so the lower level code is async safe.

* [aot] Make the memory allocation in the loading of llvm EH info async safe. Remove fields from MonoLLVMFDEInfo which are passed as parameters.

* [runtime] Add the ability to store the unwind info address/length in the MonoJitInfo structure instead of referencing it using the 'unwind_info' field.

* [aot] Make the loading of llvm EH information async safe.

[mdoc] Integrate api-doc-tools version

[runtime] Disable some tests in full-aot mode which cannot be AOTed because of type loading errors.

[metadata] Lock around special static fields hash access

If a type is loaded in another thread, the fields hash can get resized, leading to failure.

[amd64] Make inline function in header static

This was actually failing the osx build for me, probably due to symbol conflicts.

[metadata] Don't use static_data_array

Memory allocated with mono_gc_alloc_fixed is uncollectable both on sgen and boehm. This array serves no other purpose. It should probably be killed entirely, but it's part of the API.

Merge pull request #4701 from kumpera/jit_improvements

[JIT] Last two optimizations to Roslyn.

Merge pull request #4723 from lambdageek/bug-54485

[metadata] expand uninstantiated generic type definitions used as generic type args

[metadata] If a GTD appears uninstantiated in a generic instantiation argument,
replace it by its canonical open instantiation.

If we have
```
  class Foo<A,B,C> { ... }
  class Bar<T,U> { ... }
```

Then if someone tries to construct `Bar<Foo, int>` (or more precisely, the
MonoGenericInst corresponding to `<Foo, int>` instead return `Bar< Foo<A,B,C>, int >`.

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

[metadata] Add a canonical_inst MonoType to MonoClassGtd

The canonical instantiation of a generic type definition is the open
instantiation where the GTD is applied to its own generic parameters in order.

Suppose we have `class T<A,B> { ... }` then the canonical instantiation is just "T<A,B>".

[Mono.Options] CommandSet and PCL/NetStandard Support (#4719)

When `PCL` is defined, the intent is that the resulting
`Mono.Options.dll` assembly will be usable on all PCL profiles.
This includes "silly" profiles such as Profile259, which lacks the
`System.Console` type, meaning e.g. `Console.Out` doesn't exist.

Unfortunately, the
`CommandSet(string, Converter<string, string, TextWriter, TextWriter)`
constructor would use `Console.Out` and `Console.Error` when the
`TextWriter` parameters were `null`, meaning this constructor can't
be provided for all PCL profiles.

Altering the constructor to instead use `TextWriter.Null` would allow
the constructor to be provided, but would be a *worse* experience,
IMHO: anybody (everybody!) expecting the default values to
"do something useful" will be sorely disappointed, as output and
error messages would instead be dropped. :-(

Instead, remove this constructor:

public CommandSet (string suite, MessageLocalizerConverter localizer = null, TextWriter output = null, TextWriter error = null);

and replace it with these *two* constructors:

public CommandSet (string suite, MessageLocalizerConverter localizer = null)
public CommandSet (string suite, TextWriter output, TextWriter error, MessageLocalizerConverter localizer = null)

The first constructor, which accepts no `TextWriter` parameters, is
only provided in non-PCL builds, allowing it to use `Console.Out`.

The second constructor is available in PCL profiles, and *requires*
that `output` and `error` be non-`null`.

Documentation has been updated accordingly.

[tests] System.Type.MakeGenericType with a GTD as argument

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

Merge pull request #4722 from alexanderkyte/checked_after

[scripts] Don't build in checked mode

Fix README.md to show correct values for --with-libgc

External boehm is no longer supported.

[corlib/System.Data] Test fixes for xammac 4.5 and watchos

The TrimSpecial() method and Windows-specific code was removed from
xammac 4.5 so reflection can't find it anymore, handle this case.

Also add a workaround for SqlParameterCollectionTest.CopyToTest()
which fails on watchos since the SqlCommand would throw a PNSE.
It's a workaround because it just makes the test ignored since we don't
set the env var that ConnectionManager.Instance.Sql reads in XI tests,
but it makes the test consistent with the other tests in the suite.
We should fix this properly later on.

Add #ifdef checks for code that might not exist

* Some platforms don't support these features, so guard them.
* For platforms that do support them, no changes should occur.

[msvc] Update csproj files

[scripts] Don't build in checked mode

Add autoconf checks for platforms without IPv6

* There is an existing check for the `sockaddr_in6` struct.
* Use it to wrap code that implements IPv6.
* This makes porting to platforms that don't support IPv6 easier.
* It should have no impact on platforms that do support IPv6.

[System.Data] Call SqlConnection.ClearAllPools() only when we had a connection

When ConnectionManager.Instance.Sql.ConnectionString throws in
SetUp() we'd still call SqlConnection.ClearAllPools() in the TearDown()
which happens for example when the connection string environment
variable is not set.

This is a problem on WatchOS since SqlConnection.ClearAllPools()
throws PlatformNotSupportedException there, making the tests fail.
Instead we now check whether the connectionString is set and only
call the method in that case.

Bump cecil

Fix RemoteEndpoint with async+DualMode (#4717)

[Mono.CSharp] Fixes parallel build

[runtime] Shadow copy the .pdb files as well. Fixes #55095. (#4715)

[System] Don't include AlsaDevice on xammac_net_4_5

(cherry picked from commit 79ad720d5d6a595672012535116246920d32fba1)

[runtime] Forgot to update strelemref_wrapper_name with the new wrapper type.

[Mono.CSharp] Fixes parallel build

Merge pull request #4708 from xmcclure/orbis-system-net

Minor orbis fix in System.Net

[jit] Disable handler block trampolines+aot support on ios, it does an async stack walk, and the eh info for llvm compiled methods cannot be loaded in async context currently. Works around #54658. (#4713)

[configure] Fix docs for --with-libgc option (#4707)

It was renamed from --with-gc to --with-libgc in 66ca91f10b0fdd61824e318f659b807cbf45aef3,
but a few places were missed.

[msvc] Update csproj files (#4711)

Merge pull request #4712 from alexanderkyte/master-circular-fix

[runtime] Fix circular types in SRE

[runtime] Handle circular references in SRE types

We previously recursed through the entire directed cyclic graph
of the type passed to SRE. When given a (legal) circular type reference,
we encountered stack overflows. This was particularly easy to trigger
with fsharpi and with FAKE.

By creating everything about the type before searching for the parent,
and by defering all type hierarchy establishment until the type must be
surfaced to manage code, we tease out the data dependencies caught up in
the cycle.

Note the bounded lifetime of the unmanaged object hidden in the SRE
module, which acts as a global reference for all types reachable from
the TypeBuilder passed to the runtime through the icall.

[runtime] Test circular references in SRE

Minor orbis fix in System.Net

Bump nuget-buildtasks

[runtime] Blacklist System.Threading.Overlapped nugets

Update nuget-buildtasks from upstream's `dev15.1` branch (#4703)

* Update nuget-buildtasks from upstream's `dev15.1` branch

dev15.1 branch + fixup import paths

This was prompted by needing to support nuget restore for projects with
`PackageReference`.

* [nuget] Bump reference to NuGet.BuildTasks to pick up a build fix

Remove the use of ExpectedException in Mono.Posix Tests (#4698)

* Remove the use of ExpectedException in Mono.Posix Tests

ExpectedException is not available in later versions of
NUnit.  To test the NetCore 2.0 version of Mono.Posix I
need a newer version of NUnit that does not contain ExpectedException.

Using Assert.Throws should be equivalent.

* Mono.Posix Tests move CanUseRealTimeSignals checks

Moving the CanUseRealTimeSignals check outside of the
Assert.Throws callback function.

This is a change in test behavior requested by @akoeplinger
in the PR review.

* Fixing Mono.Posix.UnixSignalTest.TestSignumPropertyThrows

This test throws InvalidOperationException and we need to check
for that in the Assert.Throws type.

Add System.Net.Http 4.3.1 to blacklist

[corlib] Fixes static builds (#4687)

* [corlib] Fixes static builds

Place break inside if block

Bug 55148 - Debugger checks type which is possibly never used (#4690)

[JIT/GC] Optimize mono_gc_wbarrier_value_copy_bitmap.

mono_gc_wbarrier_value_copy_bitmap shown up when profiling Roslyn and this PR applies the following set of optimizations to it:

- Drop the bitmap, the extra branch in the loop costs more than the extra cardtable store itself
- Rename it to mono_gc_wbarrier_range_copy to reflect its new meaning
- Remove the JIT wrapper, this function doesn't need it
- Use mono_gc_get_range_copy_func as a way to punch through all layers and get the actual implementation
- Move the implementation to sgen-cardtable.c where everything can be inlined
- Exploit the fact we only need to mark the first card of a given range and thus we can hoist the address calculation outside of the loop

All of this speeds up a microbenchmark by 2x and Roslyn wallclock by around 2%.

[runtime] Optimize virtual_stelemref for classes even further.

This optimization exploits the fact that if the element type idepth is <= MONO_DEFAULT_SUPERTABLE_SIZE we can skip the idepth check.

This improves an array store only benchmark by 11%.

On Roslyn, all arrays falls into this case.

[w32error] Fix g_error args in mono_w32error_unix_to_win32

Empty commit to force a wrench build

Merge pull request #4569 from lambdageek/dev-coop-handles-march

Misc coop handles work

[appletls]: Disable the new keychain code on XI. (#4695)

(cherry picked from commit b6255d35cd224bc2ed63e52689fc336c31e63a34)

[WebConnection] Fix race condition between Close and BeginWrite (#4693)

* [WebConnection] Make ReadDone and InitRead instance methods to avoid passing cnc around

* [WebConnection] Inline only call to InitConnection

* [WebConnection] Fix race condition between Close and BeginWrite

[msvc] Update csproj files (#4696)

[custom-attrs] Use coop handles for mono_reflection_get_custom_attrs_info_checked

* use handles in icalls
  - ves_icall_MonoCustomAttrs_GetCustomAttributesDataInternal
  - ves_icall_MonoCustomAttrs_GetCustomAttributesInternal

[reflection] Move mono_reflection_type_get_handle to sre-internals.h

It's basically deprecated and only use in reflection until reflection is
converted to use handles.

[custom-attrs] Rename custom_attrs_defined_internal to ves_icall_MonoCustomAttrs_IsDefinedInternal

and use coop handles.

[remoting] Use coop handles for S.R.Remoting.Contexts.Context.{Register,Release}Context

* ves_icall_System_Runtime_Remoting_Contexts_Context_RegisterContext
* ves_icall_System_Runtime_Remoting_Contexts_Context_ReleaseContext

Added new mono_threads_{register,release}_app_context to be used internally in
the runtime.

[runtime] Use coop handles for ves_icall_System_AppDomain_DoUnhandledException

[domain] Mark mono_context_set external only

runtime should use mono_context_set_handle

[remoting] Move mono_context_{get,set} icall registration to remoting

It was previously in marshal

[domain] Use handles for MonoAppContext icalls

* ves_icall_System_AppDomain_InternalGetContext
* ves_icall_System_AppDomain_InternalGetDefaultContext
* ves_icall_System_AppDomain_InternalSetContext

[reflection] Use coop handles for MonoEventInfo.get_event_info

Bump CoreRT

[runtime] Avoid freeing a string passed to mono_mb_emit_exception_full (), it doesn't assume ownership of it. Fixes random wrench failures after 91ae1664061e8ac2d370da57c57d450bb00d64de.

Revert "[acceptance-tests] Disable a regressed coreclr test"

This reverts commit 5e8012a6787e4dc1bb6577c4e3572a2c8070dd05.

The bug was fixed.

[jit] Make newarr [void] unverified. Fixes #55083.

Merge pull request #4680 from lambdageek/bug-49721-take2

[loader] Ignore public key token in strong name match, if requested framework assembly was remapped

[btls]: Cleanup certificate store initialization (#4683)

* [BTLS]: Cleanup certificate store initialization.

* Kill unused MonoBtlsX509Store.AddTrustedRoots().

* In server-mode, MonoBtlsProvider.SetupCertificateStore() now only adds
  certificates explicitly trused via MonoTlsSettings.TrustAnchors.

* MonoTlsProvider.ValidateCertificate() - which is called from
  X509Certificate2.Verify() via X509CertificateImplBtls.Verify() - now
  uses MonoTlsSettings.DefaultSettings and assumes client-mode.

* [appletls]: Actually use the new code that was added in PR #4671.

* [btls]: Add new MonoTlsSettings.CertificateValidationTime property.

This allows you to set a custom time for certificate expiration checks.

* [appletls]: Bind SecTrustSetVerifyDate() and check MonoTlsSettings.CertificateValidationTime.

Fix check for file in gensources.sh

It didn't correctly check whether the file exists so we got
"tail: no such file or directory" messages during the build.

[runtime] Fix some small domain related memory leaks. (#4685)

[System.Core][System.ComponentModel.DataAnnotations] Update strings

Merge pull request #4675 from kumpera/runtime-scalability-take1

[runtime] Reduce contention on some top locks when running Roslyn.

[appletls]: Actually use the new code that was added in PR #4671.

Merge pull request #4615 from alexanderkyte/string_error_handling

[runtime] Don't discard string conversion errors

[ppdb] Fix a crash if mono_ppdb_get_seq_points () is only passed a 'source_file' argument.

[msvc] Update csproj files

[loader] Ignore public key token in strong name match, if requested framework assembly was remapped

When framework assemblies are compiled for the mobile runtime, they are
generally signed with a different key than the normal desktop one.
Unfortunately each assembly has different logic for which key is
used (typically silverlight.pub or corefx.pub in place of ecma.pub), and short
of hardcoding a table in the runtime, there's no good way to know.

[assembly] Add option to ignore public key tokens when comparing MonoAssemblyNames

Revert "[loader] Remap public key tokens of framework assemblies when running with --runtime=mobile"

This reverts commit af7195e3493bc8c3e49da9e411ec7c8da1efcbef.

[acceptance-tests] Disable a regressed coreclr test

I filed a tracking bug.

Merge pull request #4670 from BrzVlad/fix-sgen-testing

[tests] sgen testing tweaks

[tests] Improve configurations of sgen tests

Main pointers :
 - explicit minor and bridge configuration (add some for missing major)
 - reduce duplicated testing or of low relevance (like split nursery)
 - intensive testing of latest new configurations combined (parallel minor + major + future experiments)
 - include testing with a small and a large nursery for additional variance

[tests] Add timeout limits to some longer sgen tests