Merge pull request #4716 from Unity-Technologies/eglib-msvc-targets

Separate the eglib source and build instructions

[simd] Add support for Vector.IsHardwareAccelerated in the System.Numerics.Vectors assembly as well. Fixes #55577. (#4762)

[runtime] Add a variant of mono_string_new_wrapper () and use it from marshalling code to avoid asserting when encountering invalid strings. (#4753)

Fixes build

[Facades] Move more asseblies to Facades to support implicit reference for _._ like nugets and ignore version mismatch

[System]: Fix bug #52508 in HttpListener when doing multiple Https request.

Move the call to SslStream.AuthenticateAsServer() into the constructor because
we don't want to call it again when Init() is called from Close().

Revert changes that I accidentally pushed on top of PR #4756.

[Mono.Security.Interface]: Improve synergy between `SslStream` and `IMonoSslStream` (#4756)

* [Mono.Security.Interface]: Improve synergy between `MonoTlsProvider`, `SslStream` and `IMonoSslStream`.

* Add `IMonoSslStream.SslStream` property; the `IMonoSslStream` is now only created my the `SslStream`
  constructor, so `SslStream` "owns" the `IMonoSslStream`.

* Add internal `MonoTlsProvider.CreateSslStreamInternal()`.

* `SslStream` does not need to implement any additional interfaces; we can do without after
   getting rid of the wrapper classes.

* Add internal `SslStream.CreateMonoSslStream(Stream,bool,MonoTlsProvider,MonoTlsSettings)`.

* `MonoTlsProvider.CreateSslStream()` implementations should use this new method.

* Remove ancient obsolete MonoTlsProviderFactory APIs.

* `HttpListener` and `HttpConnection` now use `SslStream` instead of `IMonoSslStream`.

* [System]: Cleanup internal TLS Provider registration.

* Make it build.

Define HAVE_STRUCT_SOCKADDR_IN6 on Windows (#4755)

This enables IPv6 support on Windows.

[Mono.Security.Interface]: Add MonoTlsProviderFactory.GetMonoSslStream(HttpListenerContext). (#4749)

* [System]: Cleanup 'SECURITY_DEP' conditionals in the internal TLS code.

* [System]: Remove 'Mono.Net.Security.IMonoSslStream'; use 'Mono.Security.Interface.IMonoSslStream' instead.

* [System]: Remove 'Mono.Net.Security.IMonoTlsProvider'; use 'Mono.Security.Interface.MonoTlsProvider' instead.

* [System]: Cleanup internal TLS code.

* [Mono.Security]: Add MonoTlsProviderFactory.GetSslStream(HttpListenerContext).

* [System]: `SslStream' should not implement any additional interfaces.

Make changes as per review to make the code more thread safe.

Ensure that when an async read is performed on a response content the
errors are correctly dealt with.

Some network errors in iOS are exposed as Stream errors, specially those
related with the lost of the connection. This patch ensures that if such
an issue happens an exception is raised in the async read task.

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

Merge pull request #4730 from lambdageek/facades-and-bug-580

* Check strong names on a couple more codepaths during assembly loading.  Fixes [bugzilla #580](https://bugzilla.xamarin.com/show_bug.cgi?id=580) and [bugzilla #55436](https://bugzilla.xamarin.com/show_bug.cgi?id=55436)
* Add `--apply-bindings=FILE` option.  Allows us to specify a configuration file for assembly binding redirections when AOTing.
* Use the above flag when AOTing `System.ReflectionMetadata.dll` to pick up `csc.exe.config`
* Add all the Facades to the `framework_assemblies` list in the runtime.  Facade and non-Facade framework assemblies behave differently:
  * for Facades, don't do any version remapping, and any version is okay.
  * for non-Facades, consult the version map to remap depdending on the --runtime used.

Merge pull request #4745 from xmcclure/ipv6-cleanup

Fix some style issues introduced by PR#4720

[man] Document --apply-bindings=FILE option

[assembly] Add Facades dir to default_path on all runtimes

Previous it was just the desktop runtime, but now that we have special handling
for framework facade assemblies in the loader, we should do it on other
runtimes, too.

[roslyn] Pass --apply-bindings=$(csc_exe).config when AOT-ing System.Reflection.Metadata

To pick up assembly binding redirections from csc.exe.config

[assembly] Add Facades to framework_assemblies list

[assembly] When loading framework facade assemblies, any version is ok in mono_assembly_candidate_predicate_sn_same_name

As long as the name and public key token matches.

This is different from the non-facade framework assemblies for which the
name and version must match but the public key token is ignored.

[assembly] add ANAME_EQ_IGNORE_VERSION flag to assembly_names_equal_flags()

[aot] Add --apply-bindings=FILE option

Apply bindings from configuration file FILE when running the AOT compiler.

This is useful if you need to AOT a module bar.dll that is loaded when the main
assembly foo.exe has a configuration file foo.exe.config that specifies
assembly bindings that bar.dll should respect.

[domain] Factor out mono_domain_parse_assembly_bindings

Move to a separate function the code that parses assembly bindings from a configuration file.

[msvc] Update csproj files

Remove unused facade

Make the filters file match the targets files

* Create one filters file for each targets file
* Import each filters file into the project-level filters file
* Add folders for Common, Win32, and Posix targets files

[loader] Pass sn_same_name predicate when loading from basedir and from default_path, too.

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

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

[System.Data.Common] Add IDbColumnSchemaGenerator

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

Fix some style issues introduced by PR#4720

Existing code in this file mixing ifdefs with if {} else {} places the ifdef after the } but before the else.

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.

Separate the eglib source and build instructions

This change extracts the eglib source files from the build instuctions
for builds via MSVC on Windows. It also separates the source files into
platform-specific and common .targets files. This provides more
flexibility for builds from MSVC in a few important ways:

1. We can do MSVC-based builds for platforms that will not be pushed to
public repos by checking out the Mono code externally and importing the
.targets files into a custom .vcxproj for the platform.
2. We can replace the platform specific .targets files with a custom set
of source files to implement a different platform abstraction layer
that will not be public without the need to merge changes to the
.vcxproj from the public repo.

This allows for ease of synchronizing changes to the public Mono
repo with private forks, while _not_ changing the behavior of the code
in the public repo.

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