2009-12-04 Zoltan Varga <vargaz@gmail.com>

[mono.git] / mono / mini / jit-icalls.c

diff --git a/mono/mini/jit-icalls.c b/mono/mini/jit-icalls.c
index 37b2ae992d80e5c6f4267ad3acdfd90ed37cb089..6accfcbe0fde6b25e80483a106b8e57e4bde9fb5 100644 (file)
--- a/mono/mini/jit-icalls.c
+++ b/mono/mini/jit-icalls.c
@@ -7,9 +7,12 @@
  *
  * (C) 2002 Ximian, Inc.
  */
-
+#include <config.h>
 #include <math.h>
 #include <limits.h>
+#ifdef HAVE_ALLOCA_H
+#include <alloca.h>
+#endif
 
 #include "jit-icalls.h"
 
@@ -20,22 +23,6 @@ mono_ldftn (MonoMethod *method)
 
        MONO_ARCH_SAVE_REGS;
 
-       addr = mono_create_jump_trampoline (mono_domain_get (), method, TRUE);
-
-       return mono_create_ftnptr (mono_domain_get (), addr);
-}
-
-/*
- * Same as mono_ldftn, but do not add a synchronized wrapper. Used in the
- * synchronized wrappers to avoid infinite recursion.
- */
-void*
-mono_ldftn_nosync (MonoMethod *method)
-{
-       gpointer addr;
-
-       MONO_ARCH_SAVE_REGS;
-
        addr = mono_create_jump_trampoline (mono_domain_get (), method, FALSE);
 
        return mono_create_ftnptr (mono_domain_get (), addr);
@@ -65,11 +52,7 @@ ldvirtfn_internal (MonoObject *obj, MonoMethod *method, gboolean gshared)
                res = mono_class_inflate_generic_method (res, &context);
        }
 
-       /* FIXME: only do this for methods which can be shared! */
-       if (res->is_inflated && mono_method_get_context (res)->method_inst &&
-                       mono_class_generic_sharing_enabled (res->klass)) {
-               res = mono_marshal_get_static_rgctx_invoke (res);
-       }
+       /* An rgctx wrapper is added by the trampolines no need to do it here */
 
        return mono_ldftn (res);
 }
@@ -304,7 +287,7 @@ mono_irem_un (guint32 a, guint32 b)
 
 #endif
 
-#ifdef MONO_ARCH_EMULATE_MUL_DIV
+#if defined(MONO_ARCH_EMULATE_MUL_DIV) || defined(MONO_ARCH_EMULATE_MUL_OVF)
 
 gint32
 mono_imul (gint32 a, gint32 b)
@@ -323,7 +306,7 @@ mono_imul_ovf (gint32 a, gint32 b)
 
        res = (gint64)a * (gint64)b;
 
-       if ((res > 0x7fffffffL) || (res < -2147483648))
+       if ((res > 0x7fffffffL) || (res < -2147483648LL))
                mono_raise_exception (mono_get_exception_overflow ());
 
        return res;
@@ -616,6 +599,17 @@ mono_fclt_un (double a, double b)
        return isunordered (a, b) || a < b;
 }
 
+gboolean
+mono_isfinite (double a)
+{
+#ifdef HAVE_ISFINITE
+       return isfinite (a);
+#else
+       g_assert_not_reached ();
+       return TRUE;
+#endif
+}
+
 double
 mono_fload_r4 (float *ptr)
 {
@@ -749,7 +743,7 @@ mono_class_static_field_address (MonoDomain *domain, MonoClassField *field)
 
        mono_class_init (field->parent);
 
-       vtable = mono_class_vtable (domain, field->parent);
+       vtable = mono_class_vtable_full (domain, field->parent, TRUE);
        if (!vtable->initialized)
                mono_runtime_class_init (vtable);
 
@@ -903,34 +897,25 @@ mono_lconv_to_r8_un (guint64 a)
 #endif
 
 gpointer
-mono_helper_compile_generic_method (MonoObject *obj, MonoMethod *method, MonoGenericContext *context, gpointer *this_arg)
+mono_helper_compile_generic_method (MonoObject *obj, MonoMethod *method, gpointer *this_arg)
 {
-       MonoMethod *vmethod, *inflated;
+       MonoMethod *vmethod;
        gpointer addr;
+       MonoGenericContext *context = mono_method_get_context (method);
 
        mono_jit_stats.generic_virtual_invocations++;
 
        if (obj == NULL)
                mono_raise_exception (mono_get_exception_null_reference ());
        vmethod = mono_object_get_virtual_method (obj, method);
-
-       /* 'vmethod' is partially inflated.  All the blanks corresponding to the type parameters of the
-          declaring class have been inflated.  We still need to fully inflate the method parameters.
-
-          FIXME: This code depends on the declaring class being fully inflated, since we inflate it twice with 
-          the same context.
-       */
        g_assert (!vmethod->klass->generic_container);
        g_assert (!vmethod->klass->generic_class || !vmethod->klass->generic_class->context.class_inst->is_open);
        g_assert (!context->method_inst || !context->method_inst->is_open);
-       inflated = mono_class_inflate_generic_method (vmethod, context);
-       if (mono_class_generic_sharing_enabled (inflated->klass) &&
-                       mono_method_is_generic_sharable_impl (method, FALSE)) {
-               /* The method is shared generic code, so it needs a
-                  MRGCTX. */
-               inflated = mono_marshal_get_static_rgctx_invoke (inflated);
-       }
-       addr = mono_compile_method (inflated);
+
+       addr = mono_compile_method (vmethod);
+
+       if (mono_method_needs_static_rgctx_invoke (vmethod, FALSE))
+               addr = mono_create_static_rgctx_trampoline (vmethod, addr);
 
        /* Since this is a virtual call, have to unbox vtypes */
        if (obj->vtable->klass->valuetype)
@@ -941,14 +926,6 @@ mono_helper_compile_generic_method (MonoObject *obj, MonoMethod *method, MonoGen
        return addr;
 }
 
-gpointer
-mono_helper_compile_generic_method_wo_context (MonoObject *obj, MonoMethod *method, gpointer *this_arg)
-{
-       MonoGenericContext *context = mono_method_get_context (method);
-
-       return mono_helper_compile_generic_method (obj, method, context, this_arg);
-}
-
 MonoString*
 mono_helper_ldstr (MonoImage *image, guint32 idx)
 {