Merge pull request #1857 from slluis/fix-assembly-resolver
[mono.git] / mono / mini / tramp-x86.c
index 4b3b5b6192d8b8c10ae9936660b2ee96c260f444..38705c1eb06c91f6dacf6146b21e85c8fab180c9 100644 (file)
@@ -18,6 +18,7 @@
 #include <mono/metadata/mono-debug.h>
 #include <mono/metadata/mono-debug-debugger.h>
 #include <mono/metadata/monitor.h>
+#include <mono/metadata/profiler-private.h>
 #include <mono/metadata/gc-internal.h>
 #include <mono/arch/x86/x86-codegen.h>
 
@@ -51,6 +52,7 @@ mono_arch_get_unbox_trampoline (MonoMethod *m, gpointer addr)
        g_assert ((code - start) < size);
 
        nacl_domain_code_validate (domain, &start, size, &code);
+       mono_profiler_code_buffer_new (start, code - start, MONO_PROFILER_CODE_BUFFER_UNBOX_TRAMPOLINE, m);
 
        return start;
 }
@@ -73,6 +75,7 @@ mono_arch_get_static_rgctx_trampoline (MonoMethod *m, MonoMethodRuntimeGenericCo
 
        nacl_domain_code_validate (domain, &start, buf_len, &code);
        mono_arch_flush_icache (start, code - start);
+       mono_profiler_code_buffer_new (start, code - start, MONO_PROFILER_CODE_BUFFER_GENERICS_TRAMPOLINE, NULL);
 
        return start;
 }
@@ -103,6 +106,7 @@ mono_arch_get_llvm_imt_trampoline (MonoDomain *domain, MonoMethod *m, int vt_off
        nacl_domain_code_validate (domain, &start, buf_len, &code);
 
        mono_arch_flush_icache (start, code - start);
+       mono_profiler_code_buffer_new (start, code - start, MONO_PROFILER_CODE_BUFFER_IMT_TRAMPOLINE, NULL);
 
        return start;
 }
@@ -489,12 +493,11 @@ mono_arch_create_generic_trampoline (MonoTrampolineType tramp_type, MonoTrampInf
 
        nacl_global_codeman_validate (&buf, 256, &code);
        g_assert ((code - buf) <= 256);
+       mono_profiler_code_buffer_new (buf, code - buf, MONO_PROFILER_CODE_BUFFER_HELPER, NULL);
 
-       if (info) {
-               tramp_name = mono_get_generic_trampoline_name (tramp_type);
-               *info = mono_tramp_info_create (tramp_name, buf, code - buf, ji, unwind_ops);
-               g_free (tramp_name);
-       }
+       tramp_name = mono_get_generic_trampoline_name (tramp_type);
+       *info = mono_tramp_info_create (tramp_name, buf, code - buf, ji, unwind_ops);
+       g_free (tramp_name);
 
        return buf;
 }
@@ -511,9 +514,9 @@ mono_arch_get_nullified_class_init_trampoline (MonoTrampInfo **info)
        nacl_global_codeman_validate (&buf, tramp_size, &code);
 
        mono_arch_flush_icache (buf, code - buf);
+       mono_profiler_code_buffer_new (buf, code - buf, MONO_PROFILER_CODE_BUFFER_HELPER, NULL);
 
-       if (info)
-               *info = mono_tramp_info_create ("nullified_class_init_trampoline", buf, code - buf, NULL, NULL);
+       *info = mono_tramp_info_create ("nullified_class_init_trampoline", buf, code - buf, NULL, NULL);
 
        return buf;
 }
@@ -536,6 +539,7 @@ mono_arch_create_specific_trampoline (gpointer arg1, MonoTrampolineType tramp_ty
        nacl_domain_code_validate (domain, &code, NACL_SIZE (4, kNaClAlignment), &buf);
 
        mono_arch_flush_icache (code, buf - code);
+       mono_profiler_code_buffer_new (code, buf - code, MONO_PROFILER_CODE_BUFFER_SPECIFIC_TRAMPOLINE, mono_get_generic_trampoline_simple_name (tramp_type));
 
        if (code_len)
                *code_len = buf - code;
@@ -635,14 +639,13 @@ mono_arch_create_rgctx_lazy_fetch_trampoline (guint32 slot, MonoTrampInfo **info
 
        nacl_global_codeman_validate (&buf, tramp_size, &code);
        mono_arch_flush_icache (buf, code - buf);
+       mono_profiler_code_buffer_new (buf, code - buf, MONO_PROFILER_CODE_BUFFER_GENERICS_TRAMPOLINE, NULL);
 
        g_assert (code - buf <= tramp_size);
 
-       if (info) {
-               char *name = mono_get_rgctx_fetch_trampoline_name (slot);
-               *info = mono_tramp_info_create (name, buf, code - buf, ji, unwind_ops);
-               g_free (name);
-       }
+       char *name = mono_get_rgctx_fetch_trampoline_name (slot);
+       *info = mono_tramp_info_create (name, buf, code - buf, ji, unwind_ops);
+       g_free (name);
 
        return buf;
 }
@@ -680,68 +683,11 @@ mono_arch_create_general_rgctx_lazy_fetch_trampoline (MonoTrampInfo **info, gboo
 
        nacl_global_codeman_validate (&buf, tramp_size, &code);
        mono_arch_flush_icache (buf, code - buf);
+       mono_profiler_code_buffer_new (buf, code - buf, MONO_PROFILER_CODE_BUFFER_GENERICS_TRAMPOLINE, NULL);
 
        g_assert (code - buf <= tramp_size);
 
-       if (info)
-               *info = mono_tramp_info_create ("rgctx_fetch_trampoline_general", buf, code - buf, ji, unwind_ops);
-
-       return buf;
-}
-
-gpointer
-mono_arch_create_generic_class_init_trampoline (MonoTrampInfo **info, gboolean aot)
-{
-       guint8 *tramp;
-       guint8 *code, *buf;
-       static int byte_offset = -1;
-       static guint8 bitmask;
-       guint8 *jump;
-       int tramp_size;
-       GSList *unwind_ops = NULL;
-       MonoJumpInfo *ji = NULL;
-
-       tramp_size = 64;
-
-       code = buf = mono_global_codeman_reserve (tramp_size);
-
-       unwind_ops = mono_arch_get_cie_program ();
-
-       if (byte_offset < 0)
-               mono_marshal_find_bitfield_offset (MonoVTable, initialized, &byte_offset, &bitmask);
-
-       x86_test_membase_imm (code, MONO_ARCH_VTABLE_REG, byte_offset, bitmask);
-       jump = code;
-       x86_branch8 (code, X86_CC_Z, -1, 1);
-
-       x86_ret (code);
-
-       x86_patch (jump, code);
-
-       /* Push the vtable so the stack is the same as in a specific trampoline */
-       x86_push_reg (code, MONO_ARCH_VTABLE_REG);
-
-       if (aot) {
-               code = mono_arch_emit_load_aotconst (buf, code, &ji, MONO_PATCH_INFO_JIT_ICALL_ADDR, "generic_trampoline_generic_class_init");
-               x86_jump_reg (code, X86_EAX);
-       } else {
-               tramp = mono_get_trampoline_code (MONO_TRAMPOLINE_GENERIC_CLASS_INIT);
-
-               /* jump to the actual trampoline */
-               x86_jump_code (code, tramp);
-       }
-
-       mono_arch_flush_icache (code, code - buf);
-
-       g_assert (code - buf <= tramp_size);
-#ifdef __native_client_codegen__
-       g_assert (code - buf <= kNaClAlignment);
-#endif
-
-       nacl_global_codeman_validate (&buf, tramp_size, &code);
-
-       if (info)
-               *info = mono_tramp_info_create ("generic_class_init_trampoline", buf, code - buf, ji, unwind_ops);
+       *info = mono_tramp_info_create ("rgctx_fetch_trampoline_general", buf, code - buf, ji, unwind_ops);
 
        return buf;
 }
@@ -766,29 +712,42 @@ mono_arch_create_generic_class_init_trampoline (MonoTrampInfo **info, gboolean a
  *
  */
 gpointer
-mono_arch_create_monitor_enter_trampoline (MonoTrampInfo **info, gboolean aot)
+mono_arch_create_monitor_enter_trampoline (MonoTrampInfo **info, gboolean is_v4, gboolean aot)
 {
-       guint8 *tramp = mono_get_trampoline_code (MONO_TRAMPOLINE_MONITOR_ENTER);
        guint8 *code, *buf;
        guint8 *jump_obj_null, *jump_sync_null, *jump_other_owner, *jump_cmpxchg_failed, *jump_tid, *jump_sync_thin_hash = NULL;
+       guint8 *jump_lock_taken_true = NULL;
        int tramp_size;
-       int owner_offset, nest_offset, dummy;
+       int status_offset, nest_offset;
        MonoJumpInfo *ji = NULL;
        GSList *unwind_ops = NULL;
 
        g_assert (MONO_ARCH_MONITOR_OBJECT_REG == X86_EAX);
+#ifdef MONO_ARCH_MONITOR_LOCK_TAKEN_REG
+       g_assert (MONO_ARCH_MONITOR_LOCK_TAKEN_REG == X86_EDX);
+#else
+       g_assert (!is_v4);
+#endif
 
-       mono_monitor_threads_sync_members_offset (&owner_offset, &nest_offset, &dummy);
-       g_assert (MONO_THREADS_SYNC_MEMBER_SIZE (owner_offset) == sizeof (gpointer));
+       mono_monitor_threads_sync_members_offset (&status_offset, &nest_offset);
+       g_assert (MONO_THREADS_SYNC_MEMBER_SIZE (status_offset) == sizeof (guint32));
        g_assert (MONO_THREADS_SYNC_MEMBER_SIZE (nest_offset) == sizeof (guint32));
-       owner_offset = MONO_THREADS_SYNC_MEMBER_OFFSET (owner_offset);
+       status_offset = MONO_THREADS_SYNC_MEMBER_OFFSET (status_offset);
        nest_offset = MONO_THREADS_SYNC_MEMBER_OFFSET (nest_offset);
 
-       tramp_size = NACL_SIZE (96, 128);
+       tramp_size = NACL_SIZE (128, 192);
 
        code = buf = mono_global_codeman_reserve (tramp_size);
 
+       x86_push_reg (code, X86_EAX);
        if (mono_thread_get_tls_offset () != -1) {
+               if (is_v4) {
+                       x86_test_membase_imm (code, X86_EDX, 0, 1);
+                       /* if *lock_taken is 1, jump to actual trampoline */
+                       jump_lock_taken_true = code;
+                       x86_branch8 (code, X86_CC_NZ, -1, 1);
+                       x86_push_reg (code, X86_EDX);
+               }
                /* MonoObject* obj is in EAX */
                /* is obj null? */
                x86_test_reg_reg (code, X86_EAX, X86_EAX);
@@ -828,76 +787,89 @@ mono_arch_create_monitor_enter_trampoline (MonoTrampInfo **info, gboolean aot)
                        code = mono_x86_emit_tls_get (code, X86_EDX, mono_thread_get_tls_offset ());
                }
                /* load TID into EDX */
-               x86_mov_reg_membase (code, X86_EDX, X86_EDX, MONO_STRUCT_OFFSET (MonoInternalThread, tid), 4);
+               x86_mov_reg_membase (code, X86_EDX, X86_EDX, MONO_STRUCT_OFFSET (MonoInternalThread, small_id), 4);
 
-               /* is synchronization->owner null? */
-               x86_alu_membase_imm (code, X86_CMP, X86_ECX, owner_offset, 0);
+               /* is synchronization->owner free */
+               x86_mov_reg_membase (code, X86_EAX, X86_ECX, status_offset, 4);
+               x86_test_reg_imm (code, X86_EAX, OWNER_MASK);
                /* if not, jump to next case */
                jump_tid = code;
                x86_branch8 (code, X86_CC_NZ, -1, 1);
 
                /* if yes, try a compare-exchange with the TID */
-               /* free up register EAX, needed for the zero */
-               x86_push_reg (code, X86_EAX);
-               /* zero EAX */
-               x86_alu_reg_reg (code, X86_XOR, X86_EAX, X86_EAX);
+               /* Form new status */
+               x86_alu_reg_reg (code, X86_OR, X86_EDX, X86_EAX);
                /* compare and exchange */
                x86_prefix (code, X86_LOCK_PREFIX);
-               x86_cmpxchg_membase_reg (code, X86_ECX, owner_offset, X86_EDX);
+               x86_cmpxchg_membase_reg (code, X86_ECX, status_offset, X86_EDX);
                /* if not successful, jump to actual trampoline */
                jump_cmpxchg_failed = code;
                x86_branch8 (code, X86_CC_NZ, -1, 1);
                /* if successful, pop and return */
+               if (is_v4) {
+                       x86_pop_reg (code, X86_EDX);
+                       x86_mov_membase_imm (code, X86_EDX, 0, 1, 1);
+               }
                x86_pop_reg (code, X86_EAX);
                x86_ret (code);
 
                /* next case: synchronization->owner is not null */
                x86_patch (jump_tid, code);
                /* is synchronization->owner == TID? */
-               x86_alu_membase_reg (code, X86_CMP, X86_ECX, owner_offset, X86_EDX);
+               x86_alu_reg_imm (code, X86_AND, X86_EAX, OWNER_MASK);
+               x86_alu_reg_reg (code, X86_CMP, X86_EAX, X86_EDX);
                /* if not, jump to actual trampoline */
                jump_other_owner = code;
                x86_branch8 (code, X86_CC_NZ, -1, 1);
                /* if yes, increment nest */
                x86_inc_membase (code, X86_ECX, nest_offset);
+               if (is_v4) {
+                       x86_pop_reg (code, X86_EDX);
+                       x86_mov_membase_imm (code, X86_EDX, 0, 1, 1);
+               }
+               x86_pop_reg (code, X86_EAX);
                /* return */
                x86_ret (code);
 
-               /* push obj */
+               /* obj is pushed, jump to the actual trampoline */
                x86_patch (jump_obj_null, code);
                if (jump_sync_thin_hash)
                        x86_patch (jump_sync_thin_hash, code);
                x86_patch (jump_sync_null, code);
                x86_patch (jump_other_owner, code);
-               x86_push_reg (code, X86_EAX);
-               /* jump to the actual trampoline */
                x86_patch (jump_cmpxchg_failed, code);
-               if (aot) {
-                       /* We are calling the generic trampoline directly, the argument is pushed
-                        * on the stack just like a specific trampoline.
-                        */
-                       code = mono_arch_emit_load_aotconst (buf, code, &ji, MONO_PATCH_INFO_JIT_ICALL_ADDR, "generic_trampoline_monitor_enter");
-                       x86_jump_reg (code, X86_EAX);
-               } else {
-                       x86_jump_code (code, tramp);
+
+               if (is_v4) {
+                       x86_pop_reg (code, X86_EDX);
+                       x86_patch (jump_lock_taken_true, code);
                }
-       } else {
-               /* push obj and jump to the actual trampoline */
-               x86_push_reg (code, X86_EAX);
-               if (aot) {
+       }
+
+       if (aot) {
+               /* We are calling the generic trampoline directly, the argument is pushed
+                * on the stack just like a specific trampoline.
+                */
+               if (is_v4)
+                       code = mono_arch_emit_load_aotconst (buf, code, &ji, MONO_PATCH_INFO_JIT_ICALL_ADDR, "generic_trampoline_monitor_enter_v4");
+               else
                        code = mono_arch_emit_load_aotconst (buf, code, &ji, MONO_PATCH_INFO_JIT_ICALL_ADDR, "generic_trampoline_monitor_enter");
-                       x86_jump_reg (code, X86_EAX);
-               } else {
-                       x86_jump_code (code, tramp);
-               }
+               x86_jump_reg (code, X86_EAX);
+       } else {
+               if (is_v4)
+                       x86_jump_code (code, mono_get_trampoline_code (MONO_TRAMPOLINE_MONITOR_ENTER_V4));
+               else
+                       x86_jump_code (code, mono_get_trampoline_code (MONO_TRAMPOLINE_MONITOR_ENTER));
        }
 
        mono_arch_flush_icache (buf, code - buf);
        g_assert (code - buf <= tramp_size);
 
        nacl_global_codeman_validate (&buf, tramp_size, &code);
+       mono_profiler_code_buffer_new (buf, code - buf, MONO_PROFILER_CODE_BUFFER_MONITOR, NULL);
 
-       if (info)
+       if (is_v4)
+               *info = mono_tramp_info_create ("monitor_enter_v4_trampoline", buf, code - buf, ji, unwind_ops);
+       else
                *info = mono_tramp_info_create ("monitor_enter_trampoline", buf, code - buf, ji, unwind_ops);
 
        return buf;
@@ -909,26 +881,25 @@ mono_arch_create_monitor_exit_trampoline (MonoTrampInfo **info, gboolean aot)
        guint8 *tramp = mono_get_trampoline_code (MONO_TRAMPOLINE_MONITOR_EXIT);
        guint8 *code, *buf;
        guint8 *jump_obj_null, *jump_have_waiters, *jump_sync_null, *jump_not_owned, *jump_sync_thin_hash = NULL;
-       guint8 *jump_next;
+       guint8 *jump_next, *jump_cmpxchg_failed;
        int tramp_size;
-       int owner_offset, nest_offset, entry_count_offset;
+       int status_offset, nest_offset;
        MonoJumpInfo *ji = NULL;
        GSList *unwind_ops = NULL;
 
        g_assert (MONO_ARCH_MONITOR_OBJECT_REG == X86_EAX);
 
-       mono_monitor_threads_sync_members_offset (&owner_offset, &nest_offset, &entry_count_offset);
-       g_assert (MONO_THREADS_SYNC_MEMBER_SIZE (owner_offset) == sizeof (gpointer));
+       mono_monitor_threads_sync_members_offset (&status_offset, &nest_offset);
+       g_assert (MONO_THREADS_SYNC_MEMBER_SIZE (status_offset) == sizeof (guint32));
        g_assert (MONO_THREADS_SYNC_MEMBER_SIZE (nest_offset) == sizeof (guint32));
-       g_assert (MONO_THREADS_SYNC_MEMBER_SIZE (entry_count_offset) == sizeof (gint32));
-       owner_offset = MONO_THREADS_SYNC_MEMBER_OFFSET (owner_offset);
+       status_offset = MONO_THREADS_SYNC_MEMBER_OFFSET (status_offset);
        nest_offset = MONO_THREADS_SYNC_MEMBER_OFFSET (nest_offset);
-       entry_count_offset = MONO_THREADS_SYNC_MEMBER_OFFSET (entry_count_offset);
 
-       tramp_size = NACL_SIZE (96, 128);
+       tramp_size = NACL_SIZE (128, 192);
 
        code = buf = mono_global_codeman_reserve (tramp_size);
 
+       x86_push_reg (code, X86_EAX);
        if (mono_thread_get_tls_offset () != -1) {
                /* MonoObject* obj is in EAX */
                /* is obj null? */
@@ -969,9 +940,11 @@ mono_arch_create_monitor_exit_trampoline (MonoTrampInfo **info, gboolean aot)
                        code = mono_x86_emit_tls_get (code, X86_EDX, mono_thread_get_tls_offset ());
                }
                /* load TID into EDX */
-               x86_mov_reg_membase (code, X86_EDX, X86_EDX, MONO_STRUCT_OFFSET (MonoInternalThread, tid), 4);
+               x86_mov_reg_membase (code, X86_EDX, X86_EDX, MONO_STRUCT_OFFSET (MonoInternalThread, small_id), 4);
                /* is synchronization->owner == TID */
-               x86_alu_membase_reg (code, X86_CMP, X86_ECX, owner_offset, X86_EDX);
+               x86_mov_reg_membase (code, X86_EAX, X86_ECX, status_offset, 4);
+               x86_alu_reg_reg (code, X86_XOR, X86_EDX, X86_EAX);
+               x86_test_reg_imm (code, X86_EDX, OWNER_MASK);
                /* if no, jump to actual trampoline */
                jump_not_owned = code;
                x86_branch8 (code, X86_CC_NZ, -1, 1);
@@ -982,19 +955,30 @@ mono_arch_create_monitor_exit_trampoline (MonoTrampInfo **info, gboolean aot)
                /* if not, jump to next case */
                jump_next = code;
                x86_branch8 (code, X86_CC_NZ, -1, 1);
-               /* if yes, is synchronization->entry_count zero? */
-               x86_alu_membase_imm (code, X86_CMP, X86_ECX, entry_count_offset, 0);
-               /* if not, jump to actual trampoline */
+               /* if yes, is synchronization->entry_count greater than zero? */
+               x86_test_reg_imm (code, X86_EAX, ENTRY_COUNT_WAITERS);
+               /* if yes, jump to actual trampoline */
                jump_have_waiters = code;
                x86_branch8 (code, X86_CC_NZ, -1 , 1);
-               /* if yes, set synchronization->owner to null and return */
-               x86_mov_membase_imm (code, X86_ECX, owner_offset, 0, 4);
+               /* if not, try to set synchronization->owner to null and return */
+               x86_mov_reg_reg (code, X86_EDX, X86_EAX, 4);
+               x86_alu_reg_imm (code, X86_AND, X86_EDX, ENTRY_COUNT_MASK); 
+               /* compare and exchange */
+               x86_prefix (code, X86_LOCK_PREFIX);
+               /* EAX contains the previous status */
+               x86_cmpxchg_membase_reg (code, X86_ECX, status_offset, X86_EDX);
+               /* if not successful, jump to actual trampoline */
+               jump_cmpxchg_failed = code;
+               x86_branch8 (code, X86_CC_NZ, -1, 1);
+
+               x86_pop_reg (code, X86_EAX);
                x86_ret (code);
 
                /* next case: synchronization->nest is not 1 */
                x86_patch (jump_next, code);
                /* decrease synchronization->nest and return */
                x86_dec_membase (code, X86_ECX, nest_offset);
+               x86_pop_reg (code, X86_EAX);
                x86_ret (code);
 
                /* push obj and jump to the actual trampoline */
@@ -1002,12 +986,12 @@ mono_arch_create_monitor_exit_trampoline (MonoTrampInfo **info, gboolean aot)
                if (jump_sync_thin_hash)
                        x86_patch (jump_sync_thin_hash, code);
                x86_patch (jump_have_waiters, code);
+               x86_patch (jump_cmpxchg_failed, code);
                x86_patch (jump_not_owned, code);
                x86_patch (jump_sync_null, code);
        }
 
-       /* push obj and jump to the actual trampoline */
-       x86_push_reg (code, X86_EAX);
+       /* obj is pushed, jump to the actual trampoline */
        if (aot) {
                code = mono_arch_emit_load_aotconst (buf, code, &ji, MONO_PATCH_INFO_JIT_ICALL_ADDR, "generic_trampoline_monitor_exit");
                x86_jump_reg (code, X86_EAX);
@@ -1019,9 +1003,9 @@ mono_arch_create_monitor_exit_trampoline (MonoTrampInfo **info, gboolean aot)
 
        mono_arch_flush_icache (buf, code - buf);
        g_assert (code - buf <= tramp_size);
+       mono_profiler_code_buffer_new (buf, code - buf, MONO_PROFILER_CODE_BUFFER_MONITOR, NULL);
 
-       if (info)
-               *info = mono_tramp_info_create ("monitor_exit_trampoline", buf, code - buf, ji, unwind_ops);
+       *info = mono_tramp_info_create ("monitor_exit_trampoline", buf, code - buf, ji, unwind_ops);
 
        return buf;
 }
@@ -1029,7 +1013,7 @@ mono_arch_create_monitor_exit_trampoline (MonoTrampInfo **info, gboolean aot)
 #else
 
 gpointer
-mono_arch_create_monitor_enter_trampoline (MonoTrampInfo **info, gboolean aot)
+mono_arch_create_monitor_enter_trampoline (MonoTrampInfo **info, gboolean is_v4, gboolean aot)
 {
        g_assert_not_reached ();
        return NULL;
@@ -1101,10 +1085,10 @@ mono_arch_create_handler_block_trampoline (MonoTrampInfo **info, gboolean aot)
        nacl_global_codeman_validate (&buf, tramp_size, &code);
 
        mono_arch_flush_icache (buf, code - buf);
+       mono_profiler_code_buffer_new (buf, code - buf, MONO_PROFILER_CODE_BUFFER_HELPER, NULL);
        g_assert (code - buf <= tramp_size);
 
-       if (info)
-               *info = mono_tramp_info_create ("handler_block_trampoline", buf, code - buf, ji, unwind_ops);
+       *info = mono_tramp_info_create ("handler_block_trampoline", buf, code - buf, ji, unwind_ops);
 
        return buf;
 }
@@ -1149,6 +1133,7 @@ mono_arch_get_gsharedvt_arg_trampoline (MonoDomain *domain, gpointer arg, gpoint
 
        nacl_domain_code_validate (domain, &start, buf_len, &code);
        mono_arch_flush_icache (start, code - start);
+       mono_profiler_code_buffer_new (start, code - start, MONO_PROFILER_CODE_BUFFER_GENERICS_TRAMPOLINE, NULL);
 
        return start;
 }
@@ -1162,8 +1147,7 @@ mono_arch_get_gsharedvt_arg_trampoline (MonoDomain *domain, gpointer arg, gpoint
 gpointer
 mono_arch_get_gsharedvt_trampoline (MonoTrampInfo **info, gboolean aot)
 {
-       if (info)
-               *info = NULL;
+       *info = NULL;
        return NULL;
 }