Aleksey Kliger [Wed, 24 May 2017 16:25:26 +0000 (12:25 -0400)]
[sgen] Scan the handle stack for dangling ptrs during domain unload
There should be no handles pointing to objects in the domain that is unloading
in any thread's handle stack once mono_gc_clear_domain () is called.
Aleksey Kliger [Wed, 24 May 2017 15:41:44 +0000 (11:41 -0400)]
[coop handles] Fix wrong #ifdef
Aleksey Kliger [Tue, 23 May 2017 21:24:18 +0000 (17:24 -0400)]
[appdomain] Don't leak MonoReflectionAssemblyHandles in callbacks to managed
Aleksey Kliger [Wed, 24 May 2017 15:18:36 +0000 (11:18 -0400)]
[tests] Regression test for #56694
When an AssemblyLoad event fires in a domain, a System.Reflection.Assembly
object is created in that domain for the assembly being loaded. The bug left
this object alive after the domain was unloaded, which lead to a GC crash.
Zoltan Varga [Wed, 24 May 2017 17:58:18 +0000 (13:58 -0400)]
[runtime] Implement interior handles as a special case to simplify and speed up the more common handle code. (#4921)
Marek Safar [Wed, 24 May 2017 17:28:29 +0000 (19:28 +0200)]
[corlib] Fixes warning
Marek Safar [Wed, 24 May 2017 17:26:46 +0000 (19:26 +0200)]
[mcs] Parsing of nested unbound types. Fixes #55400
Marek Safar [Wed, 24 May 2017 13:39:37 +0000 (15:39 +0200)]
[corlib] Remove unused field
Miguel de Icaza [Wed, 24 May 2017 17:19:58 +0000 (13:19 -0400)]
Improve error messages to help users diagnose problems when using custom mode (#4922)
* Improve error messages to help users diagnose problems when using the custom mode
* Update mkbundle.cs
Zoltan Varga [Wed, 24 May 2017 13:35:18 +0000 (09:35 -0400)]
[runtime] Convert a g_warning to a type load failure. (#4920)
Zoltan Varga [Wed, 24 May 2017 02:51:09 +0000 (22:51 -0400)]
Fix make dist.
Rodrigo Kumpera [Tue, 23 May 2017 23:59:07 +0000 (16:59 -0700)]
Fix the CpBlk and InitBlk intrinsics.
Rodrigo Kumpera [Tue, 23 May 2017 21:59:02 +0000 (14:59 -0700)]
Update test for correct behavior.
Rodrigo Kumpera [Tue, 23 May 2017 21:46:53 +0000 (14:46 -0700)]
[runtime] Fix multiple issues around symlink handling.
After fixing my windows setup, I managed to properly test MS's behavior, which is a lot saner and it's the following:
Opening fails with ERROR_CANT_RESOLVE_FILENAME (1921).
Testing and Deleting works.
Fixing File.Delete required removing the lstat we do before deleting, which no longer serves any purpose.
Fixing File.Exists requires handling ELOOP when stat fails and we retry with lstat.
Martin Baulig [Tue, 23 May 2017 19:42:05 +0000 (15:42 -0400)]
[System]: Don't call OnCompleted(this) twice in SocketAsyncEventArgs.FinishWrapperConnectSuccess(). (#4894)
Alexander Köplinger [Tue, 23 May 2017 17:54:35 +0000 (19:54 +0200)]
[bcl] Fix compiling xunit-based tests
We moved System.Runtime.InteropServices.RuntimeInformation to a facade.
Zoltan Varga [Tue, 23 May 2017 15:46:59 +0000 (11:46 -0400)]
[runtime] Add specialized versions of mono_icall_handle_new () for internal/non-internal pointers. (#4918)
Zoltan Varga [Tue, 23 May 2017 15:36:43 +0000 (11:36 -0400)]
Fix full aot (#4919)
* [runtime] Fix the generation of invalid IL in the return marshalling of stringbuilders.
* [runtime] Disable some failing fullaot tests.
Aleksey Kliger [Fri, 12 May 2017 16:30:05 +0000 (12:30 -0400)]
[runtime] Use coop handles for System.Delegate.AllocDelegateLike_internal
Aleksey Kliger [Thu, 11 May 2017 21:35:42 +0000 (17:35 -0400)]
[marshal] Use coop handles for System.Delegate.GetVirtualMethod_internal
Aleksey Kliger [Thu, 11 May 2017 21:06:29 +0000 (17:06 -0400)]
[marshal] Use coop handles in mono_delegate_to_ftnptr
* Create mono_delegate_handle_to_ftnptr for the implementation
* Use coop handles for
System.Runtime.InteropServices.Marshal.GetFunctionPointerForDelegateInternal
Aleksey Kliger [Thu, 11 May 2017 18:18:26 +0000 (14:18 -0400)]
[runtime] Remove temporary fcn mono_delegate_ctor_with_method_fixme
Aleksey Kliger [Thu, 11 May 2017 18:17:22 +0000 (14:17 -0400)]
[marshal] Change mono_delegate_ctor to use coop handles
Aleksey Kliger [Thu, 11 May 2017 17:44:45 +0000 (13:44 -0400)]
[marshal] Use handles in mono_ftnptr_to_delegate
* Move most of the logic to mono_ftnptr_to_delegate_handle
* Use handles for
System.Runtime.InteropServices.Marshal.GetDelegateForFunctionPointerInternal
Aleksey Kliger [Wed, 10 May 2017 22:18:44 +0000 (18:18 -0400)]
[runtime] Use coop handles in mono_delegate_ctor_with_method
Use coop handles in icall System.Delegate.CreateDelegate_internal
TODO: change callsites fo mono_delegate_ctor_with_method_fixme to use handles
Aleksey Kliger [Wed, 10 May 2017 19:26:29 +0000 (15:26 -0400)]
[marshal] Use coop handles for OffsetOf, SizeOf, Prelink, PrelinkAll
System.Runtime.InteropServices.Marshal methods
* OffsetOf
* Prelink
* PrelinkAll
* SizeOf
Alexander Köplinger [Tue, 23 May 2017 12:27:42 +0000 (14:27 +0200)]
Remove outdated info and unused files from mcs/ (#4916)
Most of the removed files weren't touched in the last decade
and are unused so it's better to remove them to avoid confusion
about what they do.
I also went over and removed the most outdated info from readme
files, we probably want to do a more thorough overhaul in the future.
Marek Safar [Tue, 23 May 2017 06:35:58 +0000 (08:35 +0200)]
[mcs] When setting struct empty layout consider compiler generated fields. Fixes #55604
Alexander Köplinger [Tue, 23 May 2017 00:32:20 +0000 (02:32 +0200)]
Bump bockbuild to fix build
Zoltan Varga [Tue, 23 May 2017 00:24:14 +0000 (20:24 -0400)]
[runtime] Add a fastpath to mono_object_isinst_icall (). (#4917)
Alexander Köplinger [Mon, 22 May 2017 22:16:22 +0000 (00:16 +0200)]
Use nuget 4.3.0-preview1 to fix build
4.5.0 is not yet available.
Also use lowercase nuget.exe to match the URL on https://dist.nuget.org/
Marek Safar [Mon, 22 May 2017 21:29:54 +0000 (23:29 +0200)]
[mcs] Add expression bodied syntax for accessors. Fixes #54991
Bernhard Urban [Mon, 22 May 2017 20:13:01 +0000 (22:13 +0200)]
[interp] fix unaligned.
Alexis Christoforides [Mon, 22 May 2017 20:04:30 +0000 (16:04 -0400)]
[MacSDK] Update Nuget to 4.5, switch to binary release instead of building
Alexander Köplinger [Mon, 22 May 2017 16:09:44 +0000 (18:09 +0200)]
[bcl] Make monolite platform specific (#4910)
mscorlib.dll etc will be platform specific in the future. This also
means that we need to adapt monolite since it'll become platform
specific once we start the divergence.
monolite is essentially a repackaging of the "build" profile which
means we need to turn that profile into multi-platform profile
like net_4_x. We'll still have a symlink from build -> build-darwin
and the monolite which is downloaded or packaged in the tarball
is expanded into mcs/lib/monolite-<platform> depending on the platform
you're building on.
I had to move the get-monolite-latest target from the top-level
Makefile.am into mcs/class/Makefile so it has access to HOST_PLATFORM,
but it arguably should've been there anyway before.
Alexander Köplinger [Mon, 22 May 2017 16:02:12 +0000 (18:02 +0200)]
[bcl] Exclude Mono.CSharp.dll from api diff
We don't guarantee a stable API there and it changes often
when mcs internals change.
Alex Rønne Petersen [Mon, 22 May 2017 15:02:22 +0000 (17:02 +0200)]
[profiler] Fall back to CLOCK_MONOTONIC if CLOCK_PROCESS_CPUTIME_ID results in EINVAL. (#4913)
This is the case on WSL.
Marek Safar [Mon, 22 May 2017 14:58:26 +0000 (16:58 +0200)]
[mcs] Inflate referenced current types for imported type definitions. Fixes #56462
Marek Safar [Mon, 22 May 2017 14:23:11 +0000 (16:23 +0200)]
[mcs] Removes TypeManager calls from MoreSpecific
vkargov [Mon, 22 May 2017 12:43:28 +0000 (05:43 -0700)]
[ci] Add the collect-coverage tag for use on Jenkins. (#4912)
* [ci] Add the collect-coverage tag for use on Jenkins.
* Move collect-coverage up so we don't set different values for -O
Bernhard Urban [Mon, 22 May 2017 10:40:19 +0000 (12:40 +0200)]
[mcs] extend API for loading types and consuming source file from a stream (#4905)
Rodrigo Kumpera [Sat, 20 May 2017 02:47:51 +0000 (19:47 -0700)]
[jit] Fix x86 failures and handle unaligned load with offset
Rodrigo Kumpera [Fri, 19 May 2017 22:06:19 +0000 (15:06 -0700)]
[corlib] Address reviewer comments to new test.
Alexander Köplinger [Fri, 19 May 2017 20:51:54 +0000 (22:51 +0200)]
Bump api-snapshot
Marek Safar [Fri, 19 May 2017 15:33:32 +0000 (17:33 +0200)]
Bump Roslyn to 2.3 preview
Alexis Christoforides [Fri, 19 May 2017 19:24:13 +0000 (15:24 -0400)]
Bump bockbuild
Alexander Köplinger [Fri, 19 May 2017 17:55:41 +0000 (19:55 +0200)]
Bump the btls submodule
Bring in the change that makes building the "tool" subdirectory
in BTLS off by default - we don't need it in Mono and it fails
to build on some platforms with older C++ compiler.
Marek Safar [Fri, 19 May 2017 16:43:59 +0000 (18:43 +0200)]
[mcs] Report more unsafe code errors in iterators. Fixes #56616
Marek Safar [Fri, 19 May 2017 13:11:29 +0000 (15:11 +0200)]
[build] Remove platforms from xbuild build
Zoltan Varga [Fri, 19 May 2017 15:37:38 +0000 (11:37 -0400)]
[amd64] Fix the allocation of the caller area in filter clauses. Fixes #56567. (#4903)
Alexander Köplinger [Fri, 19 May 2017 10:59:00 +0000 (12:59 +0200)]
[bcl] Dispose Cecil AssemblyDefinition after usage (#4900)
Follow-up to https://github.com/mono/mono/pull/4899. I audited
all the places in BCL where we made use of Cecil's AssemblyDefinition
and made sure we're properly disposing them and not keeping files
unnecessarily open.
I used InMemory=true in cases where tracking the lifetime would
be complicated. There are also some tests in mcs/tests which I
didn't bother to fix.
Katelyn Gadd [Fri, 19 May 2017 07:24:49 +0000 (00:24 -0700)]
Warn on failure to create default Reflection.Emit SymbolWriter instead of throwing and failing (#4748)
* Bug #53038: Warn on failure to create default Reflection.Emit SymbolWriter (on platforms like Android that don't ship one) instead of throwing and failing
Rodrigo Kumpera [Fri, 19 May 2017 03:13:38 +0000 (20:13 -0700)]
[jit] Add unaligned tests.
Zoltan Varga [Fri, 19 May 2017 02:47:56 +0000 (22:47 -0400)]
[amd64] Remove the restriction on the locals stack size. Fixes #56452. (#4901)
Rodrigo Kumpera [Thu, 18 May 2017 23:38:05 +0000 (16:38 -0700)]
[jit] Properly handle unaligned memory access.
Rodrigo Kumpera [Thu, 18 May 2017 23:19:22 +0000 (16:19 -0700)]
[jit] Assorted small changes.
Remove 10000 size check in mini_emit_memory_copy_bytes, this is no longer needed now that the memcpy expansion logic in mini_emit_memcpy_internal was fixes.
Use a define for 10000.
Remove useless debug spew.
Rodrigo Kumpera [Thu, 18 May 2017 22:56:54 +0000 (15:56 -0700)]
[mini] Remove assert and document reason it can't be used.
Bernhard Urban [Wed, 17 May 2017 19:52:18 +0000 (21:52 +0200)]
[interp] respect 64bit alignment on armv7
Alexander Köplinger [Thu, 18 May 2017 19:41:08 +0000 (21:41 +0200)]
[pdb2mdb] Dispose Cecil AssemblyDefinition on pdb2mdb (#4899)
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=56275
Recent Cecil 0.10 changed to not reading the assembly in memory but
instead reads directly from the underlying stream [0]. This however
means that the file handle is not freed until the AssemblyDefinition
is disposed.
Xamarin.Android is using the pdb2mdb source as part of
Xamarin.Android.Build.Tasks.dll and hit an issue where the file
would be locked inside of VS.
[0] http://cecil.pe/post/
149243207656/mono-cecil-010-beta-1
(cherry picked from commit
5077205a44a7dc97edf6b67072bea53f043cf815)
Alexander Köplinger [Thu, 18 May 2017 18:12:44 +0000 (20:12 +0200)]
[configure] Add stdalign.h checks for BTLS to ARM platforms too
We added it for x86/amd64 in
488b550ac0275c944e0a2c23be93173382084e5f
but missed adding that check when ARM was added recently.
vkargov [Thu, 18 May 2017 18:08:58 +0000 (11:08 -0700)]
[aot] Fix a memory leak related to constraint allocation/deallocation. (#4880)
* [jit] Make a copy of the constraint in mini_get_shared_gparam().
* [aot] Free the temporary gshared constraint in decode_class_ref().
Katelyn Gadd [Tue, 11 Apr 2017 09:34:43 +0000 (02:34 -0700)]
Fixes for bug#54388 "InternalsVisibleTo is case sensitive" * Fix the JIT so that access checks are performed on constructors * Use ASCII case insensitivity when performing cross-assembly access checks for internals * Update mcs to use case-insensitive comparisons when evaluating InternalsVisibleTo attributes for access checks (matching the new runtime behavior, and the behavior of roslyn)
This issue is actually a pair of bugs:
1. We don't perform any access checks for constructors (which can give a false impression of our behavior for this issue). This means that you can also invoke private constructors from other assemblies.
2. InternalsVisibleTo checks are done using strcmp instead of a case-insensitive comparison.
Marek Safar [Thu, 18 May 2017 15:43:09 +0000 (17:43 +0200)]
[corlib] Bump DoubleWait test timeouts
vkargov [Thu, 18 May 2017 15:04:58 +0000 (08:04 -0700)]
Fix eglibc remapping and revive the test for its completeness. (#4896)
* [eglib] Add the missing eglib remappings.
* [tests] Revive the eglib remapping completeness test.
Marek Safar [Thu, 18 May 2017 07:48:51 +0000 (09:48 +0200)]
Adds System.Net.Http 4.3.2 to blacklist
Alexander Köplinger [Thu, 18 May 2017 10:58:57 +0000 (12:58 +0200)]
[configure] Remove the --enable-dynamic-btls switch, it is the only option
Marek Safar [Tue, 16 May 2017 18:53:32 +0000 (20:53 +0200)]
[ci] Enable linker tests on PRs
Marek Safar [Thu, 11 May 2017 19:08:25 +0000 (21:08 +0200)]
Add linker tests
Marek Safar [Tue, 9 May 2017 15:50:58 +0000 (17:50 +0200)]
Add LinkerDescriptor
Marek Safar [Wed, 10 May 2017 13:00:15 +0000 (15:00 +0200)]
Bump linker
Rodrigo Kumpera [Thu, 11 May 2017 23:36:26 +0000 (16:36 -0700)]
[jit] Remove mini_emit_stobj from method-to-ir.c and move mini_emit_wb_aware_memcpy to memory-access.c
Rodrigo Kumpera [Thu, 11 May 2017 23:34:16 +0000 (16:34 -0700)]
[jit] Replace usage of mini_emit_stobj with mini_emit_memory_copy and simplify usage since the later can handle ref types.
Rodrigo Kumpera [Thu, 11 May 2017 23:32:09 +0000 (16:32 -0700)]
[jit] Fix wording of the function description
Rodrigo Kumpera [Thu, 11 May 2017 23:27:51 +0000 (16:27 -0700)]
[jit] Fix mono_debug_count breakage introduced by the g_getenv change.
Rodrigo Kumpera [Thu, 11 May 2017 22:52:56 +0000 (15:52 -0700)]
[mini] Add mini_emit_memory_copy to replace mini_emit_stobj
Rodrigo Kumpera [Thu, 11 May 2017 22:52:09 +0000 (15:52 -0700)]
[jit] Fix large const memcpy/memset when when the intrins pass is disabled.
Rodrigo Kumpera [Wed, 10 May 2017 20:38:40 +0000 (13:38 -0700)]
[runtime] Rework how unaligned memcpy/memset are generated.
Problems with the current code:
It's wrong on 64bits if align == 4
It's wrong on 64bits if offset is not 0 or 4.
It generates bad code if align > 1 && align < SIZEOF_VOIDP
This PR implements unaligned based on the following set of assumptions that are true for the current set of platforms we support:
- A load/store of size X works with data aligned on X. IE, a short store can handle addresses aligned to 2
- Unaligned access is either disallowed or plain slower.
- Both source and dest abide to the supplied alignment.
The way we emit unaligned access is by emitting the widest possible access based on alignment and offset.
TBH: use unaligned access on targets like x86 that support it and are profitable.
Zoltan Varga [Wed, 17 May 2017 16:33:08 +0000 (12:33 -0400)]
[runtime] Make RuntimeHelpers:SufficientExecutionStack () a no-op on android for now. Fixes #56240. (#4889)
Aleksey Kliger [Mon, 15 May 2017 22:47:14 +0000 (18:47 -0400)]
[coop handles] Add mechanism to track handle leaks
When MONO_HANDLE_TRACK_SP is defined, each handle records the native C stack pointer
at the time of its nearest enclosing HANDLE_FUNCTION_ENTER. If a subsequent
handle is allocated with a lower stack pointer (in memory; ie higher in the
call tree) then a HANDLE_FUNCTION_RETURN is missing and we're leaking handles.
Bernhard Urban [Wed, 17 May 2017 12:40:31 +0000 (14:40 +0200)]
[interp] assume EABI for passing arguments to arm trampoline
Bernhard Urban [Wed, 17 May 2017 11:40:58 +0000 (13:40 +0200)]
[interp] use helper function to check if generic inst is a value type
Bernhard Urban [Wed, 17 May 2017 11:40:58 +0000 (13:40 +0200)]
[arm] respect thumb mode in mixed mode unwinding
Bernhard Urban [Wed, 17 May 2017 11:40:58 +0000 (13:40 +0200)]
[interp] properly pass 64bit integers to native code on 32bit arches
Bernhard Urban [Wed, 17 May 2017 11:40:58 +0000 (13:40 +0200)]
[interp] disable some tests that fail on CI only
can't reproduce the crash on my dev device, will investigate later.
Bernhard Urban [Wed, 17 May 2017 11:40:58 +0000 (13:40 +0200)]
[interp] make mixed mode work on arm
Bernhard Urban [Wed, 17 May 2017 11:40:58 +0000 (13:40 +0200)]
[interp] generics/vt fix regarding elem size for ldelem.vt
Bernhard Urban [Wed, 17 May 2017 11:40:58 +0000 (13:40 +0200)]
[interp] intrinsify System.Array.UnsafeLoad
Bernhard Urban [Wed, 17 May 2017 11:40:58 +0000 (13:40 +0200)]
[interp] add a conversion that is only required for 32bit
Bernhard Urban [Wed, 17 May 2017 11:40:58 +0000 (13:40 +0200)]
[interp] icall trampoline for arm
Bernhard Urban [Wed, 17 May 2017 11:40:58 +0000 (13:40 +0200)]
[interp] make mixed mode work on arm64
Bernhard Urban [Wed, 17 May 2017 11:40:58 +0000 (13:40 +0200)]
[interp] fix gcc warnings
Bernhard Urban [Wed, 17 May 2017 11:40:57 +0000 (13:40 +0200)]
[interp] fix conv.u4.r8 on arm64
Bernhard Urban [Wed, 17 May 2017 11:40:57 +0000 (13:40 +0200)]
[interp/tramp] use MONO_STRUCT_OFFSET infrastructure and extract constants
Bernhard Urban [Wed, 17 May 2017 11:40:57 +0000 (13:40 +0200)]
[interp] implement icall_trampoline for arm64
Alexander Köplinger [Wed, 17 May 2017 12:14:04 +0000 (14:14 +0200)]
Rename Microsoft.NuGet.Build.Tasks.dll.sources to exclude profile
The earlier name sent gensources.sh into an infinite loop,
not sure why yet. The directory is only built for xbuild_14
anyway so this should be safe.
Alexander Köplinger [Wed, 17 May 2017 03:39:08 +0000 (05:39 +0200)]
Fix tarball build
The itanium backend was removed.
David Naylor [Wed, 17 May 2017 00:45:24 +0000 (02:45 +0200)]
w32handle: double the available slots (#4856)
Mono crashes if the system has more than ~128G of RAM. Doubling the
number of slots fixes the SIGSEGV error on FreeBSD.
Alexander Köplinger [Tue, 16 May 2017 22:27:43 +0000 (00:27 +0200)]
[MacSDK] Remove some outdated info from ReadMe.rtf
uninstallMono.sh is no longer installed.
The website has instructions for removing Mono:
http://www.mono-project.com/docs/about-mono/supported-platforms/osx/#uninstalling-mono-on-mac-os-x
Jo Shields [Tue, 16 May 2017 20:24:29 +0000 (21:24 +0100)]
[btls] Add "desktop" ARM Linux configurations (for IoT, mostly) (#4875)
* [btls] Add "desktop" ARM Linux configurations (for IoT, mostly)
* [btls] Disable optimized asm on soft-float ARM
It depends on a NEON FPU, which we don't target on soft float
Alexander Köplinger [Tue, 16 May 2017 19:29:49 +0000 (21:29 +0200)]
[btls] Add MonoPInvokeCallback attribute on all full aot profiles (#4879)
It was #ifdef'ed to only be available on monotouch before,
but the testing_aot_full profile in CI runs into this test failure:
```
MESSAGE:
System.ExecutionEngineException : Attempting to JIT compile method '(wrapper native-to-managed) Mono.Btls.MonoBtlsSslCtx:NativeVerifyCallback (intptr,int,intptr)' while running in aot-only mode. See https://developer.xamarin.com/guides/ios/advanced_topics/limitations/ for more information.
+++++++++++++++++++
STACK TRACE:
at (wrapper managed-to-native) System.Object:__icall_wrapper_mono_thread_interruption_checkpoint ()
at (wrapper managed-to-native) System.Runtime.InteropServices.Marshal:GetFunctionPointerForDelegateInternal (System.Delegate)
at System.Runtime.InteropServices.Marshal.GetFunctionPointerForDelegate[TDelegate] (TDelegate d) [0x00013] in <
fa55a8c31ba0490383239e5850d81686>:0
at Mono.Btls.MonoBtlsSslCtx..ctor (Mono.Btls.MonoBtlsSslCtx+BoringSslCtxHandle handle) [0x00059] in <
18810f39bffc41dcb0263eb2043cc10e>:0
at Mono.Btls.MonoBtlsSslCtx..ctor () [0x0000b] in <
18810f39bffc41dcb0263eb2043cc10e>:0
at Mono.Btls.MonoBtlsContext.InitializeConnection () [0x00000] in <
18810f39bffc41dcb0263eb2043cc10e>:0
at Mono.Btls.MonoBtlsContext.StartHandshake () [0x00000] in <
18810f39bffc41dcb0263eb2043cc10e>:0
at Mono.Net.Security.MobileAuthenticatedStream.ProcessHandshake (Mono.Net.Security.AsyncProtocolRequest asyncRequest, Mono.Net.Security.AsyncOperationStatus status) [0x00004] in <
18810f39bffc41dcb0263eb2043cc10e>:0
at Mono.Net.Security.AsyncProtocolRequest.ProcessOperation (Mono.Net.Security.AsyncOperationStatus status) [0x0006b] in <
18810f39bffc41dcb0263eb2043cc10e>:0
at Mono.Net.Security.AsyncProtocolRequest.ProcessOperation () [0x0000d] in <
18810f39bffc41dcb0263eb2043cc10e>:0
at Mono.Net.Security.AsyncProtocolRequest.StartOperation () [0x0003c] in <
18810f39bffc41dcb0263eb2043cc10e>:0
at Mono.Net.Security.AsyncProtocolRequest.StartOperation (Mono.Net.Security.AsyncOperation operation) [0x00024] in <
18810f39bffc41dcb0263eb2043cc10e>:0
at Mono.Net.Security.MobileAuthenticatedStream.ProcessAuthentication (System.Net.LazyAsyncResult lazyResult) [0x00057] in <
18810f39bffc41dcb0263eb2043cc10e>:0
```
The attribute itself is already conditional on MONOTOUCH and
FULL_AOT_RUNTIME so simply removing the ifdef should fix the issue.