[msvc] Update csproj files (#4226)

Merge pull request #4224 from akoeplinger/fix-46602

[btls] Create a native BTLS key handle for X509CertificateImplBtls.PrivateKey

Lldb support code for mono (#4225)

* Initial version of lldb support code.

* [lldb] Keep all the debug entries in a linked list so lldb can read it after attaching to the process.

* [lldb] Add an --enable-minimal=lldb option to disable the code. Add locking.

* [lldb] Add support for dynamic methods by adding a new protocol entry for unloading code regions.

* [lldb] Add beginnings of a test suite using the lldb python bindings.

* [lldb] Emit line number info for methods, not used yet.

* [lldb] Add dummy fields to structures to make sure 64 bit fields are 64 bit aligned.

* [lldb] Finish emission of line number info.

* [lldb] Use mono_method_full_name () for computing method names.

* [lldb] Emit info for specific trampolines as well.

* [lldb] Add tests for line number info.

* [lldb] Fix unwind info encoding on x86.

* [lldb] Skip seq points with no native offset.

* [lldb] Add tests for breakpoints.

[remoting] Coop handles for proxy objects (#4208)

* [remoting] Use coop handles in several functions

  - icall System.Runtime.Remoting.Proxies.RealProxy.InternalGetTransparentProxy
  - mono_remote_class
  - mono_remote_class_vtable
  - mono_upgrade_remote_class
  - mono_object_handle_get_virtual_method
  - mono_object_handle_isinst
  - mono_object_handle_isinst_mbyref

* [reflection] Use handles in System.RuntimeTypeHandle.IsInstanceOfType

* [runtime] Rewrite mono_object_castclass_mbyref
  in terms of mono_object_handle_isinst_mbyref.

  This is an external API function that isn't used inside the runtime.

* [runtime] Remove mono_object_isinst_mbyref_checked

  It was a runtime internal funciton.

  All of its former callers have been converted to use mono_object_handle_isinst_mbyref.

Merge pull request #4219 from marek-safar/corert-bump

Update ValueTuple implementation

[corlib] Update ValueTuple implementation

Bump corert

[jit] Extends an array in mono_spill_global_vars () instead of asserting. Fixes #50789. (#4201)

Merge pull request #4222 from alexanderkyte/fix_mismatch_com_disabled

[runtime] Fix corlib out of date error with disabled COM

Mono BCL test fixes so XM can run them (#4210)

[btls] Create a native BTLS key handle for X509CertificateImplBtls.PrivateKey

When setting X509CertificateImplBtls.PrivateKey we'd previously just
update the fallback instance. This causes problems if other code
internal to System.dll like MonoBtlsContext.SetPrivateCertificate()
tries to fetch the NativePrivateKey since it'd be null at that point.

This scenario happens e.g. with HttpListener, it supports a special
directory in ~/.config/.mono/httplistener which contains <port>.cer
and <port>.pvk files (public and private key) that will be used
as the server certificate. In that code we're reading the private
key from the file and assigning it to X509CertificateImplBtls.PrivateKey
without ever having a native BTLS handle.

The fix is to create such a native BTLS handle from the managed
private key when needed.

Note that we can't simply always do this e.g. in the PrivateKey
setter because the setter will be used with instances that
don't actually contain a private key. We can't throw in those
cases so instead we do it only when NativePrivateKey is accessed.

To make the code clearer and easier to read, `privateKey` was
renamed to `nativePrivateKey` in X509CertificateImplBtls.

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

[btls] Call Reset() in X509CertificateImplBtls.Import()

Otherwise we'd not clear the fields like subjectName, issuerName, etc.
and the properties they're backing would continue to return old values.

[btls] Free handle returned by EVP_PKEY_get1_RSA

From the header docs:

> The |get1| functions return a fresh reference to the underlying
> object or NULL if |pkey| is not of the correct type.

[System] Use non-obsolete API in SslStream

[msvc] Update csproj files (#4221)

[runtime] Fix corlib out of date error with disabled COM

[bcl] Remove System.Activities.Presentation folder from referencesource

We won't import it into Mono anytime soon and the long paths
cause issues when checking out Mono on Windows.

[reflection] Coop handles icalls in System.Reflection and System.RuntimeTypeHandle (#4217)

* [sre] Add typed coop handle declarations

* [reflection] Implement mono_reflection_get_token_handle

  Rewrite mono_reflection_get_token_checked to use it.

* [reflection] Use handles for ves_icall_reflection_get_token

  - Get rid of mono_reflection_get_token_checked
  - Rename mono_reflection_get_token_handle to mono_reflection_get_token_checked

* [reflection] Use handles for ves_icall_GetCurrentMethod

* [reflection] Use handles for ves_icall_System_Reflection_MethodBase_GetMethodFromHandleInternalType_native

* [reflection] Use handles for ves_icall_System_Reflection_Module_ResolveSignature

* [reflection] Use handles for ves_icall_System_Type_internal_from_name

* [reflection] Use handles for ves_icall_System_Type_internal_from_handle

* [runtime] Use coop handles for System.Object.GetType ()

* [reflection] Use handles for ves_icall_MonoField_GetParentType

* [reflection] Use handles for ves_icall_MonoField_ResolveType

* [reflection] Use handles for most icalls in System.RuntimeTypeHandle

  - GetArrayRank
  - GetAttributes
  - GetBaseType
  - GetElementType
  - HasInstantiation
  - IsArray
  - IsByRef
  - IsComObject
  - IsGenericTypeDefinition
  - IsGenericVariable
  - IsPointer
  - IsPrimitive
  - type_is_assignable_from

* [reflection] Use handles for ves_icall_RuntimeTypeHandle_GetGenericTypeDefinition_impl

  and ves_icall_RuntimeTypeHandle_GetGenericParameterInfo

[runtime] Set the pinvoke bit for handles-using icall callsites.

The MonoMethodSignature:pinvoke bit causes the IR translation to clear the
upper bits of return values that are smaller than a register. (See
mono_emit_widen_call_res)

Without this change, any icall returning a boolean (8 bits on .NET) might have
non-zero upper bits which will cause branches to misbehave.

[build] Add WIN_PLATFORM define

Revert "[configure] avoid 'no symbols' warnings on darwin"

This reverts commit c1f39f3cb4bb853bb42051f08fcd1aad980f2ea9.

Reverted this to get CI back.

[aot] Rewrite the AOT profiler. (#4176)

* [aot] Rewrite the AOT profiler.

The AOT profiler is used to collect the methods/instances used at runtime.
Usage:
* mono --profile=aot:output=out.aotprof <app>
* mono --aot=profile=out.aotprof <assembly>

The first command will run <app>, collecting the list of methods it uses into the out.aotprof file.
The second command passes the profile file to the AOT compiler to have it generate additional generic instances.

* [docs] Add a section for AOT profiling to the mono(1) man page.

* [build] Collect AOT profile info from building mscorlib in the build profile, and use it when aot-ing the csc assemblies.

* [runtime] Add missing mono-profiler-aot.h file.

* [runtime] Tidy up usage of MONO_PROFILER_API.

* [aot] Improve error handling in the aot profiler.

* [aot] Improve error handling, add more diagnostics.

[btls] Capture inner exception in MobileAuthenticatedStream

Previously we'd just throw the catched exception again
which overwrites its stacktrace, making it less useful
since all you can see is e.g. a NullReferenceException
(re)thrown at MobileAuthenticatedStream:218.

We now capture the original exception information so
the stacktrace is more complete.

[btls] Use MONODROID ifdef instead of ANDROID

We use the former everywhere else in the codebase.

[btls] Fix wrong path returned from MonoBtlsProvider.GetSystemStoreLocation()

The path ".mono/certs/NewTrust" doesn't actually exist,
it's called ".mono/new-certs/Trust". Look it up from
MonoBtlsX509StoreManager instead.

This API isn't used anywhere in the BTLS code, that's
probably why it wasn't noticed.

[Mono.Security] Add message to obsolete APIs with replacements

It's very confusing to get an "obsolete" warning
without knowing what to replace the API with.

[System] Fixes ios unresolved symbols

Merge pull request #4195 from lateralusX/jlorenss/win-build-dependency

Fix build_init vcxproj to correctly detect changes in config.h.

Add code manager callbacks so JIT can track memory chunk ranges and register stack walk handlers on Win64 (#4213)

Bump nunit-lite

[tools] Add C# strings encoding to resx2sr

[build] Update mini tests tools to use correct path

[corlib] Clean up some warnings

[aot] Avoid a crash in the aot compiler if a delegate doesn't have a BeginInvoke ()/EndInvoke () method. Fixes #51206.

[jit] Fix the check for calling mono_get_vtable_var (), check for 'cfg->method' instead of 'method', the latter is the inlined method. (#4209)

[aot] Aot System.Numerics.dll as well for the make fullaotcheck/llvmonlycheck targets.

Revert "[reflection] Additional coop handles changes (#4197)"

This reverts commit 59ed24831a31d1e88027e8e717b61c53edface58.

[msvc] Update csproj files (#4207)

[reflection] Additional coop handles changes (#4197)

* [sre] Add typed coop handle declarations

* [reflection] Implement mono_reflection_get_token_handle

  Rewrite mono_reflection_get_token_checked to use it.

* [reflection] Use handles for ves_icall_reflection_get_token

  - Get rid of mono_reflection_get_token_checked
  - Rename mono_reflection_get_token_handle to mono_reflection_get_token_checked

* [reflection] Use handles for ves_icall_GetCurrentMethod

* [reflection] Use handles for ves_icall_System_Reflection_MethodBase_GetMethodFromHandleInternalType_native

* [reflection] Use handles for ves_icall_System_Reflection_Module_ResolveSignature

* [reflection] Use handles for ves_icall_System_Type_internal_from_name

* [reflection] Use handles for ves_icall_System_Type_internal_from_handle

* [runtime] Use coop handles for System.Object.GetType ()

* [reflection] Use handles for ves_icall_MonoField_GetParentType

* [reflection] Use handles for ves_icall_MonoField_ResolveType

* [reflection] Use handles for most icalls in System.RuntimeTypeHandle

  - GetArrayRank
  - GetAttributes
  - GetBaseType
  - GetElementType
  - HasInstantiation
  - IsArray
  - IsByRef
  - IsComObject
  - IsGenericTypeDefinition
  - IsGenericVariable
  - IsPointer
  - IsPrimitive
  - type_is_assignable_from

* [reflection] Use handles for ves_icall_RuntimeTypeHandle_GetGenericTypeDefinition_impl

  and ves_icall_RuntimeTypeHandle_GetGenericParameterInfo

[System] Cleanup some warnings

Merge pull request #4205 from marek-safar/compression

[System] Use GZipStream from corefx

[bcl] Look in all .sources files when gathering files to dist

Previously we would only look into files ending in libraryname.sources.
This breaks down in cases where we used e.g. "#include common.sources"
because those files don't match the pattern and would be left out.

To fix this we could either start resolving includes in the .sources,
but the simpler fix is to just dist files from all normal .sources files.

[configure] avoid 'no symbols' warnings on darwin

This helps to get rid of the messages like these:

```
/Applications/Xcode8.1-beta1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: .libs/libmonosgen-2.0.a(libmonoruntimesgen_la-coree.o) has no symbols
/Applications/Xcode8.1-beta1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: .libs/libmonosgen-2.0.a(libmonoruntimesgen_la-file-mmap-windows.o) has no symbols
/Applications/Xcode8.1-beta1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: .libs/libmonosgen-2.0.a(libmonoruntimesgen_la-lock-tracer.o) has no symbols
/Applications/Xcode8.1-beta1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: .libs/libmonosgen-2.0.a(libmonoruntimesgen_la-mono-endian.o) has no symbols
[...]
```

That is what is in `libtool`:

```
# Commands used to build an old-style archive.
old_archive_cmds="\$AR \$AR_FLAGS \$oldlib\$oldobjs~\$RANLIB \$tool_oldlib"
```

`ar` is calling `ranlib`, and then `libtool` calls `ranlib` again. We
cannot control how `ar` calls `ranlib` (that is, we can't tell it to
pass `-no_warning_for_no_symbols`), but we can control to not call it
at all.  Let's to that then.

As a reference, source code of `ar`:
https://opensource.apple.com/source/cctools/cctools-895/ar/ar.c.auto.html

[msvc] Update csproj files (#4204)

[mcs] Accept and ignore /debug:portable option

We'll just generate a normal .mdb instead.

Decided not to update the help text since
technically we don't support generating a ppdb.

[System] Use GZipStream from corefx

Merge pull request #4202 from marek-safar/compression

[System.IO.Compression] Replaced by corefx implementation

[llvmonly] Use right IMT slow path for lazy array interfaces (#4199)

[System.Core] Fix infinite loop in csproj-local target

This was introduced in 9d5de96b0f21b3602e7489c8d63c8db3034228fe since
we would now call the csproj-local target without `intermediate`
being set which results in an infinite loop.

Overriding the target doesn't seem to be necessary anymore, so remove it.

[btls] Pass CC and CXX to the BTLS CMake and disable BTLS on older GCC (#4200)

CMake doesn't pick up the compiler from the PATH by default,
it only looks in default locations so without this we might
end up using a different compiler for BTLS than for the main
Mono build which we don't want.

BTLS also requires a fairly new GCC which includes C++11
features, added a quick check for stdalign.h which BTLS
requires and disable it if the header isn't found.

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

[build] Fixes monotouch_tv_runtime build

[build] Fixes monotouch_runtime build

[System.IO.Compression] Replaced by corefx implementation

Revert "Bump corlib version"

This reverts commit acf7d8597142b558a3ec83c952b456ae81841eff.

[System] Compression enums from corefx

[build] Fixes tarball build on systems with old mono

Merge pull request #4196 from DavidKarlas/stepAndCatchException

[Debugger] Fixed step-in and step-out in case of exception + unit tests

[System] Include common.sources in the tarball

Bump corlib version

Somehow mcs.exe disappeared from monolite even though it was added
in 68d56e8027809791b7ff64153bd0a77f9b408075 and worked for a while.

See if bumping corlib version (i.e. generating a new monolite) fixes it.

[xbuild] Fix test that got broken by 248fded92a107a556d08fa77a9e458f0a990a28a

Merge pull request #4152 from BrzVlad/misc-gc-altstack

[sgen] Reenable gc-altstack test

[System] Warnings cleanup

[System] Keep single MonoPInvokeCallbackAttribute implementation

[System] Unify Mono.Security dependency to be same for all profiles

[xbuild] More tweaks for portable debug mode

[mcs] Disable more array interpolation on user code. Fixes #51166

Merge pull request #4194 from DavidKarlas/bug44974

Bug 44974 - [pdb] stepping over is not working

Roslyn 2.0 (rc2) upgrade

[Debugger] Fixed step-in and step-out in case of exception + unit tests
Before this change step-in and step-out didn’t place breakpoints on catch handlers which caused code to continue execution instead of stopping since it missed all breakpoints…

Fix build_init vcxproj to correctly detect changes in config.h.

build_init project doesn't detect changes done in config.h. This is needed in order
for build_init to replace config.h with winconfig.h and backup cygconfig.h. This is
normally an issue if you do a reconfigure and build using make and then msbuild. The msbuild
will then fail, since config.h is still the cygwin version. Solution is to do a rebuild all
that will trigger build_init. This fix will make sure that build_init gets invoked if there has been a change to config.h.

There was also a very old issue with the first copy of config.h -> cygconfig.h. This is only done
if there is no cygconfig.h in the directory. This means that the cygconfig.h will not be update until
deleted and a new build_init has been invoked. This fix will make sure cygconfig.h is always updated
if config.h is the cygwin generated version (not already replaced with winconfig.h).

Merge pull request #4193 from DavidKarlas/stepinSpeedup

[Debugger] Fixed bug where we loop randomly long(+1) and optimised stepping in(over) non-user code

[Debugger] Fixed bug where we loop randomly long(+1 part) and optimised stepping in(over) non-user code

Bug 44974 - [pdb] stepping over is not working
mcs.exe didn’t emit some NOPs after method calls needed when doing StepOut, now that Roslyn emits this we correctly step out to line that called hence, some unit tests needed correcting

[simd] Enable simd support when 32 bit floats are enabled.

[llvm] Fix compilation/working with recent llvm versions. This should work with 4d6d2b05a267f9f4e46fdbb302f380add938b0cd from the mono/llvm repo.

Fix a warning.

[reflection] Use handles for several more icalls (#4098)

* [runtime] Remove MonoReflectionGenericMethod typedef.

  - It's not used in the runtime,
  - Unlike other defs in object-internals.h it doesn't seem to have a
    corresponding managed object.

* [reflection] Add mono_field_get_object_handle.

  Reimplement mono_field_get_object and mono_field_get_object_checked in
  terms of it.

* [reflection] Add mono_property_get_object_handle

  Reimplement mono_property_get_object and
  mono_property_get_object_checked in terms of it.

* [reflection] Use handles for FieldInfo,MonoPropertyInfo,ParameterInfo.GetTypeModifiers

* [reflection] Use handles for MonoMethodInfo.get_parameter_info

  - Change mono_param_get_objects_internal to use handles

* [reflection] Cleanup in method_object_construct

* [reflection] Use handles for System.Refleciton.EventInfo.internal_from_handle_type

  - Replace mono_event_get_object_checked by mono_event_get_object_handle
    which uses handles.

* [reflection] Use handles for System.Reflection.MethodBase.GetMethodBodyInternal

  - Replace mono_method_body_get_object_checked by
    mono_method_body_get_object_handle

* [reflection] Remove CHECK_OR_CONSTRUCT, it's dead code.

  All former callers now use CHECK_OR_CONSTRUCT_HANDLE

* [reflection] Use handles for FieldInfo.get_marshal_info and MonoMethodInfo.get_retval_marshal

  - Change mono_reflection_marshal_as_attribute_from_marshal_spec to
    return a handle; fix callers

* [metadata] Add mono_metadata_get_canonical_generic_inst utility function

* [reflection] Use handles for System.Reflection.Module.ResolveMemberToken and 3 more icalls

  - ResolveFieldToken
  - ResolveMethodToken
  - ResolveTypeToken

[threadpool] Restore handle stack mark after a worker PerformWaitCallback

If a task calls native code that uses handles before completing, we should
dispose of any handles that it may have used before we park the worker.

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

[sgen] Assert that skipped threads have empty an HandleStack

If we skip scanning a thread with a non-empty handle stack, we may move an
object but fail to update the reference in the handle.

[coop handles] Add mono_handle_stack_is_empty

[coop handles] Add MONO_HANDLE_TRACK_OWNER mode.

If MONO_HANDLE_TRACK_OWNER is defined, each coop handle records the source location
where it was first allocated.

[System] Fix basic/build profile build

It was broken by 37875ca6f3b0e65eb5750ebf556d5355da8715e9.

[runtime] Add a few missing object offsets for arm.

[jit] Fix a test which fails on big endian.

[System] Remove duplicated sources info

[System] remove unused file

configure.ac: set CSC variable to mcs.exe when using mcs as the compiler

Otherwise it'd still be using Roslyn e.g. in mono/mini as $(CSC)
would continue to point to the Roslyn csc.exe which is wrong.

Pump hazard pointer queue in unregister_thread while thread is still alive. (#4183)

Some of the destructors (for example mono_jit_info_table_free) may try to take
a lock, which requires that the current thread should be attached.

Fix MonoBtlsX509StoreManager machine store paths (#4027)

The machineXYZPath fields are initialized with machinePath instead of
userPath which caused the machine stores not to work at all

Fix a regression introduced by 98d7ad045e6661e4c2e428fe4acc4fc733d44fdc.

[build] More mcs hacking

[build] install csc aot images from aot folder

[System.Xml] Disable more failing tests on old xslt implementation

[System.Xml] Disable failing tests on old xslt implementation

[build] Pass --aot-path to the runtime when building the tests in tests/.

[build] Generate all aot images for csc into the profile directory instead of next to the corresponding assembly so the external/roslyn-binaries submodule doesn't get dirty. Pass the --aot-path argument to the runtime so it picks up the aot images.

Fix warnings.

[runtime] Remove some dead code.

[aot] Cleanup the AOT loading code a bit. Add a --aot-path= command line option to specify additional directories to search for AOT images. (#4191)

[bcl] Clean up old run-test.bat and *.csproj files

They .bat files haven't been used for a long time and don't work anymore.
The csproj files for the facades have a different name now.