Merge pull request #1659 from alexanderkyte/stringbuilder-referencesource
[mono.git] / mono / metadata / marshal.c
index 0c8fe33f219deadaed0965bed82c58d1a3c880ba..f688ff41b8da4eac788cfccd4bf6aa08ddbdb384 100644 (file)
@@ -36,6 +36,7 @@
 #include "mono/metadata/attrdefs.h"
 #include "mono/metadata/gc-internal.h"
 #include "mono/metadata/cominterop.h"
+#include "mono/metadata/remoting.h"
 #include "mono/metadata/reflection-internals.h"
 #include "mono/utils/mono-counters.h"
 #include "mono/utils/mono-tls.h"
 #define OPDEF(a,b,c,d,e,f,g,h,i,j) \
        a = i,
 
-typedef enum {
-       MONO_MARSHAL_NONE,                      /* No marshalling needed */
-       MONO_MARSHAL_COPY,                      /* Can be copied by value to the new domain */
-       MONO_MARSHAL_COPY_OUT,          /* out parameter that needs to be copied back to the original instance */
-       MONO_MARSHAL_SERIALIZE          /* Value needs to be serialized into the new domain */
-} MonoXDomainMarshalType;
-
 enum {
 #include "mono/cil/opcode.def"
        LAST = 0xff
 };
 #undef OPDEF
 
-struct _MonoRemotingMethods {
-       MonoMethod *invoke;
-       MonoMethod *invoke_with_check;
-       MonoMethod *xdomain_invoke;
-       MonoMethod *xdomain_dispatch;
-};
-
-typedef struct _MonoRemotingMethods MonoRemotingMethods;
-
 /* 
  * This mutex protects the various marshalling related caches in MonoImage
  * and a few other data structures static to this file.
@@ -143,27 +128,6 @@ mono_delegate_begin_invoke (MonoDelegate *delegate, gpointer *params);
 static MonoObject *
 mono_delegate_end_invoke (MonoDelegate *delegate, gpointer *params);
 
-static void
-mono_marshal_xdomain_copy_out_value (MonoObject *src, MonoObject *dst);
-
-static gint32
-mono_marshal_set_domain_by_id (gint32 id, MonoBoolean push);
-
-static gboolean
-mono_marshal_check_domain_image (gint32 domain_id, MonoImage *image);
-
-#ifndef DISABLE_REMOTING
-static MonoObject *
-mono_remoting_wrapper (MonoMethod *method, gpointer *params);
-
-MONO_API void
-mono_upgrade_remote_class_wrapper (MonoReflectionType *rtype, MonoTransparentProxy *tproxy);
-
-#endif
-
-static MonoReflectionType *
-type_from_handle (MonoType *handle);
-
 static void
 mono_marshal_set_last_error_windows (int error);
 
@@ -274,22 +238,14 @@ mono_marshal_init (void)
                register_icall (mono_struct_delete_old, "mono_struct_delete_old", "void ptr ptr", FALSE);
                register_icall (mono_delegate_begin_invoke, "mono_delegate_begin_invoke", "object object ptr", FALSE);
                register_icall (mono_delegate_end_invoke, "mono_delegate_end_invoke", "object object ptr", FALSE);
-               register_icall (mono_marshal_xdomain_copy_value, "mono_marshal_xdomain_copy_value", "object object", FALSE);
-               register_icall (mono_marshal_xdomain_copy_out_value, "mono_marshal_xdomain_copy_out_value", "void object object", FALSE);
-               register_icall (mono_marshal_set_domain_by_id, "mono_marshal_set_domain_by_id", "int32 int32 int32", FALSE);
-               register_icall (mono_marshal_check_domain_image, "mono_marshal_check_domain_image", "int32 int32 ptr", FALSE);
                register_icall (mono_compile_method, "mono_compile_method", "ptr ptr", FALSE);
                register_icall (mono_context_get, "mono_context_get", "object", FALSE);
                register_icall (mono_context_set, "mono_context_set", "void object", FALSE);
-               register_icall (type_from_handle, "type_from_handle", "object ptr", FALSE);
                register_icall (mono_gc_wbarrier_generic_nostore, "wb_generic", "void ptr", FALSE);
                register_icall (mono_gchandle_get_target, "mono_gchandle_get_target", "object int32", TRUE);
 
-#ifndef DISABLE_REMOTING
-               register_icall (mono_remoting_wrapper, "mono_remoting_wrapper", "object ptr ptr", FALSE);
-               register_icall (mono_upgrade_remote_class_wrapper, "mono_upgrade_remote_class_wrapper", "void object object", FALSE);
-#endif
                mono_cominterop_init ();
+               mono_remoting_init ();
        }
 }
 
@@ -304,42 +260,17 @@ mono_marshal_cleanup (void)
        marshal_mutex_initialized = FALSE;
 }
 
-#ifndef DISABLE_REMOTING
-static MonoClass *byte_array_class;
-static MonoMethod *method_rs_serialize, *method_rs_deserialize, *method_exc_fixexc, *method_rs_appdomain_target;
-static MonoMethod *method_set_call_context, *method_needs_context_sink, *method_rs_serialize_exc;
-
-static void
-mono_remoting_marshal_init (void)
+void
+mono_marshal_lock_internal (void)
 {
-       MonoClass *klass;
-
-       static gboolean module_initialized = FALSE;
-
-       if (!module_initialized) {
-               klass = mono_class_from_name (mono_defaults.corlib, "System.Runtime.Remoting", "RemotingServices");
-               method_rs_serialize = mono_class_get_method_from_name (klass, "SerializeCallData", -1);
-               method_rs_deserialize = mono_class_get_method_from_name (klass, "DeserializeCallData", -1);
-               method_rs_serialize_exc = mono_class_get_method_from_name (klass, "SerializeExceptionData", -1);
-       
-               klass = mono_defaults.real_proxy_class;
-               method_rs_appdomain_target = mono_class_get_method_from_name (klass, "GetAppDomainTarget", -1);
-       
-               klass = mono_defaults.exception_class;
-               method_exc_fixexc = mono_class_get_method_from_name (klass, "FixRemotingException", -1);
-       
-               byte_array_class = mono_array_class_get (mono_defaults.byte_class, 1);
-       
-               klass = mono_class_from_name (mono_defaults.corlib, "System.Runtime.Remoting.Messaging", "CallContext");
-               method_set_call_context = mono_class_get_method_from_name (klass, "SetCurrentCallContext", -1);
-       
-               klass = mono_class_from_name (mono_defaults.corlib, "System.Runtime.Remoting.Contexts", "Context");
-               method_needs_context_sink = mono_class_get_method_from_name (klass, "get_NeedsContextSink", -1);
+       mono_marshal_lock ();
+}
 
-               module_initialized = TRUE;
-       }
+void
+mono_marshal_unlock_internal (void)
+{
+       mono_marshal_unlock ();
 }
-#endif
 
 gpointer
 mono_delegate_to_ftnptr (MonoDelegate *delegate)
@@ -697,13 +628,11 @@ mono_free_lparray (MonoArray *array, gpointer* nativeArray)
                return;
        klass = array->obj.vtable->klass;
 
-       switch (klass->element_class->byval_arg.type) {
-               case MONO_TYPE_CLASS:
-                       for(i = 0; i < array->max_length; ++i)  
-                               mono_marshal_free_ccw(nativeArray[i]);
-                       free(nativeArray);
-               break;
-       }               
+       if (klass->element_class->byval_arg.type == MONO_TYPE_CLASS) {
+               for(i = 0; i < array->max_length; ++i)
+                       mono_marshal_free_ccw (mono_array_get (array, MonoObject*, i));
+               free(nativeArray);
+       }
 #endif
 }
 
@@ -765,49 +694,16 @@ mono_array_to_byte_byvalarray (gpointer native_arr, MonoArray *arr, guint32 elnu
        mono_array_to_byvalarray (native_arr, arr, mono_defaults.byte_class, elnum);
 }
 
-void
-mono_string_utf8_to_builder (MonoStringBuilder *sb, char *text)
-{
-       GError *error = NULL;
-       guint16 *ut;
-       glong items_written;
-       int l;
-
-       if (!sb || !text)
-               return;
-
-       l = strlen (text);
-
-       ut = g_utf8_to_utf16 (text, l, NULL, &items_written, &error);
-       
-       if (items_written > mono_stringbuilder_capacity (sb))
-               items_written = mono_stringbuilder_capacity (sb);
-       
-       if (!error) {
-               if (! sb->str || sb->str == sb->cached_str)
-                       MONO_OBJECT_SETREF (sb, str, mono_string_new_size (mono_domain_get (), items_written));
-               
-               memcpy (mono_string_chars (sb->str), ut, items_written * 2);
-               sb->length = items_written;
-               sb->cached_str = NULL;
-       } else 
-               g_error_free (error);
-
-       g_free (ut);
-}
-
-MonoStringBuilder *
-mono_string_utf8_to_builder2 (char *text)
+static MonoStringBuilder *
+mono_string_builder_new (int starting_string_length)
 {
-       int l;
-       MonoStringBuilder *sb;
        static MonoClass *string_builder_class;
        static MonoMethod *sb_ctor;
-       void *args [1];
-       MonoObject *exc;
+       static void *args [1];
+       int initial_len = starting_string_length;
 
-       if (!text)
-               return NULL;
+       if (initial_len < 0)
+               initial_len = 0;
 
        if (!string_builder_class) {
                MonoMethodDesc *desc;
@@ -818,78 +714,101 @@ mono_string_utf8_to_builder2 (char *text)
                sb_ctor = mono_method_desc_search_in_class (desc, string_builder_class);
                g_assert (sb_ctor);
                mono_method_desc_free (desc);
-       }
 
-       l = strlen (text);
+               // We make a new array in the _to_builder function, so this
+               // array will always be garbage collected.
+               args [0] = &initial_len;
+       }
 
-       sb = (MonoStringBuilder*)mono_object_new (mono_domain_get (), string_builder_class);
+       MonoStringBuilder *sb = (MonoStringBuilder*)mono_object_new (mono_domain_get (), string_builder_class);
+       MonoObject *exc;
        g_assert (sb);
-       args [0] = &l;
+
        mono_runtime_invoke (sb_ctor, sb, args, &exc);
+
+       g_assert (sb->chunkChars->max_length >= initial_len);
        g_assert (!exc);
 
-       mono_string_utf8_to_builder (sb, text);
+       return sb;
+}
+
+static void
+mono_string_utf16_to_builder_copy (MonoStringBuilder *sb, gunichar2 *text, size_t string_len)
+{
+       gunichar2 *charDst = (gunichar2 *)sb->chunkChars->vector;
+       gunichar2 *charSrc = (gunichar2 *)text;
+       memcpy (charDst, charSrc, sizeof (gunichar2) * string_len);
+
+       sb->chunkLength = string_len;
+
+       return;
+}
+
+MonoStringBuilder *
+mono_string_utf16_to_builder2 (gunichar2 *text)
+{
+       if (!text)
+               return NULL;
+
+       int len;
+       for (len = 0; text [len] != 0; ++len);
+
+       MonoStringBuilder *sb = mono_string_builder_new (len);
+       mono_string_utf16_to_builder (sb, text);
 
        return sb;
 }
 
-/*
- * FIXME: This routine does not seem to do what it seems to do
- * the @text is never copied into the string builder
- */
 void
-mono_string_utf16_to_builder (MonoStringBuilder *sb, gunichar2 *text)
+mono_string_utf8_to_builder (MonoStringBuilder *sb, char *text)
 {
-       guint32 len;
-
        if (!sb || !text)
                return;
 
-       g_assert (mono_string_chars (sb->str) == text);
+       int len = strlen (text);
+       if (len > mono_string_builder_capacity (sb))
+               len = mono_string_builder_capacity (sb);
+
+       GError *error = NULL;
+       glong copied;
+       gunichar2* ut = g_utf8_to_utf16 (text, len, NULL, &copied, &error);
 
-       for (len = 0; text [len] != 0; ++len)
-               ;
+       if (!error) {
+               MONO_OBJECT_SETREF (sb, chunkPrevious, NULL);
+               mono_string_utf16_to_builder_copy (sb, ut, copied);
+       } else
+               g_error_free (error);
 
-       sb->length = len;
+       g_free (ut);
 }
 
 MonoStringBuilder *
-mono_string_utf16_to_builder2 (gunichar2 *text)
+mono_string_utf8_to_builder2 (char *text)
 {
-       int len;
-       MonoStringBuilder *sb;
-       static MonoClass *string_builder_class;
-       static MonoMethod *sb_ctor;
-       void *args [1];
-       MonoObject *exc;
-
        if (!text)
                return NULL;
 
-       if (!string_builder_class) {
-               MonoMethodDesc *desc;
+       int len = strlen (text);
+       MonoStringBuilder *sb = mono_string_builder_new (len);
+       mono_string_utf8_to_builder (sb, text);
 
-               string_builder_class = mono_class_from_name (mono_defaults.corlib, "System.Text", "StringBuilder");
-               g_assert (string_builder_class);
-               desc = mono_method_desc_new (":.ctor(int)", FALSE);
-               sb_ctor = mono_method_desc_search_in_class (desc, string_builder_class);
-               g_assert (sb_ctor);
-               mono_method_desc_free (desc);
-       }
+       return sb;
+}
 
-       for (len = 0; text [len] != 0; ++len)
-               ;
 
-       sb = (MonoStringBuilder*)mono_object_new (mono_domain_get (), string_builder_class);
-       g_assert (sb);
-       args [0] = &len;
-       mono_runtime_invoke (sb_ctor, sb, args, &exc);
-       g_assert (!exc);
+void
+mono_string_utf16_to_builder (MonoStringBuilder *sb, gunichar2 *text)
+{
+       if (!sb || !text)
+               return;
 
-       sb->length = len;
-       memcpy (mono_string_chars (sb->str), text, len * 2);
+       guint32 len;
+       for (len = 0; text [len] != 0; ++len);
+       
+       if (len > mono_string_builder_capacity (sb))
+               len = mono_string_builder_capacity (sb);
 
-       return sb;
+       mono_string_utf16_to_builder_copy (sb, text, len);
 }
 
 /**
@@ -902,35 +821,38 @@ mono_string_utf16_to_builder2 (gunichar2 *text)
  *
  * The return value must be released with g_free.
  */
-gpointer
+gchar*
 mono_string_builder_to_utf8 (MonoStringBuilder *sb)
 {
        GError *error = NULL;
-       gchar *tmp, *res = NULL;
 
        if (!sb)
                return NULL;
 
-       if ((sb->str == sb->cached_str) && (sb->str->length == 0)) {
-               /* 
-                * The sb could have been allocated with the default capacity and be empty.
-                * we need to alloc a buffer of the default capacity in this case.
-                */
-               MONO_OBJECT_SETREF (sb, str, mono_string_new_size (mono_domain_get (), 16));
-               sb->cached_str = NULL;
-       }
 
-       tmp = g_utf16_to_utf8 (mono_string_chars (sb->str), sb->length, NULL, NULL, &error);
+       gunichar2 *str_utf16 = mono_string_builder_to_utf16 (sb);
+
+       guint str_len = mono_string_builder_string_length (sb);
+
+       gchar *tmp = g_utf16_to_utf8 (str_utf16, str_len, NULL, NULL, &error);
+
        if (error) {
                g_error_free (error);
+               g_free (str_utf16);
                mono_raise_exception (mono_get_exception_execution_engine ("Failed to convert StringBuilder from utf16 to utf8"));
+               return NULL;
        } else {
-               res = mono_marshal_alloc (mono_stringbuilder_capacity (sb) + 1);
-               memcpy (res, tmp, sb->length + 1);
+               guint len = mono_string_builder_capacity (sb) + 1;
+               gchar *res = mono_marshal_alloc (len * sizeof (gchar));
+               g_assert (str_len < len);
+               memcpy (res, tmp, str_len * sizeof (gchar));
+               res[str_len] = '\0';
+
+
+               g_free (str_utf16);
                g_free (tmp);
+               return res;
        }
-
-       return res;
 }
 
 /**
@@ -943,35 +865,43 @@ mono_string_builder_to_utf8 (MonoStringBuilder *sb)
  *
  * The return value must not be freed.
  */
-gpointer
+gunichar2*
 mono_string_builder_to_utf16 (MonoStringBuilder *sb)
 {
        if (!sb)
                return NULL;
 
-       g_assert (sb->str);
+       g_assert (sb->chunkChars);
 
-       /*
-        * The stringbuilder might not have ownership of this string. If this is
-        * the case, we must duplicate the string, so that we don't munge immutable
-        * strings
-        */
-       if (sb->str == sb->cached_str) {
-               /* 
-                * The sb could have been allocated with the default capacity and be empty.
-                * we need to alloc a buffer of the default capacity in this case.
-                */
-               if (sb->str->length == 0)
-                       MONO_OBJECT_SETREF (sb, str, mono_string_new_size (mono_domain_get (), 16));
-               else
-                       MONO_OBJECT_SETREF (sb, str, mono_string_new_utf16 (mono_domain_get (), mono_string_chars (sb->str), mono_stringbuilder_capacity (sb)));
-               sb->cached_str = NULL;
-       }
-       
-       if (sb->length == 0)
-               *(mono_string_chars (sb->str)) = '\0';
+       guint len = mono_string_builder_capacity (sb);
+
+       if (len == 0)
+               len = 1;
+
+       gunichar2 *str = mono_marshal_alloc ((len + 1) * sizeof (gunichar2));
+       str[len] = '\0';
+
+       if (len == 0)
+               return str;
+
+       MonoStringBuilder* chunk = sb;
+       do {
+               if (chunk->chunkLength > 0) {
+                       // Check that we will not overrun our boundaries.
+                       gunichar2 *source = (gunichar2 *)chunk->chunkChars->vector;
+
+                       if (chunk->chunkLength <= len) {
+                               memcpy (str + chunk->chunkOffset, source, chunk->chunkLength * sizeof(gunichar2));
+                       } else {
+                               g_error ("A chunk in the StringBuilder had a length longer than expected from the offset.");
+                       }
+
+                       len -= chunk->chunkLength;
+               }
+               chunk = chunk->chunkPrevious;
+       } while (chunk != NULL);
 
-       return mono_string_chars (sb->str);
+       return str;
 }
 
 static gpointer
@@ -1082,57 +1012,6 @@ mono_string_new_len_wrapper (const char *text, guint length)
 }
 
 #ifndef DISABLE_JIT
-#ifndef DISABLE_REMOTING
-static int
-mono_mb_emit_proxy_check (MonoMethodBuilder *mb, int branch_code)
-{
-       int pos;
-       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoObject, vtable));
-       mono_mb_emit_byte (mb, CEE_LDIND_I);
-       mono_mb_emit_icon (mb, MONO_STRUCT_OFFSET (MonoVTable, klass));
-       mono_mb_emit_byte (mb, CEE_ADD);
-       mono_mb_emit_byte (mb, CEE_LDIND_I);
-       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
-       mono_mb_emit_byte (mb, CEE_MONO_CLASSCONST);
-       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, mono_defaults.transparent_proxy_class));
-       pos = mono_mb_emit_branch (mb, branch_code);
-       return pos;
-}
-
-static int
-mono_mb_emit_xdomain_check (MonoMethodBuilder *mb, int branch_code)
-{
-       int pos;
-       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoTransparentProxy, rp));
-       mono_mb_emit_byte (mb, CEE_LDIND_REF);
-       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoRealProxy, target_domain_id));
-       mono_mb_emit_byte (mb, CEE_LDIND_I4);
-       mono_mb_emit_icon (mb, -1);
-       pos = mono_mb_emit_branch (mb, branch_code);
-       return pos;
-}
-
-static int
-mono_mb_emit_contextbound_check (MonoMethodBuilder *mb, int branch_code)
-{
-       static int offset = -1;
-       static guint8 mask;
-
-       if (offset < 0)
-               mono_marshal_find_bitfield_offset (MonoClass, contextbound, &offset, &mask);
-
-       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoTransparentProxy, remote_class));
-       mono_mb_emit_byte (mb, CEE_LDIND_REF);
-       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoRemoteClass, proxy_class));
-       mono_mb_emit_byte (mb, CEE_LDIND_REF);
-       mono_mb_emit_ldflda (mb, offset);
-       mono_mb_emit_byte (mb, CEE_LDIND_U1);
-       mono_mb_emit_icon (mb, mask);
-       mono_mb_emit_byte (mb, CEE_AND);
-       mono_mb_emit_icon (mb, 0);
-       return mono_mb_emit_branch (mb, branch_code);
-}
-#endif /* DISABLE_REMOTING */
 
 /*
  * mono_mb_emit_exception_marshal_directive:
@@ -1828,27 +1707,8 @@ emit_object_to_ptr_conv (MonoMethodBuilder *mb, MonoType *type, MonoMarshalConv
 #endif /* DISABLE_COM */
 
        case MONO_MARSHAL_CONV_SAFEHANDLE: {
-               int dar_release_slot, pos;
+               int pos;
                
-               dar_release_slot = mono_mb_add_local (mb, &mono_defaults.boolean_class->byval_arg);
-
-               /*
-                * The following is ifdefed-out, because I have no way of doing the
-                * DangerousRelease when destroying the structure
-                */
-#if 0
-               /* set release = false */
-               mono_mb_emit_icon (mb, 0);
-               mono_mb_emit_stloc (mb, dar_release_slot);
-               if (!sh_dangerous_add_ref)
-                       init_safe_handle ();
-
-               /* safehandle.DangerousAddRef (ref release) */
-               mono_mb_emit_ldloc (mb, 0); /* the source */
-               mono_mb_emit_byte (mb, CEE_LDIND_I);
-               mono_mb_emit_ldloc_addr (mb, dar_release_slot);
-               mono_mb_emit_managed_call (mb, sh_dangerous_add_ref, NULL);
-#endif
                mono_mb_emit_ldloc (mb, 0);
                mono_mb_emit_byte (mb, CEE_LDIND_I);
                pos = mono_mb_emit_branch (mb, CEE_BRTRUE);
@@ -2130,9 +1990,10 @@ emit_struct_free (MonoMethodBuilder *mb, MonoClass *klass, int struct_var)
 static void
 emit_thread_interrupt_checkpoint_call (MonoMethodBuilder *mb, gpointer checkpoint_func)
 {
-       int pos_noabort;
+       int pos_noabort, pos_noex;
 
-       mono_mb_emit_ptr (mb, (gpointer) mono_thread_interruption_request_flag ());
+       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+       mono_mb_emit_byte (mb, CEE_MONO_LDPTR_INT_REQ_FLAG);
        mono_mb_emit_byte (mb, CEE_LDIND_U4);
        pos_noabort = mono_mb_emit_branch (mb, CEE_BRFALSE);
 
@@ -2140,6 +2001,12 @@ emit_thread_interrupt_checkpoint_call (MonoMethodBuilder *mb, gpointer checkpoin
        mono_mb_emit_byte (mb, CEE_MONO_NOT_TAKEN);
 
        mono_mb_emit_icall (mb, checkpoint_func);
+       /* Throw the exception returned by the checkpoint function, if any */
+       mono_mb_emit_byte (mb, CEE_DUP);
+       pos_noex = mono_mb_emit_branch (mb, CEE_BRFALSE);
+       mono_mb_emit_byte (mb, CEE_THROW);
+       mono_mb_patch_branch (mb, pos_noex);
+       mono_mb_emit_byte (mb, CEE_POP);
        
        mono_mb_patch_branch (mb, pos_noabort);
 }
@@ -2156,7 +2023,7 @@ emit_thread_interrupt_checkpoint (MonoMethodBuilder *mb)
 static void
 emit_thread_force_interrupt_checkpoint (MonoMethodBuilder *mb)
 {
-       emit_thread_interrupt_checkpoint_call (mb, mono_thread_force_interruption_checkpoint);
+       emit_thread_interrupt_checkpoint_call (mb, mono_thread_force_interruption_checkpoint_noraise);
 }
 
 void
@@ -2165,6 +2032,12 @@ mono_marshal_emit_thread_interrupt_checkpoint (MonoMethodBuilder *mb)
        emit_thread_interrupt_checkpoint (mb);
 }
 
+void
+mono_marshal_emit_thread_force_interrupt_checkpoint (MonoMethodBuilder *mb)
+{
+       emit_thread_force_interrupt_checkpoint (mb);
+}
+
 #endif /* DISABLE_JIT */
 
 static MonoAsyncResult *
@@ -2229,7 +2102,7 @@ mono_delegate_begin_invoke (MonoDelegate *delegate, gpointer *params)
 
 #ifndef DISABLE_JIT
 
-static int
+int
 mono_mb_emit_save_args (MonoMethodBuilder *mb, MonoMethodSignature *sig, gboolean save_this)
 {
        int i, params_var, tmp_var;
@@ -2434,7 +2307,6 @@ static gboolean
 mono_marshal_need_free (MonoType *t, MonoMethodPInvoke *piinfo, MonoMarshalSpec *spec)
 {
        MonoMarshalNative encoding;
-       MonoMarshalConv conv;
 
        switch (t->type) {
        case MONO_TYPE_VALUETYPE:
@@ -2444,7 +2316,7 @@ mono_marshal_need_free (MonoType *t, MonoMethodPInvoke *piinfo, MonoMarshalSpec
        case MONO_TYPE_CLASS:
                if (t->data.klass == mono_defaults.stringbuilder_class) {
                        gboolean need_free;
-                       conv = mono_marshal_get_ptr_to_stringbuilder_conv (piinfo, spec, &need_free);
+                       mono_marshal_get_ptr_to_stringbuilder_conv (piinfo, spec, &need_free);
                        return need_free;
                }
                return FALSE;
@@ -2475,22 +2347,6 @@ get_cache (GHashTable **var, GHashFunc hash_func, GCompareFunc equal_func)
        return *var;
 }
 
-static GHashTable*
-get_cache_full (GHashTable **var, GHashFunc hash_func, GCompareFunc equal_func, GDestroyNotify key_destroy_func, GDestroyNotify value_destroy_func)
-{
-       if (!(*var)) {
-               mono_marshal_lock ();
-               if (!(*var)) {
-                       GHashTable *cache = 
-                               g_hash_table_new_full (hash_func, equal_func, key_destroy_func, value_destroy_func);
-                       mono_memory_barrier ();
-                       *var = cache;
-               }
-               mono_marshal_unlock ();
-       }
-       return *var;
-}
-
 GHashTable*
 mono_marshal_get_cache (GHashTable **var, GHashFunc hash_func, GCompareFunc equal_func)
 {
@@ -2513,7 +2369,7 @@ mono_marshal_find_in_cache (GHashTable *cache, gpointer key)
  *
  *   Create a MonoMethod from MB, set INFO as wrapper info.
  */
-static MonoMethod*
+MonoMethod*
 mono_mb_create (MonoMethodBuilder *mb, MonoMethodSignature *sig,
                                int max_stack, WrapperInfo *info)
 {
@@ -2526,7 +2382,7 @@ mono_mb_create (MonoMethodBuilder *mb, MonoMethodSignature *sig,
 }
 
 /* Create the method from the builder and place it in the cache */
-static MonoMethod*
+MonoMethod*
 mono_mb_create_and_cache_full (GHashTable *cache, gpointer key,
                                                           MonoMethodBuilder *mb, MonoMethodSignature *sig,
                                                           int max_stack, WrapperInfo *info, gboolean *out_found)
@@ -2571,87 +2427,15 @@ mono_mb_create_and_cache (GHashTable *cache, gpointer key,
        return mono_mb_create_and_cache_full (cache, key, mb, sig, max_stack, NULL, NULL);
 }
 
-static inline MonoMethod*
-mono_marshal_remoting_find_in_cache (MonoMethod *method, int wrapper_type)
+MonoMethod *
+mono_marshal_method_from_wrapper (MonoMethod *wrapper)
 {
-       MonoMethod *res = NULL;
-       MonoRemotingMethods *wrps;
-
-       mono_marshal_lock ();
-       if (method->klass->image->remoting_invoke_cache)
-               wrps = g_hash_table_lookup (method->klass->image->remoting_invoke_cache, method);
-       else
-               wrps = NULL;
+       gpointer res;
+       int wrapper_type = wrapper->wrapper_type;
+       WrapperInfo *info;
 
-       if (wrps) {
-               switch (wrapper_type) {
-               case MONO_WRAPPER_REMOTING_INVOKE: res = wrps->invoke; break;
-               case MONO_WRAPPER_REMOTING_INVOKE_WITH_CHECK: res = wrps->invoke_with_check; break;
-               case MONO_WRAPPER_XDOMAIN_INVOKE: res = wrps->xdomain_invoke; break;
-               case MONO_WRAPPER_XDOMAIN_DISPATCH: res = wrps->xdomain_dispatch; break;
-               }
-       }
-       
-       /* it is important to do the unlock after the load from wrps, since in
-        * mono_remoting_mb_create_and_cache () we drop the marshal lock to be able
-        * to take the loader lock and some other thread may set the fields.
-        */
-       mono_marshal_unlock ();
-       return res;
-}
-
-/* Create the method from the builder and place it in the cache */
-static inline MonoMethod*
-mono_remoting_mb_create_and_cache (MonoMethod *key, MonoMethodBuilder *mb, 
-                                                               MonoMethodSignature *sig, int max_stack)
-{
-       MonoMethod **res = NULL;
-       MonoRemotingMethods *wrps;
-       GHashTable *cache = get_cache_full (&key->klass->image->remoting_invoke_cache, mono_aligned_addr_hash, NULL, NULL, g_free);
-
-       mono_marshal_lock ();
-       wrps = g_hash_table_lookup (cache, key);
-       if (!wrps) {
-               wrps = g_new0 (MonoRemotingMethods, 1);
-               g_hash_table_insert (cache, key, wrps);
-       }
-
-       switch (mb->method->wrapper_type) {
-       case MONO_WRAPPER_REMOTING_INVOKE: res = &wrps->invoke; break;
-       case MONO_WRAPPER_REMOTING_INVOKE_WITH_CHECK: res = &wrps->invoke_with_check; break;
-       case MONO_WRAPPER_XDOMAIN_INVOKE: res = &wrps->xdomain_invoke; break;
-       case MONO_WRAPPER_XDOMAIN_DISPATCH: res = &wrps->xdomain_dispatch; break;
-       default: g_assert_not_reached (); break;
-       }
-       mono_marshal_unlock ();
-
-       if (*res == NULL) {
-               MonoMethod *newm;
-               newm = mono_mb_create_method (mb, sig, max_stack);
-
-               mono_marshal_lock ();
-               if (!*res) {
-                       *res = newm;
-                       mono_marshal_set_wrapper_info (*res, key);
-                       mono_marshal_unlock ();
-               } else {
-                       mono_marshal_unlock ();
-                       mono_free_method (newm);
-               }
-       }
-
-       return *res;
-}              
-
-MonoMethod *
-mono_marshal_method_from_wrapper (MonoMethod *wrapper)
-{
-       gpointer res;
-       int wrapper_type = wrapper->wrapper_type;
-       WrapperInfo *info;
-
-       if (wrapper_type == MONO_WRAPPER_NONE || wrapper_type == MONO_WRAPPER_DYNAMIC_METHOD)
-               return wrapper;
+       if (wrapper_type == MONO_WRAPPER_NONE || wrapper_type == MONO_WRAPPER_DYNAMIC_METHOD)
+               return wrapper;
 
        switch (wrapper_type) {
        case MONO_WRAPPER_REMOTING_INVOKE:
@@ -2662,14 +2446,18 @@ mono_marshal_method_from_wrapper (MonoMethod *wrapper)
                res = mono_marshal_get_wrapper_info (wrapper);
                if (res == NULL)
                        return wrapper;
-               if (wrapper->is_inflated)
+               if (wrapper->is_inflated) {
+                       MonoError error;
+                       MonoMethod *result;
                        /*
                         * A method cannot be inflated and a wrapper at the same time, so the wrapper info
                         * contains an uninflated method.
                         */
-                       return mono_class_inflate_generic_method (res, mono_method_get_context (wrapper));
-               else
-                       return res;
+                       result = mono_class_inflate_generic_method_checked (res, mono_method_get_context (wrapper), &error);
+                       g_assert (mono_error_ok (&error)); /* FIXME don't swallow the error */
+                       return result;
+               }
+               return res;
        case MONO_WRAPPER_MANAGED_TO_NATIVE:
                info = mono_marshal_get_wrapper_info (wrapper);
                if (info && (info->subtype == WRAPPER_SUBTYPE_NONE || info->subtype == WRAPPER_SUBTYPE_NATIVE_FUNC_AOT || info->subtype == WRAPPER_SUBTYPE_PINVOKE))
@@ -2721,7 +2509,7 @@ mono_marshal_set_wrapper_info (MonoMethod *method, gpointer data)
        datav [1] = data;
 }
 
-static WrapperInfo*
+WrapperInfo*
 mono_wrapper_info_create (MonoMethodBuilder *mb, WrapperSubtype subtype)
 {
        WrapperInfo *info;
@@ -2784,10 +2572,8 @@ check_generic_wrapper_cache (GHashTable *cache, MonoMethod *orig_method, gpointe
        MonoMethod *res;
        MonoMethod *inst, *def;
        MonoGenericContext *ctx;
-       MonoMethod *def_method;
 
        g_assert (orig_method->is_inflated);
-       def_method = ((MonoMethodInflated*)orig_method)->declaring;
        ctx = mono_method_get_context (orig_method);
 
        /*
@@ -2802,7 +2588,9 @@ check_generic_wrapper_cache (GHashTable *cache, MonoMethod *orig_method, gpointe
         */
        def = mono_marshal_find_in_cache (cache, def_key);
        if (def) {
-               inst = mono_class_inflate_generic_method (def, ctx);
+               MonoError error;
+               inst = mono_class_inflate_generic_method_checked (def, ctx, &error);
+               g_assert (mono_error_ok (&error)); /* FIXME don't swallow the error */
                /* Cache it */
                mono_memory_barrier ();
                mono_marshal_lock ();
@@ -2820,12 +2608,14 @@ check_generic_wrapper_cache (GHashTable *cache, MonoMethod *orig_method, gpointe
 static MonoMethod*
 cache_generic_wrapper (GHashTable *cache, MonoMethod *orig_method, MonoMethod *def, MonoGenericContext *ctx, gpointer key)
 {
+       MonoError error;
        MonoMethod *inst, *res;
 
        /*
         * We use the same cache for the generic definition and the instances.
         */
-       inst = mono_class_inflate_generic_method (def, ctx);
+       inst = mono_class_inflate_generic_method_checked (def, ctx, &error);
+       g_assert (mono_error_ok (&error)); /* FIXME don't swallow the error */
        mono_memory_barrier ();
        mono_marshal_lock ();
        res = g_hash_table_lookup (cache, key);
@@ -2840,6 +2630,7 @@ cache_generic_wrapper (GHashTable *cache, MonoMethod *orig_method, MonoMethod *d
 static MonoMethod*
 check_generic_delegate_wrapper_cache (GHashTable *cache, MonoMethod *orig_method, MonoMethod *def_method, MonoGenericContext *ctx)
 {
+       MonoError error;
        MonoMethod *res;
        MonoMethod *inst, *def;
 
@@ -2855,7 +2646,9 @@ check_generic_delegate_wrapper_cache (GHashTable *cache, MonoMethod *orig_method
         */
        def = mono_marshal_find_in_cache (cache, def_method->klass);
        if (def) {
-               inst = mono_class_inflate_generic_method (def, ctx);
+               inst = mono_class_inflate_generic_method_checked (def, ctx, &error);
+               g_assert (mono_error_ok (&error)); /* FIXME don't swallow the error */
+
                /* Cache it */
                mono_memory_barrier ();
                mono_marshal_lock ();
@@ -2873,12 +2666,15 @@ check_generic_delegate_wrapper_cache (GHashTable *cache, MonoMethod *orig_method
 static MonoMethod*
 cache_generic_delegate_wrapper (GHashTable *cache, MonoMethod *orig_method, MonoMethod *def, MonoGenericContext *ctx)
 {
+       MonoError error;
        MonoMethod *inst, *res;
 
        /*
         * We use the same cache for the generic definition and the instances.
         */
-       inst = mono_class_inflate_generic_method (def, ctx);
+       inst = mono_class_inflate_generic_method_checked (def, ctx, &error);
+       g_assert (mono_error_ok (&error)); /* FIXME don't swallow the error */
+
        mono_memory_barrier ();
        mono_marshal_lock ();
        res = g_hash_table_lookup (cache, orig_method->klass);
@@ -3037,7 +2833,7 @@ mono_delegate_end_invoke (MonoDelegate *delegate, gpointer *params)
 
 #ifndef DISABLE_JIT
 
-static void
+void
 mono_mb_emit_restore_result (MonoMethodBuilder *mb, MonoType *return_type)
 {
        MonoType *t = mono_type_get_underlying_type (return_type);
@@ -3178,2712 +2974,1114 @@ mono_marshal_get_delegate_end_invoke (MonoMethod *method)
        return res;
 }
 
-#ifndef DISABLE_REMOTING
-
-static MonoObject *
-mono_remoting_wrapper (MonoMethod *method, gpointer *params)
+typedef struct
 {
-       MonoMethodMessage *msg;
-       MonoTransparentProxy *this;
-       MonoObject *res, *exc;
-       MonoArray *out_args;
-
-       this = *((MonoTransparentProxy **)params [0]);
-
-       g_assert (this);
-       g_assert (((MonoObject *)this)->vtable->klass == mono_defaults.transparent_proxy_class);
-       
-       /* skip the this pointer */
-       params++;
-
-       if (mono_class_is_contextbound (this->remote_class->proxy_class) && this->rp->context == (MonoObject *) mono_context_get ())
-       {
-               int i;
-               MonoMethodSignature *sig = mono_method_signature (method);
-               int count = sig->param_count;
-               gpointer* mparams = (gpointer*) alloca(count*sizeof(gpointer));
-
-               for (i=0; i<count; i++) {
-                       MonoClass *class = mono_class_from_mono_type (sig->params [i]);
-                       if (class->valuetype) {
-                               if (sig->params [i]->byref) {
-                                       mparams[i] = *((gpointer *)params [i]);
-                               } else {
-                                       /* runtime_invoke expects a boxed instance */
-                                       if (mono_class_is_nullable (mono_class_from_mono_type (sig->params [i])))
-                                               mparams[i] = mono_nullable_box (params [i], class);
-                                       else
-                                               mparams[i] = params [i];
-                               }
-                       } else {
-                               mparams[i] = *((gpointer**)params [i]);
-                       }
-               }
+       MonoMethodSignature *sig;
+       gpointer pointer;
+} SignaturePointerPair;
 
-               return mono_runtime_invoke (method, method->klass->valuetype? mono_object_unbox ((MonoObject*)this): this, mparams, NULL);
-       }
+static guint
+signature_pointer_pair_hash (gconstpointer data)
+{
+       SignaturePointerPair *pair = (SignaturePointerPair*)data;
 
-       msg = mono_method_call_message_new (method, params, NULL, NULL, NULL);
+       return mono_signature_hash (pair->sig) ^ mono_aligned_addr_hash (pair->pointer);
+}
 
-       res = mono_remoting_invoke ((MonoObject *)this->rp, msg, &exc, &out_args);
+static gboolean
+signature_pointer_pair_equal (gconstpointer data1, gconstpointer data2)
+{
+       SignaturePointerPair *pair1 = (SignaturePointerPair*) data1, *pair2 = (SignaturePointerPair*) data2;
+       return mono_metadata_signature_equal (pair1->sig, pair2->sig) && (pair1->pointer == pair2->pointer);
+}
 
-       if (exc)
-               mono_raise_exception ((MonoException *)exc);
+static gboolean
+signature_pointer_pair_matches_pointer (gpointer key, gpointer value, gpointer user_data)
+{
+       SignaturePointerPair *pair = (SignaturePointerPair*)key;
 
-       mono_method_return_message_restore (method, params, out_args);
+       return pair->pointer == user_data;
+}
 
-       return res;
-} 
+static void
+free_signature_pointer_pair (SignaturePointerPair *pair)
+{
+       g_free (pair);
+}
 
-MonoMethod *
-mono_marshal_get_remoting_invoke (MonoMethod *method)
+static MonoMethod *
+mono_marshal_get_delegate_invoke_internal (MonoMethod *method, gboolean callvirt, gboolean static_method_with_first_arg_bound, MonoMethod *target_method)
 {
-       MonoMethodSignature *sig;
+       MonoMethodSignature *sig, *static_sig, *invoke_sig;
+       int i;
        MonoMethodBuilder *mb;
        MonoMethod *res;
-       int params_var;
+       GHashTable *cache;
+       gpointer cache_key = NULL;
+       SignaturePointerPair key;
+       SignaturePointerPair *new_key;
+       int local_prev, local_target;
+       int pos0;
+       char *name;
+       MonoClass *target_class = NULL;
+       gboolean closed_over_null = FALSE;
+       MonoGenericContext *ctx = NULL;
+       MonoGenericContainer *container = NULL;
+       MonoMethod *orig_method = NULL;
+       WrapperInfo *info;
+       WrapperSubtype subtype = WRAPPER_SUBTYPE_NONE;
+       gboolean found;
 
-       g_assert (method);
+       g_assert (method && method->klass->parent == mono_defaults.multicastdelegate_class &&
+                 !strcmp (method->name, "Invoke"));
 
-       if (method->wrapper_type == MONO_WRAPPER_REMOTING_INVOKE || method->wrapper_type == MONO_WRAPPER_XDOMAIN_INVOKE)
-               return method;
+       invoke_sig = sig = mono_signature_no_pinvoke (method);
 
-       /* this seems to be the best plase to put this, as all remoting invokes seem to get filtered through here */
-#ifndef DISABLE_COM
-       if (mono_class_is_com_object (method->klass) || method->klass == mono_class_get_com_object_class ()) {
-               MonoVTable *vtable = mono_class_vtable (mono_domain_get (), method->klass);
-               g_assert (vtable); /*FIXME do proper error handling*/
+       /*
+        * If the delegate target is null, and the target method is not static, a virtual 
+        * call is made to that method with the first delegate argument as this. This is 
+        * a non-documented .NET feature.
+        */
+       if (callvirt) {
+               subtype = WRAPPER_SUBTYPE_DELEGATE_INVOKE_VIRTUAL;
+               if (target_method->is_inflated) {
+                       MonoType *target_type;
 
-               if (!mono_vtable_is_remote (vtable)) {
-                       return mono_cominterop_get_invoke (method);
+                       g_assert (method->signature->hasthis);
+                       target_type = mono_class_inflate_generic_type (method->signature->params [0],
+                               mono_method_get_context (method));
+                       target_class = mono_class_from_mono_type (target_type);
+               } else {
+                       target_class = target_method->klass;
                }
-       }
-#endif
 
-       sig = mono_signature_no_pinvoke (method);
+               closed_over_null = sig->param_count == mono_method_signature (target_method)->param_count;
+       }
 
-       /* we cant remote methods without this pointer */
-       if (!sig->hasthis)
-               return method;
+       if (static_method_with_first_arg_bound) {
+               subtype = WRAPPER_SUBTYPE_DELEGATE_INVOKE_BOUND;
+               g_assert (!callvirt);
+               invoke_sig = mono_method_signature (target_method);
+       }
 
-       if ((res = mono_marshal_remoting_find_in_cache (method, MONO_WRAPPER_REMOTING_INVOKE)))
-               return res;
+       /*
+        * For generic delegates, create a generic wrapper, and return an instance to help AOT.
+        */
+       if (method->is_inflated && subtype == WRAPPER_SUBTYPE_NONE) {
+               orig_method = method;
+               ctx = &((MonoMethodInflated*)method)->context;
+               method = ((MonoMethodInflated*)method)->declaring;
 
-       mb = mono_mb_new (method->klass, method->name, MONO_WRAPPER_REMOTING_INVOKE);
+               container = mono_method_get_generic_container (method);
+               if (!container)
+                       container = method->klass->generic_container;
+               g_assert (container);
 
-#ifndef DISABLE_JIT
-       mb->method->save_lmf = 1;
+               invoke_sig = sig = mono_signature_no_pinvoke (method);
+       }
 
-       params_var = mono_mb_emit_save_args (mb, sig, TRUE);
+       /*
+        * Check cache
+        */
+       if (ctx) {
+               cache = get_cache (&method->klass->image->delegate_invoke_generic_cache, mono_aligned_addr_hash, NULL);
+               res = check_generic_delegate_wrapper_cache (cache, orig_method, method, ctx);
+               if (res)
+                       return res;
+               cache_key = method->klass;
+       } else if (static_method_with_first_arg_bound) {
+               cache = get_cache (&method->klass->image->delegate_bound_static_invoke_cache,
+                                                  (GHashFunc)mono_signature_hash, 
+                                                  (GCompareFunc)mono_metadata_signature_equal);
+               /*
+                * The wrapper is based on sig+invoke_sig, but sig can be derived from invoke_sig.
+                */
+               res = mono_marshal_find_in_cache (cache, invoke_sig);
+               if (res)
+                       return res;
+               cache_key = invoke_sig;
+       } else if (callvirt) {
+               GHashTable **cache_ptr;
 
-       mono_mb_emit_ptr (mb, method);
-       mono_mb_emit_ldloc (mb, params_var);
-       mono_mb_emit_icall (mb, mono_remoting_wrapper);
-       emit_thread_interrupt_checkpoint (mb);
+               cache_ptr = &method->klass->image->delegate_abstract_invoke_cache;
 
-       if (sig->ret->type == MONO_TYPE_VOID) {
-               mono_mb_emit_byte (mb, CEE_POP);
-               mono_mb_emit_byte (mb, CEE_RET);
+               /* We need to cache the signature+method pair */
+               mono_marshal_lock ();
+               if (!*cache_ptr)
+                       *cache_ptr = g_hash_table_new_full (signature_pointer_pair_hash, (GEqualFunc)signature_pointer_pair_equal, (GDestroyNotify)free_signature_pointer_pair, NULL);
+               cache = *cache_ptr;
+               key.sig = invoke_sig;
+               key.pointer = target_method;
+               res = g_hash_table_lookup (cache, &key);
+               mono_marshal_unlock ();
+               if (res)
+                       return res;
        } else {
-                mono_mb_emit_restore_result (mb, sig->ret);
+               cache = get_cache (&method->klass->image->delegate_invoke_cache,
+                                                  (GHashFunc)mono_signature_hash, 
+                                                  (GCompareFunc)mono_metadata_signature_equal);
+               res = mono_marshal_find_in_cache (cache, sig);
+               if (res)
+                       return res;
+               cache_key = sig;
        }
-#endif
 
-       res = mono_remoting_mb_create_and_cache (method, mb, sig, sig->param_count + 16);
-       mono_mb_free (mb);
+       static_sig = signature_dup (method->klass->image, sig);
+       static_sig->hasthis = 0;
+       if (!static_method_with_first_arg_bound)
+               invoke_sig = static_sig;
 
-       return res;
-}
+       if (static_method_with_first_arg_bound)
+               name = mono_signature_to_name (invoke_sig, "invoke_bound");
+       else if (closed_over_null)
+               name = mono_signature_to_name (invoke_sig, "invoke_closed_over_null");
+       else if (callvirt)
+               name = mono_signature_to_name (invoke_sig, "invoke_callvirt");
+       else
+               name = mono_signature_to_name (invoke_sig, "invoke");
+       if (ctx)
+               mb = mono_mb_new (method->klass, name, MONO_WRAPPER_DELEGATE_INVOKE);
+       else
+               mb = mono_mb_new (get_wrapper_target_class (method->klass->image), name, MONO_WRAPPER_DELEGATE_INVOKE);
+       g_free (name);
 
-#endif /* DISABLE_REMOTING */
+#ifndef DISABLE_JIT
+       /* allocate local 0 (object) */
+       local_target = mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
+       local_prev = mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
 
+       g_assert (sig->hasthis);
+       
+       /*
+        * if (prev != null)
+         *     prev.Invoke( args .. );
+        * return this.<target>( args .. );
+         */
+       
+       /* this wrapper can be used in unmanaged-managed transitions */
+       emit_thread_interrupt_checkpoint (mb);
+       
+       /* get this->prev */
+       mono_mb_emit_ldarg (mb, 0);
+       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoMulticastDelegate, prev));
+       mono_mb_emit_byte (mb, CEE_LDIND_REF);
+       mono_mb_emit_stloc (mb, local_prev);
+       mono_mb_emit_ldloc (mb, local_prev);
 
-/* mono_get_xdomain_marshal_type()
- * Returns the kind of marshalling that a type needs for cross domain calls.
- */
-static MonoXDomainMarshalType
-mono_get_xdomain_marshal_type (MonoType *t)
-{
-       switch (t->type) {
-       case MONO_TYPE_VOID:
-               g_assert_not_reached ();
-               break;
-       case MONO_TYPE_U1:
-       case MONO_TYPE_I1:
-       case MONO_TYPE_BOOLEAN:
-       case MONO_TYPE_U2:
-       case MONO_TYPE_I2:
-       case MONO_TYPE_CHAR:
-       case MONO_TYPE_U4:
-       case MONO_TYPE_I4:
-       case MONO_TYPE_I8:
-       case MONO_TYPE_U8:
-       case MONO_TYPE_R4:
-       case MONO_TYPE_R8:
-               return MONO_MARSHAL_NONE;
-       case MONO_TYPE_STRING:
-               return MONO_MARSHAL_COPY;
-       case MONO_TYPE_ARRAY:
-       case MONO_TYPE_SZARRAY: {
-               MonoClass *elem_class = mono_class_from_mono_type (t)->element_class;
-               if (mono_get_xdomain_marshal_type (&(elem_class->byval_arg)) != MONO_MARSHAL_SERIALIZE)
-                       return MONO_MARSHAL_COPY;
-               break;
-       }
+       /* if prev != null */
+       pos0 = mono_mb_emit_branch (mb, CEE_BRFALSE);
+
+       /* then recurse */
+
+       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+       mono_mb_emit_byte (mb, CEE_MONO_NOT_TAKEN);
+
+       mono_mb_emit_ldloc (mb, local_prev);
+       for (i = 0; i < sig->param_count; i++)
+               mono_mb_emit_ldarg (mb, i + 1);
+       if (ctx) {
+               MonoError error;
+               mono_mb_emit_op (mb, CEE_CALLVIRT, mono_class_inflate_generic_method_checked (method, &container->context, &error));
+               g_assert (mono_error_ok (&error)); /* FIXME don't swallow the error */
+       } else {
+               mono_mb_emit_op (mb, CEE_CALLVIRT, method);
        }
+       if (sig->ret->type != MONO_TYPE_VOID)
+               mono_mb_emit_byte (mb, CEE_POP);
 
-       return MONO_MARSHAL_SERIALIZE;
-}
+       /* continued or prev == null */
+       mono_mb_patch_branch (mb, pos0);
 
+       /* get this->target */
+       mono_mb_emit_ldarg (mb, 0);
+       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoDelegate, target));
+       mono_mb_emit_byte (mb, CEE_LDIND_REF);
+       mono_mb_emit_stloc (mb, local_target);
 
-/* mono_marshal_xdomain_copy_value
- * Makes a copy of "val" suitable for the current domain.
- */
-MonoObject *
-mono_marshal_xdomain_copy_value (MonoObject *val)
-{
-       MonoDomain *domain;
-       if (val == NULL) return NULL;
+       /*static methods with bound first arg can have null target and still be bound*/
+       if (!static_method_with_first_arg_bound) {
+               /* if target != null */
+               mono_mb_emit_ldloc (mb, local_target);
+               pos0 = mono_mb_emit_branch (mb, CEE_BRFALSE);
 
-       domain = mono_domain_get ();
+               /* then call this->method_ptr nonstatic */
+               if (callvirt) {
+                       // FIXME:
+                       mono_mb_emit_exception_full (mb, "System", "NotImplementedException", "");
+               } else {
+                       mono_mb_emit_ldloc (mb, local_target);
+                       for (i = 0; i < sig->param_count; ++i)
+                               mono_mb_emit_ldarg (mb, i + 1);
+                       mono_mb_emit_ldarg (mb, 0);
+                       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoDelegate, method_ptr));
+                       mono_mb_emit_byte (mb, CEE_LDIND_I );
+                       mono_mb_emit_op (mb, CEE_CALLI, sig);
 
-       switch (mono_object_class (val)->byval_arg.type) {
-       case MONO_TYPE_VOID:
-               g_assert_not_reached ();
-               break;
-       case MONO_TYPE_U1:
-       case MONO_TYPE_I1:
-       case MONO_TYPE_BOOLEAN:
-       case MONO_TYPE_U2:
-       case MONO_TYPE_I2:
-       case MONO_TYPE_CHAR:
-       case MONO_TYPE_U4:
-       case MONO_TYPE_I4:
-       case MONO_TYPE_I8:
-       case MONO_TYPE_U8:
-       case MONO_TYPE_R4:
-       case MONO_TYPE_R8: {
-               return mono_value_box (domain, mono_object_class (val), ((char*)val) + sizeof(MonoObject));
-       }
-       case MONO_TYPE_STRING: {
-               MonoString *str = (MonoString *) val;
-               return (MonoObject *) mono_string_new_utf16 (domain, mono_string_chars (str), mono_string_length (str));
-       }
-       case MONO_TYPE_ARRAY:
-       case MONO_TYPE_SZARRAY: {
-               MonoArray *acopy;
-               MonoXDomainMarshalType mt = mono_get_xdomain_marshal_type (&(mono_object_class (val)->element_class->byval_arg));
-               if (mt == MONO_MARSHAL_SERIALIZE) return NULL;
-               acopy = mono_array_clone_in_domain (domain, (MonoArray *) val);
-               if (mt == MONO_MARSHAL_COPY) {
-                       int i, len = mono_array_length (acopy);
-                       for (i = 0; i < len; i++) {
-                               MonoObject *item = mono_array_get (acopy, gpointer, i);
-                               mono_array_setref (acopy, i, mono_marshal_xdomain_copy_value (item));
-                       }
+                       mono_mb_emit_byte (mb, CEE_RET);
                }
-               return (MonoObject *) acopy;
+       
+               /* else [target == null] call this->method_ptr static */
+               mono_mb_patch_branch (mb, pos0);
        }
+
+       if (callvirt) {
+               if (!closed_over_null) {
+                       mono_mb_emit_ldarg (mb, 1);
+                       mono_mb_emit_op (mb, CEE_CASTCLASS, target_class);
+                       for (i = 1; i < sig->param_count; ++i)
+                               mono_mb_emit_ldarg (mb, i + 1);
+                       mono_mb_emit_op (mb, CEE_CALLVIRT, target_method);
+               } else {
+                       mono_mb_emit_byte (mb, CEE_LDNULL);
+                       for (i = 0; i < sig->param_count; ++i)
+                               mono_mb_emit_ldarg (mb, i + 1);
+                       mono_mb_emit_op (mb, CEE_CALL, target_method);
+               }
+       } else {
+               if (static_method_with_first_arg_bound) {
+                       mono_mb_emit_ldloc (mb, local_target);
+                       if (!MONO_TYPE_IS_REFERENCE (invoke_sig->params[0]))
+                               mono_mb_emit_op (mb, CEE_UNBOX_ANY, mono_class_from_mono_type (invoke_sig->params[0]));
+               }
+               for (i = 0; i < sig->param_count; ++i)
+                       mono_mb_emit_ldarg (mb, i + 1);
+               mono_mb_emit_ldarg (mb, 0);
+               mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoDelegate, method_ptr));
+               mono_mb_emit_byte (mb, CEE_LDIND_I );
+               mono_mb_emit_op (mb, CEE_CALLI, invoke_sig);
        }
 
-       if (mono_object_class (val) == mono_defaults.stringbuilder_class) {
-               MonoStringBuilder *oldsb = (MonoStringBuilder *) val;
-               MonoStringBuilder *newsb = (MonoStringBuilder *) mono_object_new (domain, mono_defaults.stringbuilder_class);
-               MONO_OBJECT_SETREF (newsb, str, mono_string_new_utf16 (domain, mono_string_chars (oldsb->str), mono_string_length (oldsb->str)));
-               newsb->length = oldsb->length;
-               newsb->max_capacity = (gint32)0x7fffffff;
-               return (MonoObject *) newsb;
+       mono_mb_emit_byte (mb, CEE_RET);
+
+       mb->skip_visibility = 1;
+#endif /* DISABLE_JIT */
+
+       if (ctx) {
+               MonoMethod *def;
+
+               def = mono_mb_create_and_cache (cache, cache_key, mb, sig, sig->param_count + 16);
+               res = cache_generic_delegate_wrapper (cache, orig_method, def, ctx);
+       } else if (callvirt) {
+               new_key = g_new0 (SignaturePointerPair, 1);
+               *new_key = key;
+
+               info = mono_wrapper_info_create (mb, subtype);
+
+               res = mono_mb_create_and_cache_full (cache, new_key, mb, sig, sig->param_count + 16, info, &found);
+               if (found)
+                       g_free (new_key);
+       } else {
+               info = mono_wrapper_info_create (mb, subtype);
+
+               res = mono_mb_create_and_cache_full (cache, cache_key, mb, sig, sig->param_count + 16, info, NULL);
        }
-       return NULL;
+       mono_mb_free (mb);
+
+       return res;     
 }
 
-/* mono_marshal_xdomain_copy_out_value()
- * Copies the contents of the src instance into the dst instance. src and dst
- * must have the same type, and if they are arrays, the same size.
+/*
+ * the returned method invokes all methods in a multicast delegate.
  */
-static void
-mono_marshal_xdomain_copy_out_value (MonoObject *src, MonoObject *dst)
+MonoMethod *
+mono_marshal_get_delegate_invoke (MonoMethod *method, MonoDelegate *del)
 {
-       if (src == NULL || dst == NULL) return;
-       
-       g_assert (mono_object_class (src) == mono_object_class (dst));
+       gboolean callvirt = FALSE;
+       gboolean static_method_with_first_arg_bound = FALSE;
+       MonoMethod *target_method = NULL;
+       MonoMethodSignature *sig;
 
-       switch (mono_object_class (src)->byval_arg.type) {
-       case MONO_TYPE_ARRAY:
-       case MONO_TYPE_SZARRAY: {
-               int mt = mono_get_xdomain_marshal_type (&(mono_object_class (src)->element_class->byval_arg));
-               if (mt == MONO_MARSHAL_SERIALIZE) return;
-               if (mt == MONO_MARSHAL_COPY) {
-                       int i, len = mono_array_length ((MonoArray *)dst);
-                       for (i = 0; i < len; i++) {
-                               MonoObject *item = mono_array_get ((MonoArray *)src, gpointer, i);
-                               mono_array_setref ((MonoArray *)dst, i, mono_marshal_xdomain_copy_value (item));
-                       }
-               } else {
-                       mono_array_full_copy ((MonoArray *)src, (MonoArray *)dst);
-               }
-               return;
-       }
-       }
+       sig = mono_signature_no_pinvoke (method);
 
-       if (mono_object_class (src) == mono_defaults.stringbuilder_class) {
-               MonoStringBuilder *src_sb = (MonoStringBuilder *) src;
-               MonoStringBuilder *dst_sb = (MonoStringBuilder *) dst;
-       
-               MONO_OBJECT_SETREF (dst_sb, str, mono_string_new_utf16 (mono_object_domain (dst), mono_string_chars (src_sb->str), mono_string_length (src_sb->str)));
-               dst_sb->cached_str = NULL;
-               dst_sb->length = src_sb->length;
+       if (del && !del->target && del->method && mono_method_signature (del->method)->hasthis) {
+               callvirt = TRUE;
+               target_method = del->method;
        }
-}
-
 
-#if !(defined (DISABLE_JIT) || defined (DISABLE_REMOTING))
-
-static void
-mono_marshal_emit_xdomain_copy_value (MonoMethodBuilder *mb, MonoClass *pclass)
-{
-       mono_mb_emit_icall (mb, mono_marshal_xdomain_copy_value);
-       mono_mb_emit_op (mb, CEE_CASTCLASS, pclass);
-}
+       if (del && del->method && mono_method_signature (del->method)->param_count == sig->param_count + 1 && (del->method->flags & METHOD_ATTRIBUTE_STATIC)) {
+               static_method_with_first_arg_bound = TRUE;
+               target_method = del->method;
+       }
 
-static void
-mono_marshal_emit_xdomain_copy_out_value (MonoMethodBuilder *mb, MonoClass *pclass)
-{
-       mono_mb_emit_icall (mb, mono_marshal_xdomain_copy_out_value);
+       return mono_marshal_get_delegate_invoke_internal (method, callvirt, static_method_with_first_arg_bound, target_method);
 }
-#endif
 
-#ifndef DISABLE_REMOTING
-/* mono_marshal_supports_fast_xdomain()
- * Returns TRUE if the method can use the fast xdomain wrapper.
+/*
+ * signature_dup_add_this:
+ *
+ *  Make a copy of @sig, adding an explicit this argument.
  */
-static gboolean
-mono_marshal_supports_fast_xdomain (MonoMethod *method)
+static MonoMethodSignature*
+signature_dup_add_this (MonoImage *image, MonoMethodSignature *sig, MonoClass *klass)
 {
-       return !mono_class_is_contextbound (method->klass) &&
-                  !((method->flags & METHOD_ATTRIBUTE_SPECIAL_NAME) && (strcmp (".ctor", method->name) == 0));
+       MonoMethodSignature *res;
+       int i;
+
+       res = mono_metadata_signature_alloc (image, sig->param_count + 1);
+       memcpy (res, sig, MONO_SIZEOF_METHOD_SIGNATURE);
+       res->param_count = sig->param_count + 1;
+       res->hasthis = FALSE;
+       for (i = sig->param_count - 1; i >= 0; i --)
+               res->params [i + 1] = sig->params [i];
+       res->params [0] = klass->valuetype ? &klass->this_arg : &klass->byval_arg;
+
+       return res;
 }
-#endif
 
-static gint32
-mono_marshal_set_domain_by_id (gint32 id, MonoBoolean push)
-{
-       MonoDomain *current_domain = mono_domain_get ();
-       MonoDomain *domain = mono_domain_get_by_id (id);
+typedef struct {
+       MonoMethodSignature *ctor_sig;
+       MonoMethodSignature *sig;
+} CtorSigPair;
 
-       if (!domain || !mono_domain_set (domain, FALSE))        
-               mono_raise_exception (mono_get_exception_appdomain_unloaded ());
+/* protected by the marshal lock, contains CtorSigPair pointers */
+static GSList *strsig_list = NULL;
 
-       if (push)
-               mono_thread_push_appdomain_ref (domain);
-       else
-               mono_thread_pop_appdomain_ref ();
+static MonoMethodSignature *
+lookup_string_ctor_signature (MonoMethodSignature *sig)
+{
+       MonoMethodSignature *callsig;
+       CtorSigPair *cs;
+       GSList *item;
 
-       return current_domain->domain_id;
+       mono_marshal_lock ();
+       callsig = NULL;
+       for (item = strsig_list; item; item = item->next) {
+               cs = item->data;
+               /* mono_metadata_signature_equal () is safe to call with the marshal lock
+                * because it is lock-free.
+                */
+               if (mono_metadata_signature_equal (sig, cs->ctor_sig)) {
+                       callsig = cs->sig;
+                       break;
+               }
+       }
+       mono_marshal_unlock ();
+       return callsig;
 }
 
-#if !(defined (DISABLE_JIT) || defined (DISABLE_REMOTING))
-static void
-mono_marshal_emit_switch_domain (MonoMethodBuilder *mb)
+static MonoMethodSignature *
+add_string_ctor_signature (MonoMethod *method)
 {
-       mono_mb_emit_icall (mb, mono_marshal_set_domain_by_id);
+       MonoMethodSignature *callsig;
+       CtorSigPair *cs;
+
+       callsig = signature_dup (method->klass->image, mono_method_signature (method));
+       callsig->ret = &mono_defaults.string_class->byval_arg;
+       cs = g_new (CtorSigPair, 1);
+       cs->sig = callsig;
+       cs->ctor_sig = mono_method_signature (method);
+
+       mono_marshal_lock ();
+       strsig_list = g_slist_prepend (strsig_list, cs);
+       mono_marshal_unlock ();
+       return callsig;
 }
 
-/* mono_marshal_emit_load_domain_method ()
- * Loads into the stack a pointer to the code of the provided method for
- * the current domain.
+/*
+ * mono_marshal_get_string_ctor_signature:
+ *
+ *   Return the modified signature used by string ctors (they return the newly created
+ * string).
  */
-static void
-mono_marshal_emit_load_domain_method (MonoMethodBuilder *mb, MonoMethod *method)
+MonoMethodSignature*
+mono_marshal_get_string_ctor_signature (MonoMethod *method)
 {
-       /* We need a pointer to the method for the running domain (not the domain
-        * that compiles the method).
-        */
-       mono_mb_emit_ptr (mb, method);
-       mono_mb_emit_icall (mb, mono_compile_method);
+       MonoMethodSignature *sig = lookup_string_ctor_signature (mono_method_signature (method));
+       if (!sig)
+               sig = add_string_ctor_signature (method);
+
+       return sig;
 }
-#endif
 
-/* mono_marshal_check_domain_image ()
- * Returns TRUE if the image is loaded in the specified
- * application domain.
- */
-static gboolean
-mono_marshal_check_domain_image (gint32 domain_id, MonoImage *image)
+static MonoType*
+get_runtime_invoke_type (MonoType *t, gboolean ret)
 {
-       MonoAssembly* ass;
-       GSList *tmp;
-       
-       MonoDomain *domain = mono_domain_get_by_id (domain_id);
-       if (!domain)
-               return FALSE;
-       
-       mono_domain_assemblies_lock (domain);
-       for (tmp = domain->domain_assemblies; tmp; tmp = tmp->next) {
-               ass = tmp->data;
-               if (ass->image == image)
-                       break;
+       if (t->byref) {
+               if (t->type == MONO_TYPE_GENERICINST && mono_class_is_nullable (mono_class_from_mono_type (t)))
+                       return t;
+               /* Can't share this with 'I' as that needs another indirection */
+               return &mono_defaults.int_class->this_arg;
        }
-       mono_domain_assemblies_unlock (domain);
-       
-       return tmp != NULL;
-}
 
-#ifndef DISABLE_REMOTING
+       if (MONO_TYPE_IS_REFERENCE (t))
+               return &mono_defaults.object_class->byval_arg;
 
-/* mono_marshal_get_xappdomain_dispatch ()
- * Generates a method that dispatches a method call from another domain into
- * the current domain.
- */
-static MonoMethod *
-mono_marshal_get_xappdomain_dispatch (MonoMethod *method, int *marshal_types, int complex_count, int complex_out_count, int ret_marshal_type)
-{
-       MonoMethodSignature *sig, *csig;
-       MonoMethodBuilder *mb;
-       MonoMethod *res;
-       int i, j, param_index, copy_locals_base;
-       MonoClass *ret_class = NULL;
-       int loc_array=0, loc_return=0, loc_serialized_exc=0;
-       MonoExceptionClause *main_clause;
-       int pos, pos_leave;
-       gboolean copy_return;
-
-       if ((res = mono_marshal_remoting_find_in_cache (method, MONO_WRAPPER_XDOMAIN_DISPATCH)))
-               return res;
-
-       sig = mono_method_signature (method);
-       copy_return = (sig->ret->type != MONO_TYPE_VOID && ret_marshal_type != MONO_MARSHAL_SERIALIZE);
-
-       j = 0;
-       csig = mono_metadata_signature_alloc (mono_defaults.corlib, 3 + sig->param_count - complex_count);
-       csig->params [j++] = &mono_defaults.object_class->byval_arg;
-       csig->params [j++] = &byte_array_class->this_arg;
-       csig->params [j++] = &byte_array_class->this_arg;
-       for (i = 0; i < sig->param_count; i++) {
-               if (marshal_types [i] != MONO_MARSHAL_SERIALIZE)
-                       csig->params [j++] = sig->params [i];
-       }
-       if (copy_return)
-               csig->ret = sig->ret;
-       else
-               csig->ret = &mono_defaults.void_class->byval_arg;
-       csig->pinvoke = 1;
-       csig->hasthis = FALSE;
-
-       mb = mono_mb_new (method->klass, method->name, MONO_WRAPPER_XDOMAIN_DISPATCH);
-       mb->method->save_lmf = 1;
-
-#ifndef DISABLE_JIT
-       /* Locals */
-
-       loc_serialized_exc = mono_mb_add_local (mb, &byte_array_class->byval_arg);
-       if (complex_count > 0)
-               loc_array = mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
-       if (sig->ret->type != MONO_TYPE_VOID) {
-               loc_return = mono_mb_add_local (mb, sig->ret);
-               ret_class = mono_class_from_mono_type (sig->ret);
-       }
-
-       /* try */
-
-       main_clause = mono_image_alloc0 (method->klass->image, sizeof (MonoExceptionClause));
-       main_clause->try_offset = mono_mb_get_label (mb);
-
-       /* Clean the call context */
-
-       mono_mb_emit_byte (mb, CEE_LDNULL);
-       mono_mb_emit_managed_call (mb, method_set_call_context, NULL);
-       mono_mb_emit_byte (mb, CEE_POP);
-
-       /* Deserialize call data */
-
-       mono_mb_emit_ldarg (mb, 1);
-       mono_mb_emit_byte (mb, CEE_LDIND_REF);
-       mono_mb_emit_byte (mb, CEE_DUP);
-       pos = mono_mb_emit_short_branch (mb, CEE_BRFALSE_S);
-       
-       mono_marshal_emit_xdomain_copy_value (mb, byte_array_class);
-       mono_mb_emit_managed_call (mb, method_rs_deserialize, NULL);
-       
-       if (complex_count > 0)
-               mono_mb_emit_stloc (mb, loc_array);
-       else
-               mono_mb_emit_byte (mb, CEE_POP);
-
-       mono_mb_patch_short_branch (mb, pos);
-
-       /* Get the target object */
-       
-       mono_mb_emit_ldarg (mb, 0);
-       mono_mb_emit_managed_call (mb, method_rs_appdomain_target, NULL);
-
-       /* Load the arguments */
-       
-       copy_locals_base = mb->locals;
-       param_index = 3;        // Index of the first non-serialized parameter of this wrapper
-       j = 0;
-       for (i = 0; i < sig->param_count; i++) {
-               MonoType *pt = sig->params [i];
-               MonoClass *pclass = mono_class_from_mono_type (pt);
-               switch (marshal_types [i]) {
-               case MONO_MARSHAL_SERIALIZE: {
-                       /* take the value from the serialized array */
-                       mono_mb_emit_ldloc (mb, loc_array);
-                       mono_mb_emit_icon (mb, j++);
-                       if (pt->byref) {
-                               if (pclass->valuetype) {
-                                       mono_mb_emit_byte (mb, CEE_LDELEM_REF);
-                                       mono_mb_emit_op (mb, CEE_UNBOX, pclass);
-                               } else {
-                                       mono_mb_emit_op (mb, CEE_LDELEMA, pclass);
-                               }
-                       } else {
-                               if (pclass->valuetype) {
-                                       mono_mb_emit_byte (mb, CEE_LDELEM_REF);
-                                       mono_mb_emit_op (mb, CEE_UNBOX, pclass);
-                                       mono_mb_emit_op (mb, CEE_LDOBJ, pclass);
-                               } else {
-                                       mono_mb_emit_byte (mb, CEE_LDELEM_REF);
-                                       if (pclass != mono_defaults.object_class) {
-                                               mono_mb_emit_op (mb, CEE_CASTCLASS, pclass);
-                                       }
-                               }
-                       }
-                       break;
-               }
-               case MONO_MARSHAL_COPY_OUT: {
-                       /* Keep a local copy of the value since we need to copy it back after the call */
-                       int copy_local = mono_mb_add_local (mb, &(pclass->byval_arg));
-                       mono_mb_emit_ldarg (mb, param_index++);
-                       mono_marshal_emit_xdomain_copy_value (mb, pclass);
-                       mono_mb_emit_byte (mb, CEE_DUP);
-                       mono_mb_emit_stloc (mb, copy_local);
-                       break;
-               }
-               case MONO_MARSHAL_COPY: {
-                       mono_mb_emit_ldarg (mb, param_index);
-                       if (pt->byref) {
-                               mono_mb_emit_byte (mb, CEE_DUP);
-                               mono_mb_emit_byte (mb, CEE_DUP);
-                               mono_mb_emit_byte (mb, CEE_LDIND_REF);
-                               mono_marshal_emit_xdomain_copy_value (mb, pclass);
-                               mono_mb_emit_byte (mb, CEE_STIND_REF);
-                       } else {
-                               mono_marshal_emit_xdomain_copy_value (mb, pclass);
-                       }
-                       param_index++;
-                       break;
-               }
-               case MONO_MARSHAL_NONE:
-                       mono_mb_emit_ldarg (mb, param_index++);
-                       break;
-               }
-       }
-
-       /* Make the call to the real object */
-
-       emit_thread_force_interrupt_checkpoint (mb);
-       
-       mono_mb_emit_op (mb, CEE_CALLVIRT, method);
-
-       if (sig->ret->type != MONO_TYPE_VOID)
-               mono_mb_emit_stloc (mb, loc_return);
-
-       /* copy back MONO_MARSHAL_COPY_OUT parameters */
-
-       j = 0;
-       param_index = 3;
-       for (i = 0; i < sig->param_count; i++) {
-               if (marshal_types [i] == MONO_MARSHAL_SERIALIZE) continue;
-               if (marshal_types [i] == MONO_MARSHAL_COPY_OUT) {
-                       mono_mb_emit_ldloc (mb, copy_locals_base + (j++));
-                       mono_mb_emit_ldarg (mb, param_index);
-                       mono_marshal_emit_xdomain_copy_out_value (mb, mono_class_from_mono_type (sig->params [i]));
-               }
-               param_index++;
-       }
-
-       /* Serialize the return values */
-       
-       if (complex_out_count > 0) {
-               /* Reset parameters in the array that don't need to be serialized back */
-               j = 0;
-               for (i = 0; i < sig->param_count; i++) {
-                       if (marshal_types[i] != MONO_MARSHAL_SERIALIZE) continue;
-                       if (!sig->params [i]->byref) {
-                               mono_mb_emit_ldloc (mb, loc_array);
-                               mono_mb_emit_icon (mb, j);
-                               mono_mb_emit_byte (mb, CEE_LDNULL);
-                               mono_mb_emit_byte (mb, CEE_STELEM_REF);
-                       }
-                       j++;
-               }
-       
-               /* Add the return value to the array */
-       
-               if (ret_marshal_type == MONO_MARSHAL_SERIALIZE) {
-                       mono_mb_emit_ldloc (mb, loc_array);
-                       mono_mb_emit_icon (mb, complex_count);  /* The array has an additional slot to hold the ret value */
-                       mono_mb_emit_ldloc (mb, loc_return);
-
-                       g_assert (ret_class); /*FIXME properly fail here*/
-                       if (ret_class->valuetype) {
-                               mono_mb_emit_op (mb, CEE_BOX, ret_class);
-                       }
-                       mono_mb_emit_byte (mb, CEE_STELEM_REF);
-               }
-       
-               /* Serialize */
-       
-               mono_mb_emit_ldarg (mb, 1);
-               mono_mb_emit_ldloc (mb, loc_array);
-               mono_mb_emit_managed_call (mb, method_rs_serialize, NULL);
-               mono_mb_emit_byte (mb, CEE_STIND_REF);
-       } else if (ret_marshal_type == MONO_MARSHAL_SERIALIZE) {
-               mono_mb_emit_ldarg (mb, 1);
-               mono_mb_emit_ldloc (mb, loc_return);
-               if (ret_class->valuetype) {
-                       mono_mb_emit_op (mb, CEE_BOX, ret_class);
-               }
-               mono_mb_emit_managed_call (mb, method_rs_serialize, NULL);
-               mono_mb_emit_byte (mb, CEE_STIND_REF);
-       } else {
-               mono_mb_emit_ldarg (mb, 1);
-               mono_mb_emit_byte (mb, CEE_LDNULL);
-               mono_mb_emit_managed_call (mb, method_rs_serialize, NULL);
-               mono_mb_emit_byte (mb, CEE_STIND_REF);
-       }
-
-       mono_mb_emit_ldarg (mb, 2);
-       mono_mb_emit_byte (mb, CEE_LDNULL);
-       mono_mb_emit_byte (mb, CEE_STIND_REF);
-       pos_leave = mono_mb_emit_branch (mb, CEE_LEAVE);
-
-       /* Main exception catch */
-       main_clause->flags = MONO_EXCEPTION_CLAUSE_NONE;
-       main_clause->try_len = mono_mb_get_pos (mb) - main_clause->try_offset;
-       main_clause->data.catch_class = mono_defaults.object_class;
-       
-       /* handler code */
-       main_clause->handler_offset = mono_mb_get_label (mb);
-       mono_mb_emit_managed_call (mb, method_rs_serialize_exc, NULL);
-       mono_mb_emit_stloc (mb, loc_serialized_exc);
-       mono_mb_emit_ldarg (mb, 2);
-       mono_mb_emit_ldloc (mb, loc_serialized_exc);
-       mono_mb_emit_byte (mb, CEE_STIND_REF);
-       mono_mb_emit_branch (mb, CEE_LEAVE);
-       main_clause->handler_len = mono_mb_get_pos (mb) - main_clause->handler_offset;
-       /* end catch */
-
-       mono_mb_patch_branch (mb, pos_leave);
-       
-       if (copy_return)
-               mono_mb_emit_ldloc (mb, loc_return);
-
-       mono_mb_emit_byte (mb, CEE_RET);
-
-       mono_mb_set_clauses (mb, 1, main_clause);
-#endif
-
-       res = mono_remoting_mb_create_and_cache (method, mb, csig, csig->param_count + 16);
-       mono_mb_free (mb);
-
-       return res;
-}
-
-/* mono_marshal_get_xappdomain_invoke ()
- * Generates a fast remoting wrapper for cross app domain calls.
- */
-MonoMethod *
-mono_marshal_get_xappdomain_invoke (MonoMethod *method)
-{
-       MonoMethodSignature *sig;
-       MonoMethodBuilder *mb;
-       MonoMethod *res;
-       int i, j, complex_count, complex_out_count, copy_locals_base;
-       int *marshal_types;
-       MonoClass *ret_class = NULL;
-       MonoMethod *xdomain_method;
-       int ret_marshal_type = MONO_MARSHAL_NONE;
-       int loc_array=0, loc_serialized_data=-1, loc_real_proxy;
-       int loc_old_domainid, loc_domainid, loc_return=0, loc_serialized_exc=0, loc_context;
-       int pos, pos_dispatch, pos_noex;
-       gboolean copy_return = FALSE;
-
-       g_assert (method);
-       
-       if (method->wrapper_type == MONO_WRAPPER_REMOTING_INVOKE || method->wrapper_type == MONO_WRAPPER_XDOMAIN_INVOKE)
-               return method;
-
-       /* we cant remote methods without this pointer */
-       if (!mono_method_signature (method)->hasthis)
-               return method;
-
-       if (!mono_marshal_supports_fast_xdomain (method))
-               return mono_marshal_get_remoting_invoke (method);
-       
-       mono_remoting_marshal_init ();
-
-       if ((res = mono_marshal_remoting_find_in_cache (method, MONO_WRAPPER_XDOMAIN_INVOKE)))
-               return res;
-       
-       sig = mono_signature_no_pinvoke (method);
-
-       mb = mono_mb_new (method->klass, method->name, MONO_WRAPPER_XDOMAIN_INVOKE);
-       mb->method->save_lmf = 1;
-
-       /* Count the number of parameters that need to be serialized */
-
-       marshal_types = alloca (sizeof (int) * sig->param_count);
-       complex_count = complex_out_count = 0;
-       for (i = 0; i < sig->param_count; i++) {
-               MonoType *ptype = sig->params[i];
-               int mt = mono_get_xdomain_marshal_type (ptype);
-               
-               /* If the [Out] attribute is applied to a parameter that can be internally copied,
-                * the copy will be made by reusing the original object instance
-                */
-               if ((ptype->attrs & PARAM_ATTRIBUTE_OUT) != 0 && mt == MONO_MARSHAL_COPY && !ptype->byref)
-                       mt = MONO_MARSHAL_COPY_OUT;
-               else if (mt == MONO_MARSHAL_SERIALIZE) {
-                       complex_count++;
-                       if (ptype->byref) complex_out_count++;
-               }
-               marshal_types [i] = mt;
-       }
-
-       if (sig->ret->type != MONO_TYPE_VOID) {
-               ret_marshal_type = mono_get_xdomain_marshal_type (sig->ret);
-               ret_class = mono_class_from_mono_type (sig->ret);
-               copy_return = ret_marshal_type != MONO_MARSHAL_SERIALIZE;
-       }
-       
-       /* Locals */
-
-#ifndef DISABLE_JIT
-       if (complex_count > 0)
-               loc_array = mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
-       loc_serialized_data = mono_mb_add_local (mb, &byte_array_class->byval_arg);
-       loc_real_proxy = mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
-       if (copy_return)
-               loc_return = mono_mb_add_local (mb, sig->ret);
-       loc_old_domainid = mono_mb_add_local (mb, &mono_defaults.int32_class->byval_arg);
-       loc_domainid = mono_mb_add_local (mb, &mono_defaults.int32_class->byval_arg);
-       loc_serialized_exc = mono_mb_add_local (mb, &byte_array_class->byval_arg);
-       loc_context = mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
-
-       /* Save thread domain data */
-
-       mono_mb_emit_icall (mb, mono_context_get);
-       mono_mb_emit_byte (mb, CEE_DUP);
-       mono_mb_emit_stloc (mb, loc_context);
-
-       /* If the thread is not running in the default context, it needs to go
-        * through the whole remoting sink, since the context is going to change
-        */
-       mono_mb_emit_managed_call (mb, method_needs_context_sink, NULL);
-       pos = mono_mb_emit_short_branch (mb, CEE_BRTRUE_S);
-       
-       /* Another case in which the fast path can't be used: when the target domain
-        * has a different image for the same assembly.
-        */
-
-       /* Get the target domain id */
-
-       mono_mb_emit_ldarg (mb, 0);
-       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoTransparentProxy, rp));
-       mono_mb_emit_byte (mb, CEE_LDIND_REF);
-       mono_mb_emit_byte (mb, CEE_DUP);
-       mono_mb_emit_stloc (mb, loc_real_proxy);
-
-       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoRealProxy, target_domain_id));
-       mono_mb_emit_byte (mb, CEE_LDIND_I4);
-       mono_mb_emit_stloc (mb, loc_domainid);
-
-       /* Check if the target domain has the same image for the required assembly */
-
-       mono_mb_emit_ldloc (mb, loc_domainid);
-       mono_mb_emit_ptr (mb, method->klass->image);
-       mono_mb_emit_icall (mb, mono_marshal_check_domain_image);
-       pos_dispatch = mono_mb_emit_short_branch (mb, CEE_BRTRUE_S);
-
-       /* Use the whole remoting sink to dispatch this message */
-
-       mono_mb_patch_short_branch (mb, pos);
-
-       mono_mb_emit_ldarg (mb, 0);
-       for (i = 0; i < sig->param_count; i++)
-               mono_mb_emit_ldarg (mb, i + 1);
-       
-       mono_mb_emit_managed_call (mb, mono_marshal_get_remoting_invoke (method), NULL);
-       mono_mb_emit_byte (mb, CEE_RET);
-       mono_mb_patch_short_branch (mb, pos_dispatch);
-
-       /* Create the array that will hold the parameters to be serialized */
-
-       if (complex_count > 0) {
-               mono_mb_emit_icon (mb, (ret_marshal_type == MONO_MARSHAL_SERIALIZE && complex_out_count > 0) ? complex_count + 1 : complex_count);      /* +1 for the return type */
-               mono_mb_emit_op (mb, CEE_NEWARR, mono_defaults.object_class);
-       
-               j = 0;
-               for (i = 0; i < sig->param_count; i++) {
-                       MonoClass *pclass;
-                       if (marshal_types [i] != MONO_MARSHAL_SERIALIZE) continue;
-                       pclass = mono_class_from_mono_type (sig->params[i]);
-                       mono_mb_emit_byte (mb, CEE_DUP);
-                       mono_mb_emit_icon (mb, j);
-                       mono_mb_emit_ldarg (mb, i + 1);         /* 0=this */
-                       if (sig->params[i]->byref) {
-                               if (pclass->valuetype)
-                                       mono_mb_emit_op (mb, CEE_LDOBJ, pclass);
-                               else
-                                       mono_mb_emit_byte (mb, CEE_LDIND_REF);
-                       }
-                       if (pclass->valuetype)
-                               mono_mb_emit_op (mb, CEE_BOX, pclass);
-                       mono_mb_emit_byte (mb, CEE_STELEM_REF);
-                       j++;
-               }
-               mono_mb_emit_stloc (mb, loc_array);
-
-               /* Serialize parameters */
-       
-               mono_mb_emit_ldloc (mb, loc_array);
-               mono_mb_emit_managed_call (mb, method_rs_serialize, NULL);
-               mono_mb_emit_stloc (mb, loc_serialized_data);
-       } else {
-               mono_mb_emit_byte (mb, CEE_LDNULL);
-               mono_mb_emit_managed_call (mb, method_rs_serialize, NULL);
-               mono_mb_emit_stloc (mb, loc_serialized_data);
-       }
-
-       /* switch domain */
-
-       mono_mb_emit_ldloc (mb, loc_domainid);
-       mono_mb_emit_byte (mb, CEE_LDC_I4_1);
-       mono_marshal_emit_switch_domain (mb);
-       mono_mb_emit_stloc (mb, loc_old_domainid);
-
-       /* Load the arguments */
-       
-       mono_mb_emit_ldloc (mb, loc_real_proxy);
-       mono_mb_emit_ldloc_addr (mb, loc_serialized_data);
-       mono_mb_emit_ldloc_addr (mb, loc_serialized_exc);
-
-       copy_locals_base = mb->locals;
-       for (i = 0; i < sig->param_count; i++) {
-               switch (marshal_types [i]) {
-               case MONO_MARSHAL_SERIALIZE:
-                       continue;
-               case MONO_MARSHAL_COPY: {
-                       mono_mb_emit_ldarg (mb, i+1);
-                       if (sig->params [i]->byref) {
-                               /* make a local copy of the byref parameter. The real parameter
-                                * will be updated after the xdomain call
-                                */
-                               MonoClass *pclass = mono_class_from_mono_type (sig->params [i]);
-                               int copy_local = mono_mb_add_local (mb, &(pclass->byval_arg));
-                               mono_mb_emit_byte (mb, CEE_LDIND_REF);
-                               mono_mb_emit_stloc (mb, copy_local);
-                               mono_mb_emit_ldloc_addr (mb, copy_local);
-                       }
-                       break;
-               }
-               case MONO_MARSHAL_COPY_OUT:
-               case MONO_MARSHAL_NONE:
-                       mono_mb_emit_ldarg (mb, i+1);
-                       break;
-               }
-       }
-
-       /* Make the call to the invoke wrapper in the target domain */
-
-       xdomain_method = mono_marshal_get_xappdomain_dispatch (method, marshal_types, complex_count, complex_out_count, ret_marshal_type);
-       mono_marshal_emit_load_domain_method (mb, xdomain_method);
-       mono_mb_emit_calli (mb, mono_method_signature (xdomain_method));
-
-       if (copy_return)
-               mono_mb_emit_stloc (mb, loc_return);
-
-       /* Switch domain */
-
-       mono_mb_emit_ldloc (mb, loc_old_domainid);
-       mono_mb_emit_byte (mb, CEE_LDC_I4_0);
-       mono_marshal_emit_switch_domain (mb);
-       mono_mb_emit_byte (mb, CEE_POP);
-       
-       /* Restore thread domain data */
-       
-       mono_mb_emit_ldloc (mb, loc_context);
-       mono_mb_emit_icall (mb, mono_context_set);
-       
-       /* if (loc_serialized_exc != null) ... */
-
-       mono_mb_emit_ldloc (mb, loc_serialized_exc);
-       pos_noex = mono_mb_emit_short_branch (mb, CEE_BRFALSE_S);
-
-       mono_mb_emit_ldloc (mb, loc_serialized_exc);
-       mono_marshal_emit_xdomain_copy_value (mb, byte_array_class);
-       mono_mb_emit_managed_call (mb, method_rs_deserialize, NULL);
-       mono_mb_emit_op (mb, CEE_CASTCLASS, mono_defaults.exception_class);
-       mono_mb_emit_managed_call (mb, method_exc_fixexc, NULL);
-       mono_mb_emit_byte (mb, CEE_THROW);
-       mono_mb_patch_short_branch (mb, pos_noex);
-
-       /* copy back non-serialized output parameters */
-
-       j = 0;
-       for (i = 0; i < sig->param_count; i++) {
-               if (!sig->params [i]->byref || marshal_types [i] != MONO_MARSHAL_COPY) continue;
-               mono_mb_emit_ldarg (mb, i + 1);
-               mono_mb_emit_ldloc (mb, copy_locals_base + (j++));
-               mono_marshal_emit_xdomain_copy_value (mb, mono_class_from_mono_type (sig->params [i]));
-               mono_mb_emit_byte (mb, CEE_STIND_REF);
-       }
-
-       /* Deserialize out parameters */
-
-       if (complex_out_count > 0) {
-               mono_mb_emit_ldloc (mb, loc_serialized_data);
-               mono_marshal_emit_xdomain_copy_value (mb, byte_array_class);
-               mono_mb_emit_managed_call (mb, method_rs_deserialize, NULL);
-               mono_mb_emit_stloc (mb, loc_array);
-       
-               /* Copy back output parameters and return type */
-               
-               j = 0;
-               for (i = 0; i < sig->param_count; i++) {
-                       if (marshal_types [i] != MONO_MARSHAL_SERIALIZE) continue;
-                       if (sig->params[i]->byref) {
-                               MonoClass *pclass = mono_class_from_mono_type (sig->params [i]);
-                               mono_mb_emit_ldarg (mb, i + 1);
-                               mono_mb_emit_ldloc (mb, loc_array);
-                               mono_mb_emit_icon (mb, j);
-                               mono_mb_emit_byte (mb, CEE_LDELEM_REF);
-                               if (pclass->valuetype) {
-                                       mono_mb_emit_op (mb, CEE_UNBOX, pclass);
-                                       mono_mb_emit_op (mb, CEE_LDOBJ, pclass);
-                                       mono_mb_emit_op (mb, CEE_STOBJ, pclass);
-                               } else {
-                                       if (pclass != mono_defaults.object_class)
-                                               mono_mb_emit_op (mb, CEE_CASTCLASS, pclass);
-                                       mono_mb_emit_byte (mb, CEE_STIND_REF);
-                               }
-                       }
-                       j++;
-               }
-       
-               if (ret_marshal_type == MONO_MARSHAL_SERIALIZE) {
-                       mono_mb_emit_ldloc (mb, loc_array);
-                       mono_mb_emit_icon (mb, complex_count);
-                       mono_mb_emit_byte (mb, CEE_LDELEM_REF);
-                       if (ret_class->valuetype) {
-                               mono_mb_emit_op (mb, CEE_UNBOX, ret_class);
-                               mono_mb_emit_op (mb, CEE_LDOBJ, ret_class);
-                       }
-               }
-       } else if (ret_marshal_type == MONO_MARSHAL_SERIALIZE) {
-               mono_mb_emit_ldloc (mb, loc_serialized_data);
-               mono_marshal_emit_xdomain_copy_value (mb, byte_array_class);
-               mono_mb_emit_managed_call (mb, method_rs_deserialize, NULL);
-               if (ret_class->valuetype) {
-                       mono_mb_emit_op (mb, CEE_UNBOX, ret_class);
-                       mono_mb_emit_op (mb, CEE_LDOBJ, ret_class);
-               } else if (ret_class != mono_defaults.object_class) {
-                       mono_mb_emit_op (mb, CEE_CASTCLASS, ret_class);
-               }
-       } else {
-               mono_mb_emit_ldloc (mb, loc_serialized_data);
-               mono_mb_emit_byte (mb, CEE_DUP);
-               pos = mono_mb_emit_short_branch (mb, CEE_BRFALSE_S);
-               mono_marshal_emit_xdomain_copy_value (mb, byte_array_class);
-       
-               mono_mb_patch_short_branch (mb, pos);
-               mono_mb_emit_managed_call (mb, method_rs_deserialize, NULL);
-               mono_mb_emit_byte (mb, CEE_POP);
-       }
-
-       if (copy_return) {
-               mono_mb_emit_ldloc (mb, loc_return);
-               if (ret_marshal_type == MONO_MARSHAL_COPY)
-                       mono_marshal_emit_xdomain_copy_value (mb, ret_class);
-       }
-
-       mono_mb_emit_byte (mb, CEE_RET);
-#endif /* DISABLE_JIT */
-
-       res = mono_remoting_mb_create_and_cache (method, mb, sig, sig->param_count + 16);
-       mono_mb_free (mb);
-
-       return res;
-}
-
-MonoMethod *
-mono_marshal_get_remoting_invoke_for_target (MonoMethod *method, MonoRemotingTarget target_type)
-{
-       if (target_type == MONO_REMOTING_TARGET_APPDOMAIN) {
-               return mono_marshal_get_xappdomain_invoke (method);
-       } else if (target_type == MONO_REMOTING_TARGET_COMINTEROP) {
-#ifndef DISABLE_COM
-               return mono_cominterop_get_invoke (method);
-#else
-               g_assert_not_reached ();
-#endif
-       } else {
-               return mono_marshal_get_remoting_invoke (method);
-       }
-       /* Not erached */
-       return NULL;
-}
-
-G_GNUC_UNUSED static gpointer
-mono_marshal_load_remoting_wrapper (MonoRealProxy *rp, MonoMethod *method)
-{
-       if (rp->target_domain_id != -1)
-               return mono_compile_method (mono_marshal_get_xappdomain_invoke (method));
-       else
-               return mono_compile_method (mono_marshal_get_remoting_invoke (method));
-}
-
-MonoMethod *
-mono_marshal_get_remoting_invoke_with_check (MonoMethod *method)
-{
-       MonoMethodSignature *sig;
-       MonoMethodBuilder *mb;
-       MonoMethod *res, *native;
-       int i, pos, pos_rem;
-
-       g_assert (method);
-
-       if (method->wrapper_type == MONO_WRAPPER_REMOTING_INVOKE_WITH_CHECK)
-               return method;
-
-       /* we cant remote methods without this pointer */
-       g_assert (mono_method_signature (method)->hasthis);
-
-       if ((res = mono_marshal_remoting_find_in_cache (method, MONO_WRAPPER_REMOTING_INVOKE_WITH_CHECK)))
-               return res;
-
-       sig = mono_signature_no_pinvoke (method);
-       
-       mb = mono_mb_new (method->klass, method->name, MONO_WRAPPER_REMOTING_INVOKE_WITH_CHECK);
-
-#ifndef DISABLE_JIT
-       for (i = 0; i <= sig->param_count; i++)
-               mono_mb_emit_ldarg (mb, i);
-       
-       mono_mb_emit_ldarg (mb, 0);
-       pos = mono_mb_emit_proxy_check (mb, CEE_BNE_UN);
-
-       if (mono_marshal_supports_fast_xdomain (method)) {
-               mono_mb_emit_ldarg (mb, 0);
-               pos_rem = mono_mb_emit_xdomain_check (mb, CEE_BEQ);
-               
-               /* wrapper for cross app domain calls */
-               native = mono_marshal_get_xappdomain_invoke (method);
-               mono_mb_emit_managed_call (mb, native, mono_method_signature (native));
-               mono_mb_emit_byte (mb, CEE_RET);
-               
-               mono_mb_patch_branch (mb, pos_rem);
-       }
-       /* wrapper for normal remote calls */
-       native = mono_marshal_get_remoting_invoke (method);
-       mono_mb_emit_managed_call (mb, native, mono_method_signature (native));
-       mono_mb_emit_byte (mb, CEE_RET);
-
-       /* not a proxy */
-       mono_mb_patch_branch (mb, pos);
-       mono_mb_emit_managed_call (mb, method, mono_method_signature (method));
-       mono_mb_emit_byte (mb, CEE_RET);
-#endif
-
-       res = mono_remoting_mb_create_and_cache (method, mb, sig, sig->param_count + 16);
-       mono_mb_free (mb);
-
-       return res;
-}
-
-#endif /* DISABLE_REMOTING */
-
-typedef struct
-{
-       MonoMethodSignature *sig;
-       MonoMethod *method;
-} SignatureMethodPair;
-
-static guint
-signature_method_pair_hash (gconstpointer data)
-{
-       SignatureMethodPair *pair = (SignatureMethodPair*)data;
-
-       return mono_signature_hash (pair->sig) ^ mono_aligned_addr_hash (pair->method);
-}
-
-static gboolean
-signature_method_pair_equal (SignatureMethodPair *pair1, SignatureMethodPair *pair2)
-{
-       return mono_metadata_signature_equal (pair1->sig, pair2->sig) && (pair1->method == pair2->method);
-}
-
-static gboolean
-signature_method_pair_matches_method (gpointer key, gpointer value, gpointer user_data)
-{
-       SignatureMethodPair *pair = (SignatureMethodPair*)key;
-       MonoMethod *method = (MonoMethod*)user_data;
-
-       return pair->method == method;
-}
-
-static void
-free_signature_method_pair (SignatureMethodPair *pair)
-{
-       g_free (pair);
-}
-
-static MonoMethod *
-mono_marshal_get_delegate_invoke_internal (MonoMethod *method, gboolean callvirt, gboolean static_method_with_first_arg_bound, MonoMethod *target_method)
-{
-       MonoMethodSignature *sig, *static_sig, *invoke_sig;
-       int i;
-       MonoMethodBuilder *mb;
-       MonoMethod *res;
-       GHashTable *cache;
-       gpointer cache_key = NULL;
-       SignatureMethodPair key;
-       SignatureMethodPair *new_key;
-       int local_prev, local_target;
-       int pos0;
-       char *name;
-       MonoClass *target_class = NULL;
-       gboolean closed_over_null = FALSE;
-       MonoGenericContext *ctx = NULL;
-       MonoGenericContainer *container = NULL;
-       MonoMethod *orig_method = NULL;
-       WrapperInfo *info;
-       WrapperSubtype subtype = WRAPPER_SUBTYPE_NONE;
-       gboolean found;
-
-       g_assert (method && method->klass->parent == mono_defaults.multicastdelegate_class &&
-                 !strcmp (method->name, "Invoke"));
-
-       invoke_sig = sig = mono_signature_no_pinvoke (method);
-
-       /*
-        * If the delegate target is null, and the target method is not static, a virtual 
-        * call is made to that method with the first delegate argument as this. This is 
-        * a non-documented .NET feature.
-        */
-       if (callvirt) {
-               subtype = WRAPPER_SUBTYPE_DELEGATE_INVOKE_VIRTUAL;
-               if (target_method->is_inflated) {
-                       MonoType *target_type;
-
-                       g_assert (method->signature->hasthis);
-                       target_type = mono_class_inflate_generic_type (method->signature->params [0],
-                               mono_method_get_context (method));
-                       target_class = mono_class_from_mono_type (target_type);
-               } else {
-                       target_class = target_method->klass;
-               }
-
-               closed_over_null = sig->param_count == mono_method_signature (target_method)->param_count;
-       }
-
-       if (static_method_with_first_arg_bound) {
-               subtype = WRAPPER_SUBTYPE_DELEGATE_INVOKE_BOUND;
-               g_assert (!callvirt);
-               invoke_sig = mono_method_signature (target_method);
-       }
-
-       /*
-        * For generic delegates, create a generic wrapper, and return an instance to help AOT.
-        */
-       if (method->is_inflated && subtype == WRAPPER_SUBTYPE_NONE) {
-               orig_method = method;
-               ctx = &((MonoMethodInflated*)method)->context;
-               method = ((MonoMethodInflated*)method)->declaring;
-
-               container = mono_method_get_generic_container (method);
-               if (!container)
-                       container = method->klass->generic_container;
-               g_assert (container);
-
-               invoke_sig = sig = mono_signature_no_pinvoke (method);
-       }
-
-       /*
-        * Check cache
-        */
-       if (ctx) {
-               cache = get_cache (&method->klass->image->delegate_invoke_generic_cache, mono_aligned_addr_hash, NULL);
-               res = check_generic_delegate_wrapper_cache (cache, orig_method, method, ctx);
-               if (res)
-                       return res;
-               cache_key = method->klass;
-       } else if (static_method_with_first_arg_bound) {
-               cache = get_cache (&method->klass->image->delegate_bound_static_invoke_cache,
-                                                  (GHashFunc)mono_signature_hash, 
-                                                  (GCompareFunc)mono_metadata_signature_equal);
-               /*
-                * The wrapper is based on sig+invoke_sig, but sig can be derived from invoke_sig.
-                */
-               res = mono_marshal_find_in_cache (cache, invoke_sig);
-               if (res)
-                       return res;
-               cache_key = invoke_sig;
-       } else if (callvirt) {
-               GHashTable **cache_ptr;
-
-               cache_ptr = &method->klass->image->delegate_abstract_invoke_cache;
-
-               /* We need to cache the signature+method pair */
-               mono_marshal_lock ();
-               if (!*cache_ptr)
-                       *cache_ptr = g_hash_table_new_full (signature_method_pair_hash, (GEqualFunc)signature_method_pair_equal, (GDestroyNotify)free_signature_method_pair, NULL);
-               cache = *cache_ptr;
-               key.sig = invoke_sig;
-               key.method = target_method;
-               res = g_hash_table_lookup (cache, &key);
-               mono_marshal_unlock ();
-               if (res)
-                       return res;
-       } else {
-               cache = get_cache (&method->klass->image->delegate_invoke_cache,
-                                                  (GHashFunc)mono_signature_hash, 
-                                                  (GCompareFunc)mono_metadata_signature_equal);
-               res = mono_marshal_find_in_cache (cache, sig);
-               if (res)
-                       return res;
-               cache_key = sig;
-       }
-
-       static_sig = signature_dup (method->klass->image, sig);
-       static_sig->hasthis = 0;
-       if (!static_method_with_first_arg_bound)
-               invoke_sig = static_sig;
-
-       if (static_method_with_first_arg_bound)
-               name = mono_signature_to_name (invoke_sig, "invoke_bound");
-       else if (closed_over_null)
-               name = mono_signature_to_name (invoke_sig, "invoke_closed_over_null");
-       else if (callvirt)
-               name = mono_signature_to_name (invoke_sig, "invoke_callvirt");
-       else
-               name = mono_signature_to_name (invoke_sig, "invoke");
-       if (ctx)
-               mb = mono_mb_new (method->klass, name, MONO_WRAPPER_DELEGATE_INVOKE);
-       else
-               mb = mono_mb_new (get_wrapper_target_class (method->klass->image), name, MONO_WRAPPER_DELEGATE_INVOKE);
-       g_free (name);
-
-#ifndef DISABLE_JIT
-       /* allocate local 0 (object) */
-       local_target = mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
-       local_prev = mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
-
-       g_assert (sig->hasthis);
-       
-       /*
-        * if (prev != null)
-         *     prev.Invoke( args .. );
-        * return this.<target>( args .. );
-         */
-       
-       /* this wrapper can be used in unmanaged-managed transitions */
-       emit_thread_interrupt_checkpoint (mb);
-       
-       /* get this->prev */
-       mono_mb_emit_ldarg (mb, 0);
-       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoMulticastDelegate, prev));
-       mono_mb_emit_byte (mb, CEE_LDIND_REF);
-       mono_mb_emit_stloc (mb, local_prev);
-       mono_mb_emit_ldloc (mb, local_prev);
-
-       /* if prev != null */
-       pos0 = mono_mb_emit_branch (mb, CEE_BRFALSE);
-
-       /* then recurse */
-
-       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
-       mono_mb_emit_byte (mb, CEE_MONO_NOT_TAKEN);
-
-       mono_mb_emit_ldloc (mb, local_prev);
-       for (i = 0; i < sig->param_count; i++)
-               mono_mb_emit_ldarg (mb, i + 1);
-       if (ctx)
-               mono_mb_emit_op (mb, CEE_CALLVIRT, mono_class_inflate_generic_method (method, &container->context));
-       else
-               mono_mb_emit_op (mb, CEE_CALLVIRT, method);
-       if (sig->ret->type != MONO_TYPE_VOID)
-               mono_mb_emit_byte (mb, CEE_POP);
-
-       /* continued or prev == null */
-       mono_mb_patch_branch (mb, pos0);
-
-       /* get this->target */
-       mono_mb_emit_ldarg (mb, 0);
-       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoDelegate, target));
-       mono_mb_emit_byte (mb, CEE_LDIND_REF);
-       mono_mb_emit_stloc (mb, local_target);
-
-       /*static methods with bound first arg can have null target and still be bound*/
-       if (!static_method_with_first_arg_bound) {
-               /* if target != null */
-               mono_mb_emit_ldloc (mb, local_target);
-               pos0 = mono_mb_emit_branch (mb, CEE_BRFALSE);
-
-               /* then call this->method_ptr nonstatic */
-               if (callvirt) {
-                       // FIXME:
-                       mono_mb_emit_exception_full (mb, "System", "NotImplementedException", "");
-               } else {
-                       mono_mb_emit_ldloc (mb, local_target);
-                       for (i = 0; i < sig->param_count; ++i)
-                               mono_mb_emit_ldarg (mb, i + 1);
-                       mono_mb_emit_ldarg (mb, 0);
-                       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoDelegate, method_ptr));
-                       mono_mb_emit_byte (mb, CEE_LDIND_I );
-                       mono_mb_emit_op (mb, CEE_CALLI, sig);
-
-                       mono_mb_emit_byte (mb, CEE_RET);
-               }
-       
-               /* else [target == null] call this->method_ptr static */
-               mono_mb_patch_branch (mb, pos0);
-       }
-
-       if (callvirt) {
-               if (!closed_over_null) {
-                       mono_mb_emit_ldarg (mb, 1);
-                       mono_mb_emit_op (mb, CEE_CASTCLASS, target_class);
-                       for (i = 1; i < sig->param_count; ++i)
-                               mono_mb_emit_ldarg (mb, i + 1);
-                       mono_mb_emit_op (mb, CEE_CALLVIRT, target_method);
-               } else {
-                       mono_mb_emit_byte (mb, CEE_LDNULL);
-                       for (i = 0; i < sig->param_count; ++i)
-                               mono_mb_emit_ldarg (mb, i + 1);
-                       mono_mb_emit_op (mb, CEE_CALL, target_method);
-               }
-       } else {
-               if (static_method_with_first_arg_bound) {
-                       mono_mb_emit_ldloc (mb, local_target);
-                       if (!MONO_TYPE_IS_REFERENCE (invoke_sig->params[0]))
-                               mono_mb_emit_op (mb, CEE_UNBOX_ANY, mono_class_from_mono_type (invoke_sig->params[0]));
-               }
-               for (i = 0; i < sig->param_count; ++i)
-                       mono_mb_emit_ldarg (mb, i + 1);
-               mono_mb_emit_ldarg (mb, 0);
-               mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoDelegate, method_ptr));
-               mono_mb_emit_byte (mb, CEE_LDIND_I );
-               mono_mb_emit_op (mb, CEE_CALLI, invoke_sig);
-       }
-
-       mono_mb_emit_byte (mb, CEE_RET);
-
-       mb->skip_visibility = 1;
-#endif /* DISABLE_JIT */
-
-       if (ctx) {
-               MonoMethod *def;
-
-               def = mono_mb_create_and_cache (cache, cache_key, mb, sig, sig->param_count + 16);
-               res = cache_generic_delegate_wrapper (cache, orig_method, def, ctx);
-       } else if (callvirt) {
-               new_key = g_new0 (SignatureMethodPair, 1);
-               *new_key = key;
-
-               info = mono_wrapper_info_create (mb, subtype);
-
-               res = mono_mb_create_and_cache_full (cache, new_key, mb, sig, sig->param_count + 16, info, &found);
-               if (found)
-                       g_free (new_key);
-       } else {
-               info = mono_wrapper_info_create (mb, subtype);
-
-               res = mono_mb_create_and_cache_full (cache, cache_key, mb, sig, sig->param_count + 16, info, NULL);
-       }
-       mono_mb_free (mb);
-
-       return res;     
-}
-
-/*
- * the returned method invokes all methods in a multicast delegate.
- */
-MonoMethod *
-mono_marshal_get_delegate_invoke (MonoMethod *method, MonoDelegate *del)
-{
-       gboolean callvirt = FALSE;
-       gboolean static_method_with_first_arg_bound = FALSE;
-       MonoMethod *target_method = NULL;
-       MonoMethodSignature *sig;
-
-       sig = mono_signature_no_pinvoke (method);
-
-       if (del && !del->target && del->method && mono_method_signature (del->method)->hasthis) {
-               callvirt = TRUE;
-               target_method = del->method;
-       }
-
-       if (del && del->method && mono_method_signature (del->method)->param_count == sig->param_count + 1 && (del->method->flags & METHOD_ATTRIBUTE_STATIC)) {
-               static_method_with_first_arg_bound = TRUE;
-               target_method = del->method;
-       }
-
-       return mono_marshal_get_delegate_invoke_internal (method, callvirt, static_method_with_first_arg_bound, target_method);
-}
-
-/*
- * signature_dup_add_this:
- *
- *  Make a copy of @sig, adding an explicit this argument.
- */
-static MonoMethodSignature*
-signature_dup_add_this (MonoImage *image, MonoMethodSignature *sig, MonoClass *klass)
-{
-       MonoMethodSignature *res;
-       int i;
-
-       res = mono_metadata_signature_alloc (image, sig->param_count + 1);
-       memcpy (res, sig, MONO_SIZEOF_METHOD_SIGNATURE);
-       res->param_count = sig->param_count + 1;
-       res->hasthis = FALSE;
-       for (i = sig->param_count - 1; i >= 0; i --)
-               res->params [i + 1] = sig->params [i];
-       res->params [0] = klass->valuetype ? &klass->this_arg : &klass->byval_arg;
-
-       return res;
-}
-
-typedef struct {
-       MonoMethodSignature *ctor_sig;
-       MonoMethodSignature *sig;
-} CtorSigPair;
-
-/* protected by the marshal lock, contains CtorSigPair pointers */
-static GSList *strsig_list = NULL;
-
-static MonoMethodSignature *
-lookup_string_ctor_signature (MonoMethodSignature *sig)
-{
-       MonoMethodSignature *callsig;
-       CtorSigPair *cs;
-       GSList *item;
-
-       mono_marshal_lock ();
-       callsig = NULL;
-       for (item = strsig_list; item; item = item->next) {
-               cs = item->data;
-               /* mono_metadata_signature_equal () is safe to call with the marshal lock
-                * because it is lock-free.
-                */
-               if (mono_metadata_signature_equal (sig, cs->ctor_sig)) {
-                       callsig = cs->sig;
-                       break;
-               }
-       }
-       mono_marshal_unlock ();
-       return callsig;
-}
-
-static MonoMethodSignature *
-add_string_ctor_signature (MonoMethod *method)
-{
-       MonoMethodSignature *callsig;
-       CtorSigPair *cs;
-
-       callsig = signature_dup (method->klass->image, mono_method_signature (method));
-       callsig->ret = &mono_defaults.string_class->byval_arg;
-       cs = g_new (CtorSigPair, 1);
-       cs->sig = callsig;
-       cs->ctor_sig = mono_method_signature (method);
-
-       mono_marshal_lock ();
-       strsig_list = g_slist_prepend (strsig_list, cs);
-       mono_marshal_unlock ();
-       return callsig;
-}
-
-/*
- * mono_marshal_get_string_ctor_signature:
- *
- *   Return the modified signature used by string ctors (they return the newly created
- * string).
- */
-MonoMethodSignature*
-mono_marshal_get_string_ctor_signature (MonoMethod *method)
-{
-       MonoMethodSignature *sig = lookup_string_ctor_signature (mono_method_signature (method));
-       if (!sig)
-               sig = add_string_ctor_signature (method);
-
-       return sig;
-}
-
-static MonoType*
-get_runtime_invoke_type (MonoType *t, gboolean ret)
-{
-       if (t->byref) {
-               if (t->type == MONO_TYPE_GENERICINST && mono_class_is_nullable (mono_class_from_mono_type (t)))
-                       return t;
-               /* Can't share this with 'I' as that needs another indirection */
-               return &mono_defaults.int_class->this_arg;
-       }
-
-       if (MONO_TYPE_IS_REFERENCE (t))
-               return &mono_defaults.object_class->byval_arg;
-
-       if (ret)
-               /* The result needs to be boxed */
-               return t;
-
-handle_enum:
-       switch (t->type) {
-               /* Can't share these as the argument needs to be loaded using sign/zero extension */
-               /*
-       case MONO_TYPE_U1:
-               return &mono_defaults.sbyte_class->byval_arg;
-       case MONO_TYPE_U2:
-               return &mono_defaults.int16_class->byval_arg;
-       case MONO_TYPE_U4:
-               return &mono_defaults.int32_class->byval_arg;
-               */
-       case MONO_TYPE_U8:
-               return &mono_defaults.int64_class->byval_arg;
-       case MONO_TYPE_BOOLEAN:
-               return &mono_defaults.byte_class->byval_arg;
-       case MONO_TYPE_CHAR:
-               return &mono_defaults.uint16_class->byval_arg;
-       case MONO_TYPE_U:
-       case MONO_TYPE_PTR:
-               return &mono_defaults.int_class->byval_arg;
-       case MONO_TYPE_VALUETYPE:
-               if (t->data.klass->enumtype) {
-                       t = mono_class_enum_basetype (t->data.klass);
-                       goto handle_enum;
-               }
-               return t;
-       default:
-               return t;
-       }
-}
-
-/*
- * mono_marshal_get_runtime_invoke_sig:
- *
- *   Return a common signature used for sharing runtime invoke wrappers.
- */
-static MonoMethodSignature*
-mono_marshal_get_runtime_invoke_sig (MonoMethodSignature *sig)
-{
-       MonoMethodSignature *res = mono_metadata_signature_dup (sig);
-       int i;
-
-       res->generic_param_count = 0;
-       res->ret = get_runtime_invoke_type (sig->ret, TRUE);
-       for (i = 0; i < res->param_count; ++i)
-               res->params [i] = get_runtime_invoke_type (sig->params [i], FALSE);
-
-       return res;
-}
-
-static gboolean
-runtime_invoke_signature_equal (MonoMethodSignature *sig1, MonoMethodSignature *sig2)
-{
-       /* Can't share wrappers which return a vtype since it needs to be boxed */
-       if (sig1->ret != sig2->ret && !(MONO_TYPE_IS_REFERENCE (sig1->ret) && MONO_TYPE_IS_REFERENCE (sig2->ret)) && !mono_metadata_type_equal (sig1->ret, sig2->ret))
-               return FALSE;
-       else
-               return mono_metadata_signature_equal (sig1, sig2);
-}
-
-#ifndef DISABLE_JIT
-
-/*
- * emit_invoke_call:
- *
- *   Emit the call to the wrapper method from a runtime invoke wrapper.
- */
-static void
-emit_invoke_call (MonoMethodBuilder *mb, MonoMethod *method,
-                                 MonoMethodSignature *sig, MonoMethodSignature *callsig,
-                                 int loc_res,
-                                 gboolean virtual, gboolean need_direct_wrapper)
-{
-       static MonoString *string_dummy = NULL;
-       int i;
-       int *tmp_nullable_locals;
-       gboolean void_ret = FALSE;
-
-       /* to make it work with our special string constructors */
-       if (!string_dummy) {
-               MONO_GC_REGISTER_ROOT_SINGLE (string_dummy);
-               string_dummy = mono_string_new_wrapper ("dummy");
-       }
-
-       if (virtual) {
-               g_assert (sig->hasthis);
-               g_assert (method->flags & METHOD_ATTRIBUTE_VIRTUAL);
-       }
-
-       if (sig->hasthis) {
-               if (method->string_ctor) {
-                       if (mono_gc_is_moving ()) {
-                               mono_mb_emit_ptr (mb, &string_dummy);
-                               mono_mb_emit_byte (mb, CEE_LDIND_REF);
-                       } else {
-                               mono_mb_emit_ptr (mb, string_dummy);
-                       }
-               } else {
-                       mono_mb_emit_ldarg (mb, 0);
-               }
-       }
-
-       tmp_nullable_locals = g_new0 (int, sig->param_count);
-
-       for (i = 0; i < sig->param_count; i++) {
-               MonoType *t = sig->params [i];
-               int type;
-
-               mono_mb_emit_ldarg (mb, 1);
-               if (i) {
-                       mono_mb_emit_icon (mb, sizeof (gpointer) * i);
-                       mono_mb_emit_byte (mb, CEE_ADD);
-               }
-
-               if (t->byref) {
-                       mono_mb_emit_byte (mb, CEE_LDIND_I);
-                       /* A Nullable<T> type don't have a boxed form, it's either null or a boxed T.
-                        * So to make this work we unbox it to a local variablee and push a reference to that.
-                        */
-                       if (t->type == MONO_TYPE_GENERICINST && mono_class_is_nullable (mono_class_from_mono_type (t))) {
-                               tmp_nullable_locals [i] = mono_mb_add_local (mb, &mono_class_from_mono_type (t)->byval_arg);
-
-                               mono_mb_emit_op (mb, CEE_UNBOX_ANY, mono_class_from_mono_type (t));
-                               mono_mb_emit_stloc (mb, tmp_nullable_locals [i]);
-                               mono_mb_emit_ldloc_addr (mb, tmp_nullable_locals [i]);
-                       }
-                       continue;
-               }
-
-               /*FIXME 'this doesn't handle generic enums. Shouldn't we?*/
-               type = sig->params [i]->type;
-handle_enum:
-               switch (type) {
-               case MONO_TYPE_I1:
-               case MONO_TYPE_BOOLEAN:
-               case MONO_TYPE_U1:
-               case MONO_TYPE_I2:
-               case MONO_TYPE_U2:
-               case MONO_TYPE_CHAR:
-               case MONO_TYPE_I:
-               case MONO_TYPE_U:
-               case MONO_TYPE_I4:
-               case MONO_TYPE_U4:
-               case MONO_TYPE_R4:
-               case MONO_TYPE_R8:
-               case MONO_TYPE_I8:
-               case MONO_TYPE_U8:
-                       mono_mb_emit_byte (mb, CEE_LDIND_I);
-                       mono_mb_emit_byte (mb, mono_type_to_ldind (sig->params [i]));
-                       break;
-               case MONO_TYPE_STRING:
-               case MONO_TYPE_CLASS:  
-               case MONO_TYPE_ARRAY:
-               case MONO_TYPE_PTR:
-               case MONO_TYPE_SZARRAY:
-               case MONO_TYPE_OBJECT:
-                       mono_mb_emit_byte (mb, mono_type_to_ldind (sig->params [i]));
-                       break;
-               case MONO_TYPE_GENERICINST:
-                       if (!mono_type_generic_inst_is_valuetype (sig->params [i])) {
-                               mono_mb_emit_byte (mb, mono_type_to_ldind (sig->params [i]));
-                               break;
-                       }
-
-                       /* fall through */
-               case MONO_TYPE_VALUETYPE:
-                       if (type == MONO_TYPE_VALUETYPE && t->data.klass->enumtype) {
-                               type = mono_class_enum_basetype (t->data.klass)->type;
-                               goto handle_enum;
-                       }
-                       mono_mb_emit_byte (mb, CEE_LDIND_I);
-                       if (mono_class_is_nullable (mono_class_from_mono_type (sig->params [i]))) {
-                               /* Need to convert a boxed vtype to an mp to a Nullable struct */
-                               mono_mb_emit_op (mb, CEE_UNBOX, mono_class_from_mono_type (sig->params [i]));
-                               mono_mb_emit_op (mb, CEE_LDOBJ, mono_class_from_mono_type (sig->params [i]));
-                       } else {
-                               mono_mb_emit_op (mb, CEE_LDOBJ, mono_class_from_mono_type (sig->params [i]));
-                       }
-                       break;
-               default:
-                       g_assert_not_reached ();
-               }
-       }
-       
-       if (virtual) {
-               mono_mb_emit_op (mb, CEE_CALLVIRT, method);
-       } else if (need_direct_wrapper) {
-               mono_mb_emit_op (mb, CEE_CALL, method);
-       } else {
-               mono_mb_emit_ldarg (mb, 3);
-               mono_mb_emit_calli (mb, callsig);
-       }
-
-       if (sig->ret->byref) {
-               /* fixme: */
-               g_assert_not_reached ();
-       }
-
-       switch (sig->ret->type) {
-       case MONO_TYPE_VOID:
-               if (!method->string_ctor)
-                       void_ret = TRUE;
-               break;
-       case MONO_TYPE_BOOLEAN:
-       case MONO_TYPE_CHAR:
-       case MONO_TYPE_I1:
-       case MONO_TYPE_U1:
-       case MONO_TYPE_I2:
-       case MONO_TYPE_U2:
-       case MONO_TYPE_I4:
-       case MONO_TYPE_U4:
-       case MONO_TYPE_I:
-       case MONO_TYPE_U:
-       case MONO_TYPE_R4:
-       case MONO_TYPE_R8:
-       case MONO_TYPE_I8:
-       case MONO_TYPE_U8:
-       case MONO_TYPE_VALUETYPE:
-       case MONO_TYPE_TYPEDBYREF:
-       case MONO_TYPE_GENERICINST:
-               /* box value types */
-               mono_mb_emit_op (mb, CEE_BOX, mono_class_from_mono_type (sig->ret));
-               break;
-       case MONO_TYPE_STRING:
-       case MONO_TYPE_CLASS:  
-       case MONO_TYPE_ARRAY:
-       case MONO_TYPE_SZARRAY:
-       case MONO_TYPE_OBJECT:
-               /* nothing to do */
-               break;
-       case MONO_TYPE_PTR:
-               /* The result is an IntPtr */
-               mono_mb_emit_op (mb, CEE_BOX, mono_defaults.int_class);
-               break;
-       default:
-               g_assert_not_reached ();
-       }
-
-       if (!void_ret)
-               mono_mb_emit_stloc (mb, loc_res);
-
-       /* Convert back nullable-byref arguments */
-       for (i = 0; i < sig->param_count; i++) {
-               MonoType *t = sig->params [i];
-
-               /* 
-                * Box the result and put it back into the array, the caller will have
-                * to obtain it from there.
-                */
-               if (t->byref && t->type == MONO_TYPE_GENERICINST && mono_class_is_nullable (mono_class_from_mono_type (t))) {
-                       mono_mb_emit_ldarg (mb, 1);                     
-                       mono_mb_emit_icon (mb, sizeof (gpointer) * i);
-                       mono_mb_emit_byte (mb, CEE_ADD);
-
-                       mono_mb_emit_ldloc (mb, tmp_nullable_locals [i]);
-                       mono_mb_emit_op (mb, CEE_BOX, mono_class_from_mono_type (t));
-
-                       mono_mb_emit_byte (mb, CEE_STIND_REF);
-               }
-       }
-
-       g_free (tmp_nullable_locals);
-}
-
-static void
-emit_runtime_invoke_body (MonoMethodBuilder *mb, MonoClass *target_klass, MonoMethod *method,
-                                                 MonoMethodSignature *sig, MonoMethodSignature *callsig,
-                                                 gboolean virtual, gboolean need_direct_wrapper)
-{
-       gint32 labels [16];
-       MonoExceptionClause *clause;
-       int loc_res, loc_exc;
-
-       /* The wrapper looks like this:
-        *
-        * <interrupt check>
-        * if (exc) {
-        *       try {
-        *         return <call>
-        *       } catch (Exception e) {
-        *     *exc = e;
-        *   }
-        * } else {
-        *     return <call>
-        * }
-        */
-
-       /* allocate local 0 (object) tmp */
-       loc_res = mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
-       /* allocate local 1 (object) exc */
-       loc_exc = mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
-
-       /* *exc is assumed to be initialized to NULL by the caller */
-
-       mono_mb_emit_byte (mb, CEE_LDARG_2);
-       labels [0] = mono_mb_emit_branch (mb, CEE_BRFALSE);
-
-       /*
-        * if (exc) case
-        */
-       labels [1] = mono_mb_get_label (mb);
-       emit_thread_force_interrupt_checkpoint (mb);
-       emit_invoke_call (mb, method, sig, callsig, loc_res, virtual, need_direct_wrapper);
-
-       labels [2] = mono_mb_emit_branch (mb, CEE_LEAVE);
-
-       /* Add a try clause around the call */
-       clause = mono_image_alloc0 (target_klass->image, sizeof (MonoExceptionClause));
-       clause->flags = MONO_EXCEPTION_CLAUSE_NONE;
-       clause->data.catch_class = mono_defaults.exception_class;
-       clause->try_offset = labels [1];
-       clause->try_len = mono_mb_get_label (mb) - labels [1];
-
-       clause->handler_offset = mono_mb_get_label (mb);
-
-       /* handler code */
-       mono_mb_emit_stloc (mb, loc_exc);       
-       mono_mb_emit_byte (mb, CEE_LDARG_2);
-       mono_mb_emit_ldloc (mb, loc_exc);
-       mono_mb_emit_byte (mb, CEE_STIND_REF);
-
-       mono_mb_emit_branch (mb, CEE_LEAVE);
-
-       clause->handler_len = mono_mb_get_pos (mb) - clause->handler_offset;
-
-       mono_mb_set_clauses (mb, 1, clause);
-
-       mono_mb_patch_branch (mb, labels [2]);
-       mono_mb_emit_ldloc (mb, loc_res);
-       mono_mb_emit_byte (mb, CEE_RET);
-
-       /*
-        * if (!exc) case
-        */
-       mono_mb_patch_branch (mb, labels [0]);
-       emit_thread_force_interrupt_checkpoint (mb);
-       emit_invoke_call (mb, method, sig, callsig, loc_res, virtual, need_direct_wrapper);
-
-       mono_mb_emit_ldloc (mb, 0);
-       mono_mb_emit_byte (mb, CEE_RET);
-}
-#endif
-
-/*
- * generates IL code for the runtime invoke function 
- * MonoObject *runtime_invoke (MonoObject *this, void **params, MonoObject **exc, void* method)
- *
- * we also catch exceptions if exc != null
- * If VIRTUAL is TRUE, then METHOD is invoked virtually on THIS. This is useful since
- * it means that the compiled code for METHOD does not have to be looked up 
- * before calling the runtime invoke wrapper. In this case, the wrapper ignores
- * its METHOD argument.
- */
-MonoMethod *
-mono_marshal_get_runtime_invoke (MonoMethod *method, gboolean virtual)
-{
-       MonoMethodSignature *sig, *csig, *callsig;
-       MonoMethodBuilder *mb;
-       GHashTable *cache = NULL;
-       MonoClass *target_klass;
-       MonoMethod *res = NULL;
-       static MonoMethodSignature *cctor_signature = NULL;
-       static MonoMethodSignature *finalize_signature = NULL;
-       char *name;
-       const char *param_names [16];
-       gboolean need_direct_wrapper = FALSE;
-       WrapperInfo *info;
-
-       g_assert (method);
-
-       if (!cctor_signature) {
-               cctor_signature = mono_metadata_signature_alloc (mono_defaults.corlib, 0);
-               cctor_signature->ret = &mono_defaults.void_class->byval_arg;
-       }
-       if (!finalize_signature) {
-               finalize_signature = mono_metadata_signature_alloc (mono_defaults.corlib, 0);
-               finalize_signature->ret = &mono_defaults.void_class->byval_arg;
-               finalize_signature->hasthis = 1;
-       }
-
-       if (virtual)
-               need_direct_wrapper = TRUE;
-
-       /* 
-        * Use a separate cache indexed by methods to speed things up and to avoid the
-        * boundless mempool growth caused by the signature_dup stuff below.
-        */
-       if (virtual)
-               cache = get_cache (&method->klass->image->runtime_invoke_vcall_cache, mono_aligned_addr_hash, NULL);
-       else
-               cache = get_cache (&method->klass->image->runtime_invoke_direct_cache, mono_aligned_addr_hash, NULL);
-       res = mono_marshal_find_in_cache (cache, method);
-       if (res)
-               return res;
-               
-       if (method->klass->rank && (method->iflags & METHOD_IMPL_ATTRIBUTE_INTERNAL_CALL) &&
-               (method->iflags & METHOD_IMPL_ATTRIBUTE_NATIVE)) {
-               /* 
-                * Array Get/Set/Address methods. The JIT implements them using inline code
-                * so we need to create an invoke wrapper which calls the method directly.
-                */
-               need_direct_wrapper = TRUE;
-       }
-               
-       if (method->string_ctor) {
-               callsig = lookup_string_ctor_signature (mono_method_signature (method));
-               if (!callsig)
-                       callsig = add_string_ctor_signature (method);
-               /* Can't share this as we push a string as this */
-               need_direct_wrapper = TRUE;
-       } else {
-               if (method_is_dynamic (method))
-                       callsig = signature_dup (method->klass->image, mono_method_signature (method));
-               else
-                       callsig = mono_method_signature (method);
-       }
-
-       target_klass = get_wrapper_target_class (method->klass->image);
-
-       /* Try to share wrappers for non-corlib methods with simple signatures */
-       if (mono_metadata_signature_equal (callsig, cctor_signature)) {
-               callsig = cctor_signature;
-               target_klass = mono_defaults.object_class;
-       } else if (mono_metadata_signature_equal (callsig, finalize_signature)) {
-               callsig = finalize_signature;
-               target_klass = mono_defaults.object_class;
-       }
-
-       if (need_direct_wrapper) {
-               /* Already searched at the start */
-       } else {
-               MonoMethodSignature *tmp_sig;
-
-               callsig = mono_marshal_get_runtime_invoke_sig (callsig);
-
-               if (method->klass->valuetype && mono_method_signature (method)->hasthis)
-                       /* These have a different csig */
-                       cache = get_cache (&target_klass->image->runtime_invoke_vtype_cache,
-                                                          (GHashFunc)mono_signature_hash,
-                                                          (GCompareFunc)runtime_invoke_signature_equal);
-               else
-                       cache = get_cache (&target_klass->image->runtime_invoke_cache,
-                                                          (GHashFunc)mono_signature_hash,
-                                                          (GCompareFunc)runtime_invoke_signature_equal);
-
-               /* from mono_marshal_find_in_cache */
-               mono_marshal_lock ();
-               res = g_hash_table_lookup (cache, callsig);
-               mono_marshal_unlock ();
-
-               if (res) {
-                       g_free (callsig);
-                       return res;
-               }
-
-               /* Make a copy of the signature from the image mempool */
-               tmp_sig = callsig;
-               callsig = mono_metadata_signature_dup_full (target_klass->image, callsig);
-               g_free (tmp_sig);
-       }
-       
-       sig = mono_method_signature (method);
-
-       csig = mono_metadata_signature_alloc (target_klass->image, 4);
-
-       csig->ret = &mono_defaults.object_class->byval_arg;
-       if (method->klass->valuetype && mono_method_signature (method)->hasthis)
-               csig->params [0] = get_runtime_invoke_type (&method->klass->this_arg, FALSE);
-       else
-               csig->params [0] = &mono_defaults.object_class->byval_arg;
-       csig->params [1] = &mono_defaults.int_class->byval_arg;
-       csig->params [2] = &mono_defaults.int_class->byval_arg;
-       csig->params [3] = &mono_defaults.int_class->byval_arg;
-       csig->pinvoke = 1;
-#if TARGET_WIN32
-       /* This is called from runtime code so it has to be cdecl */
-       csig->call_convention = MONO_CALL_C;
-#endif
-
-       name = mono_signature_to_name (callsig, virtual ? "runtime_invoke_virtual" : "runtime_invoke");
-       mb = mono_mb_new (target_klass, name,  MONO_WRAPPER_RUNTIME_INVOKE);
-       g_free (name);
-
-#ifndef DISABLE_JIT
-       param_names [0] = "this";
-       param_names [1] = "params";
-       param_names [2] = "exc";
-       param_names [3] = "method";
-       mono_mb_set_param_names (mb, param_names);
-
-       emit_runtime_invoke_body (mb, target_klass, method, sig, callsig, virtual, need_direct_wrapper);
-#endif
-
-       if (need_direct_wrapper) {
-#ifndef DISABLE_JIT
-               mb->skip_visibility = 1;
-#endif
-               info = mono_wrapper_info_create (mb, virtual ? WRAPPER_SUBTYPE_RUNTIME_INVOKE_VIRTUAL : WRAPPER_SUBTYPE_RUNTIME_INVOKE_DIRECT);
-               info->d.runtime_invoke.method = method;
-               res = mono_mb_create_and_cache_full (cache, method, mb, csig, sig->param_count + 16, info, NULL);
-       } else {
-               /* taken from mono_mb_create_and_cache */
-               mono_marshal_lock ();
-               res = g_hash_table_lookup (cache, callsig);
-               mono_marshal_unlock ();
-
-               info = mono_wrapper_info_create (mb, WRAPPER_SUBTYPE_RUNTIME_INVOKE_NORMAL);
-               info->d.runtime_invoke.sig = callsig;
-
-               /* Somebody may have created it before us */
-               if (!res) {
-                       MonoMethod *newm;
-                       newm = mono_mb_create (mb, csig, sig->param_count + 16, info);
-
-                       mono_marshal_lock ();
-                       res = g_hash_table_lookup (cache, callsig);
-                       if (!res) {
-                               res = newm;
-                               g_hash_table_insert (cache, callsig, res);
-                               /* Can't insert it into wrapper_hash since the key is a signature */
-                               g_hash_table_insert (method->klass->image->runtime_invoke_direct_cache, method, res);
-                       } else {
-                               mono_free_method (newm);
-                       }
-                       mono_marshal_unlock ();
-               }
-
-               /* end mono_mb_create_and_cache */
-       }
-
-       mono_mb_free (mb);
-
-       return res;     
-}
-
-/*
- * mono_marshal_get_runtime_invoke_dynamic:
- *
- *   Return a method which can be used to invoke managed methods from native code
- * dynamically.
- * The signature of the returned method is given by RuntimeInvokeDynamicFunction:
- * void runtime_invoke (void *args, MonoObject **exc, void *compiled_method)
- * ARGS should point to an architecture specific structure containing 
- * the arguments and space for the return value.
- * The other arguments are the same as for runtime_invoke (), except that
- * ARGS should contain the this argument too.
- * This wrapper serves the same purpose as the runtime-invoke wrappers, but there
- * is only one copy of it, which is useful in full-aot.
- * The wrapper info for the wrapper is a WrapperInfo structure.
- */
-MonoMethod*
-mono_marshal_get_runtime_invoke_dynamic (void)
-{
-       static MonoMethod *method;
-       MonoMethodSignature *csig;
-       MonoExceptionClause *clause;
-       MonoMethodBuilder *mb;
-       int pos, posna;
-       char *name;
-       WrapperInfo *info;
-
-       if (method)
-               return method;
-
-       csig = mono_metadata_signature_alloc (mono_defaults.corlib, 4);
-
-       csig->ret = &mono_defaults.void_class->byval_arg;
-       csig->params [0] = &mono_defaults.int_class->byval_arg;
-       csig->params [1] = &mono_defaults.int_class->byval_arg;
-       csig->params [2] = &mono_defaults.int_class->byval_arg;
-       csig->params [3] = &mono_defaults.int_class->byval_arg;
-
-       name = g_strdup ("runtime_invoke_dynamic");
-       mb = mono_mb_new (mono_defaults.object_class, name, MONO_WRAPPER_RUNTIME_INVOKE);
-       g_free (name);
-
-#ifndef DISABLE_JIT
-       /* allocate local 0 (object) tmp */
-       mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
-       /* allocate local 1 (object) exc */
-       mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
-
-       /* cond set *exc to null */
-       mono_mb_emit_byte (mb, CEE_LDARG_1);
-       mono_mb_emit_byte (mb, CEE_BRFALSE_S);
-       mono_mb_emit_byte (mb, 3);      
-       mono_mb_emit_byte (mb, CEE_LDARG_1);
-       mono_mb_emit_byte (mb, CEE_LDNULL);
-       mono_mb_emit_byte (mb, CEE_STIND_REF);
-
-       emit_thread_force_interrupt_checkpoint (mb);
-
-       mono_mb_emit_byte (mb, CEE_LDARG_0);
-       mono_mb_emit_byte (mb, CEE_LDARG_2);
-       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
-       mono_mb_emit_byte (mb, CEE_MONO_DYN_CALL);
-
-       pos = mono_mb_emit_branch (mb, CEE_LEAVE);
-
-       clause = mono_image_alloc0 (mono_defaults.corlib, sizeof (MonoExceptionClause));
-       clause->flags = MONO_EXCEPTION_CLAUSE_FILTER;
-       clause->try_len = mono_mb_get_label (mb);
-
-       /* filter code */
-       clause->data.filter_offset = mono_mb_get_label (mb);
-       
-       mono_mb_emit_byte (mb, CEE_POP);
-       mono_mb_emit_byte (mb, CEE_LDARG_1);
-       mono_mb_emit_byte (mb, CEE_LDC_I4_0);
-       mono_mb_emit_byte (mb, CEE_PREFIX1);
-       mono_mb_emit_byte (mb, CEE_CGT_UN);
-       mono_mb_emit_byte (mb, CEE_PREFIX1);
-       mono_mb_emit_byte (mb, CEE_ENDFILTER);
-
-       clause->handler_offset = mono_mb_get_label (mb);
-
-       /* handler code */
-       /* store exception */
-       mono_mb_emit_stloc (mb, 1);
-       
-       mono_mb_emit_byte (mb, CEE_LDARG_1);
-       mono_mb_emit_ldloc (mb, 1);
-       mono_mb_emit_byte (mb, CEE_STIND_REF);
-
-       mono_mb_emit_byte (mb, CEE_LDNULL);
-       mono_mb_emit_stloc (mb, 0);
-
-       /* Check for the abort exception */
-       mono_mb_emit_ldloc (mb, 1);
-       mono_mb_emit_op (mb, CEE_ISINST, mono_defaults.threadabortexception_class);
-       posna = mono_mb_emit_short_branch (mb, CEE_BRFALSE_S);
-
-       /* Delay the abort exception */
-       mono_mb_emit_icall (mb, ves_icall_System_Threading_Thread_ResetAbort);
-
-       mono_mb_patch_short_branch (mb, posna);
-       mono_mb_emit_branch (mb, CEE_LEAVE);
-
-       clause->handler_len = mono_mb_get_pos (mb) - clause->handler_offset;
-
-       mono_mb_set_clauses (mb, 1, clause);
-
-       /* return result */
-       mono_mb_patch_branch (mb, pos);
-       //mono_mb_emit_ldloc (mb, 0);
-       mono_mb_emit_byte (mb, CEE_RET);
-#endif /* DISABLE_JIT */
-
-       info = mono_wrapper_info_create (mb, WRAPPER_SUBTYPE_RUNTIME_INVOKE_DYNAMIC);
-
-       mono_marshal_lock ();
-       /* double-checked locking */
-       if (!method)
-               method = mono_mb_create (mb, csig, 16, info);
-
-       mono_marshal_unlock ();
-
-       mono_mb_free (mb);
-
-       return method;
-}
-
-#ifndef DISABLE_JIT
-static void
-mono_mb_emit_auto_layout_exception (MonoMethodBuilder *mb, MonoClass *klass)
-{
-       char *msg = g_strdup_printf ("The type `%s.%s' layout needs to be Sequential or Explicit",
-                                    klass->name_space, klass->name);
+       if (ret)
+               /* The result needs to be boxed */
+               return t;
 
-       mono_mb_emit_exception_marshal_directive (mb, msg);
+handle_enum:
+       switch (t->type) {
+               /* Can't share these as the argument needs to be loaded using sign/zero extension */
+               /*
+       case MONO_TYPE_U1:
+               return &mono_defaults.sbyte_class->byval_arg;
+       case MONO_TYPE_U2:
+               return &mono_defaults.int16_class->byval_arg;
+       case MONO_TYPE_U4:
+               return &mono_defaults.int32_class->byval_arg;
+               */
+       case MONO_TYPE_U8:
+               return &mono_defaults.int64_class->byval_arg;
+       case MONO_TYPE_BOOLEAN:
+               return &mono_defaults.byte_class->byval_arg;
+       case MONO_TYPE_CHAR:
+               return &mono_defaults.uint16_class->byval_arg;
+       case MONO_TYPE_U:
+       case MONO_TYPE_PTR:
+               return &mono_defaults.int_class->byval_arg;
+       case MONO_TYPE_VALUETYPE:
+               if (t->data.klass->enumtype) {
+                       t = mono_class_enum_basetype (t->data.klass);
+                       goto handle_enum;
+               }
+               return t;
+       default:
+               return t;
+       }
 }
-#endif
 
-#ifndef DISABLE_REMOTING
 /*
- * mono_marshal_get_ldfld_remote_wrapper:
- * @klass: The return type
+ * mono_marshal_get_runtime_invoke_sig:
  *
- * This method generates a wrapper for calling mono_load_remote_field_new.
- * The return type is ignored for now, as mono_load_remote_field_new () always
- * returns an object. In the future, to optimize some codepaths, we might
- * call a different function that takes a pointer to a valuetype, instead.
+ *   Return a common signature used for sharing runtime invoke wrappers.
  */
-MonoMethod *
-mono_marshal_get_ldfld_remote_wrapper (MonoClass *klass)
+static MonoMethodSignature*
+mono_marshal_get_runtime_invoke_sig (MonoMethodSignature *sig)
 {
-       MonoMethodSignature *sig, *csig;
-       MonoMethodBuilder *mb;
-       MonoMethod *res;
-       static MonoMethod* cached = NULL;
-
-       mono_marshal_lock ();
-       if (cached) {
-               mono_marshal_unlock ();
-               return cached;
-       }
-       mono_marshal_unlock ();
+       MonoMethodSignature *res = mono_metadata_signature_dup (sig);
+       int i;
 
-       mb = mono_mb_new_no_dup_name (mono_defaults.object_class, "__mono_load_remote_field_new_wrapper", MONO_WRAPPER_LDFLD_REMOTE);
+       res->generic_param_count = 0;
+       res->ret = get_runtime_invoke_type (sig->ret, TRUE);
+       for (i = 0; i < res->param_count; ++i)
+               res->params [i] = get_runtime_invoke_type (sig->params [i], FALSE);
 
-       mb->method->save_lmf = 1;
+       return res;
+}
 
-       sig = mono_metadata_signature_alloc (mono_defaults.corlib, 3);
-       sig->params [0] = &mono_defaults.object_class->byval_arg;
-       sig->params [1] = &mono_defaults.int_class->byval_arg;
-       sig->params [2] = &mono_defaults.int_class->byval_arg;
-       sig->ret = &mono_defaults.object_class->byval_arg;
+static gboolean
+runtime_invoke_signature_equal (MonoMethodSignature *sig1, MonoMethodSignature *sig2)
+{
+       /* Can't share wrappers which return a vtype since it needs to be boxed */
+       if (sig1->ret != sig2->ret && !(MONO_TYPE_IS_REFERENCE (sig1->ret) && MONO_TYPE_IS_REFERENCE (sig2->ret)) && !mono_metadata_type_equal (sig1->ret, sig2->ret))
+               return FALSE;
+       else
+               return mono_metadata_signature_equal (sig1, sig2);
+}
 
 #ifndef DISABLE_JIT
-       mono_mb_emit_ldarg (mb, 0);
-       mono_mb_emit_ldarg (mb, 1);
-       mono_mb_emit_ldarg (mb, 2);
-
-       csig = mono_metadata_signature_alloc (mono_defaults.corlib, 3);
-       csig->params [0] = &mono_defaults.object_class->byval_arg;
-       csig->params [1] = &mono_defaults.int_class->byval_arg;
-       csig->params [2] = &mono_defaults.int_class->byval_arg;
-       csig->ret = &mono_defaults.object_class->byval_arg;
-       csig->pinvoke = 1;
-
-       mono_mb_emit_native_call (mb, csig, mono_load_remote_field_new);
-       emit_thread_interrupt_checkpoint (mb);
-
-       mono_mb_emit_byte (mb, CEE_RET);
-#endif
-
-       mono_marshal_lock ();
-       res = cached;
-       mono_marshal_unlock ();
-       if (!res) {
-               MonoMethod *newm;
-               newm = mono_mb_create (mb, sig, 4, NULL);
-               mono_marshal_lock ();
-               res = cached;
-               if (!res) {
-                       res = newm;
-                       cached = res;
-                       mono_marshal_unlock ();
-               } else {
-                       mono_marshal_unlock ();
-                       mono_free_method (newm);
-               }
-       }
-       mono_mb_free (mb);
-
-       return res;
-}
 
 /*
- * mono_marshal_get_ldfld_wrapper:
- * @type: the type of the field
+ * emit_invoke_call:
  *
- * This method generates a function which can be use to load a field with type
- * @type from an object. The generated function has the following signature:
- * <@type> ldfld_wrapper (MonoObject *this, MonoClass *class, MonoClassField *field, int offset)
+ *   Emit the call to the wrapper method from a runtime invoke wrapper.
  */
-MonoMethod *
-mono_marshal_get_ldfld_wrapper (MonoType *type)
+static void
+emit_invoke_call (MonoMethodBuilder *mb, MonoMethod *method,
+                                 MonoMethodSignature *sig, MonoMethodSignature *callsig,
+                                 int loc_res,
+                                 gboolean virtual, gboolean need_direct_wrapper)
 {
-       MonoMethodSignature *sig;
-       MonoMethodBuilder *mb;
-       MonoMethod *res;
-       MonoClass *klass;
-       GHashTable *cache;
-       WrapperInfo *info;
-       char *name;
-       int t, pos0, pos1 = 0;
+       static MonoString *string_dummy = NULL;
+       int i;
+       int *tmp_nullable_locals;
+       gboolean void_ret = FALSE;
 
-       type = mono_type_get_underlying_type (type);
+       /* to make it work with our special string constructors */
+       if (!string_dummy) {
+               MONO_GC_REGISTER_ROOT_SINGLE (string_dummy);
+               string_dummy = mono_string_new_wrapper ("dummy");
+       }
 
-       t = type->type;
+       if (virtual) {
+               g_assert (sig->hasthis);
+               g_assert (method->flags & METHOD_ATTRIBUTE_VIRTUAL);
+       }
 
-       if (!type->byref) {
-               if (type->type == MONO_TYPE_SZARRAY) {
-                       klass = mono_defaults.array_class;
-               } else if (type->type == MONO_TYPE_VALUETYPE) {
-                       klass = type->data.klass;
-               } else if (t == MONO_TYPE_OBJECT || t == MONO_TYPE_CLASS || t == MONO_TYPE_STRING) {
-                       klass = mono_defaults.object_class;
-               } else if (t == MONO_TYPE_PTR || t == MONO_TYPE_FNPTR) {
-                       klass = mono_defaults.int_class;
-               } else if (t == MONO_TYPE_GENERICINST) {
-                       if (mono_type_generic_inst_is_valuetype (type))
-                               klass = mono_class_from_mono_type (type);
-                       else
-                               klass = mono_defaults.object_class;
+       if (sig->hasthis) {
+               if (method->string_ctor) {
+                       if (mono_gc_is_moving ()) {
+                               mono_mb_emit_ptr (mb, &string_dummy);
+                               mono_mb_emit_byte (mb, CEE_LDIND_REF);
+                       } else {
+                               mono_mb_emit_ptr (mb, string_dummy);
+                       }
                } else {
-                       klass = mono_class_from_mono_type (type);                       
+                       mono_mb_emit_ldarg (mb, 0);
                }
-       } else {
-               klass = mono_defaults.int_class;
        }
 
-       cache = get_cache (&klass->image->ldfld_wrapper_cache, mono_aligned_addr_hash, NULL);
-       if ((res = mono_marshal_find_in_cache (cache, klass)))
-               return res;
-
-       /* we add the %p pointer value of klass because class names are not unique */
-       name = g_strdup_printf ("__ldfld_wrapper_%p_%s.%s", klass, klass->name_space, klass->name); 
-       mb = mono_mb_new (mono_defaults.object_class, name, MONO_WRAPPER_LDFLD);
-       g_free (name);
-
-       sig = mono_metadata_signature_alloc (mono_defaults.corlib, 4);
-       sig->params [0] = &mono_defaults.object_class->byval_arg;
-       sig->params [1] = &mono_defaults.int_class->byval_arg;
-       sig->params [2] = &mono_defaults.int_class->byval_arg;
-       sig->params [3] = &mono_defaults.int_class->byval_arg;
-       sig->ret = &klass->byval_arg;
+       tmp_nullable_locals = g_new0 (int, sig->param_count);
 
-#ifndef DISABLE_JIT
-       mono_mb_emit_ldarg (mb, 0);
-       pos0 = mono_mb_emit_proxy_check (mb, CEE_BNE_UN);
+       for (i = 0; i < sig->param_count; i++) {
+               MonoType *t = sig->params [i];
+               int type;
 
-       mono_mb_emit_ldarg (mb, 0);
-       mono_mb_emit_ldarg (mb, 1);
-       mono_mb_emit_ldarg (mb, 2);
+               mono_mb_emit_ldarg (mb, 1);
+               if (i) {
+                       mono_mb_emit_icon (mb, sizeof (gpointer) * i);
+                       mono_mb_emit_byte (mb, CEE_ADD);
+               }
 
-       mono_mb_emit_managed_call (mb, mono_marshal_get_ldfld_remote_wrapper (klass), NULL);
+               if (t->byref) {
+                       mono_mb_emit_byte (mb, CEE_LDIND_I);
+                       /* A Nullable<T> type don't have a boxed form, it's either null or a boxed T.
+                        * So to make this work we unbox it to a local variablee and push a reference to that.
+                        */
+                       if (t->type == MONO_TYPE_GENERICINST && mono_class_is_nullable (mono_class_from_mono_type (t))) {
+                               tmp_nullable_locals [i] = mono_mb_add_local (mb, &mono_class_from_mono_type (t)->byval_arg);
 
-       /*
-       csig = mono_metadata_signature_alloc (mono_defaults.corlib, 3);
-       csig->params [0] = &mono_defaults.object_class->byval_arg;
-       csig->params [1] = &mono_defaults.int_class->byval_arg;
-       csig->params [2] = &mono_defaults.int_class->byval_arg;
-       csig->ret = &klass->this_arg;
-       csig->pinvoke = 1;
+                               mono_mb_emit_op (mb, CEE_UNBOX_ANY, mono_class_from_mono_type (t));
+                               mono_mb_emit_stloc (mb, tmp_nullable_locals [i]);
+                               mono_mb_emit_ldloc_addr (mb, tmp_nullable_locals [i]);
+                       }
+                       continue;
+               }
 
-       mono_mb_emit_native_call (mb, csig, mono_load_remote_field_new);
-       emit_thread_interrupt_checkpoint (mb);
-       */
+               /*FIXME 'this doesn't handle generic enums. Shouldn't we?*/
+               type = sig->params [i]->type;
+handle_enum:
+               switch (type) {
+               case MONO_TYPE_I1:
+               case MONO_TYPE_BOOLEAN:
+               case MONO_TYPE_U1:
+               case MONO_TYPE_I2:
+               case MONO_TYPE_U2:
+               case MONO_TYPE_CHAR:
+               case MONO_TYPE_I:
+               case MONO_TYPE_U:
+               case MONO_TYPE_I4:
+               case MONO_TYPE_U4:
+               case MONO_TYPE_R4:
+               case MONO_TYPE_R8:
+               case MONO_TYPE_I8:
+               case MONO_TYPE_U8:
+                       mono_mb_emit_byte (mb, CEE_LDIND_I);
+                       mono_mb_emit_byte (mb, mono_type_to_ldind (sig->params [i]));
+                       break;
+               case MONO_TYPE_STRING:
+               case MONO_TYPE_CLASS:  
+               case MONO_TYPE_ARRAY:
+               case MONO_TYPE_PTR:
+               case MONO_TYPE_SZARRAY:
+               case MONO_TYPE_OBJECT:
+                       mono_mb_emit_byte (mb, mono_type_to_ldind (sig->params [i]));
+                       break;
+               case MONO_TYPE_GENERICINST:
+                       if (!mono_type_generic_inst_is_valuetype (sig->params [i])) {
+                               mono_mb_emit_byte (mb, mono_type_to_ldind (sig->params [i]));
+                               break;
+                       }
 
-       if (klass->valuetype) {
-               mono_mb_emit_op (mb, CEE_UNBOX, klass);
-               pos1 = mono_mb_emit_branch (mb, CEE_BR);
+                       /* fall through */
+               case MONO_TYPE_VALUETYPE:
+                       if (type == MONO_TYPE_VALUETYPE && t->data.klass->enumtype) {
+                               type = mono_class_enum_basetype (t->data.klass)->type;
+                               goto handle_enum;
+                       }
+                       mono_mb_emit_byte (mb, CEE_LDIND_I);
+                       if (mono_class_is_nullable (mono_class_from_mono_type (sig->params [i]))) {
+                               /* Need to convert a boxed vtype to an mp to a Nullable struct */
+                               mono_mb_emit_op (mb, CEE_UNBOX, mono_class_from_mono_type (sig->params [i]));
+                               mono_mb_emit_op (mb, CEE_LDOBJ, mono_class_from_mono_type (sig->params [i]));
+                       } else {
+                               mono_mb_emit_op (mb, CEE_LDOBJ, mono_class_from_mono_type (sig->params [i]));
+                       }
+                       break;
+               default:
+                       g_assert_not_reached ();
+               }
+       }
+       
+       if (virtual) {
+               mono_mb_emit_op (mb, CEE_CALLVIRT, method);
+       } else if (need_direct_wrapper) {
+               mono_mb_emit_op (mb, CEE_CALL, method);
        } else {
-               mono_mb_emit_byte (mb, CEE_RET);
+               mono_mb_emit_ldarg (mb, 3);
+               mono_mb_emit_calli (mb, callsig);
        }
 
+       if (sig->ret->byref) {
+               /* fixme: */
+               g_assert_not_reached ();
+       }
 
-       mono_mb_patch_branch (mb, pos0);
-
-       mono_mb_emit_ldarg (mb, 0);
-        mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
-        mono_mb_emit_byte (mb, CEE_MONO_OBJADDR);
-       mono_mb_emit_ldarg (mb, 3);
-       mono_mb_emit_byte (mb, CEE_ADD);
-
-       if (klass->valuetype)
-               mono_mb_patch_branch (mb, pos1);
-
-       switch (t) {
-       case MONO_TYPE_I1:
-       case MONO_TYPE_U1:
+       switch (sig->ret->type) {
+       case MONO_TYPE_VOID:
+               if (!method->string_ctor)
+                       void_ret = TRUE;
+               break;
        case MONO_TYPE_BOOLEAN:
        case MONO_TYPE_CHAR:
+       case MONO_TYPE_I1:
+       case MONO_TYPE_U1:
        case MONO_TYPE_I2:
        case MONO_TYPE_U2:
        case MONO_TYPE_I4:
        case MONO_TYPE_U4:
-       case MONO_TYPE_I8:
-       case MONO_TYPE_U8:
+       case MONO_TYPE_I:
+       case MONO_TYPE_U:
        case MONO_TYPE_R4:
        case MONO_TYPE_R8:
+       case MONO_TYPE_I8:
+       case MONO_TYPE_U8:
+       case MONO_TYPE_VALUETYPE:
+       case MONO_TYPE_TYPEDBYREF:
+       case MONO_TYPE_GENERICINST:
+               /* box value types */
+               mono_mb_emit_op (mb, CEE_BOX, mono_class_from_mono_type (sig->ret));
+               break;
+       case MONO_TYPE_STRING:
+       case MONO_TYPE_CLASS:  
        case MONO_TYPE_ARRAY:
        case MONO_TYPE_SZARRAY:
        case MONO_TYPE_OBJECT:
-       case MONO_TYPE_CLASS:
-       case MONO_TYPE_STRING:
-       case MONO_TYPE_I:
-       case MONO_TYPE_U:
-       case MONO_TYPE_PTR:
-       case MONO_TYPE_FNPTR:
-               mono_mb_emit_byte (mb, mono_type_to_ldind (type));
-               break;
-       case MONO_TYPE_VALUETYPE:
-               g_assert (!klass->enumtype);
-               mono_mb_emit_op (mb, CEE_LDOBJ, klass);
-               break;
-       case MONO_TYPE_GENERICINST:
-               if (mono_type_generic_inst_is_valuetype (type)) {
-                       mono_mb_emit_op (mb, CEE_LDOBJ, klass);
-               } else {
-                       mono_mb_emit_byte (mb, CEE_LDIND_REF);
-               }
+               /* nothing to do */
                break;
-       case MONO_TYPE_VAR:
-       case MONO_TYPE_MVAR:
-               mono_mb_emit_op (mb, CEE_LDOBJ, klass);
+       case MONO_TYPE_PTR:
+               /* The result is an IntPtr */
+               mono_mb_emit_op (mb, CEE_BOX, mono_defaults.int_class);
                break;
        default:
-               g_warning ("type %x not implemented", type->type);
                g_assert_not_reached ();
        }
 
-       mono_mb_emit_byte (mb, CEE_RET);
-#endif /* DISABLE_JIT */
+       if (!void_ret)
+               mono_mb_emit_stloc (mb, loc_res);
 
-       info = mono_wrapper_info_create (mb, WRAPPER_SUBTYPE_NONE);
-       info->d.proxy.klass = klass;
-       res = mono_mb_create_and_cache_full (cache, klass,
-                                                                                mb, sig, sig->param_count + 16, info, NULL);
-       mono_mb_free (mb);
-       
-       return res;
-}
+       /* Convert back nullable-byref arguments */
+       for (i = 0; i < sig->param_count; i++) {
+               MonoType *t = sig->params [i];
 
-/*
- * mono_marshal_get_ldflda_wrapper:
- * @type: the type of the field
- *
- * This method generates a function which can be used to load a field address
- * from an object. The generated function has the following signature:
- * gpointer ldflda_wrapper (MonoObject *this, MonoClass *class, MonoClassField *field, int offset);
- */
-MonoMethod *
-mono_marshal_get_ldflda_wrapper (MonoType *type)
-{
-       MonoMethodSignature *sig;
-       MonoMethodBuilder *mb;
-       MonoMethod *res;
-       MonoClass *klass;
-       GHashTable *cache;
-       WrapperInfo *info;
-       char *name;
-       int t, pos0, pos1, pos2, pos3;
+               /* 
+                * Box the result and put it back into the array, the caller will have
+                * to obtain it from there.
+                */
+               if (t->byref && t->type == MONO_TYPE_GENERICINST && mono_class_is_nullable (mono_class_from_mono_type (t))) {
+                       mono_mb_emit_ldarg (mb, 1);                     
+                       mono_mb_emit_icon (mb, sizeof (gpointer) * i);
+                       mono_mb_emit_byte (mb, CEE_ADD);
 
-       type = mono_type_get_underlying_type (type);
-       t = type->type;
+                       mono_mb_emit_ldloc (mb, tmp_nullable_locals [i]);
+                       mono_mb_emit_op (mb, CEE_BOX, mono_class_from_mono_type (t));
 
-       if (!type->byref) {
-               if (type->type == MONO_TYPE_SZARRAY) {
-                       klass = mono_defaults.array_class;
-               } else if (type->type == MONO_TYPE_VALUETYPE) {
-                       klass = type->data.klass;
-               } else if (t == MONO_TYPE_OBJECT || t == MONO_TYPE_CLASS || t == MONO_TYPE_STRING ||
-                          t == MONO_TYPE_CLASS) { 
-                       klass = mono_defaults.object_class;
-               } else if (t == MONO_TYPE_PTR || t == MONO_TYPE_FNPTR) {
-                       klass = mono_defaults.int_class;
-               } else if (t == MONO_TYPE_GENERICINST) {
-                       if (mono_type_generic_inst_is_valuetype (type))
-                               klass = mono_class_from_mono_type (type);
-                       else
-                               klass = mono_defaults.object_class;
-               } else {
-                       klass = mono_class_from_mono_type (type);                       
+                       mono_mb_emit_byte (mb, CEE_STIND_REF);
                }
-       } else {
-               klass = mono_defaults.int_class;
        }
 
-       cache = get_cache (&klass->image->ldflda_wrapper_cache, mono_aligned_addr_hash, NULL);
-       if ((res = mono_marshal_find_in_cache (cache, klass)))
-               return res;
+       g_free (tmp_nullable_locals);
+}
 
-       /* we add the %p pointer value of klass because class names are not unique */
-       name = g_strdup_printf ("__ldflda_wrapper_%p_%s.%s", klass, klass->name_space, klass->name); 
-       mb = mono_mb_new (mono_defaults.object_class, name, MONO_WRAPPER_LDFLDA);
-       g_free (name);
+static void
+emit_runtime_invoke_body (MonoMethodBuilder *mb, MonoClass *target_klass, MonoMethod *method,
+                                                 MonoMethodSignature *sig, MonoMethodSignature *callsig,
+                                                 gboolean virtual, gboolean need_direct_wrapper)
+{
+       gint32 labels [16];
+       MonoExceptionClause *clause;
+       int loc_res, loc_exc;
 
-       sig = mono_metadata_signature_alloc (mono_defaults.corlib, 4);
-       sig->params [0] = &mono_defaults.object_class->byval_arg;
-       sig->params [1] = &mono_defaults.int_class->byval_arg;
-       sig->params [2] = &mono_defaults.int_class->byval_arg;
-       sig->params [3] = &mono_defaults.int_class->byval_arg;
-       sig->ret = &mono_defaults.int_class->byval_arg;
+       /* The wrapper looks like this:
+        *
+        * <interrupt check>
+        * if (exc) {
+        *       try {
+        *         return <call>
+        *       } catch (Exception e) {
+        *     *exc = e;
+        *   }
+        * } else {
+        *     return <call>
+        * }
+        */
 
-#ifndef DISABLE_JIT
-       /* if typeof (this) != transparent_proxy goto pos0 */
-       mono_mb_emit_ldarg (mb, 0);
-       pos0 = mono_mb_emit_proxy_check (mb, CEE_BNE_UN);
+       /* allocate local 0 (object) tmp */
+       loc_res = mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
+       /* allocate local 1 (object) exc */
+       loc_exc = mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
 
-       /* if same_appdomain goto pos1 */
-       mono_mb_emit_ldarg (mb, 0);
-       pos1 = mono_mb_emit_xdomain_check (mb, CEE_BEQ);
+       /* *exc is assumed to be initialized to NULL by the caller */
+
+       mono_mb_emit_byte (mb, CEE_LDARG_2);
+       labels [0] = mono_mb_emit_branch (mb, CEE_BRFALSE);
+
+       /*
+        * if (exc) case
+        */
+       labels [1] = mono_mb_get_label (mb);
+       emit_thread_force_interrupt_checkpoint (mb);
+       emit_invoke_call (mb, method, sig, callsig, loc_res, virtual, need_direct_wrapper);
 
-       mono_mb_emit_exception_full (mb, "System", "InvalidOperationException", "Attempt to load field address from object in another appdomain.");
+       labels [2] = mono_mb_emit_branch (mb, CEE_LEAVE);
 
-       /* same app domain */
-       mono_mb_patch_branch (mb, pos1);
+       /* Add a try clause around the call */
+       clause = mono_image_alloc0 (target_klass->image, sizeof (MonoExceptionClause));
+       clause->flags = MONO_EXCEPTION_CLAUSE_NONE;
+       clause->data.catch_class = mono_defaults.exception_class;
+       clause->try_offset = labels [1];
+       clause->try_len = mono_mb_get_label (mb) - labels [1];
 
-       /* if typeof (this) != contextbound goto pos2 */
-       mono_mb_emit_ldarg (mb, 0);
-       pos2 = mono_mb_emit_contextbound_check (mb, CEE_BEQ);
+       clause->handler_offset = mono_mb_get_label (mb);
 
-       /* if this->rp->context == mono_context_get goto pos3 */
-       mono_mb_emit_ldarg (mb, 0);
-       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoTransparentProxy, rp));
-       mono_mb_emit_byte (mb, CEE_LDIND_REF);
-       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoRealProxy, context));
-       mono_mb_emit_byte (mb, CEE_LDIND_REF);
-       mono_mb_emit_icall (mb, mono_context_get);
-       pos3 = mono_mb_emit_branch (mb, CEE_BEQ);
+       /* handler code */
+       mono_mb_emit_stloc (mb, loc_exc);       
+       mono_mb_emit_byte (mb, CEE_LDARG_2);
+       mono_mb_emit_ldloc (mb, loc_exc);
+       mono_mb_emit_byte (mb, CEE_STIND_REF);
 
-       mono_mb_emit_exception_full (mb, "System", "InvalidOperationException", "Attempt to load field address from object in another context.");
+       mono_mb_emit_branch (mb, CEE_LEAVE);
 
-       mono_mb_patch_branch (mb, pos2);
-       mono_mb_patch_branch (mb, pos3);
+       clause->handler_len = mono_mb_get_pos (mb) - clause->handler_offset;
 
-       /* return the address of the field from this->rp->unwrapped_server */
-       mono_mb_emit_ldarg (mb, 0);
-       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoTransparentProxy, rp));
-       mono_mb_emit_byte (mb, CEE_LDIND_REF);
-       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoRealProxy, unwrapped_server));
-       mono_mb_emit_byte (mb, CEE_LDIND_REF);
-       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
-       mono_mb_emit_byte (mb, CEE_MONO_OBJADDR);
-       mono_mb_emit_ldarg (mb, 3);
-       mono_mb_emit_byte (mb, CEE_ADD);
+       mono_mb_set_clauses (mb, 1, clause);
+
+       mono_mb_patch_branch (mb, labels [2]);
+       mono_mb_emit_ldloc (mb, loc_res);
        mono_mb_emit_byte (mb, CEE_RET);
 
-       /* not a proxy: return the address of the field directly */
-       mono_mb_patch_branch (mb, pos0);
+       /*
+        * if (!exc) case
+        */
+       mono_mb_patch_branch (mb, labels [0]);
+       emit_thread_force_interrupt_checkpoint (mb);
+       emit_invoke_call (mb, method, sig, callsig, loc_res, virtual, need_direct_wrapper);
 
-       mono_mb_emit_ldarg (mb, 0);
-        mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
-        mono_mb_emit_byte (mb, CEE_MONO_OBJADDR);
-       mono_mb_emit_ldarg (mb, 3);
-       mono_mb_emit_byte (mb, CEE_ADD);
+       mono_mb_emit_ldloc (mb, 0);
+       mono_mb_emit_byte (mb, CEE_RET);
+}
+#endif
+
+/*
+ * generates IL code for the runtime invoke function 
+ * MonoObject *runtime_invoke (MonoObject *this, void **params, MonoObject **exc, void* method)
+ *
+ * we also catch exceptions if exc != null
+ * If VIRTUAL is TRUE, then METHOD is invoked virtually on THIS. This is useful since
+ * it means that the compiled code for METHOD does not have to be looked up 
+ * before calling the runtime invoke wrapper. In this case, the wrapper ignores
+ * its METHOD argument.
+ */
+MonoMethod *
+mono_marshal_get_runtime_invoke (MonoMethod *method, gboolean virtual)
+{
+       MonoMethodSignature *sig, *csig, *callsig;
+       MonoMethodBuilder *mb;
+       GHashTable *cache = NULL;
+       MonoClass *target_klass;
+       MonoMethod *res = NULL;
+       static MonoMethodSignature *cctor_signature = NULL;
+       static MonoMethodSignature *finalize_signature = NULL;
+       char *name;
+       const char *param_names [16];
+       gboolean need_direct_wrapper = FALSE;
+       WrapperInfo *info;
+
+       g_assert (method);
+
+       if (!cctor_signature) {
+               cctor_signature = mono_metadata_signature_alloc (mono_defaults.corlib, 0);
+               cctor_signature->ret = &mono_defaults.void_class->byval_arg;
+       }
+       if (!finalize_signature) {
+               finalize_signature = mono_metadata_signature_alloc (mono_defaults.corlib, 0);
+               finalize_signature->ret = &mono_defaults.void_class->byval_arg;
+               finalize_signature->hasthis = 1;
+       }
+
+       if (virtual)
+               need_direct_wrapper = TRUE;
+
+       /* 
+        * Use a separate cache indexed by methods to speed things up and to avoid the
+        * boundless mempool growth caused by the signature_dup stuff below.
+        */
+       if (virtual)
+               cache = get_cache (&method->klass->image->runtime_invoke_vcall_cache, mono_aligned_addr_hash, NULL);
+       else
+               cache = get_cache (&method->klass->image->runtime_invoke_direct_cache, mono_aligned_addr_hash, NULL);
+       res = mono_marshal_find_in_cache (cache, method);
+       if (res)
+               return res;
+               
+       if (method->klass->rank && (method->iflags & METHOD_IMPL_ATTRIBUTE_INTERNAL_CALL) &&
+               (method->iflags & METHOD_IMPL_ATTRIBUTE_NATIVE)) {
+               /* 
+                * Array Get/Set/Address methods. The JIT implements them using inline code
+                * so we need to create an invoke wrapper which calls the method directly.
+                */
+               need_direct_wrapper = TRUE;
+       }
+               
+       if (method->string_ctor) {
+               callsig = lookup_string_ctor_signature (mono_method_signature (method));
+               if (!callsig)
+                       callsig = add_string_ctor_signature (method);
+               /* Can't share this as we push a string as this */
+               need_direct_wrapper = TRUE;
+       } else {
+               if (method_is_dynamic (method))
+                       callsig = signature_dup (method->klass->image, mono_method_signature (method));
+               else
+                       callsig = mono_method_signature (method);
+       }
+
+       target_klass = get_wrapper_target_class (method->klass->image);
+
+       /* Try to share wrappers for non-corlib methods with simple signatures */
+       if (mono_metadata_signature_equal (callsig, cctor_signature)) {
+               callsig = cctor_signature;
+               target_klass = mono_defaults.object_class;
+       } else if (mono_metadata_signature_equal (callsig, finalize_signature)) {
+               callsig = finalize_signature;
+               target_klass = mono_defaults.object_class;
+       }
 
-       mono_mb_emit_byte (mb, CEE_RET);
-#endif
+       if (need_direct_wrapper) {
+               /* Already searched at the start */
+       } else {
+               MonoMethodSignature *tmp_sig;
 
-       info = mono_wrapper_info_create (mb, WRAPPER_SUBTYPE_NONE);
-       info->d.proxy.klass = klass;
-       res = mono_mb_create_and_cache_full (cache, klass,
-                                                                                mb, sig, sig->param_count + 16,
-                                                                                info, NULL);
-       mono_mb_free (mb);
-       
-       return res;
-}
+               callsig = mono_marshal_get_runtime_invoke_sig (callsig);
 
-/*
- * mono_marshal_get_stfld_remote_wrapper:
- * klass: The type of the field
- *
- *  This function generates a wrapper for calling mono_store_remote_field_new
- * with the appropriate signature.
- * Similarly to mono_marshal_get_ldfld_remote_wrapper () this doesn't depend on the
- * klass argument anymore.
- */
-MonoMethod *
-mono_marshal_get_stfld_remote_wrapper (MonoClass *klass)
-{
-       MonoMethodSignature *sig, *csig;
-       MonoMethodBuilder *mb;
-       MonoMethod *res;
-       static MonoMethod *cached = NULL;
+               if (method->klass->valuetype && mono_method_signature (method)->hasthis)
+                       /* These have a different csig */
+                       cache = get_cache (&target_klass->image->runtime_invoke_vtype_cache,
+                                                          (GHashFunc)mono_signature_hash,
+                                                          (GCompareFunc)runtime_invoke_signature_equal);
+               else
+                       cache = get_cache (&target_klass->image->runtime_invoke_cache,
+                                                          (GHashFunc)mono_signature_hash,
+                                                          (GCompareFunc)runtime_invoke_signature_equal);
 
-       mono_marshal_lock ();
-       if (cached) {
+               /* from mono_marshal_find_in_cache */
+               mono_marshal_lock ();
+               res = g_hash_table_lookup (cache, callsig);
                mono_marshal_unlock ();
-               return cached;
-       }
-       mono_marshal_unlock ();
 
-       mb = mono_mb_new_no_dup_name (mono_defaults.object_class, "__mono_store_remote_field_new_wrapper", MONO_WRAPPER_STFLD_REMOTE);
-
-       mb->method->save_lmf = 1;
+               if (res) {
+                       g_free (callsig);
+                       return res;
+               }
 
-       sig = mono_metadata_signature_alloc (mono_defaults.corlib, 4);
-       sig->params [0] = &mono_defaults.object_class->byval_arg;
-       sig->params [1] = &mono_defaults.int_class->byval_arg;
-       sig->params [2] = &mono_defaults.int_class->byval_arg;
-       sig->params [3] = &mono_defaults.object_class->byval_arg;
-       sig->ret = &mono_defaults.void_class->byval_arg;
+               /* Make a copy of the signature from the image mempool */
+               tmp_sig = callsig;
+               callsig = mono_metadata_signature_dup_full (target_klass->image, callsig);
+               g_free (tmp_sig);
+       }
+       
+       sig = mono_method_signature (method);
 
-#ifndef DISABLE_JIT
-       mono_mb_emit_ldarg (mb, 0);
-       mono_mb_emit_ldarg (mb, 1);
-       mono_mb_emit_ldarg (mb, 2);
-       mono_mb_emit_ldarg (mb, 3);
+       csig = mono_metadata_signature_alloc (target_klass->image, 4);
 
-       csig = mono_metadata_signature_alloc (mono_defaults.corlib, 4);
-       csig->params [0] = &mono_defaults.object_class->byval_arg;
+       csig->ret = &mono_defaults.object_class->byval_arg;
+       if (method->klass->valuetype && mono_method_signature (method)->hasthis)
+               csig->params [0] = get_runtime_invoke_type (&method->klass->this_arg, FALSE);
+       else
+               csig->params [0] = &mono_defaults.object_class->byval_arg;
        csig->params [1] = &mono_defaults.int_class->byval_arg;
        csig->params [2] = &mono_defaults.int_class->byval_arg;
-       csig->params [3] = &mono_defaults.object_class->byval_arg;
-       csig->ret = &mono_defaults.void_class->byval_arg;
+       csig->params [3] = &mono_defaults.int_class->byval_arg;
        csig->pinvoke = 1;
+#if TARGET_WIN32
+       /* This is called from runtime code so it has to be cdecl */
+       csig->call_convention = MONO_CALL_C;
+#endif
 
-       mono_mb_emit_native_call (mb, csig, mono_store_remote_field_new);
-       emit_thread_interrupt_checkpoint (mb);
+       name = mono_signature_to_name (callsig, virtual ? "runtime_invoke_virtual" : "runtime_invoke");
+       mb = mono_mb_new (target_klass, name,  MONO_WRAPPER_RUNTIME_INVOKE);
+       g_free (name);
 
-       mono_mb_emit_byte (mb, CEE_RET);
+#ifndef DISABLE_JIT
+       param_names [0] = "this";
+       param_names [1] = "params";
+       param_names [2] = "exc";
+       param_names [3] = "method";
+       mono_mb_set_param_names (mb, param_names);
+
+       emit_runtime_invoke_body (mb, target_klass, method, sig, callsig, virtual, need_direct_wrapper);
 #endif
 
-       mono_marshal_lock ();
-       res = cached;
-       mono_marshal_unlock ();
-       if (!res) {
-               MonoMethod *newm;
-               newm = mono_mb_create (mb, sig, 6, NULL);
+       if (need_direct_wrapper) {
+#ifndef DISABLE_JIT
+               mb->skip_visibility = 1;
+#endif
+               info = mono_wrapper_info_create (mb, virtual ? WRAPPER_SUBTYPE_RUNTIME_INVOKE_VIRTUAL : WRAPPER_SUBTYPE_RUNTIME_INVOKE_DIRECT);
+               info->d.runtime_invoke.method = method;
+               res = mono_mb_create_and_cache_full (cache, method, mb, csig, sig->param_count + 16, info, NULL);
+       } else {
+               /* taken from mono_mb_create_and_cache */
                mono_marshal_lock ();
-               res = cached;
+               res = g_hash_table_lookup (cache, callsig);
+               mono_marshal_unlock ();
+
+               info = mono_wrapper_info_create (mb, WRAPPER_SUBTYPE_RUNTIME_INVOKE_NORMAL);
+               info->d.runtime_invoke.sig = callsig;
+
+               /* Somebody may have created it before us */
                if (!res) {
-                       res = newm;
-                       cached = res;
-                       mono_marshal_unlock ();
-               } else {
+                       MonoMethod *newm;
+                       newm = mono_mb_create (mb, csig, sig->param_count + 16, info);
+
+                       mono_marshal_lock ();
+                       res = g_hash_table_lookup (cache, callsig);
+                       if (!res) {
+                               res = newm;
+                               g_hash_table_insert (cache, callsig, res);
+                               /* Can't insert it into wrapper_hash since the key is a signature */
+                               g_hash_table_insert (method->klass->image->runtime_invoke_direct_cache, method, res);
+                       } else {
+                               mono_free_method (newm);
+                       }
                        mono_marshal_unlock ();
-                       mono_free_method (newm);
                }
+
+               /* end mono_mb_create_and_cache */
        }
+
        mono_mb_free (mb);
-       
-       return res;
+
+       return res;     
 }
 
 /*
- * mono_marshal_get_stfld_wrapper:
- * @type: the type of the field
+ * mono_marshal_get_runtime_invoke_dynamic:
  *
- * This method generates a function which can be use to store a field with type
- * @type. The generated function has the following signature:
- * void stfld_wrapper (MonoObject *this, MonoClass *class, MonoClassField *field, int offset, <@type> val)
+ *   Return a method which can be used to invoke managed methods from native code
+ * dynamically.
+ * The signature of the returned method is given by RuntimeInvokeDynamicFunction:
+ * void runtime_invoke (void *args, MonoObject **exc, void *compiled_method)
+ * ARGS should point to an architecture specific structure containing 
+ * the arguments and space for the return value.
+ * The other arguments are the same as for runtime_invoke (), except that
+ * ARGS should contain the this argument too.
+ * This wrapper serves the same purpose as the runtime-invoke wrappers, but there
+ * is only one copy of it, which is useful in full-aot.
+ * The wrapper info for the wrapper is a WrapperInfo structure.
  */
-MonoMethod *
-mono_marshal_get_stfld_wrapper (MonoType *type)
+MonoMethod*
+mono_marshal_get_runtime_invoke_dynamic (void)
 {
-       MonoMethodSignature *sig;
+       static MonoMethod *method;
+       MonoMethodSignature *csig;
+       MonoExceptionClause *clause;
        MonoMethodBuilder *mb;
-       MonoMethod *res;
-       MonoClass *klass;
-       GHashTable *cache;
-       WrapperInfo *info;
+       int pos, posna;
        char *name;
-       int t, pos;
+       WrapperInfo *info;
 
-       type = mono_type_get_underlying_type (type);
-       t = type->type;
+       if (method)
+               return method;
 
-       if (!type->byref) {
-               if (type->type == MONO_TYPE_SZARRAY) {
-                       klass = mono_defaults.array_class;
-               } else if (type->type == MONO_TYPE_VALUETYPE) {
-                       klass = type->data.klass;
-               } else if (t == MONO_TYPE_OBJECT || t == MONO_TYPE_CLASS || t == MONO_TYPE_STRING) {
-                       klass = mono_defaults.object_class;
-               } else if (t == MONO_TYPE_PTR || t == MONO_TYPE_FNPTR) {
-                       klass = mono_defaults.int_class;
-               } else if (t == MONO_TYPE_GENERICINST) {
-                       if (mono_type_generic_inst_is_valuetype (type))
-                               klass = mono_class_from_mono_type (type);
-                       else
-                               klass = mono_defaults.object_class;
-               } else {
-                       klass = mono_class_from_mono_type (type);                       
-               }
-       } else {
-               klass = mono_defaults.int_class;
-       }
+       csig = mono_metadata_signature_alloc (mono_defaults.corlib, 4);
 
-       cache = get_cache (&klass->image->stfld_wrapper_cache, mono_aligned_addr_hash, NULL);
-       if ((res = mono_marshal_find_in_cache (cache, klass)))
-               return res;
+       csig->ret = &mono_defaults.void_class->byval_arg;
+       csig->params [0] = &mono_defaults.int_class->byval_arg;
+       csig->params [1] = &mono_defaults.int_class->byval_arg;
+       csig->params [2] = &mono_defaults.int_class->byval_arg;
+       csig->params [3] = &mono_defaults.int_class->byval_arg;
 
-       /* we add the %p pointer value of klass because class names are not unique */
-       name = g_strdup_printf ("__stfld_wrapper_%p_%s.%s", klass, klass->name_space, klass->name); 
-       mb = mono_mb_new (mono_defaults.object_class, name, MONO_WRAPPER_STFLD);
+       name = g_strdup ("runtime_invoke_dynamic");
+       mb = mono_mb_new (mono_defaults.object_class, name, MONO_WRAPPER_RUNTIME_INVOKE);
        g_free (name);
 
-       sig = mono_metadata_signature_alloc (mono_defaults.corlib, 5);
-       sig->params [0] = &mono_defaults.object_class->byval_arg;
-       sig->params [1] = &mono_defaults.int_class->byval_arg;
-       sig->params [2] = &mono_defaults.int_class->byval_arg;
-       sig->params [3] = &mono_defaults.int_class->byval_arg;
-       sig->params [4] = &klass->byval_arg;
-       sig->ret = &mono_defaults.void_class->byval_arg;
-
 #ifndef DISABLE_JIT
-       mono_mb_emit_ldarg (mb, 0);
-       pos = mono_mb_emit_proxy_check (mb, CEE_BNE_UN);
+       /* allocate local 0 (object) tmp */
+       mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
+       /* allocate local 1 (object) exc */
+       mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
 
-       mono_mb_emit_ldarg (mb, 0);
-       mono_mb_emit_ldarg (mb, 1);
-       mono_mb_emit_ldarg (mb, 2);
-       mono_mb_emit_ldarg (mb, 4);
-       if (klass->valuetype)
-               mono_mb_emit_op (mb, CEE_BOX, klass);
+       /* cond set *exc to null */
+       mono_mb_emit_byte (mb, CEE_LDARG_1);
+       mono_mb_emit_byte (mb, CEE_BRFALSE_S);
+       mono_mb_emit_byte (mb, 3);      
+       mono_mb_emit_byte (mb, CEE_LDARG_1);
+       mono_mb_emit_byte (mb, CEE_LDNULL);
+       mono_mb_emit_byte (mb, CEE_STIND_REF);
+
+       emit_thread_force_interrupt_checkpoint (mb);
+
+       mono_mb_emit_byte (mb, CEE_LDARG_0);
+       mono_mb_emit_byte (mb, CEE_LDARG_2);
+       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+       mono_mb_emit_byte (mb, CEE_MONO_DYN_CALL);
+
+       pos = mono_mb_emit_branch (mb, CEE_LEAVE);
+
+       clause = mono_image_alloc0 (mono_defaults.corlib, sizeof (MonoExceptionClause));
+       clause->flags = MONO_EXCEPTION_CLAUSE_FILTER;
+       clause->try_len = mono_mb_get_label (mb);
+
+       /* filter code */
+       clause->data.filter_offset = mono_mb_get_label (mb);
+       
+       mono_mb_emit_byte (mb, CEE_POP);
+       mono_mb_emit_byte (mb, CEE_LDARG_1);
+       mono_mb_emit_byte (mb, CEE_LDC_I4_0);
+       mono_mb_emit_byte (mb, CEE_PREFIX1);
+       mono_mb_emit_byte (mb, CEE_CGT_UN);
+       mono_mb_emit_byte (mb, CEE_PREFIX1);
+       mono_mb_emit_byte (mb, CEE_ENDFILTER);
+
+       clause->handler_offset = mono_mb_get_label (mb);
+
+       /* handler code */
+       /* store exception */
+       mono_mb_emit_stloc (mb, 1);
+       
+       mono_mb_emit_byte (mb, CEE_LDARG_1);
+       mono_mb_emit_ldloc (mb, 1);
+       mono_mb_emit_byte (mb, CEE_STIND_REF);
+
+       mono_mb_emit_byte (mb, CEE_LDNULL);
+       mono_mb_emit_stloc (mb, 0);
+
+       /* Check for the abort exception */
+       mono_mb_emit_ldloc (mb, 1);
+       mono_mb_emit_op (mb, CEE_ISINST, mono_defaults.threadabortexception_class);
+       posna = mono_mb_emit_short_branch (mb, CEE_BRFALSE_S);
+
+       /* Delay the abort exception */
+       mono_mb_emit_icall (mb, ves_icall_System_Threading_Thread_ResetAbort);
+
+       mono_mb_patch_short_branch (mb, posna);
+       mono_mb_emit_branch (mb, CEE_LEAVE);
+
+       clause->handler_len = mono_mb_get_pos (mb) - clause->handler_offset;
 
-       mono_mb_emit_managed_call (mb, mono_marshal_get_stfld_remote_wrapper (klass), NULL);
+       mono_mb_set_clauses (mb, 1, clause);
 
+       /* return result */
+       mono_mb_patch_branch (mb, pos);
+       //mono_mb_emit_ldloc (mb, 0);
        mono_mb_emit_byte (mb, CEE_RET);
+#endif /* DISABLE_JIT */
+
+       info = mono_wrapper_info_create (mb, WRAPPER_SUBTYPE_RUNTIME_INVOKE_DYNAMIC);
+
+       mono_marshal_lock ();
+       /* double-checked locking */
+       if (!method)
+               method = mono_mb_create (mb, csig, 16, info);
 
-       mono_mb_patch_branch (mb, pos);
+       mono_marshal_unlock ();
 
-       mono_mb_emit_ldarg (mb, 0);
-        mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
-        mono_mb_emit_byte (mb, CEE_MONO_OBJADDR);
-       mono_mb_emit_ldarg (mb, 3);
-       mono_mb_emit_byte (mb, CEE_ADD);
-       mono_mb_emit_ldarg (mb, 4);
+       mono_mb_free (mb);
 
-       switch (t) {
-       case MONO_TYPE_I1:
-       case MONO_TYPE_U1:
-       case MONO_TYPE_BOOLEAN:
-       case MONO_TYPE_CHAR:
-       case MONO_TYPE_I2:
-       case MONO_TYPE_U2:
-       case MONO_TYPE_I4:
-       case MONO_TYPE_U4:
-       case MONO_TYPE_I8:
-       case MONO_TYPE_U8:
-       case MONO_TYPE_R4:
-       case MONO_TYPE_R8:
-       case MONO_TYPE_ARRAY:
-       case MONO_TYPE_SZARRAY:
-       case MONO_TYPE_OBJECT:
-       case MONO_TYPE_CLASS:
-       case MONO_TYPE_STRING:
-       case MONO_TYPE_I:
-       case MONO_TYPE_U:
-       case MONO_TYPE_PTR:
-       case MONO_TYPE_FNPTR:
-               mono_mb_emit_byte (mb, mono_type_to_stind (type));
-               break;
-       case MONO_TYPE_VALUETYPE:
-               g_assert (!klass->enumtype);
-               mono_mb_emit_op (mb, CEE_STOBJ, klass);
-               break;
-       case MONO_TYPE_GENERICINST:
-       case MONO_TYPE_VAR:
-       case MONO_TYPE_MVAR:
-               mono_mb_emit_op (mb, CEE_STOBJ, klass);
-               break;
-       default:
-               g_warning ("type %x not implemented", type->type);
-               g_assert_not_reached ();
-       }
+       return method;
+}
 
-       mono_mb_emit_byte (mb, CEE_RET);
-#endif
+#ifndef DISABLE_JIT
+static void
+mono_mb_emit_auto_layout_exception (MonoMethodBuilder *mb, MonoClass *klass)
+{
+       char *msg = g_strdup_printf ("The type `%s.%s' layout needs to be Sequential or Explicit",
+                                    klass->name_space, klass->name);
 
-       info = mono_wrapper_info_create (mb, WRAPPER_SUBTYPE_NONE);
-       info->d.proxy.klass = klass;
-       res = mono_mb_create_and_cache_full (cache, klass,
-                                                                                mb, sig, sig->param_count + 16,
-                                                                                info, NULL);
-       mono_mb_free (mb);
-       
-       return res;
+       mono_mb_emit_exception_marshal_directive (mb, msg);
 }
-#endif /* DISABLE_REMOTING */
+#endif
 
 /*
  * generates IL code for the icall wrapper (the generated method
@@ -5962,20 +4160,23 @@ emit_marshal_custom (EmitMarshalContext *m, int argnum, MonoType *t,
        int pos2;
 
        if (!ICustomMarshaler) {
-               ICustomMarshaler = mono_class_from_name (mono_defaults.corlib, "System.Runtime.InteropServices", "ICustomMarshaler");
-               if (!ICustomMarshaler) {
+               MonoClass *klass = mono_class_from_name (mono_defaults.corlib, "System.Runtime.InteropServices", "ICustomMarshaler");
+               if (!klass) {
                        exception_msg = g_strdup ("Current profile doesn't support ICustomMarshaler");
                        goto handle_exception;
                }
 
-               cleanup_native = mono_class_get_method_from_name (ICustomMarshaler, "CleanUpNativeData", 1);
+               cleanup_native = mono_class_get_method_from_name (klass, "CleanUpNativeData", 1);
                g_assert (cleanup_native);
-               cleanup_managed = mono_class_get_method_from_name (ICustomMarshaler, "CleanUpManagedData", 1);
+               cleanup_managed = mono_class_get_method_from_name (klass, "CleanUpManagedData", 1);
                g_assert (cleanup_managed);
-               marshal_managed_to_native = mono_class_get_method_from_name (ICustomMarshaler, "MarshalManagedToNative", 1);
+               marshal_managed_to_native = mono_class_get_method_from_name (klass, "MarshalManagedToNative", 1);
                g_assert (marshal_managed_to_native);
-               marshal_native_to_managed = mono_class_get_method_from_name (ICustomMarshaler, "MarshalNativeToManaged", 1);
+               marshal_native_to_managed = mono_class_get_method_from_name (klass, "MarshalNativeToManaged", 1);
                g_assert (marshal_native_to_managed);
+
+               mono_memory_barrier ();
+               ICustomMarshaler = klass;
        }
 
        if (spec->data.custom_data.image)
@@ -7777,6 +5978,31 @@ emit_marshal_array (EmitMarshalContext *m, int argnum, MonoType *t,
                need_free = mono_marshal_need_free (&klass->element_class->byval_arg, 
                                                                                        m->piinfo, spec);
 
+               if ((t->attrs & PARAM_ATTRIBUTE_OUT) && spec && spec->native == MONO_NATIVE_LPARRAY && spec->data.array_data.param_num != -1) {
+                       int param_num = spec->data.array_data.param_num;
+                       MonoType *param_type;
+
+                       param_type = m->sig->params [param_num];
+
+                       if (param_type->byref && param_type->type != MONO_TYPE_I4) {
+                               char *msg = g_strdup ("Not implemented.");
+                               mono_mb_emit_exception_marshal_directive (mb, msg);
+                               break;
+                       }
+
+                       mono_mb_emit_ldarg (mb, argnum);
+
+                       /* Create the managed array */
+                       mono_mb_emit_ldarg (mb, param_num);
+                       if (m->sig->params [param_num]->byref)
+                               // FIXME: Support other types
+                               mono_mb_emit_byte (mb, CEE_LDIND_I4);
+                       mono_mb_emit_byte (mb, CEE_CONV_OVF_I);
+                       mono_mb_emit_op (mb, CEE_NEWARR, klass->element_class);
+                       /* Store into argument */
+                       mono_mb_emit_byte (mb, CEE_STIND_I);
+               }
+
                if (need_convert || need_free) {
                        /* FIXME: Optimize blittable case */
                        MonoClass *eklass;
@@ -7892,6 +6118,8 @@ emit_marshal_array (EmitMarshalContext *m, int argnum, MonoType *t,
                        /* free memory allocated (if any) by MONO_MARSHAL_CONV_ARRAY_LPARRAY */
 
                        mono_mb_emit_ldarg (mb, argnum);
+                       if (t->byref)
+                               mono_mb_emit_byte (mb, CEE_LDIND_REF);
                        mono_mb_emit_ldloc (mb, conv_arg);
                        mono_mb_emit_icall (mb, conv_to_icall (MONO_MARSHAL_FREE_LPARRAY));
                }
@@ -7913,7 +6141,7 @@ emit_marshal_array (EmitMarshalContext *m, int argnum, MonoType *t,
        case MARSHAL_ACTION_MANAGED_CONV_IN: {
                MonoClass *eklass;
                guint32 label1, label2, label3;
-               int index_var, src_ptr, loc, esize, param_num, num_elem;
+               int index_var, src_ptr, esize, param_num, num_elem;
                MonoMarshalConv conv;
                gboolean is_string = FALSE;
                
@@ -7974,7 +6202,6 @@ emit_marshal_array (EmitMarshalContext *m, int argnum, MonoType *t,
                else
                        esize = mono_class_native_size (eklass, NULL);
                src_ptr = mono_mb_add_local (mb, &mono_defaults.int_class->byval_arg);
-               loc = mono_mb_add_local (mb, &mono_defaults.int_class->byval_arg);
 
                mono_mb_emit_byte (mb, CEE_LDNULL);
                mono_mb_emit_stloc (mb, conv_arg);
@@ -8091,7 +6318,7 @@ emit_marshal_array (EmitMarshalContext *m, int argnum, MonoType *t,
        case MARSHAL_ACTION_MANAGED_CONV_OUT: {
                MonoClass *eklass;
                guint32 label1, label2, label3;
-               int index_var, dest_ptr, loc, esize, param_num, num_elem;
+               int index_var, dest_ptr, esize, param_num, num_elem;
                MonoMarshalConv conv;
                gboolean is_string = FALSE;
 
@@ -8139,7 +6366,6 @@ emit_marshal_array (EmitMarshalContext *m, int argnum, MonoType *t,
                        esize = mono_class_native_size (eklass, NULL);
 
                dest_ptr = mono_mb_add_local (mb, &mono_defaults.int_class->byval_arg);
-               loc = mono_mb_add_local (mb, &mono_defaults.int_class->byval_arg);
 
                /* Check null */
                mono_mb_emit_ldloc (mb, conv_arg);
@@ -8686,8 +6912,9 @@ emit_marshal (EmitMarshalContext *m, int argnum, MonoType *t,
                        return emit_marshal_vtype (m, argnum, t, spec, conv_arg, conv_arg_type, action);
                else
                        return emit_marshal_object (m, argnum, t, spec, conv_arg, conv_arg_type, action);
+       default:
+               return conv_arg;
        }
-       return conv_arg;
 }
 
 #ifndef DISABLE_JIT
@@ -8714,7 +6941,9 @@ mono_marshal_emit_native_wrapper (MonoImage *image, MonoMethodBuilder *mb, MonoM
        int type, param_shift = 0;
        static MonoMethodSignature *get_last_error_sig = NULL;
 
+       memset (&m, 0, sizeof (m));
        m.mb = mb;
+       m.sig = sig;
        m.piinfo = piinfo;
 
        /* we copy the signature, so that we can set pinvoke to 0 */
@@ -8902,6 +7131,8 @@ mono_marshal_emit_native_wrapper (MonoImage *image, MonoMethodBuilder *mb, MonoM
                case MONO_TYPE_BOOLEAN:
                        emit_marshal (&m, argnum, t, spec, tmp_locals [i], NULL, MARSHAL_ACTION_CONV_OUT);
                        break;
+               default:
+                       break;
                }
        }
 
@@ -8955,6 +7186,9 @@ mono_marshal_get_native_wrapper (MonoMethod *method, gboolean check_exceptions,
                return res;
 
        if (MONO_CLASS_IS_IMPORT (method->klass)) {
+               /* The COM code is not AOT compatible, it calls mono_custom_attrs_get_attr_checked () */
+               if (aot)
+                       return method;
 #ifndef DISABLE_COM
                return mono_cominterop_get_native_wrapper (method);
 #else
@@ -9151,13 +7385,18 @@ mono_marshal_get_native_func_wrapper (MonoImage *image, MonoMethodSignature *sig
 {
        MonoMethodSignature *csig;
 
+       SignaturePointerPair key, *new_key;
        MonoMethodBuilder *mb;
        MonoMethod *res;
        GHashTable *cache;
+       gboolean found;
        char *name;
 
-       cache = get_cache (&image->native_wrapper_cache, mono_aligned_addr_hash, NULL);
-       if ((res = mono_marshal_find_in_cache (cache, func)))
+       key.sig = sig;
+       key.pointer = func;
+
+       cache = get_cache (&image->native_func_wrapper_cache, signature_pointer_pair_hash, signature_pointer_pair_equal);
+       if ((res = mono_marshal_find_in_cache (cache, &key)))
                return res;
 
        name = g_strdup_printf ("wrapper_native_%p", func);
@@ -9170,8 +7409,15 @@ mono_marshal_get_native_func_wrapper (MonoImage *image, MonoMethodSignature *sig
 
        csig = signature_dup (image, sig);
        csig->pinvoke = 0;
-       res = mono_mb_create_and_cache (cache, func,
-                                                                       mb, csig, csig->param_count + 16);
+
+       new_key = g_new (SignaturePointerPair,1);
+       new_key->sig = csig;
+       new_key->pointer = func;
+
+       res = mono_mb_create_and_cache_full (cache, new_key, mb, csig, csig->param_count + 16, NULL, &found);
+       if (found)
+               g_free (new_key);
+
        mono_mb_free (mb);
 
        mono_marshal_set_wrapper_info (res, NULL);
@@ -9438,6 +7684,8 @@ mono_marshal_emit_managed_wrapper (MonoMethodBuilder *mb, MonoMethodSignature *i
                        case MONO_TYPE_BOOLEAN:
                                emit_marshal (m, i, t, mspecs [i + 1], tmp_locals [i], NULL, MARSHAL_ACTION_MANAGED_CONV_OUT);
                                break;
+                       default:
+                               break;
                        }
                }
                else if (invoke_sig->params [i]->attrs & PARAM_ATTRIBUTE_OUT) {
@@ -9561,6 +7809,7 @@ mono_marshal_get_managed_wrapper (MonoMethod *method, MonoClass *delegate_klass,
        csig->hasthis = 0;
        csig->pinvoke = 1;
 
+       memset (&m, 0, sizeof (m));
        m.mb = mb;
        m.sig = sig;
        m.piinfo = NULL;
@@ -9602,8 +7851,6 @@ mono_marshal_get_managed_wrapper (MonoMethod *method, MonoClass *delegate_klass,
                        gint32 call_conv;
                        gint32 charset = 0;
                        MonoBoolean set_last_error = 0;
-                       MonoBoolean best_fit_mapping = 0;
-                       MonoBoolean throw_on_unmappable = 0;
                        MonoError error;
 
                        mono_reflection_create_custom_attr_data_args (mono_defaults.corlib, attr->ctor, attr->data, attr->data_size, &typed_args, &named_args, &arginfo, &error);
@@ -9626,9 +7873,9 @@ mono_marshal_get_managed_wrapper (MonoMethod *method, MonoClass *delegate_klass,
                                } else if (!strcmp (narg->field->name, "SetLastError")) {
                                        set_last_error = *(MonoBoolean*)mono_object_unbox (o);
                                } else if (!strcmp (narg->field->name, "BestFitMapping")) {
-                                       best_fit_mapping = *(MonoBoolean*)mono_object_unbox (o);
+                                       // best_fit_mapping = *(MonoBoolean*)mono_object_unbox (o);
                                } else if (!strcmp (narg->field->name, "ThrowOnUnmappableChar")) {
-                                       throw_on_unmappable = *(MonoBoolean*)mono_object_unbox (o);
+                                       // throw_on_unmappable = *(MonoBoolean*)mono_object_unbox (o);
                                } else {
                                        g_assert_not_reached ();
                                }
@@ -9707,6 +7954,7 @@ mono_marshal_get_vtfixup_ftnptr (MonoImage *image, guint32 token, guint16 type)
                csig->hasthis = 0;
                csig->pinvoke = 1;
 
+               memset (&m, 0, sizeof (m));
                m.mb = mb;
                m.sig = sig;
                m.piinfo = NULL;
@@ -9757,18 +8005,6 @@ mono_marshal_get_vtfixup_ftnptr (MonoImage *image, guint32 token, guint16 type)
        return mono_compile_method (method);
 }
 
-static MonoReflectionType *
-type_from_handle (MonoType *handle)
-{
-       MonoDomain *domain = mono_domain_get (); 
-       MonoClass *klass = mono_class_from_mono_type (handle);
-
-       MONO_ARCH_SAVE_REGS;
-
-       mono_class_init (klass);
-       return mono_type_get_object (domain, handle);
-}
-
 /*
  * This does the equivalent of mono_object_castclass_with_cache.
  * The wrapper info for the wrapper is a WrapperInfo structure.
@@ -9908,7 +8144,7 @@ mono_marshal_get_isinst_with_cache (void)
        mono_mb_emit_ldloc (mb, 1);
        mono_mb_emit_byte (mb, CEE_LDC_I4);
        mono_mb_emit_i4 (mb, ~0x1);
-       mono_mb_emit_byte (mb, CEE_CONV_U);
+       mono_mb_emit_byte (mb, CEE_CONV_I);
        mono_mb_emit_byte (mb, CEE_AND);
        mono_mb_emit_ldloc (mb, 0);
        /*if ((cached_vtable & ~0x1)== obj_vtable)*/
@@ -10174,100 +8410,6 @@ mono_marshal_get_castclass (MonoClass *klass)
        return res;
 }
 
-#ifndef DISABLE_REMOTING
-MonoMethod *
-mono_marshal_get_proxy_cancast (MonoClass *klass)
-{
-       static MonoMethodSignature *isint_sig = NULL;
-       GHashTable *cache;
-       MonoMethod *res;
-       WrapperInfo *info;
-       int pos_failed, pos_end;
-       char *name, *klass_name;
-       MonoMethod *can_cast_to;
-       MonoMethodDesc *desc;
-       MonoMethodBuilder *mb;
-
-       cache = get_cache (&klass->image->proxy_isinst_cache, mono_aligned_addr_hash, NULL);
-       if ((res = mono_marshal_find_in_cache (cache, klass)))
-               return res;
-
-       if (!isint_sig) {
-               isint_sig = mono_metadata_signature_alloc (mono_defaults.corlib, 1);
-               isint_sig->params [0] = &mono_defaults.object_class->byval_arg;
-               isint_sig->ret = &mono_defaults.object_class->byval_arg;
-               isint_sig->pinvoke = 0;
-       }
-
-       klass_name = mono_type_full_name (&klass->byval_arg);
-       name = g_strdup_printf ("__proxy_isinst_wrapper_%s", klass_name); 
-       mb = mono_mb_new (mono_defaults.object_class, name, MONO_WRAPPER_PROXY_ISINST);
-       g_free (klass_name);
-       g_free (name);
-       
-       mb->method->save_lmf = 1;
-
-#ifndef DISABLE_JIT
-       /* get the real proxy from the transparent proxy*/
-       mono_mb_emit_ldarg (mb, 0);
-       mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoTransparentProxy, rp));
-       mono_mb_emit_byte (mb, CEE_LDIND_REF);
-       
-       /* get the reflection type from the type handle */
-       mono_mb_emit_ptr (mb, &klass->byval_arg);
-       mono_mb_emit_icall (mb, type_from_handle);
-       
-       mono_mb_emit_ldarg (mb, 0);
-       
-       /* make the call to CanCastTo (type, ob) */
-       desc = mono_method_desc_new ("IRemotingTypeInfo:CanCastTo", FALSE);
-       can_cast_to = mono_method_desc_search_in_class (desc, mono_defaults.iremotingtypeinfo_class);
-       g_assert (can_cast_to);
-       mono_method_desc_free (desc);
-       mono_mb_emit_op (mb, CEE_CALLVIRT, can_cast_to);
-       
-       pos_failed = mono_mb_emit_branch (mb, CEE_BRFALSE);
-
-       /* Upgrade the proxy vtable by calling: mono_upgrade_remote_class_wrapper (type, ob)*/
-       mono_mb_emit_ptr (mb, &klass->byval_arg);
-       mono_mb_emit_icall (mb, type_from_handle);
-       mono_mb_emit_ldarg (mb, 0);
-       
-       mono_mb_emit_icall (mb, mono_upgrade_remote_class_wrapper);
-       emit_thread_interrupt_checkpoint (mb);
-       
-       mono_mb_emit_ldarg (mb, 0);
-       pos_end = mono_mb_emit_branch (mb, CEE_BR);
-       
-       /* fail */
-       
-       mono_mb_patch_branch (mb, pos_failed);
-       mono_mb_emit_byte (mb, CEE_LDNULL);
-       
-       /* the end */
-       
-       mono_mb_patch_branch (mb, pos_end);
-       mono_mb_emit_byte (mb, CEE_RET);
-#endif
-
-       info = mono_wrapper_info_create (mb, WRAPPER_SUBTYPE_NONE);
-       info->d.proxy.klass = klass;
-       res = mono_mb_create_and_cache_full (cache, klass, mb, isint_sig, isint_sig->param_count + 16, info, NULL);
-       mono_mb_free (mb);
-
-       return res;
-}
-
-void
-mono_upgrade_remote_class_wrapper (MonoReflectionType *rtype, MonoTransparentProxy *tproxy)
-{
-       MonoClass *klass;
-       MonoDomain *domain = ((MonoObject*)tproxy)->vtable->domain;
-       klass = mono_class_from_mono_type (rtype->type);
-       mono_upgrade_remote_class (domain, (MonoObject*)tproxy, klass);
-}
-#endif /*DISABLE_REMOTING */
-
 /**
  * mono_marshal_get_struct_to_ptr:
  * @klass:
@@ -10426,11 +8568,9 @@ mono_marshal_get_synchronized_inner_wrapper (MonoMethod *method)
        MonoMethodSignature *sig;
        MonoMethod *res;
        MonoGenericContext *ctx = NULL;
-       MonoMethod *orig_method = NULL;
        MonoGenericContainer *container = NULL;
 
        if (method->is_inflated && !mono_method_get_context (method)->method_inst) {
-               orig_method = method;
                ctx = &((MonoMethodInflated*)method)->context;
                method = ((MonoMethodInflated*)method)->declaring;
                container = mono_method_get_generic_container (method);
@@ -10450,8 +8590,11 @@ mono_marshal_get_synchronized_inner_wrapper (MonoMethod *method)
        info->d.synchronized_inner.method = method;
        res = mono_mb_create (mb, sig, 0, info);
        mono_mb_free (mb);
-       if (ctx)
-               res = mono_class_inflate_generic_method (res, ctx);
+       if (ctx) {
+               MonoError error;
+               res = mono_class_inflate_generic_method_checked (res, ctx, &error);
+               g_assert (mono_error_ok (&error)); /* FIXME don't swallow the error */
+       }
        return res;
 }
 
@@ -10592,10 +8735,13 @@ mono_marshal_get_synchronized_wrapper (MonoMethod *method)
        for (i = 0; i < sig->param_count; i++)
                mono_mb_emit_ldarg (mb, i + (sig->hasthis == TRUE));
 
-       if (ctx)
-               mono_mb_emit_managed_call (mb, mono_class_inflate_generic_method (method, &container->context), NULL);
-       else
+       if (ctx) {
+               MonoError error;
+               mono_mb_emit_managed_call (mb, mono_class_inflate_generic_method_checked (method, &container->context, &error), NULL);
+               g_assert (mono_error_ok (&error)); /* FIXME don't swallow the error */
+       } else {
                mono_mb_emit_managed_call (mb, method, NULL);
+       }
 
        if (!MONO_TYPE_IS_VOID (sig->ret))
                mono_mb_emit_stloc (mb, ret_local);
@@ -11412,10 +9558,11 @@ static int elem_addr_cache_next = 0;
  * @rank: rank of the array type
  * @elem_size: size in bytes of an element of an array.
  *
- * Returns a MonoMethd that implements the code to get the address
+ * Returns a MonoMethod that implements the code to get the address
  * of an element in a multi-dimenasional array of @rank dimensions.
  * The returned method takes an array as the first argument and then
  * @rank indexes for the @rank dimensions.
+ * If ELEM_SIZE is 0, read the array size from the array object.
  */
 MonoMethod*
 mono_marshal_get_array_address (int rank, int elem_size)
@@ -11424,6 +9571,7 @@ mono_marshal_get_array_address (int rank, int elem_size)
        MonoMethodBuilder *mb;
        MonoMethodSignature *sig;
        WrapperInfo *info;
+       char *name;
        int i, bounds, ind, realidx;
        int branch_pos, *branch_positions;
        int cached;
@@ -11451,7 +9599,9 @@ mono_marshal_get_array_address (int rank, int elem_size)
                sig->params [i + 1] = &mono_defaults.int32_class->byval_arg;
        }
 
-       mb = mono_mb_new (mono_defaults.object_class, "ElementAddr", MONO_WRAPPER_MANAGED_TO_MANAGED);
+       name = g_strdup_printf ("ElementAddr_%d", elem_size);
+       mb = mono_mb_new (mono_defaults.object_class, name, MONO_WRAPPER_MANAGED_TO_MANAGED);
+       g_free (name);
        
 #ifndef DISABLE_JIT
        bounds = mono_mb_add_local (mb, &mono_defaults.int_class->byval_arg);
@@ -11519,8 +9669,24 @@ mono_marshal_get_array_address (int rank, int elem_size)
        mono_mb_emit_ldarg (mb, 0);
        mono_mb_emit_ldflda (mb, MONO_STRUCT_OFFSET (MonoArray, vector));
        mono_mb_emit_ldloc (mb, ind);
-       mono_mb_emit_icon (mb, elem_size);
-       mono_mb_emit_byte (mb, CEE_MUL);
+       if (elem_size) {
+               mono_mb_emit_icon (mb, elem_size);
+       } else {
+               /* Load arr->vtable->klass->sizes.element_class */
+               mono_mb_emit_ldarg (mb, 0);
+               mono_mb_emit_byte (mb, CEE_CONV_I);
+               mono_mb_emit_icon (mb, MONO_STRUCT_OFFSET (MonoObject, vtable));
+               mono_mb_emit_byte (mb, CEE_ADD);
+               mono_mb_emit_byte (mb, CEE_LDIND_I);
+               mono_mb_emit_icon (mb, MONO_STRUCT_OFFSET (MonoVTable, klass));
+               mono_mb_emit_byte (mb, CEE_ADD);
+               mono_mb_emit_byte (mb, CEE_LDIND_I);
+               /* sizes is an union, so this reads sizes.element_size */
+               mono_mb_emit_icon (mb, MONO_STRUCT_OFFSET (MonoClass, sizes));
+               mono_mb_emit_byte (mb, CEE_ADD);
+               mono_mb_emit_byte (mb, CEE_LDIND_I4);
+       }
+               mono_mb_emit_byte (mb, CEE_MUL);
        mono_mb_emit_byte (mb, CEE_ADD);
        mono_mb_emit_byte (mb, CEE_RET);
 
@@ -11617,10 +9783,13 @@ mono_marshal_get_array_accessor_wrapper (MonoMethod *method)
        for (i = 0; i < sig->param_count; i++)
                mono_mb_emit_ldarg (mb, i + (sig->hasthis == TRUE));
 
-       if (ctx)
-               mono_mb_emit_managed_call (mb, mono_class_inflate_generic_method (method, &container->context), NULL);
-       else
+       if (ctx) {
+               MonoError error;
+               mono_mb_emit_managed_call (mb, mono_class_inflate_generic_method_checked (method, &container->context, &error), NULL);
+               g_assert (mono_error_ok (&error)); /* FIXME don't swallow the error */
+       } else {
                mono_mb_emit_managed_call (mb, method, NULL);
+       }
        mono_mb_emit_byte (mb, CEE_RET);
 #endif
 
@@ -11729,19 +9898,25 @@ ves_icall_System_Runtime_InteropServices_Marshal_copy_to_unmanaged (MonoArray *s
        int element_size;
        void *source_addr;
 
-       MONO_ARCH_SAVE_REGS;
+       MONO_CHECK_ARG_NULL (src,);
+       MONO_CHECK_ARG_NULL (dest,);
 
-       MONO_CHECK_ARG_NULL (src);
-       MONO_CHECK_ARG_NULL (dest);
-
-       if (src->obj.vtable->klass->rank != 1)
-               mono_raise_exception (mono_get_exception_argument ("array", "array is multi-dimensional"));
-       if (start_index < 0)
-               mono_raise_exception (mono_get_exception_argument ("startIndex", "Must be >= 0"));
-       if (length < 0)
-               mono_raise_exception (mono_get_exception_argument ("length", "Must be >= 0"));
-       if (start_index + length > mono_array_length (src))
-               mono_raise_exception (mono_get_exception_argument ("length", "start_index + length > array length"));
+       if (src->obj.vtable->klass->rank != 1) {
+               mono_set_pending_exception (mono_get_exception_argument ("array", "array is multi-dimensional"));
+               return;
+       }
+       if (start_index < 0) {
+               mono_set_pending_exception (mono_get_exception_argument ("startIndex", "Must be >= 0"));
+               return;
+       }
+       if (length < 0) {
+               mono_set_pending_exception (mono_get_exception_argument ("length", "Must be >= 0"));
+               return;
+       }
+       if (start_index + length > mono_array_length (src)) {
+               mono_set_pending_exception (mono_get_exception_argument ("length", "start_index + length > array length"));
+               return;
+       }
 
        element_size = mono_array_element_size (src->obj.vtable->klass);
 
@@ -11758,20 +9933,25 @@ ves_icall_System_Runtime_InteropServices_Marshal_copy_from_unmanaged (gpointer s
        int element_size;
        void *dest_addr;
 
-       MONO_ARCH_SAVE_REGS;
-
-       MONO_CHECK_ARG_NULL (src);
-       MONO_CHECK_ARG_NULL (dest);
-
-       if (dest->obj.vtable->klass->rank != 1)
-               mono_raise_exception (mono_get_exception_argument ("array", "array is multi-dimensional"));
-       if (start_index < 0)
-               mono_raise_exception (mono_get_exception_argument ("startIndex", "Must be >= 0"));
-       if (length < 0)
-               mono_raise_exception (mono_get_exception_argument ("length", "Must be >= 0"));
-       if (start_index + length > mono_array_length (dest))
-               mono_raise_exception (mono_get_exception_argument ("length", "start_index + length > array length"));
+       MONO_CHECK_ARG_NULL (src,);
+       MONO_CHECK_ARG_NULL (dest,);
 
+       if (dest->obj.vtable->klass->rank != 1) {
+               mono_set_pending_exception (mono_get_exception_argument ("array", "array is multi-dimensional"));
+               return;
+       }
+       if (start_index < 0) {
+               mono_set_pending_exception (mono_get_exception_argument ("startIndex", "Must be >= 0"));
+               return;
+       }
+       if (length < 0) {
+               mono_set_pending_exception (mono_get_exception_argument ("length", "Must be >= 0"));
+               return;
+       }
+       if (start_index + length > mono_array_length (dest)) {
+               mono_set_pending_exception (mono_get_exception_argument ("length", "start_index + length > array length"));
+               return;
+       }
        element_size = mono_array_element_size (dest->obj.vtable->klass);
          
        /* no references should be involved */
@@ -11783,8 +9963,6 @@ ves_icall_System_Runtime_InteropServices_Marshal_copy_from_unmanaged (gpointer s
 MonoString *
 ves_icall_System_Runtime_InteropServices_Marshal_PtrToStringAnsi (char *ptr)
 {
-       MONO_ARCH_SAVE_REGS;
-
        if (ptr == NULL)
                return NULL;
        else
@@ -11794,11 +9972,8 @@ ves_icall_System_Runtime_InteropServices_Marshal_PtrToStringAnsi (char *ptr)
 MonoString *
 ves_icall_System_Runtime_InteropServices_Marshal_PtrToStringAnsi_len (char *ptr, gint32 len)
 {
-       MONO_ARCH_SAVE_REGS;
-
        if (ptr == NULL) {
-               mono_raise_exception (mono_get_exception_argument_null ("ptr"));
-               g_assert_not_reached ();
+               mono_set_pending_exception (mono_get_exception_argument_null ("ptr"));
                return NULL;
        } else {
                return mono_string_new_len (mono_domain_get (), ptr, len);
@@ -11812,8 +9987,6 @@ ves_icall_System_Runtime_InteropServices_Marshal_PtrToStringUni (guint16 *ptr)
        int len = 0;
        guint16 *t = ptr;
 
-       MONO_ARCH_SAVE_REGS;
-
        if (ptr == NULL)
                return NULL;
 
@@ -11828,11 +10001,8 @@ ves_icall_System_Runtime_InteropServices_Marshal_PtrToStringUni_len (guint16 *pt
 {
        MonoDomain *domain = mono_domain_get (); 
 
-       MONO_ARCH_SAVE_REGS;
-
        if (ptr == NULL) {
-               mono_raise_exception (mono_get_exception_argument_null ("ptr"));
-               g_assert_not_reached ();
+               mono_set_pending_exception (mono_get_exception_argument_null ("ptr"));
                return NULL;
        } else {
                return mono_string_new_utf16 (domain, ptr, len);
@@ -11842,8 +10012,6 @@ ves_icall_System_Runtime_InteropServices_Marshal_PtrToStringUni_len (guint16 *pt
 guint32 
 ves_icall_System_Runtime_InteropServices_Marshal_GetLastWin32Error (void)
 {
-       MONO_ARCH_SAVE_REGS;
-
        return (GPOINTER_TO_INT (mono_native_tls_get_value (last_error_tls_id)));
 }
 
@@ -11854,14 +10022,14 @@ ves_icall_System_Runtime_InteropServices_Marshal_SizeOf (MonoReflectionType *rty
        MonoType *type;
        guint32 layout;
 
-       MONO_ARCH_SAVE_REGS;
-
-       MONO_CHECK_ARG_NULL (rtype);
+       MONO_CHECK_ARG_NULL (rtype, 0);
 
        type = rtype->type;
        klass = mono_class_from_mono_type (type);
-       if (!mono_class_init (klass))
-               mono_raise_exception (mono_class_get_exception_for_failure (klass));
+       if (!mono_class_init (klass)) {
+               mono_set_pending_exception (mono_class_get_exception_for_failure (klass));
+               return 0;
+       }
 
        layout = (klass->flags & TYPE_ATTRIBUTE_LAYOUT_MASK);
 
@@ -11874,10 +10042,10 @@ ves_icall_System_Runtime_InteropServices_Marshal_SizeOf (MonoReflectionType *rty
                msg = g_strdup_printf ("Type %s cannot be marshaled as an unmanaged structure.", klass->name);
                exc = mono_get_exception_argument ("t", msg);
                g_free (msg);
-               mono_raise_exception (exc);
+               mono_set_pending_exception (exc);
+               return 0;
        }
 
-
        return mono_class_native_size (klass, NULL);
 }
 
@@ -11887,10 +10055,8 @@ ves_icall_System_Runtime_InteropServices_Marshal_StructureToPtr (MonoObject *obj
        MonoMethod *method;
        gpointer pa [3];
 
-       MONO_ARCH_SAVE_REGS;
-
-       MONO_CHECK_ARG_NULL (obj);
-       MONO_CHECK_ARG_NULL (dst);
+       MONO_CHECK_ARG_NULL (obj,);
+       MONO_CHECK_ARG_NULL (dst,);
 
        method = mono_marshal_get_struct_to_ptr (obj->vtable->klass);
 
@@ -11920,10 +10086,8 @@ ves_icall_System_Runtime_InteropServices_Marshal_PtrToStructure (gpointer src, M
 {
        MonoType *t;
 
-       MONO_ARCH_SAVE_REGS;
-
-       MONO_CHECK_ARG_NULL (src);
-       MONO_CHECK_ARG_NULL (dst);
+       MONO_CHECK_ARG_NULL (src,);
+       MONO_CHECK_ARG_NULL (dst,);
        
        t = mono_type_get_underlying_type (mono_class_get_type (dst->vtable->klass));
 
@@ -11935,7 +10099,7 @@ ves_icall_System_Runtime_InteropServices_Marshal_PtrToStructure (gpointer src, M
                exc = mono_get_exception_argument ("dst", tmp);
                g_free (tmp);  
 
-               mono_raise_exception (exc);
+               mono_set_pending_exception (exc);
                return;
        }
 
@@ -11949,15 +10113,15 @@ ves_icall_System_Runtime_InteropServices_Marshal_PtrToStructure_type (gpointer s
        MonoDomain *domain = mono_domain_get (); 
        MonoObject *res;
 
-       MONO_ARCH_SAVE_REGS;
-
        if (src == NULL)
                return NULL;
-       MONO_CHECK_ARG_NULL (type);
+       MONO_CHECK_ARG_NULL (type, NULL);
 
        klass = mono_class_from_mono_type (type->type);
-       if (!mono_class_init (klass))
-               mono_raise_exception (mono_class_get_exception_for_failure (klass));
+       if (!mono_class_init (klass)) {
+               mono_set_pending_exception (mono_class_get_exception_for_failure (klass));
+               return NULL;
+       }
 
        res = mono_object_new (domain, klass);
 
@@ -11974,15 +10138,15 @@ ves_icall_System_Runtime_InteropServices_Marshal_OffsetOf (MonoReflectionType *t
        char *fname;
        int match_index = -1;
        
-       MONO_ARCH_SAVE_REGS;
-
-       MONO_CHECK_ARG_NULL (type);
-       MONO_CHECK_ARG_NULL (field_name);
+       MONO_CHECK_ARG_NULL (type, 0);
+       MONO_CHECK_ARG_NULL (field_name, 0);
 
        fname = mono_string_to_utf8 (field_name);
        klass = mono_class_from_mono_type (type->type);
-       if (!mono_class_init (klass))
-               mono_raise_exception (mono_class_get_exception_for_failure (klass));
+       if (!mono_class_init (klass)) {
+               mono_set_pending_exception (mono_class_get_exception_for_failure (klass));
+               return 0;
+       }
 
        while (klass && match_index == -1) {
                MonoClassField* field;
@@ -12015,7 +10179,8 @@ ves_icall_System_Runtime_InteropServices_Marshal_OffsetOf (MonoReflectionType *t
                exc = mono_get_exception_argument ("fieldName", tmp);
                g_free (tmp);
  
-               mono_raise_exception ((MonoException*)exc);
+               mono_set_pending_exception ((MonoException*)exc);
+               return 0;
        }
 
        info = mono_marshal_load_type_info (klass);     
@@ -12046,8 +10211,6 @@ ves_icall_System_Runtime_InteropServices_Marshal_StringToHGlobalAnsi (MonoString
 gpointer
 ves_icall_System_Runtime_InteropServices_Marshal_StringToHGlobalUni (MonoString *string)
 {
-       MONO_ARCH_SAVE_REGS;
-
        if (string == NULL)
                return NULL;
        else {
@@ -12072,7 +10235,6 @@ mono_struct_delete_old (MonoClass *klass, char *ptr)
        info = mono_marshal_load_type_info (klass);
 
        for (i = 0; i < info->num_fields; i++) {
-               MonoMarshalNative ntype;
                MonoMarshalConv conv;
                MonoType *ftype = info->fields [i].field->type;
                char *cpos;
@@ -12080,8 +10242,8 @@ mono_struct_delete_old (MonoClass *klass, char *ptr)
                if (ftype->attrs & FIELD_ATTRIBUTE_STATIC)
                        continue;
 
-               ntype = mono_type_to_unmanaged (ftype, info->fields [i].mspec, TRUE, 
-                                               klass->unicode, &conv);
+               mono_type_to_unmanaged (ftype, info->fields [i].mspec, TRUE, 
+                               klass->unicode, &conv);
                        
                cpos = ptr + info->fields [i].offset;
 
@@ -12119,14 +10281,14 @@ ves_icall_System_Runtime_InteropServices_Marshal_DestroyStructure (gpointer src,
 {
        MonoClass *klass;
 
-       MONO_ARCH_SAVE_REGS;
-
-       MONO_CHECK_ARG_NULL (src);
-       MONO_CHECK_ARG_NULL (type);
+       MONO_CHECK_ARG_NULL (src,);
+       MONO_CHECK_ARG_NULL (type,);
 
        klass = mono_class_from_mono_type (type->type);
-       if (!mono_class_init (klass))
-               mono_raise_exception (mono_class_get_exception_for_failure (klass));
+       if (!mono_class_init (klass)) {
+               mono_set_pending_exception (mono_class_get_exception_for_failure (klass));
+               return;
+       }
 
        mono_struct_delete_old (klass, (char *)src);
 }
@@ -12136,8 +10298,6 @@ ves_icall_System_Runtime_InteropServices_Marshal_AllocHGlobal (int size)
 {
        gpointer res;
 
-       MONO_ARCH_SAVE_REGS;
-
        if ((gulong)size == 0)
                /* This returns a valid pointer for size 0 on MS.NET */
                size = 4;
@@ -12177,8 +10337,6 @@ ves_icall_System_Runtime_InteropServices_Marshal_ReAllocHGlobal (gpointer ptr, i
 void
 ves_icall_System_Runtime_InteropServices_Marshal_FreeHGlobal (void *ptr)
 {
-       MONO_ARCH_SAVE_REGS;
-
 #ifdef HOST_WIN32
        GlobalFree (ptr);
 #else
@@ -12189,20 +10347,25 @@ ves_icall_System_Runtime_InteropServices_Marshal_FreeHGlobal (void *ptr)
 void*
 ves_icall_System_Runtime_InteropServices_Marshal_AllocCoTaskMem (int size)
 {
-       MONO_ARCH_SAVE_REGS;
+       void *res;
 
 #ifdef HOST_WIN32
-       return CoTaskMemAlloc (size);
+       res = CoTaskMemAlloc (size);
 #else
-       return g_try_malloc ((gulong)size);
+       if ((gulong)size == 0)
+               /* This returns a valid pointer for size 0 on MS.NET */
+               size = 4;
+
+       res = g_try_malloc ((gulong)size);
 #endif
+       if (!res)
+               mono_gc_out_of_memory ((gulong)size);
+       return res;
 }
 
 void
 ves_icall_System_Runtime_InteropServices_Marshal_FreeCoTaskMem (void *ptr)
 {
-       MONO_ARCH_SAVE_REGS;
-
 #ifdef HOST_WIN32
        CoTaskMemFree (ptr);
 #else
@@ -12213,13 +10376,16 @@ ves_icall_System_Runtime_InteropServices_Marshal_FreeCoTaskMem (void *ptr)
 gpointer
 ves_icall_System_Runtime_InteropServices_Marshal_ReAllocCoTaskMem (gpointer ptr, int size)
 {
-       MONO_ARCH_SAVE_REGS;
+       void *res;
 
 #ifdef HOST_WIN32
-       return CoTaskMemRealloc (ptr, size);
+       res = CoTaskMemRealloc (ptr, size);
 #else
-       return g_try_realloc (ptr, (gulong)size);
+       res = g_try_realloc (ptr, (gulong)size);
 #endif
+       if (!res)
+               mono_gc_out_of_memory ((gulong)size);
+       return res;
 }
 
 void*
@@ -12232,8 +10398,10 @@ MonoDelegate*
 ves_icall_System_Runtime_InteropServices_Marshal_GetDelegateForFunctionPointerInternal (void *ftn, MonoReflectionType *type)
 {
        MonoClass *klass = mono_type_get_class (type->type);
-       if (!mono_class_init (klass))
-               mono_raise_exception (mono_class_get_exception_for_failure (klass));
+       if (!mono_class_init (klass)) {
+               mono_set_pending_exception (mono_class_get_exception_for_failure (klass));
+               return NULL;
+       }
 
        return mono_ftnptr_to_delegate (klass, ftn);
 }
@@ -12683,10 +10851,10 @@ mono_marshal_asany (MonoObject *o, MonoMarshalNative string_encoding, int param_
 
                return res;
        }
+       default:
+               break;
        }
-
        mono_raise_exception (mono_get_exception_argument ("", "No PInvoke conversion exists for value passed to Object-typed parameter."));
-
        return NULL;
 }
 
@@ -13007,7 +11175,7 @@ mono_marshal_free_dynamic_wrappers (MonoMethod *method)
        if (image->runtime_invoke_direct_cache)
                g_hash_table_remove (image->runtime_invoke_direct_cache, method);
        if (image->delegate_abstract_invoke_cache)
-               g_hash_table_foreach_remove (image->delegate_abstract_invoke_cache, signature_method_pair_matches_method, method);
+               g_hash_table_foreach_remove (image->delegate_abstract_invoke_cache, signature_pointer_pair_matches_pointer, method);
        // FIXME: Need to clear the caches in other images as well
        if (image->delegate_bound_static_invoke_cache)
                g_hash_table_remove (image->delegate_bound_static_invoke_cache, mono_method_signature (method));
@@ -13017,9 +11185,9 @@ mono_marshal_free_dynamic_wrappers (MonoMethod *method)
 }
 
 static gboolean
-signature_method_pair_matches_signature (gpointer key, gpointer value, gpointer user_data)
+signature_pointer_pair_matches_signature (gpointer key, gpointer value, gpointer user_data)
 {
-       SignatureMethodPair *pair = (SignatureMethodPair*)key;
+       SignaturePointerPair *pair = (SignaturePointerPair*)key;
        MonoMethodSignature *sig = (MonoMethodSignature*)user_data;
 
        return mono_metadata_signature_equal (pair->sig, sig);
@@ -13063,11 +11231,11 @@ mono_marshal_free_inflated_wrappers (MonoMethod *method)
                g_hash_table_remove (method->klass->image->runtime_invoke_vtype_cache, sig);
 
         /*
-         * indexed by SignatureMethodPair
+         * indexed by SignaturePointerPair
          */
        if (sig && method->klass->image->delegate_abstract_invoke_cache)
                g_hash_table_foreach_remove (method->klass->image->delegate_abstract_invoke_cache,
-                                            signature_method_pair_matches_signature, (gpointer)sig);
+                                            signature_pointer_pair_matches_signature, (gpointer)sig);
 
         /*
          * indexed by MonoMethod pointers