2010-03-02 Rodrigo Kumpera <rkumpera@novell.com>
[mono.git] / mono / metadata / reflection.c
index adf1076f0c9e9ac55d9e01fc723a0f89afb0bcb9..f177a8a38087135337369d2196f1492c726d3815 100644 (file)
@@ -165,10 +165,11 @@ static gpointer resolve_object (MonoImage *image, MonoObject *obj, MonoClass **h
 static guint32 mono_image_get_methodref_token_for_methodbuilder (MonoDynamicImage *assembly, MonoReflectionMethodBuilder *method);
 static guint32 encode_generic_method_sig (MonoDynamicImage *assembly, MonoGenericContext *context);
 static gpointer register_assembly (MonoDomain *domain, MonoReflectionAssembly *res, MonoAssembly *assembly);
-#endif
-
 static void reflection_methodbuilder_from_method_builder (ReflectionMethodBuilder *rmb, MonoReflectionMethodBuilder *mb);
 static void reflection_methodbuilder_from_ctor_builder (ReflectionMethodBuilder *rmb, MonoReflectionCtorBuilder *mb);
+static guint32 create_generic_typespec (MonoDynamicImage *assembly, MonoReflectionTypeBuilder *tb);
+#endif
+
 static guint32 mono_image_typedef_or_ref (MonoDynamicImage *assembly, MonoType *type);
 static guint32 mono_image_typedef_or_ref_full (MonoDynamicImage *assembly, MonoType *type, gboolean try_typespec);
 static void    mono_image_get_generic_param_info (MonoReflectionGenericParam *gparam, guint32 owner, MonoDynamicImage *assembly);
@@ -177,16 +178,28 @@ static guint32 encode_constant (MonoDynamicImage *assembly, MonoObject *val, gui
 static char*   type_get_qualified_name (MonoType *type, MonoAssembly *ass);
 static void    encode_type (MonoDynamicImage *assembly, MonoType *type, SigBuffer *buf);
 static void get_default_param_value_blobs (MonoMethod *method, char **blobs, guint32 *types);
-static MonoObject *mono_get_object_from_blob (MonoDomain *domain, MonoType *type, const char *blob);
 static MonoReflectionType *mono_reflection_type_get_underlying_system_type (MonoReflectionType* t);
 static MonoType* mono_reflection_get_type_with_rootimage (MonoImage *rootimage, MonoImage* image, MonoTypeNameParse *info, gboolean ignorecase, gboolean *type_resolve);
 static MonoReflectionType* mono_reflection_type_resolve_user_types (MonoReflectionType *type);
 static gboolean is_sre_array (MonoClass *class);
 static gboolean is_sre_byref (MonoClass *class);
 static gboolean is_sre_pointer (MonoClass *class);
+static gboolean is_sre_type_builder (MonoClass *class);
+static gboolean is_sre_method_builder (MonoClass *class);
+static gboolean is_sre_ctor_builder (MonoClass *class);
+static gboolean is_sre_field_builder (MonoClass *class);
+static gboolean is_sr_mono_method (MonoClass *class);
+static gboolean is_sr_mono_cmethod (MonoClass *class);
+static gboolean is_sr_mono_generic_method (MonoClass *class);
+static gboolean is_sr_mono_generic_cmethod (MonoClass *class);
+static gboolean is_sr_mono_field (MonoClass *class);
+static gboolean is_sr_mono_property (MonoClass *class);
+static gboolean is_sre_method_on_tb_inst (MonoClass *class);
+static gboolean is_sre_ctor_on_tb_inst (MonoClass *class);
+
 static guint32 mono_image_get_methodspec_token (MonoDynamicImage *assembly, MonoMethod *method);
 static guint32 mono_image_get_inflated_method_token (MonoDynamicImage *assembly, MonoMethod *m);
-static MonoMethod * inflate_method (MonoReflectionGenericClass *type, MonoObject *obj);
+static MonoMethod * inflate_method (MonoReflectionType *type, MonoObject *obj);
 
 #define RESOLVE_TYPE(type) do { type = (void*)mono_reflection_type_resolve_user_types ((MonoReflectionType*)type); } while (0)
 #define RESOLVE_ARRAY_TYPE_ELEMENT(array, index) do {  \
@@ -602,6 +615,36 @@ default_class_from_mono_type (MonoType *type)
 }
 #endif
 
+/*
+ * mono_class_get_ref_info:
+ *
+ *   Return the type builder/generic param builder corresponding to KLASS, if it exists.
+ */
+gpointer
+mono_class_get_ref_info (MonoClass *klass)
+{
+       if (klass->ref_info_handle == 0)
+               return NULL;
+       else
+               return mono_gchandle_get_target (klass->ref_info_handle);
+}
+
+void
+mono_class_set_ref_info (MonoClass *klass, gpointer obj)
+{
+       klass->ref_info_handle = mono_gchandle_new ((MonoObject*)obj, FALSE);
+       g_assert (klass->ref_info_handle != 0);
+}
+
+void
+mono_class_free_ref_info (MonoClass *klass)
+{
+       if (klass->ref_info_handle) {
+               mono_gchandle_free (klass->ref_info_handle);
+               klass->ref_info_handle = 0;
+       }
+}
+
 static void
 encode_generic_class (MonoDynamicImage *assembly, MonoGenericClass *gclass, SigBuffer *buf)
 {
@@ -1394,6 +1437,7 @@ mono_image_basic_method (ReflectionMethodBuilder *mb, MonoDynamicImage *assembly
        }
 }
 
+#ifndef DISABLE_REFLECTION_EMIT
 static void
 reflection_methodbuilder_from_method_builder (ReflectionMethodBuilder *rmb, MonoReflectionMethodBuilder *mb)
 {
@@ -1466,7 +1510,6 @@ reflection_methodbuilder_from_ctor_builder (ReflectionMethodBuilder *rmb, MonoRe
        rmb->refs = NULL;
 }
 
-#ifndef DISABLE_REFLECTION_EMIT
 static void
 reflection_methodbuilder_from_dynamic_method (ReflectionMethodBuilder *rmb, MonoReflectionDynamicMethod *mb)
 {
@@ -1531,6 +1574,7 @@ mono_image_add_methodimpl (MonoDynamicImage *assembly, MonoReflectionMethodBuild
        values [MONO_METHODIMPL_DECLARATION] = tok;
 }
 
+#ifndef DISABLE_REFLECTION_EMIT
 static void
 mono_image_get_method_info (MonoReflectionMethodBuilder *mb, MonoDynamicImage *assembly)
 {
@@ -1594,6 +1638,7 @@ mono_image_get_ctor_info (MonoDomain *domain, MonoReflectionCtorBuilder *mb, Mon
        mono_image_basic_method (&rmb, assembly);
        mb->table_idx = *rmb.table_idx;
 }
+#endif
 
 static char*
 type_get_fully_qualified_name (MonoType *type)
@@ -1974,6 +2019,7 @@ mono_image_get_property_info (MonoReflectionPropertyBuilder *pb, MonoDynamicImag
         * PROPERTY    (rows already preallocated in _get_type_info ())
         * METHOD      (method info already done with the generic method code)
         * METHODSEMANTICS
+        * CONSTANT
         */
        table = &assembly->tables [MONO_TABLE_PROPERTY];
        pb->table_idx = table->next_idx ++;
@@ -2004,6 +2050,17 @@ mono_image_get_property_info (MonoReflectionPropertyBuilder *pb, MonoDynamicImag
                values [MONO_METHOD_SEMA_METHOD] = pb->set_method->table_idx;
                values [MONO_METHOD_SEMA_ASSOCIATION] = (pb->table_idx << MONO_HAS_SEMANTICS_BITS) | MONO_HAS_SEMANTICS_PROPERTY;
        }
+       if (pb->attrs & PROPERTY_ATTRIBUTE_HAS_DEFAULT) {
+               guint32 field_type = 0;
+               table = &assembly->tables [MONO_TABLE_CONSTANT];
+               table->rows ++;
+               alloc_table (table, table->rows);
+               values = table->values + table->rows * MONO_CONSTANT_SIZE;
+               values [MONO_CONSTANT_PARENT] = MONO_HASCONSTANT_PROPERTY | (pb->table_idx << MONO_HASCONSTANT_BITS);
+               values [MONO_CONSTANT_VALUE] = encode_constant (assembly, pb->def_value, &field_type);
+               values [MONO_CONSTANT_TYPE] = field_type;
+               values [MONO_CONSTANT_PADDING] = 0;
+       }
 }
 
 static void
@@ -2117,7 +2174,7 @@ mono_image_get_generic_param_info (MonoReflectionGenericParam *gparam, guint32 o
        /* FIXME: track where gen_params should be freed and remove the GC root as well */
        MOVING_GC_REGISTER (&entry->gparam);
        entry->gparam = gparam;
-
+       
        g_ptr_array_add (assembly->gen_params, entry);
 }
 
@@ -2290,9 +2347,9 @@ mono_image_typedef_or_ref_full (MonoDynamicImage *assembly, MonoType *type, gboo
         */
        if ((klass->image == &assembly->image) && (type->type != MONO_TYPE_VAR) && 
                        (type->type != MONO_TYPE_MVAR)) {
-               MonoReflectionTypeBuilder *tb = klass->reflection_info;
+               MonoReflectionTypeBuilder *tb = mono_class_get_ref_info (klass);
                token = MONO_TYPEDEFORREF_TYPEDEF | (tb->table_idx << MONO_TYPEDEFORREF_BITS);
-               mono_g_hash_table_insert (assembly->tokens, GUINT_TO_POINTER (token), klass->reflection_info);
+               mono_g_hash_table_insert (assembly->tokens, GUINT_TO_POINTER (token), mono_class_get_ref_info (klass));
                return token;
        }
 
@@ -2315,7 +2372,7 @@ mono_image_typedef_or_ref_full (MonoDynamicImage *assembly, MonoType *type, gboo
        token = MONO_TYPEDEFORREF_TYPEREF | (table->next_idx << MONO_TYPEDEFORREF_BITS); /* typeref */
        g_hash_table_insert (assembly->typeref, type, GUINT_TO_POINTER(token));
        table->next_idx ++;
-       mono_g_hash_table_insert (assembly->tokens, GUINT_TO_POINTER (token), klass->reflection_info);
+       mono_g_hash_table_insert (assembly->tokens, GUINT_TO_POINTER (token), mono_class_get_ref_info (klass));
        return token;
 }
 
@@ -2329,21 +2386,13 @@ mono_image_typedef_or_ref (MonoDynamicImage *assembly, MonoType *type)
 }
 
 #ifndef DISABLE_REFLECTION_EMIT
-/*
- * Insert a memberef row into the metadata: the token that point to the memberref
- * is returned. Caching is done in the caller (mono_image_get_methodref_token() or
- * mono_image_get_fieldref_token()).
- * The sig param is an index to an already built signature.
- */
 static guint32
-mono_image_get_memberref_token (MonoDynamicImage *assembly, MonoType *type, const char *name, guint32 sig)
+mono_image_add_memberef_row (MonoDynamicImage *assembly, guint32 parent, const char *name, guint32 sig)
 {
        MonoDynamicTable *table;
        guint32 *values;
        guint32 token, pclass;
-       guint32 parent;
 
-       parent = mono_image_typedef_or_ref (assembly, type);
        switch (parent & MONO_TYPEDEFORREF_MASK) {
        case MONO_TYPEDEFORREF_TYPEREF:
                pclass = MONO_MEMBERREF_PARENT_TYPEREF;
@@ -2377,6 +2426,20 @@ mono_image_get_memberref_token (MonoDynamicImage *assembly, MonoType *type, cons
        return token;
 }
 
+/*
+ * Insert a memberef row into the metadata: the token that point to the memberref
+ * is returned. Caching is done in the caller (mono_image_get_methodref_token() or
+ * mono_image_get_fieldref_token()).
+ * The sig param is an index to an already built signature.
+ */
+static guint32
+mono_image_get_memberref_token (MonoDynamicImage *assembly, MonoType *type, const char *name, guint32 sig)
+{
+       guint32 parent = mono_image_typedef_or_ref (assembly, type);
+       return mono_image_add_memberef_row (assembly, parent, name, sig);
+}
+
+
 static guint32
 mono_image_get_methodref_token (MonoDynamicImage *assembly, MonoMethod *method, gboolean create_typespec)
 {
@@ -2435,9 +2498,10 @@ mono_image_get_methodref_token (MonoDynamicImage *assembly, MonoMethod *method,
 static guint32
 mono_image_get_methodref_token_for_methodbuilder (MonoDynamicImage *assembly, MonoReflectionMethodBuilder *method)
 {
-       guint32 token;
+       guint32 token, parent, sig;
        ReflectionMethodBuilder rmb;
        char *name;
+       MonoReflectionTypeBuilder *tb = (MonoReflectionTypeBuilder *)method->type;
        
        token = GPOINTER_TO_UINT (g_hash_table_lookup (assembly->handleref, method));
        if (token)
@@ -2452,8 +2516,15 @@ mono_image_get_methodref_token_for_methodbuilder (MonoDynamicImage *assembly, Mo
        */
        if ((rmb.call_conv & ~0x60) != MONO_CALL_DEFAULT && (rmb.call_conv & ~0x60) != MONO_CALL_VARARG)
                rmb.call_conv = (rmb.call_conv & 0x60) | MONO_CALL_DEFAULT;
-       token = mono_image_get_memberref_token (assembly, mono_reflection_type_get_handle ((MonoReflectionType*)rmb.type),
-                                       name, method_builder_encode_signature (assembly, &rmb));
+
+       sig = method_builder_encode_signature (assembly, &rmb);
+
+       if (tb->generic_params)
+               parent = create_generic_typespec (assembly, tb);
+       else
+               parent = mono_image_typedef_or_ref (assembly, mono_reflection_type_get_handle ((MonoReflectionType*)rmb.type));
+
+       token = mono_image_add_memberef_row (assembly, parent, name, sig);
 
        g_free (name);
        g_hash_table_insert (assembly->handleref, method, GUINT_TO_POINTER(token));
@@ -2553,38 +2624,43 @@ static guint32
 mono_image_get_methodbuilder_token (MonoDynamicImage *assembly, MonoReflectionMethodBuilder *mb, gboolean create_methodspec)
 {
        guint32 token;
-       
+
        if (mb->generic_params && create_methodspec) 
                return mono_image_get_methodspec_token_for_generic_method_definition (assembly, mb);
 
-       token = GPOINTER_TO_UINT (g_hash_table_lookup (assembly->handleref, mb));
+       token = GPOINTER_TO_UINT (mono_g_hash_table_lookup (assembly->handleref_managed, mb));
        if (token)
                return token;
 
        token = mono_image_get_methodref_token_for_methodbuilder (assembly, mb);
-       g_hash_table_insert (assembly->handleref, mb, GUINT_TO_POINTER(token));
+       mono_g_hash_table_insert (assembly->handleref_managed, mb, GUINT_TO_POINTER(token));
        return token;
 }
 
 static guint32
 mono_image_get_ctorbuilder_token (MonoDynamicImage *assembly, MonoReflectionCtorBuilder *mb)
 {
-       guint32 token;
+       guint32 token, parent, sig;
        ReflectionMethodBuilder rmb;
        char *name;
+       MonoReflectionTypeBuilder *tb = (MonoReflectionTypeBuilder *)mb->type;
        
-       token = GPOINTER_TO_UINT (g_hash_table_lookup (assembly->handleref, mb));
+       token = GPOINTER_TO_UINT (mono_g_hash_table_lookup (assembly->handleref_managed, mb));
        if (token)
                return token;
 
+       g_assert (tb->generic_params);
+
        reflection_methodbuilder_from_ctor_builder (&rmb, mb);
 
+       parent = create_generic_typespec (assembly, tb);
        name = mono_string_to_utf8 (rmb.name);
-       token = mono_image_get_memberref_token (assembly, mono_reflection_type_get_handle ((MonoReflectionType*)rmb.type),
-               name, method_builder_encode_signature (assembly, &rmb));
+       sig = method_builder_encode_signature (assembly, &rmb);
+
+       token = mono_image_add_memberef_row (assembly, parent, name, sig);
 
        g_free (name);
-       g_hash_table_insert (assembly->handleref, mb, GUINT_TO_POINTER(token));
+       mono_g_hash_table_insert (assembly->handleref_managed, mb, GUINT_TO_POINTER(token));
        return token;
 }
 #endif
@@ -2601,16 +2677,29 @@ is_field_on_inst (MonoClassField *field)
 static MonoType*
 get_field_on_inst_generic_type (MonoClassField *field)
 {
+       MonoClass *class, *gtd;
        MonoDynamicGenericClass *dgclass;
        int field_index;
 
        g_assert (is_field_on_inst (field));
 
        dgclass = (MonoDynamicGenericClass*)field->parent->generic_class;
-       field_index = field - dgclass->fields;
 
-       g_assert (field_index >= 0 && field_index < dgclass->count_fields);
-       return dgclass->field_generic_types [field_index];
+       if (field >= dgclass->fields && field - dgclass->fields < dgclass->count_fields) {
+               field_index = field - dgclass->fields;
+               return dgclass->field_generic_types [field_index];              
+       }
+
+       class = field->parent;
+       gtd = class->generic_class->container_class;
+
+       if (field >= class->fields && field - class->fields < class->field.count) {
+               field_index = field - class->fields;
+               return gtd->fields [field_index].type;
+       }
+
+       g_assert_not_reached ();
+       return 0;
 }
 
 #ifndef DISABLE_REFLECTION_EMIT
@@ -2621,7 +2710,7 @@ mono_image_get_fieldref_token (MonoDynamicImage *assembly, MonoReflectionField *
        guint32 token;
        MonoClassField *field;
 
-       token = GPOINTER_TO_UINT (g_hash_table_lookup (assembly->handleref, f));
+       token = GPOINTER_TO_UINT (mono_g_hash_table_lookup (assembly->handleref_managed, f));
        if (token)
                return token;
        g_assert (f->field->parent);
@@ -2639,7 +2728,7 @@ mono_image_get_fieldref_token (MonoDynamicImage *assembly, MonoReflectionField *
        token = mono_image_get_memberref_token (assembly, &f->field->parent->byval_arg, 
                                                                                        mono_field_get_name (f->field),  
                                                                                        fieldref_encode_signature (assembly, field->parent->image, type));
-       g_hash_table_insert (assembly->handleref, f, GUINT_TO_POINTER(token));
+       mono_g_hash_table_insert (assembly->handleref_managed, f, GUINT_TO_POINTER(token));
        return token;
 }
 
@@ -2650,24 +2739,39 @@ mono_image_get_field_on_inst_token (MonoDynamicImage *assembly, MonoReflectionFi
        MonoClass *klass;
        MonoGenericClass *gclass;
        MonoDynamicGenericClass *dgclass;
-       MonoReflectionFieldBuilder *fb = f->fb;
        MonoType *type;
        char *name;
 
-       token = GPOINTER_TO_UINT (g_hash_table_lookup (assembly->handleref, f));
+       token = GPOINTER_TO_UINT (mono_g_hash_table_lookup (assembly->handleref_managed, f));
        if (token)
                return token;
-       type = mono_reflection_type_get_handle ((MonoReflectionType*)f->inst);
-       klass = mono_class_from_mono_type (type);
-       gclass = type->data.generic_class;
-       g_assert (gclass->is_dynamic);
-       dgclass = (MonoDynamicGenericClass *) gclass;
+       if (is_sre_field_builder (mono_object_class (f->fb))) {
+               MonoReflectionFieldBuilder *fb = (MonoReflectionFieldBuilder *)f->fb;
+               type = mono_reflection_type_get_handle ((MonoReflectionType*)f->inst);
+               klass = mono_class_from_mono_type (type);
+               gclass = type->data.generic_class;
+               g_assert (gclass->is_dynamic);
+               dgclass = (MonoDynamicGenericClass *) gclass;
+
+               name = mono_string_to_utf8 (fb->name);
+               token = mono_image_get_memberref_token (assembly, &klass->byval_arg, name, 
+                                                                                               field_encode_signature (assembly, fb));
+               g_free (name);          
+       } else if (is_sr_mono_field (mono_object_class (f->fb))) {
+               guint32 sig;
+               MonoClassField *field = ((MonoReflectionField *)f->fb)->field;
+
+               type = mono_reflection_type_get_handle ((MonoReflectionType*)f->inst);
+               klass = mono_class_from_mono_type (type);
 
-       name = mono_string_to_utf8 (fb->name);
-       token = mono_image_get_memberref_token (assembly, &klass->byval_arg, name, 
-                                                                                       field_encode_signature (assembly, fb));
-       g_free (name);
-       g_hash_table_insert (assembly->handleref, f, GUINT_TO_POINTER (token));
+               sig = fieldref_encode_signature (assembly, field->parent->image, field->type);
+               token = mono_image_get_memberref_token (assembly, &klass->byval_arg, field->name, sig);
+       } else {
+               char *name = mono_type_get_full_name (mono_object_class (f->fb));
+               g_error ("mono_image_get_method_on_inst_token: don't know how to handle %s", name);
+       }
+
+       mono_g_hash_table_insert (assembly->handleref_managed, f, GUINT_TO_POINTER (token));
        return token;
 }
 
@@ -2677,33 +2781,50 @@ mono_image_get_ctor_on_inst_token (MonoDynamicImage *assembly, MonoReflectionCto
        guint32 sig, token;
        MonoClass *klass;
        MonoGenericClass *gclass;
-       MonoDynamicGenericClass *dgclass;
-       MonoReflectionCtorBuilder *cb = c->cb;
-       ReflectionMethodBuilder rmb;
        MonoType *type;
-       char *name;
 
        /* A ctor cannot be a generic method, so we can ignore create_methodspec */
 
-       token = GPOINTER_TO_UINT (g_hash_table_lookup (assembly->handleref, c));
+       token = GPOINTER_TO_UINT (mono_g_hash_table_lookup (assembly->handleref_managed, c));
        if (token)
                return token;
-       type = mono_reflection_type_get_handle ((MonoReflectionType*)c->inst);
-       klass = mono_class_from_mono_type (type);
-       gclass = type->data.generic_class;
-       g_assert (gclass->is_dynamic);
-       dgclass = (MonoDynamicGenericClass *) gclass;
 
-       reflection_methodbuilder_from_ctor_builder (&rmb, cb);
+       if (is_sre_ctor_builder (mono_object_class (c->cb))) {
+               MonoReflectionCtorBuilder *cb = (MonoReflectionCtorBuilder *)c->cb;
+               MonoDynamicGenericClass *dgclass;
+               ReflectionMethodBuilder rmb;
+               char *name;
 
-       name = mono_string_to_utf8 (rmb.name);
+               type = mono_reflection_type_get_handle ((MonoReflectionType*)c->inst);
+               klass = mono_class_from_mono_type (type);
 
-       sig = method_builder_encode_signature (assembly, &rmb);
+               gclass = type->data.generic_class;
+               g_assert (gclass->is_dynamic);
+               dgclass = (MonoDynamicGenericClass *) gclass;
 
-       token = mono_image_get_memberref_token (assembly, &klass->byval_arg, name, sig);
-       g_free (name);
+               reflection_methodbuilder_from_ctor_builder (&rmb, cb);
 
-       g_hash_table_insert (assembly->handleref, c, GUINT_TO_POINTER (token));
+               name = mono_string_to_utf8 (rmb.name);
+
+               sig = method_builder_encode_signature (assembly, &rmb);
+
+               token = mono_image_get_memberref_token (assembly, &klass->byval_arg, name, sig);
+               g_free (name);
+       } else if (is_sr_mono_cmethod (mono_object_class (c->cb))) {
+               MonoMethod *mm = ((MonoReflectionMethod *)c->cb)->method;
+
+               type = mono_reflection_type_get_handle ((MonoReflectionType*)c->inst);
+               klass = mono_class_from_mono_type (type);
+
+               sig = method_encode_signature (assembly, mono_method_signature (mm));
+               token = mono_image_get_memberref_token (assembly, &klass->byval_arg, mm->name, sig);
+       } else {
+               char *name = mono_type_get_full_name (mono_object_class (c->cb));
+               g_error ("mono_image_get_method_on_inst_token: don't know how to handle %s", name);
+       }
+
+
+       mono_g_hash_table_insert (assembly->handleref_managed, c, GUINT_TO_POINTER (token));
        return token;
 }
 
@@ -2721,6 +2842,9 @@ mono_reflection_method_on_tb_inst_get_handle (MonoReflectionMethodOnTypeBuilderI
 
        klass = method->klass;
 
+       if (m->method_args == NULL)
+               return method;
+
        if (method->is_inflated)
                method = ((MonoMethodInflated *) method)->declaring;
 
@@ -2744,13 +2868,9 @@ mono_reflection_method_on_tb_inst_get_handle (MonoReflectionMethodOnTypeBuilderI
 static guint32
 mono_image_get_method_on_inst_token (MonoDynamicImage *assembly, MonoReflectionMethodOnTypeBuilderInst *m, gboolean create_methodspec)
 {
-       guint32 sig, token;
-       MonoClass *klass;
-       MonoGenericClass *gclass;
-       MonoReflectionMethodBuilder *mb = m->mb;
-       ReflectionMethodBuilder rmb;
+       guint32 sig, token = 0;
        MonoType *type;
-       char *name;
+       MonoClass *klass;
 
        if (m->method_args) {
                MonoMethod *inflated;
@@ -2763,24 +2883,43 @@ mono_image_get_method_on_inst_token (MonoDynamicImage *assembly, MonoReflectionM
                return token;
        }
 
-       token = GPOINTER_TO_UINT (g_hash_table_lookup (assembly->handleref, m));
+       token = GPOINTER_TO_UINT (mono_g_hash_table_lookup (assembly->handleref_managed, m));
        if (token)
                return token;
-       type = mono_reflection_type_get_handle ((MonoReflectionType*)m->inst);
-       klass = mono_class_from_mono_type (type);
-       gclass = type->data.generic_class;
-       g_assert (gclass->is_dynamic);
 
-       reflection_methodbuilder_from_method_builder (&rmb, mb);
+       if (is_sre_method_builder (mono_object_class (m->mb))) {
+               MonoReflectionMethodBuilder *mb = (MonoReflectionMethodBuilder *)m->mb;
+               MonoGenericClass *gclass;
+               ReflectionMethodBuilder rmb;
+               char *name;
 
-       name = mono_string_to_utf8 (rmb.name);
+               type = mono_reflection_type_get_handle ((MonoReflectionType*)m->inst);
+               klass = mono_class_from_mono_type (type);
+               gclass = type->data.generic_class;
+               g_assert (gclass->is_dynamic);
 
-       sig = method_builder_encode_signature (assembly, &rmb);
+               reflection_methodbuilder_from_method_builder (&rmb, mb);
 
-       token = mono_image_get_memberref_token (assembly, &klass->byval_arg, name, sig);
-       g_free (name);
+               name = mono_string_to_utf8 (rmb.name);
+
+               sig = method_builder_encode_signature (assembly, &rmb);
+
+               token = mono_image_get_memberref_token (assembly, &klass->byval_arg, name, sig);
+               g_free (name);          
+       } else if (is_sr_mono_method (mono_object_class (m->mb))) {
+               MonoMethod *mm = ((MonoReflectionMethod *)m->mb)->method;
+
+               type = mono_reflection_type_get_handle ((MonoReflectionType*)m->inst);
+               klass = mono_class_from_mono_type (type);
+
+               sig = method_encode_signature (assembly, mono_method_signature (mm));
+               token = mono_image_get_memberref_token (assembly, &klass->byval_arg, mm->name, sig);
+       } else {
+               char *name = mono_type_get_full_name (mono_object_class (m->mb));
+               g_error ("mono_image_get_method_on_inst_token: don't know how to handle %s", name);
+       }
 
-       g_hash_table_insert (assembly->handleref, m, GUINT_TO_POINTER (token));
+       mono_g_hash_table_insert (assembly->handleref_managed, m, GUINT_TO_POINTER (token));
        return token;
 }
 
@@ -3010,7 +3149,7 @@ mono_image_get_generic_field_token (MonoDynamicImage *assembly, MonoReflectionFi
        guint32 token, pclass, parent, sig;
        gchar *name;
 
-       token = GPOINTER_TO_UINT (g_hash_table_lookup (assembly->handleref, fb));
+       token = GPOINTER_TO_UINT (mono_g_hash_table_lookup (assembly->handleref_managed, fb));
        if (token)
                return token;
 
@@ -3045,7 +3184,7 @@ mono_image_get_generic_field_token (MonoDynamicImage *assembly, MonoReflectionFi
 
        token = MONO_TOKEN_MEMBER_REF | table->next_idx;
        table->next_idx ++;
-       g_hash_table_insert (assembly->handleref, fb, GUINT_TO_POINTER(token));
+       mono_g_hash_table_insert (assembly->handleref_managed, fb, GUINT_TO_POINTER(token));
        g_free (name);
        return token;
 }
@@ -3196,7 +3335,6 @@ mono_image_get_array_token (MonoDynamicImage *assembly, MonoReflectionArrayMetho
        m->table_idx = am->token & 0xffffff;
        return am->token;
 }
-#endif
 
 /*
  * Insert into the metadata tables all the info about the TypeBuilder tb.
@@ -3360,6 +3498,7 @@ mono_image_get_type_info (MonoDomain *domain, MonoReflectionTypeBuilder *tb, Mon
                }
        }
 }
+#endif
 
 static void
 collect_types (GPtrArray *types, MonoReflectionTypeBuilder *type)
@@ -4880,6 +5019,7 @@ create_dynamic_mono_image (MonoDynamicAssembly *assembly, char *assembly_name, c
        image->field_to_table_idx = g_hash_table_new (NULL, NULL);
        image->method_aux_hash = g_hash_table_new (NULL, NULL);
        image->handleref = g_hash_table_new (NULL, NULL);
+       image->handleref_managed = mono_g_hash_table_new_type ((GHashFunc)mono_object_hash, NULL, MONO_HASH_KEY_GC);
        image->tokens = mono_g_hash_table_new_type (NULL, NULL, MONO_HASH_VALUE_GC);
        image->generic_def_objects = mono_g_hash_table_new_type (NULL, NULL, MONO_HASH_VALUE_GC);
        image->methodspec = mono_g_hash_table_new_type ((GHashFunc)mono_object_hash, NULL, MONO_HASH_KEY_GC);
@@ -4942,6 +5082,8 @@ mono_dynamic_image_free (MonoDynamicImage *image)
                g_hash_table_destroy (di->typeref);
        if (di->handleref)
                g_hash_table_destroy (di->handleref);
+       if (di->handleref_managed)
+               mono_g_hash_table_destroy (di->handleref_managed);
        if (di->tokens)
                mono_g_hash_table_destroy (di->tokens);
        if (di->generic_def_objects)
@@ -5057,6 +5199,7 @@ mono_image_basic_init (MonoReflectionAssemblyBuilder *assemblyb)
 
        assembly->run = assemblyb->access != 2;
        assembly->save = assemblyb->access != 1;
+       assembly->domain = domain;
 
        image = create_dynamic_mono_image (assembly, mono_string_to_utf8 (assemblyb->name), g_strdup ("RefEmit_YouForgotToDefineAModule"));
        image->initial_image = TRUE;
@@ -6183,6 +6326,9 @@ mono_generic_class_get_object (MonoDomain *domain, MonoType *geninst)
        MonoGenericInst *ginst;
        MonoArray *type_args;
        int i;
+       MonoObject *tb;
+
+       g_assert (0); /*This code path should not be taken anymore, all MGC instantiation must happen in managed code*/
 
        if (!System_Reflection_MonoGenericClass) {
                System_Reflection_MonoGenericClass = mono_class_from_name (
@@ -6202,9 +6348,10 @@ mono_generic_class_get_object (MonoDomain *domain, MonoType *geninst)
 #endif
 
        res->type.type = geninst;
-       g_assert (gklass->reflection_info);
-       g_assert (!strcmp (((MonoObject*)gklass->reflection_info)->vtable->klass->name, "TypeBuilder"));
-       MONO_OBJECT_SETREF (res, generic_type, gklass->reflection_info);
+       tb = mono_class_get_ref_info (gklass);
+       g_assert (tb);
+       g_assert (!strcmp (tb->vtable->klass->name, "TypeBuilder"));
+       MONO_OBJECT_SETREF (res, generic_type, tb);
 
        ginst = klass->generic_class->context.class_inst;
        type_args = mono_array_new (domain, mono_defaults.systemtype_class, ginst->type_argc);
@@ -6306,13 +6453,32 @@ mono_type_get_object (MonoDomain *domain, MonoType *type)
                mono_raise_exception (mono_get_exception_invalid_operation ("This type cannot be propagated to managed space"));
        }
 
-       if (klass->reflection_info && !klass->wastypebuilder) {
+       if (mono_class_get_ref_info (klass) && !klass->wastypebuilder) {
+               gboolean is_type_done = TRUE;
+               /* Generic parameters have reflection_info set but they are not finished together with their enclosing type.
+                * We must ensure that once a type is finished we don't return a GenericTypeParameterBuilder.
+                * We can't simply close the types as this will interfere with other parts of the generics machinery.
+               */
+               if (klass->byval_arg.type == MONO_TYPE_MVAR || klass->byval_arg.type == MONO_TYPE_VAR) {
+                       MonoGenericParam *gparam = klass->byval_arg.data.generic_param;
+
+                       if (gparam->owner && gparam->owner->is_method) {
+                               MonoMethod *method = gparam->owner->owner.method;
+                               if (method && mono_class_get_generic_type_definition (method->klass)->wastypebuilder)
+                                       is_type_done = FALSE;
+                       } else if (gparam->owner && !gparam->owner->is_method) {
+                               MonoClass *klass = gparam->owner->owner.klass;
+                               if (klass && mono_class_get_generic_type_definition (klass)->wastypebuilder)
+                                       is_type_done = FALSE;
+                       }
+               } 
+
                /* g_assert_not_reached (); */
                /* should this be considered an error condition? */
-               if (!type->byref) {
+               if (is_type_done && !type->byref) {
                        mono_domain_unlock (domain);
                        mono_loader_unlock ();
-                       return klass->reflection_info;
+                       return mono_class_get_ref_info (klass);
                }
        }
        // FIXME: Get rid of this, do it in the icalls for Type
@@ -6864,7 +7030,7 @@ get_default_param_value_blobs (MonoMethod *method, char **blobs, guint32 *types)
        return;
 }
 
-static MonoObject *
+MonoObject *
 mono_get_object_from_blob (MonoDomain *domain, MonoType *type, const char *blob)
 {
        void *retval;
@@ -7337,11 +7503,12 @@ mono_reflection_get_type (MonoImage* image, MonoTypeNameParse *info, gboolean ig
 static MonoType*
 mono_reflection_get_type_internal_dynamic (MonoImage *rootimage, MonoAssembly *assembly, MonoTypeNameParse *info, gboolean ignorecase)
 {
-       MonoReflectionAssemblyBuilder *abuilder = (MonoReflectionAssemblyBuilder*)mono_assembly_get_object (mono_domain_get (), assembly);
+       MonoReflectionAssemblyBuilder *abuilder;
        MonoType *type;
        int i;
 
        g_assert (assembly->dynamic);
+       abuilder = (MonoReflectionAssemblyBuilder*)mono_assembly_get_object (((MonoDynamicAssembly*)assembly)->domain, assembly);
 
        /* Enumerate all modules */
 
@@ -7530,6 +7697,8 @@ mono_reflection_get_token (MonoObject *obj)
                token = mono_class_get_event_token (p->event);
        } else if (strcmp (klass->name, "ParameterInfo") == 0) {
                MonoReflectionParameter *p = (MonoReflectionParameter*)obj;
+               MonoClass *member_class = mono_object_class (p->MemberImpl);
+               g_assert (mono_class_is_reflection_method_or_constructor (member_class));
 
                token = mono_method_get_param_token (((MonoReflectionMethod*)p->MemberImpl)->method, p->PositionImpl);
        } else if (strcmp (klass->name, "Module") == 0) {
@@ -8426,6 +8595,8 @@ mono_custom_attrs_from_param (MonoMethod *method, guint32 param)
 
                /* Need to copy since it will be freed later */
                ainfo = aux->param_cattr [param];
+               if (!ainfo)
+                       return NULL;
                size = MONO_SIZEOF_CUSTOM_ATTR_INFO + sizeof (MonoCustomAttrEntry) * ainfo->num_attrs;
                res = g_malloc0 (size);
                memcpy (res, ainfo, size);
@@ -8541,8 +8712,40 @@ mono_reflection_get_custom_attrs_info (MonoObject *obj)
                cinfo = mono_custom_attrs_from_method (rmethod->method);
        } else if (strcmp ("ParameterInfo", klass->name) == 0) {
                MonoReflectionParameter *param = (MonoReflectionParameter*)obj;
-               MonoReflectionMethod *rmethod = (MonoReflectionMethod*)param->MemberImpl;
-               cinfo = mono_custom_attrs_from_param (rmethod->method, param->PositionImpl + 1);
+               MonoClass *member_class = mono_object_class (param->MemberImpl);
+               if (mono_class_is_reflection_method_or_constructor (member_class)) {
+                       MonoReflectionMethod *rmethod = (MonoReflectionMethod*)param->MemberImpl;
+                       cinfo = mono_custom_attrs_from_param (rmethod->method, param->PositionImpl + 1);
+               } else if (is_sr_mono_property (member_class)) {
+                       MonoReflectionProperty *prop = (MonoReflectionProperty *)param->MemberImpl;
+                       MonoMethod *method;
+                       if (!(method = prop->property->get))
+                               method = prop->property->set;
+                       g_assert (method);
+
+                       cinfo = mono_custom_attrs_from_param (method, param->PositionImpl + 1);
+               } else if (is_sre_method_on_tb_inst (member_class)) {/*XXX This is a workaround for Compiler Context*/
+                       MonoMethod *method = mono_reflection_method_on_tb_inst_get_handle ((MonoReflectionMethodOnTypeBuilderInst*)param->MemberImpl);
+                       cinfo = mono_custom_attrs_from_param (method, param->PositionImpl + 1);
+               } else if (is_sre_ctor_on_tb_inst (member_class)) { /*XX This is a workaround for Compiler Context*/
+               MonoReflectionCtorOnTypeBuilderInst *c = (MonoReflectionCtorOnTypeBuilderInst*)param->MemberImpl;
+                       MonoMethod *method = NULL;
+                       if (is_sre_ctor_builder (mono_object_class (c->cb)))
+                               method = ((MonoReflectionCtorBuilder *)c->cb)->mhandle;
+                       else if (is_sr_mono_cmethod (mono_object_class (c->cb)))
+                               method = ((MonoReflectionMethod *)c->cb)->method;
+                       else
+                               g_error ("mono_reflection_get_custom_attrs_info:: can't handle a CTBI with base_method of type %s", mono_type_get_full_name (member_class));
+
+                       cinfo = mono_custom_attrs_from_param (method, param->PositionImpl + 1);
+               } else {
+                       char *type_name = mono_type_get_full_name (member_class);
+                       char *msg = g_strdup_printf ("Custom attributes on a ParamInfo with member %s are not supported", type_name);
+                       MonoException *ex = mono_get_exception_not_supported  (msg);
+                       g_free (type_name);
+                       g_free (msg);
+                       mono_raise_exception (ex);
+               }
        } else if (strcmp ("AssemblyBuilder", klass->name) == 0) {
                MonoReflectionAssemblyBuilder *assemblyb = (MonoReflectionAssemblyBuilder*)obj;
                cinfo = mono_custom_attrs_from_builders (NULL, assemblyb->assembly.assembly->image, assemblyb->cattrs);
@@ -8697,6 +8900,84 @@ is_sre_generic_instance (MonoClass *class)
        check_corlib_type_cached (class, "System.Reflection", "MonoGenericClass");
 }
 
+static gboolean
+is_sre_type_builder (MonoClass *class)
+{
+       check_corlib_type_cached (class, "System.Reflection.Emit", "TypeBuilder");
+}
+
+static gboolean
+is_sre_method_builder (MonoClass *class)
+{
+       check_corlib_type_cached (class, "System.Reflection.Emit", "MethodBuilder");
+}
+
+static gboolean
+is_sre_ctor_builder (MonoClass *class)
+{
+       check_corlib_type_cached (class, "System.Reflection.Emit", "ConstructorBuilder");
+}
+
+static gboolean
+is_sre_field_builder (MonoClass *class)
+{
+       check_corlib_type_cached (class, "System.Reflection.Emit", "FieldBuilder");
+}
+
+static gboolean
+is_sr_mono_method (MonoClass *class)
+{
+       check_corlib_type_cached (class, "System.Reflection", "MonoMethod");
+}
+
+static gboolean
+is_sr_mono_cmethod (MonoClass *class)
+{
+       check_corlib_type_cached (class, "System.Reflection", "MonoCMethod");
+}
+
+static gboolean
+is_sr_mono_generic_method (MonoClass *class)
+{
+       check_corlib_type_cached (class, "System.Reflection", "MonoGenericMethod");
+}
+
+static gboolean
+is_sr_mono_generic_cmethod (MonoClass *class)
+{
+       check_corlib_type_cached (class, "System.Reflection", "MonoGenericCMethod");
+}
+
+static gboolean
+is_sr_mono_field (MonoClass *class)
+{
+       check_corlib_type_cached (class, "System.Reflection", "MonoField");
+}
+
+static gboolean
+is_sr_mono_property (MonoClass *class)
+{
+       check_corlib_type_cached (class, "System.Reflection", "MonoProperty");
+}
+
+static gboolean
+is_sre_method_on_tb_inst (MonoClass *class)
+{
+       check_corlib_type_cached (class, "System.Reflection.Emit", "MethodOnTypeBuilderInst");
+}
+
+static gboolean
+is_sre_ctor_on_tb_inst (MonoClass *class)
+{
+       check_corlib_type_cached (class, "System.Reflection.Emit", "ConstructorOnTypeBuilderInst");
+}
+
+gboolean
+mono_class_is_reflection_method_or_constructor (MonoClass *class)
+{
+       return is_sr_mono_method (class) || is_sr_mono_cmethod (class) || is_sr_mono_generic_method (class) || is_sr_mono_generic_cmethod (class);
+}
+
 MonoType*
 mono_reflection_type_get_handle (MonoReflectionType* ref)
 {
@@ -8708,7 +8989,8 @@ mono_reflection_type_get_handle (MonoReflectionType* ref)
 
        if (is_usertype (ref)) {
                ref = mono_reflection_type_get_underlying_system_type (ref);
-               g_assert (!is_usertype (ref)); /*FIXME fail better*/
+               if (ref == NULL || is_usertype (ref))
+                       return NULL;
                if (ref->type)
                        return ref->type;
        }
@@ -8752,9 +9034,13 @@ mono_reflection_type_get_handle (MonoReflectionType* ref)
                for (i = 0; i < count; ++i) {
                        MonoReflectionType *t = mono_array_get (gclass->type_arguments, gpointer, i);
                        types [i] = mono_reflection_type_get_handle (t);
+                       if (!types[i]) {
+                               g_free (types);
+                               return NULL;
+                       }
                }
 
-               res = mono_reflection_bind_generic_parameters ((MonoReflectionType*)gclass->generic_type, count, types);
+               res = mono_reflection_bind_generic_parameters (gclass->generic_type, count, types);
                g_free (types);
                g_assert (res);
                gclass->type.type = res;
@@ -8773,6 +9059,33 @@ mono_reflection_create_unmanaged_type (MonoReflectionType *type)
        mono_reflection_type_get_handle (type);
 }
 
+void
+mono_reflection_register_with_runtime (MonoReflectionType *type)
+{
+       MonoType *res = mono_reflection_type_get_handle (type);
+       MonoDomain *domain = mono_object_domain ((MonoObject*)type);
+       MonoClass *class;
+
+       if (!res)
+               mono_raise_exception (mono_get_exception_argument (NULL, "Invalid generic instantiation, one or more arguments are not proper user types"));
+
+       class = mono_class_from_mono_type (res);
+
+       mono_loader_lock (); /*same locking as mono_type_get_object*/
+       mono_domain_lock (domain);
+
+       if (!class->image->dynamic) {
+               mono_class_setup_supertypes (class);
+       } else {
+               if (!domain->type_hash)
+                       domain->type_hash = mono_g_hash_table_new_type ((GHashFunc)mymono_metadata_type_hash, 
+                                       (GCompareFunc)mymono_metadata_type_equal, MONO_HASH_VALUE_GC);
+               mono_g_hash_table_insert (domain->type_hash, res, type);
+       }
+       mono_domain_unlock (domain);
+       mono_loader_unlock ();
+}
+
 /**
  * LOCKING: Assumes the loader lock is held.
  */
@@ -8861,6 +9174,27 @@ get_field_name_and_type (MonoObject *field, char **name, MonoType **type)
                *type = f->field->type;
        }
 }
+
+#else /* DISABLE_REFLECTION_EMIT */
+
+void
+mono_reflection_register_with_runtime (MonoReflectionType *type)
+{
+       /* This is empty */
+}
+
+static gboolean
+is_sre_type_builder (MonoClass *class)
+{
+       return FALSE;
+}
+
+static gboolean
+is_sre_generic_instance (MonoClass *class)
+{
+       return FALSE;
+}
+
 #endif /* !DISABLE_REFLECTION_EMIT */
 
 static gboolean
@@ -9334,10 +9668,9 @@ mono_reflection_setup_internal_class (MonoReflectionTypeBuilder *tb)
 
        klass->element_class = klass;
 
-       if (klass->reflection_info == NULL) {
+       if (mono_class_get_ref_info (klass) == NULL) {
 
-               MOVING_GC_REGISTER (&klass->reflection_info);
-               klass->reflection_info = tb;
+               mono_class_set_ref_info (klass, tb);
 
                /* Put into cache so mono_class_get () will find it.
                Skip nested types as those should not be available on the global scope. */
@@ -9348,7 +9681,7 @@ mono_reflection_setup_internal_class (MonoReflectionTypeBuilder *tb)
                                g_slist_prepend (klass->image->reflection_info_unregister_classes, klass);
                }
        } else {
-               g_assert (klass->reflection_info == tb);
+               g_assert (mono_class_get_ref_info (klass) == tb);
        }
 
        mono_g_hash_table_insert (tb->module->dynamic_image->tokens,
@@ -9731,17 +10064,17 @@ reflection_methodbuilder_to_mono_method (MonoClass *klass,
 
        if (rmb->generic_params) {
                int count = mono_array_length (rmb->generic_params);
-               MonoGenericContainer *container;
+               MonoGenericContainer *container = rmb->generic_container;
+
+               g_assert (container);
 
-               container = rmb->generic_container;
-               if (container) {
-                       m->is_generic = TRUE;
-                       mono_method_set_generic_container (m, container);
-               }
                container->type_argc = count;
                container->type_params = image_g_new0 (image, MonoGenericParamFull, count);
                container->owner.method = m;
 
+               m->is_generic = TRUE;
+               mono_method_set_generic_container (m, container);
+
                for (i = 0; i < count; i++) {
                        MonoReflectionGenericParam *gp =
                                mono_array_get (rmb->generic_params, MonoReflectionGenericParam*, i);
@@ -9938,15 +10271,18 @@ mono_reflection_bind_generic_parameters (MonoReflectionType *type, int type_argc
 
        domain = mono_object_domain (type);
 
-       if (!strcmp (((MonoObject *) type)->vtable->klass->name, "TypeBuilder")) {
+       if (is_sre_type_builder (mono_object_class (type))) {
                tb = (MonoReflectionTypeBuilder *) type;
 
                is_dynamic = TRUE;
-       } else if (!strcmp (((MonoObject *) type)->vtable->klass->name, "MonoGenericClass")) {
+       } else if (is_sre_generic_instance (mono_object_class (type))) {
                MonoReflectionGenericClass *rgi = (MonoReflectionGenericClass *) type;
+               MonoReflectionType *gtd = rgi->generic_type;
 
-               tb = rgi->generic_type;
-               is_dynamic = TRUE;
+               if (is_sre_type_builder (mono_object_class (gtd))) {
+                       tb = (MonoReflectionTypeBuilder *)gtd;
+                       is_dynamic = TRUE;
+               }
        }
 
        /* FIXME: fix the CreateGenericParameters protocol to avoid the two stage setup of TypeBuilders */
@@ -9960,7 +10296,7 @@ mono_reflection_bind_generic_parameters (MonoReflectionType *type, int type_argc
        }
 
        if (klass->wastypebuilder) {
-               tb = (MonoReflectionTypeBuilder *) klass->reflection_info;
+               tb = (MonoReflectionTypeBuilder *) mono_class_get_ref_info (klass);
 
                is_dynamic = TRUE;
        }
@@ -9999,6 +10335,7 @@ mono_reflection_bind_generic_method_parameters (MonoReflectionMethod *rmethod, M
 
        MONO_ARCH_SAVE_REGS;
 
+       /*FIXME but this no longer should happen*/
        if (!strcmp (rmethod->object.vtable->klass->name, "MethodBuilder")) {
 #ifndef DISABLE_REFLECTION_EMIT
                MonoReflectionMethodBuilder *mb = NULL;
@@ -10041,6 +10378,7 @@ mono_reflection_bind_generic_method_parameters (MonoReflectionMethod *rmethod, M
        inflated = mono_class_inflate_generic_method (method, &tmp_context);
        imethod = (MonoMethodInflated *) inflated;
 
+       /*FIXME but I think this is no longer necessary*/
        if (method->klass->image->dynamic) {
                MonoDynamicImage *image = (MonoDynamicImage*)method->klass->image;
                /*
@@ -10098,12 +10436,24 @@ inflate_mono_method (MonoClass *klass, MonoMethod *method, MonoObject *obj)
 }
 
 static MonoMethod *
-inflate_method (MonoReflectionGenericClass *type, MonoObject *obj)
+inflate_method (MonoReflectionType *type, MonoObject *obj)
 {
        MonoMethod *method;
        MonoClass *gklass;
 
-       gklass = mono_class_from_mono_type (mono_reflection_type_get_handle ((MonoReflectionType*)type->generic_type));
+       MonoClass *type_class = mono_object_class (type);
+
+       if (is_sre_generic_instance (type_class)) {
+               MonoReflectionGenericClass *mgc = (MonoReflectionGenericClass*)type;
+               gklass = mono_class_from_mono_type (mono_reflection_type_get_handle ((MonoReflectionType*)mgc->generic_type));
+       } else if (is_sre_type_builder (type_class)) {
+               gklass = mono_class_from_mono_type (mono_reflection_type_get_handle (type));
+       } else if (type->type) {
+               gklass = mono_class_from_mono_type (type->type);
+               gklass = mono_class_get_generic_type_definition (gklass);
+       } else {
+               g_error ("Can't handle type %s", mono_type_get_full_name (mono_object_class (type)));
+       }
 
        if (!strcmp (obj->vtable->klass->name, "MethodBuilder"))
                if (((MonoReflectionMethodBuilder*)obj)->mhandle)
@@ -10141,7 +10491,9 @@ mono_reflection_generic_class_initialize (MonoReflectionGenericClass *type, Mono
        g_assert (gtype->type == MONO_TYPE_GENERICINST);
        gclass = gtype->data.generic_class;
 
-       g_assert (gclass->is_dynamic);
+       if (!gclass->is_dynamic)
+               return;
+
        dgclass = (MonoDynamicGenericClass *) gclass;
 
        if (dgclass->initialized)
@@ -10153,27 +10505,23 @@ mono_reflection_generic_class_initialize (MonoReflectionGenericClass *type, Mono
        dgclass->count_methods = methods ? mono_array_length (methods) : 0;
        dgclass->count_ctors = ctors ? mono_array_length (ctors) : 0;
        dgclass->count_fields = fields ? mono_array_length (fields) : 0;
-       dgclass->count_properties = properties ? mono_array_length (properties) : 0;
-       dgclass->count_events = events ? mono_array_length (events) : 0;
 
        dgclass->methods = g_new0 (MonoMethod *, dgclass->count_methods);
        dgclass->ctors = g_new0 (MonoMethod *, dgclass->count_ctors);
        dgclass->fields = g_new0 (MonoClassField, dgclass->count_fields);
-       dgclass->properties = g_new0 (MonoProperty, dgclass->count_properties);
-       dgclass->events = g_new0 (MonoEvent, dgclass->count_events);
        dgclass->field_objects = g_new0 (MonoObject*, dgclass->count_fields);
        dgclass->field_generic_types = g_new0 (MonoType*, dgclass->count_fields);
 
        for (i = 0; i < dgclass->count_methods; i++) {
                MonoObject *obj = mono_array_get (methods, gpointer, i);
 
-               dgclass->methods [i] = inflate_method (type, obj);
+               dgclass->methods [i] = inflate_method ((MonoReflectionType*)type, obj);
        }
 
        for (i = 0; i < dgclass->count_ctors; i++) {
                MonoObject *obj = mono_array_get (ctors, gpointer, i);
 
-               dgclass->ctors [i] = inflate_method (type, obj);
+               dgclass->ctors [i] = inflate_method ((MonoReflectionType*)type, obj);
        }
 
        for (i = 0; i < dgclass->count_fields; i++) {
@@ -10204,90 +10552,59 @@ mono_reflection_generic_class_initialize (MonoReflectionGenericClass *type, Mono
                }
        }
 
-       for (i = 0; i < dgclass->count_properties; i++) {
-               MonoObject *obj = mono_array_get (properties, gpointer, i);
-               MonoProperty *property = &dgclass->properties [i];
-
-               if (!strcmp (obj->vtable->klass->name, "PropertyBuilder")) {
-                       MonoReflectionPropertyBuilder *pb = (MonoReflectionPropertyBuilder *) obj;
-
-                       property->parent = klass;
-                       property->attrs = pb->attrs;
-                       property->name = mono_string_to_utf8 (pb->name);
-                       if (pb->get_method)
-                               property->get = inflate_method (type, (MonoObject *) pb->get_method);
-                       if (pb->set_method)
-                               property->set = inflate_method (type, (MonoObject *) pb->set_method);
-               } else if (!strcmp (obj->vtable->klass->name, "MonoProperty")) {
-                       *property = *((MonoReflectionProperty *) obj)->property;
-                       property->name = g_strdup (property->name);
-
-                       if (property->get)
-                               property->get = inflate_mono_method (klass, property->get, NULL);
-                       if (property->set)
-                               property->set = inflate_mono_method (klass, property->set, NULL);
-               } else
-                       g_assert_not_reached ();
-       }
-
-       for (i = 0; i < dgclass->count_events; i++) {
-               MonoObject *obj = mono_array_get (events, gpointer, i);
-               MonoEvent *event = &dgclass->events [i];
-
-               if (!strcmp (obj->vtable->klass->name, "EventBuilder")) {
-                       MonoReflectionEventBuilder *eb = (MonoReflectionEventBuilder *) obj;
-
-                       event->parent = klass;
-                       event->attrs = eb->attrs;
-                       event->name = mono_string_to_utf8 (eb->name);
-                       if (eb->add_method)
-                               event->add = inflate_method (type, (MonoObject *) eb->add_method);
-                       if (eb->remove_method)
-                               event->remove = inflate_method (type, (MonoObject *) eb->remove_method);
-               } else if (!strcmp (obj->vtable->klass->name, "MonoEvent")) {
-                       *event = *((MonoReflectionMonoEvent *) obj)->event;
-                       event->name = g_strdup (event->name);
-
-                       if (event->add)
-                               event->add = inflate_mono_method (klass, event->add, NULL);
-                       if (event->remove)
-                               event->remove = inflate_mono_method (klass, event->remove, NULL);
-               } else
-                       g_assert_not_reached ();
-       }
-
        dgclass->initialized = TRUE;
 }
 
 static void
-ensure_generic_class_runtime_vtable (MonoClass *klass)
+fix_partial_generic_class (MonoClass *klass)
 {
        MonoClass *gklass = klass->generic_class->container_class;
+       MonoDynamicGenericClass *dgclass;
        int i;
 
        if (klass->wastypebuilder)
                return;
 
-       ensure_runtime_vtable (gklass);
+       dgclass = (MonoDynamicGenericClass *)  klass->generic_class;
 
-       klass->method.count = gklass->method.count;
-       klass->methods = mono_image_alloc (klass->image, sizeof (MonoMethod*) * (klass->method.count + 1));
+       if (!dgclass->initialized)
+               return;
+
+       if (klass->method.count != gklass->method.count) {
+               klass->method.count = gklass->method.count;
+               klass->methods = mono_image_alloc (klass->image, sizeof (MonoMethod*) * (klass->method.count + 1));
 
-       for (i = 0; i < klass->method.count; i++) {
-               klass->methods [i] = mono_class_inflate_generic_method_full (
-                       gklass->methods [i], klass, mono_class_get_context (klass));
+               for (i = 0; i < klass->method.count; i++) {
+                       klass->methods [i] = mono_class_inflate_generic_method_full (
+                               gklass->methods [i], klass, mono_class_get_context (klass));
+               }
        }
 
-       klass->interface_count = gklass->interface_count;
-       klass->interfaces = mono_image_alloc (klass->image, sizeof (MonoClass*) * klass->interface_count);
-       for (i = 0; i < klass->interface_count; ++i) {
-               MonoType *iface_type = mono_class_inflate_generic_type (&gklass->interfaces [i]->byval_arg, mono_class_get_context (klass));
-               klass->interfaces [i] = mono_class_from_mono_type (iface_type);
-               mono_metadata_free_type (iface_type);
+       if (klass->interface_count && klass->interface_count != gklass->interface_count) {
+               klass->interface_count = gklass->interface_count;
+               klass->interfaces = mono_image_alloc (klass->image, sizeof (MonoClass*) * gklass->interface_count);
+               klass->interfaces_packed = NULL; /*make setup_interface_offsets happy*/
 
-               ensure_runtime_vtable (klass->interfaces [i]);
+               for (i = 0; i < gklass->interface_count; ++i) {
+                       MonoType *iface_type = mono_class_inflate_generic_type (&gklass->interfaces [i]->byval_arg, mono_class_get_context (klass));
+                       klass->interfaces [i] = mono_class_from_mono_type (iface_type);
+                       mono_metadata_free_type (iface_type);
+
+                       ensure_runtime_vtable (klass->interfaces [i]);
+               }
+               klass->interfaces_inited = 1;
+       }
+
+       if (klass->field.count != gklass->field.count) {
+               klass->field.count = gklass->field.count;
+               klass->fields = image_g_new0 (klass->image, MonoClassField, klass->field.count);
+
+               for (i = 0; i < klass->field.count; i++) {
+                       klass->fields [i] = gklass->fields [i];
+                       klass->fields [i].parent = klass;
+                       klass->fields [i].type = mono_class_inflate_generic_type (gklass->fields [i].type, mono_class_get_context (klass));
+               }
        }
-       klass->interfaces_inited = 1;
 
        /*We can only finish with this klass once it's parent has as well*/
        if (gklass->wastypebuilder)
@@ -10295,10 +10612,20 @@ ensure_generic_class_runtime_vtable (MonoClass *klass)
        return;
 }
 
+static void
+ensure_generic_class_runtime_vtable (MonoClass *klass)
+{
+       MonoClass *gklass = klass->generic_class->container_class;
+
+       ensure_runtime_vtable (gklass); 
+
+       fix_partial_generic_class (klass);
+}
+
 static void
 ensure_runtime_vtable (MonoClass *klass)
 {
-       MonoReflectionTypeBuilder *tb = klass->reflection_info;
+       MonoReflectionTypeBuilder *tb = mono_class_get_ref_info (klass);
        int i, num, j;
 
        if (!klass->image->dynamic || (!tb && !klass->generic_class) || klass->wastypebuilder)
@@ -10337,6 +10664,7 @@ ensure_runtime_vtable (MonoClass *klass)
                for (i = 0; i < klass->method.count; ++i)
                        klass->methods [i]->slot = i;
                
+               klass->interfaces_packed = NULL; /*make setup_interface_offsets happy*/
                mono_class_setup_interface_offsets (klass);
                mono_class_setup_interface_id (klass);
        }
@@ -10364,12 +10692,12 @@ mono_reflection_get_dynamic_overrides (MonoClass *klass, MonoMethod ***overrides
 
        g_assert (klass->image->dynamic);
 
-       if (!klass->reflection_info)
+       if (!mono_class_get_ref_info (klass))
                return;
 
-       g_assert (strcmp (((MonoObject*)klass->reflection_info)->vtable->klass->name, "TypeBuilder") == 0);
+       g_assert (strcmp (((MonoObject*)mono_class_get_ref_info (klass))->vtable->klass->name, "TypeBuilder") == 0);
 
-       tb = (MonoReflectionTypeBuilder*)klass->reflection_info;
+       tb = (MonoReflectionTypeBuilder*)mono_class_get_ref_info (klass);
 
        onum = 0;
        if (tb->methods) {
@@ -10409,7 +10737,7 @@ mono_reflection_get_dynamic_overrides (MonoClass *klass, MonoMethod ***overrides
 static void
 typebuilder_setup_fields (MonoClass *klass, MonoError *error)
 {
-       MonoReflectionTypeBuilder *tb = klass->reflection_info;
+       MonoReflectionTypeBuilder *tb = mono_class_get_ref_info (klass);
        MonoReflectionFieldBuilder *fb;
        MonoClassField *field;
        MonoImage *image = klass->image;
@@ -10436,6 +10764,14 @@ typebuilder_setup_fields (MonoClass *klass, MonoError *error)
        klass->fields = image_g_new0 (image, MonoClassField, klass->field.count);
        mono_class_alloc_ext (klass);
        klass->ext->field_def_values = image_g_new0 (image, MonoFieldDefaultValue, klass->field.count);
+       /*
+       This is, guess what, a hack.
+       The issue is that the runtime doesn't know how to setup the fields of a typebuider and crash.
+       On the static path no field class is resolved, only types are built. This is the right thing to do
+       but we suck.
+       Setting size_inited is harmless because we're doing the same job as mono_class_setup_fields anyway.
+       */
+       klass->size_inited = 1;
 
        for (i = 0; i < klass->field.count; ++i) {
                fb = mono_array_get (tb->fields, gpointer, i);
@@ -10477,7 +10813,7 @@ typebuilder_setup_fields (MonoClass *klass, MonoError *error)
 static void
 typebuilder_setup_properties (MonoClass *klass, MonoError *error)
 {
-       MonoReflectionTypeBuilder *tb = klass->reflection_info;
+       MonoReflectionTypeBuilder *tb = mono_class_get_ref_info (klass);
        MonoReflectionPropertyBuilder *pb;
        MonoImage *image = klass->image;
        MonoProperty *properties;
@@ -10544,7 +10880,7 @@ mono_reflection_event_builder_get_event_info (MonoReflectionTypeBuilder *tb, Mon
 static void
 typebuilder_setup_events (MonoClass *klass, MonoError *error)
 {
-       MonoReflectionTypeBuilder *tb = klass->reflection_info;
+       MonoReflectionTypeBuilder *tb = mono_class_get_ref_info (klass);
        MonoReflectionEventBuilder *eb;
        MonoImage *image = klass->image;
        MonoEvent *events;
@@ -10588,16 +10924,17 @@ typebuilder_setup_events (MonoClass *klass, MonoError *error)
 }
 
 static gboolean
-remove_instantiations_of (gpointer key,
+remove_instantiations_of_and_ensure_contents (gpointer key,
                                                  gpointer value,
                                                  gpointer user_data)
 {
        MonoType *type = (MonoType*)key;
        MonoClass *klass = (MonoClass*)user_data;
 
-       if ((type->type == MONO_TYPE_GENERICINST) && (type->data.generic_class->container_class == klass))
+       if ((type->type == MONO_TYPE_GENERICINST) && (type->data.generic_class->container_class == klass)) {
+               fix_partial_generic_class (mono_class_from_mono_type (type)); //Ensure it's safe to use it.
                return TRUE;
-       else
+       else
                return FALSE;
 }
 
@@ -10757,9 +11094,11 @@ mono_reflection_create_runtime_class (MonoReflectionTypeBuilder *tb)
         * If we are a generic TypeBuilder, there might be instantiations in the type cache
         * which have type System.Reflection.MonoGenericClass, but after the type is created, 
         * we want to return normal System.MonoType objects, so clear these out from the cache.
+        *
+        * Together with this we must ensure the contents of all instances to match the created type.
         */
        if (domain->type_hash && klass->generic_container)
-               mono_g_hash_table_foreach_remove (domain->type_hash, remove_instantiations_of, klass);
+               mono_g_hash_table_foreach_remove (domain->type_hash, remove_instantiations_of_and_ensure_contents, klass);
 
        mono_domain_unlock (domain);
        mono_loader_unlock ();
@@ -10824,8 +11163,7 @@ mono_reflection_initialize_generic_parameter (MonoReflectionGenericParam *gparam
 
        gparam->type.type = &pklass->byval_arg;
 
-       MOVING_GC_REGISTER (&pklass->reflection_info);
-       pklass->reflection_info = gparam; /* FIXME: GC pin gparam */
+       mono_class_set_ref_info (pklass, gparam);
        mono_image_lock (image);
        image->reflection_info_unregister_classes = g_slist_prepend (image->reflection_info_unregister_classes, pklass);
        mono_image_unlock (image);
@@ -11047,8 +11385,8 @@ mono_reflection_lookup_dynamic_token (MonoImage *image, guint32 token, gboolean
 static void
 ensure_complete_type (MonoClass *klass)
 {
-       if (klass->image->dynamic && !klass->wastypebuilder) {
-               MonoReflectionTypeBuilder *tb = klass->reflection_info;
+       if (klass->image->dynamic && !klass->wastypebuilder && mono_class_get_ref_info (klass)) {
+               MonoReflectionTypeBuilder *tb = mono_class_get_ref_info (klass);
 
                mono_domain_try_type_resolve (mono_domain_get (), NULL, (MonoObject*)tb);
 
@@ -11202,13 +11540,14 @@ resolve_object (MonoImage *image, MonoObject *obj, MonoClass **handle_class, Mon
                sig->explicit_this = helper->call_conv & 64 ? 1 : 0;
                sig->hasthis = helper->call_conv & 32 ? 1 : 0;
 
-               if (helper->call_conv == 0) /* unmanaged */
+               if (helper->unmanaged_call_conv) { /* unmanaged */
                        sig->call_convention = helper->unmanaged_call_conv - 1;
-               else
-                       if (helper->call_conv & 0x02)
-                               sig->call_convention = MONO_CALL_VARARG;
-               else
+                       sig->pinvoke = TRUE;
+               } else if (helper->call_conv & 0x02) {
+                       sig->call_convention = MONO_CALL_VARARG;
+               } else {
                        sig->call_convention = MONO_CALL_DEFAULT;
+               }
 
                sig->param_count = nargs;
                /* TODO: Copy type ? */
@@ -11242,12 +11581,20 @@ resolve_object (MonoImage *image, MonoObject *obj, MonoClass **handle_class, Mon
                MonoReflectionFieldOnTypeBuilderInst *f = (MonoReflectionFieldOnTypeBuilderInst*)obj;
                MonoClass *inflated;
                MonoType *type;
+               MonoClassField *field;
+
+               if (is_sre_field_builder (mono_object_class (f->fb)))
+                       field = ((MonoReflectionFieldBuilder*)f->fb)->handle;
+               else if (is_sr_mono_field (mono_object_class (f->fb)))
+                       field = ((MonoReflectionField*)f->fb)->field;
+               else
+                       g_error ("resolve_object:: can't handle a FTBI with base_method of type %s", mono_type_get_full_name (mono_object_class (f->fb)));
 
                type = mono_class_inflate_generic_type (mono_reflection_type_get_handle ((MonoReflectionType*)f->inst), context);
                inflated = mono_class_from_mono_type (type);
 
-               g_assert (f->fb->handle);
-               result = mono_class_get_field_from_name (inflated, mono_field_get_name (f->fb->handle));
+               result = field = mono_class_get_field_from_name (inflated, mono_field_get_name (field));
+               ensure_complete_type (field->parent);
                g_assert (result);
                mono_metadata_free_type (type);
                *handle_class = mono_defaults.fieldhandle_class;
@@ -11255,8 +11602,16 @@ resolve_object (MonoImage *image, MonoObject *obj, MonoClass **handle_class, Mon
                MonoReflectionCtorOnTypeBuilderInst *c = (MonoReflectionCtorOnTypeBuilderInst*)obj;
                MonoType *type = mono_class_inflate_generic_type (mono_reflection_type_get_handle ((MonoReflectionType*)c->inst), context);
                MonoClass *inflated_klass = mono_class_from_mono_type (type);
-               g_assert (c->cb->mhandle);
-               result = inflate_mono_method (inflated_klass, c->cb->mhandle, (MonoObject*)c->cb);
+               MonoMethod *method;
+
+               if (is_sre_ctor_builder (mono_object_class (c->cb)))
+                       method = ((MonoReflectionCtorBuilder *)c->cb)->mhandle;
+               else if (is_sr_mono_cmethod (mono_object_class (c->cb)))
+                       method = ((MonoReflectionMethod *)c->cb)->method;
+               else
+                       g_error ("resolve_object:: can't handle a CTBI with base_method of type %s", mono_type_get_full_name (mono_object_class (c->cb)));
+
+               result = inflate_mono_method (inflated_klass, method, (MonoObject*)c->cb);
                *handle_class = mono_defaults.methodhandle_class;
                mono_metadata_free_type (type);
        } else if (strcmp (obj->vtable->klass->name, "MethodOnTypeBuilderInst") == 0) {
@@ -11266,11 +11621,46 @@ resolve_object (MonoImage *image, MonoObject *obj, MonoClass **handle_class, Mon
                } else {
                        MonoType *type = mono_class_inflate_generic_type (mono_reflection_type_get_handle ((MonoReflectionType*)m->inst), context);
                        MonoClass *inflated_klass = mono_class_from_mono_type (type);
-                       g_assert (m->mb->mhandle);
-                       result = inflate_mono_method (inflated_klass, m->mb->mhandle, (MonoObject*)m->mb);
+                       MonoMethod *method;
+
+                       if (is_sre_method_builder (mono_object_class (m->mb)))
+                               method = ((MonoReflectionMethodBuilder *)m->mb)->mhandle;
+                       else if (is_sr_mono_method (mono_object_class (m->mb)))
+                               method = ((MonoReflectionMethod *)m->mb)->method;
+                       else
+                               g_error ("resolve_object:: can't handle a MTBI with base_method of type %s", mono_type_get_full_name (mono_object_class (m->mb)));
+
+                       result = inflate_mono_method (inflated_klass, method, (MonoObject*)m->mb);
                        mono_metadata_free_type (type);
                }
                *handle_class = mono_defaults.methodhandle_class;
+       } else if (strcmp (obj->vtable->klass->name, "MonoArrayMethod") == 0) {
+               MonoReflectionArrayMethod *m = (MonoReflectionArrayMethod*)obj;
+               MonoType *mtype;
+               MonoClass *klass;
+               MonoMethod *method;
+               gpointer iter;
+               char *name;
+
+               mtype = mono_reflection_type_get_handle (m->parent);
+               klass = mono_class_from_mono_type (mtype);
+
+               /* Find the method */
+
+               name = mono_string_to_utf8 (m->name);
+               iter = NULL;
+               while ((method = mono_class_get_methods (klass, &iter))) {
+                       if (!strcmp (method->name, name))
+                               break;
+               }
+               g_free (name);
+
+               // FIXME:
+               g_assert (method);
+               // FIXME: Check parameters/return value etc. match
+
+               result = method;
+               *handle_class = mono_defaults.methodhandle_class;
        } else if (is_sre_array (mono_object_get_class(obj)) ||
                                is_sre_byref (mono_object_get_class(obj)) ||
                                is_sre_pointer (mono_object_get_class(obj))) {
@@ -11884,14 +12274,14 @@ mono_reflection_call_is_assignable_to (MonoClass *klass, MonoClass *oklass)
 
        /* 
         * The result of mono_type_get_object () might be a System.MonoType but we
-        * need a TypeBuilder so use klass->reflection_info.
+        * need a TypeBuilder so use mono_class_get_ref_info (klass).
         */
-       g_assert (klass->reflection_info);
-       g_assert (!strcmp (((MonoObject*)(klass->reflection_info))->vtable->klass->name, "TypeBuilder"));
+       g_assert (mono_class_get_ref_info (klass));
+       g_assert (!strcmp (((MonoObject*)(mono_class_get_ref_info (klass)))->vtable->klass->name, "TypeBuilder"));
 
        params [0] = mono_type_get_object (mono_domain_get (), &oklass->byval_arg);
 
-       res = mono_runtime_invoke (method, (MonoObject*)(klass->reflection_info), params, &exc);
+       res = mono_runtime_invoke (method, (MonoObject*)(mono_class_get_ref_info (klass)), params, &exc);
        if (exc)
                return FALSE;
        else