Add support for OP_SETFRET to the llvm backend.
[mono.git] / mono / mini / mini-llvm.c
index 3798dc4ac47bb7d68a6f94ca78bcf629a7d0baf9..c459b388e1b0a424e0c605266d8d8cae7cb1acf2 100644 (file)
@@ -46,6 +46,11 @@ typedef struct {
         * needs to branch to in ENDFINALLY.
         */
        GSList *call_handler_return_bbs;
+       /*
+        * If this bblock is the start of a finally clause, this is the bblock that
+        * CALL_HANDLER needs to branch to.
+        */
+       LLVMBasicBlockRef call_handler_target_bb;
        /* The list of switch statements generated by ENDFINALLY instructions */
        GSList *endfinally_switch_ins_list;
        GSList *phi_nodes;
@@ -619,6 +624,8 @@ simd_op_to_intrins (int opcode)
                return "llvm.x86.sse41.pminuw";
        case OP_PMINB_UN:
                return "llvm.x86.sse2.pminu.b";
+       case OP_PMINW:
+               return "llvm.x86.sse2.pmins.w";
        case OP_MAXPD:
                return "llvm.x86.sse2.max.pd";
        case OP_MAXPS:
@@ -629,6 +636,16 @@ simd_op_to_intrins (int opcode)
                return "llvm.x86.sse41.pmaxuw";
        case OP_PMAXB_UN:
                return "llvm.x86.sse2.pmaxu.b";
+       case OP_PCMPEQB:
+               return "llvm.x86.sse2.pcmpeq.b";
+       case OP_PCMPEQW:
+               return "llvm.x86.sse2.pcmpeq.w";
+       case OP_PCMPEQD:
+               return "llvm.x86.sse2.pcmpeq.d";
+       case OP_PCMPEQQ:
+               return "llvm.x86.sse41.pcmpeqq";
+       case OP_PCMPGTB:
+               return "llvm.x86.sse2.pcmpgt.b";
 #endif
        default:
                g_assert_not_reached ();
@@ -636,6 +653,39 @@ simd_op_to_intrins (int opcode)
        }
 }
 
+static LLVMTypeRef
+simd_op_to_llvm_type (int opcode)
+{
+#if defined(TARGET_X86) || defined(TARGET_AMD64)
+       switch (opcode) {
+       case OP_EXTRACT_R8:
+       case OP_EXPAND_R8:
+               return LLVMVectorType (LLVMDoubleType (), 2);
+       case OP_EXTRACT_I8:
+       case OP_EXPAND_I8:
+               return LLVMVectorType (LLVMInt64Type (), 2);
+       case OP_EXTRACT_I4:
+       case OP_EXPAND_I4:
+               return LLVMVectorType (LLVMInt32Type (), 4);
+       case OP_EXTRACT_I2:
+       case OP_EXTRACT_U2:
+       case OP_EXPAND_I2:
+               return LLVMVectorType (LLVMInt16Type (), 8);
+       case OP_EXTRACT_I1:
+       case OP_EXTRACT_U1:
+       case OP_EXPAND_I1:
+               return LLVMVectorType (LLVMInt8Type (), 16);
+       case OP_EXPAND_R4:
+               return LLVMVectorType (LLVMFloatType (), 4);
+       default:
+               g_assert_not_reached ();
+               return NULL;
+       }
+#else
+       return NULL;
+#endif
+}
+
 /*
  * get_bb:
  *
@@ -728,6 +778,8 @@ convert_full (EmitContext *ctx, LLVMValueRef v, LLVMTypeRef dtype, gboolean is_u
                        return LLVMBuildTrunc (ctx->builder, v, dtype, "");
                if (stype == LLVMInt32Type () && (dtype == LLVMInt16Type () || dtype == LLVMInt8Type ()))
                        return LLVMBuildTrunc (ctx->builder, v, dtype, "");
+               if (stype == LLVMInt16Type () && dtype == LLVMInt8Type ())
+                       return LLVMBuildTrunc (ctx->builder, v, dtype, "");
                if (stype == LLVMDoubleType () && dtype == LLVMFloatType ())
                        return LLVMBuildFPTrunc (ctx->builder, v, dtype, "");
 
@@ -777,7 +829,7 @@ emit_volatile_load (EmitContext *ctx, int vreg)
                 * Might have to zero extend since llvm doesn't have 
                 * unsigned types.
                 */
-               if (t->type == MONO_TYPE_U1 || t->type == MONO_TYPE_U2)
+               if (t->type == MONO_TYPE_U1 || t->type == MONO_TYPE_U2 || t->type == MONO_TYPE_CHAR || t->type == MONO_TYPE_BOOLEAN)
                        v = LLVMBuildZExt (ctx->builder, v, LLVMInt32Type (), "");
                else if (t->type == MONO_TYPE_U8)
                        v = LLVMBuildZExt (ctx->builder, v, LLVMInt64Type (), "");
@@ -852,7 +904,7 @@ sig_to_llvm_sig_full (EmitContext *ctx, MonoMethodSignature *sig, LLVMCallInfo *
        }
 
        pindexes = g_new0 (int, sig->param_count);
-       param_types = g_new0 (LLVMTypeRef, (sig->param_count * 2) + 2);
+       param_types = g_new0 (LLVMTypeRef, (sig->param_count * 2) + 3);
        pindex = 0;
        if (cinfo && cinfo->rgctx_arg) {
                if (sinfo)
@@ -860,7 +912,7 @@ sig_to_llvm_sig_full (EmitContext *ctx, MonoMethodSignature *sig, LLVMCallInfo *
                param_types [pindex] = IntPtrType ();
                pindex ++;
        }
-       if (cinfo && cinfo->imt_arg) {
+       if (cinfo && cinfo->imt_arg && IS_LLVM_MONO_BRANCH) {
                if (sinfo)
                        sinfo->imt_arg_pindex = pindex;
                param_types [pindex] = IntPtrType ();
@@ -1031,6 +1083,10 @@ get_plt_entry (EmitContext *ctx, LLVMTypeRef llvm_sig, MonoJumpInfoType type, gc
        if (!callee_name)
                return NULL;
 
+       if (ctx->cfg->compile_aot)
+               /* Add a patch so referenced wrappers can be compiled in full aot mode */
+               mono_add_patch_info (ctx->cfg, 0, type, data);
+
        // FIXME: Locking
        callee = g_hash_table_lookup (ctx->lmodule->plt_entries, callee_name);
        if (!callee) {
@@ -1066,6 +1122,22 @@ get_handler_clause (MonoCompile *cfg, MonoBasicBlock *bb)
        return -1;
 }
 
+static void
+set_metadata_flag (LLVMValueRef v, const char *flag_name)
+{
+#if LLVM_CHECK_VERSION (2, 8)
+       LLVMValueRef md_arg;
+       int md_kind;
+       
+       if (!IS_LLVM_MONO_BRANCH)
+               return;
+
+       md_kind = LLVMGetMDKindID (flag_name, strlen (flag_name));
+       md_arg = LLVMMDString ("mono", 4);
+       LLVMSetMetadata (v, md_kind, LLVMMDNode (&md_arg, 1));
+#endif
+}
+
 /*
  * emit_call:
  *
@@ -1125,12 +1197,15 @@ static LLVMValueRef
 emit_load (EmitContext *ctx, MonoBasicBlock *bb, LLVMBuilderRef *builder_ref, int size, LLVMValueRef addr, const char *name, gboolean is_faulting)
 {
        const char *intrins_name;
-       LLVMValueRef args [16];
+       LLVMValueRef args [16], res;
+       LLVMTypeRef addr_type;
 
        if (is_faulting && bb->region != -1 && IS_LLVM_MONO_BRANCH) {
                /*
                 * We handle loads which can fault by calling a mono specific intrinsic
                 * using an invoke, so they are handled properly inside try blocks.
+                * We can't use this outside clauses, since LLVM optimizes intrinsics which
+                * are marked with IntrReadArgMem.
                 */
                switch (size) {
                case 1:
@@ -1149,17 +1224,38 @@ emit_load (EmitContext *ctx, MonoBasicBlock *bb, LLVMBuilderRef *builder_ref, in
                        g_assert_not_reached ();
                }
 
+               addr_type = LLVMTypeOf (addr);
+               if (addr_type == LLVMPointerType (LLVMDoubleType (), 0) || addr_type == LLVMPointerType (LLVMFloatType (), 0))
+                       addr = LLVMBuildBitCast (*builder_ref, addr, LLVMPointerType (LLVMIntType (size * 8), 0), "");
+
                args [0] = addr;
                args [1] = LLVMConstInt (LLVMInt32Type (), 0, FALSE);
                args [2] = LLVMConstInt (LLVMInt1Type (), TRUE, FALSE);
-               return emit_call (ctx, bb, builder_ref, LLVMGetNamedFunction (ctx->module, intrins_name), args, 3);
+               res = emit_call (ctx, bb, builder_ref, LLVMGetNamedFunction (ctx->module, intrins_name), args, 3);
+
+               if (addr_type == LLVMPointerType (LLVMDoubleType (), 0))
+                       res = LLVMBuildBitCast (*builder_ref, res, LLVMDoubleType (), "");
+               else if (addr_type == LLVMPointerType (LLVMFloatType (), 0))
+                       res = LLVMBuildBitCast (*builder_ref, res, LLVMFloatType (), "");
+               
+               return res;
        } else {
+               LLVMValueRef res;
+
                /* 
                 * We emit volatile loads for loads which can fault, because otherwise
                 * LLVM will generate invalid code when encountering a load from a
                 * NULL address.
                 */
-               return mono_llvm_build_load (*builder_ref, addr, name, is_faulting);
+                res = mono_llvm_build_load (*builder_ref, addr, name, is_faulting);
+
+                /* Mark it with a custom metadata */
+                /*
+                if (is_faulting)
+                        set_metadata_flag (res, "mono.faulting.load");
+                */
+
+                return res;
        }
 }
 
@@ -1169,7 +1265,7 @@ emit_store (EmitContext *ctx, MonoBasicBlock *bb, LLVMBuilderRef *builder_ref, i
        const char *intrins_name;
        LLVMValueRef args [16];
 
-       if (is_faulting && bb->region != -1) {
+       if (is_faulting && bb->region != -1 && IS_LLVM_MONO_BRANCH) {
                switch (size) {
                case 1:
                        intrins_name = "llvm.mono.store.i8.p0i8";
@@ -1187,6 +1283,11 @@ emit_store (EmitContext *ctx, MonoBasicBlock *bb, LLVMBuilderRef *builder_ref, i
                        g_assert_not_reached ();
                }
 
+               if (LLVMTypeOf (value) == LLVMDoubleType () || LLVMTypeOf (value) == LLVMFloatType ()) {
+                       value = LLVMBuildBitCast (*builder_ref, value, LLVMIntType (size * 8), "");
+                       addr = LLVMBuildBitCast (*builder_ref, addr, LLVMPointerType (LLVMIntType (size * 8), 0), "");
+               }
+
                args [0] = value;
                args [1] = addr;
                args [2] = LLVMConstInt (LLVMInt32Type (), 0, FALSE);
@@ -1216,7 +1317,7 @@ emit_cond_system_exception (EmitContext *ctx, MonoBasicBlock *bb, const char *ex
 
        LLVMBuildCondBr (ctx->builder, cmp, ex_bb, noex_bb);
 
-       exc_class = mono_class_from_name (mono_defaults.corlib, "System", exc_type);
+       exc_class = mono_class_from_name (mono_get_corlib (), "System", exc_type);
        g_assert (exc_class);
 
        /* Emit exception throwing code */
@@ -1228,22 +1329,22 @@ emit_cond_system_exception (EmitContext *ctx, MonoBasicBlock *bb, const char *ex
                LLVMTypeRef sig;
                const char *icall_name;
 
-               MonoMethodSignature *throw_sig = mono_metadata_signature_alloc (mono_defaults.corlib, 2);
-               throw_sig->ret = &mono_defaults.void_class->byval_arg;
-               throw_sig->params [0] = &mono_defaults.int32_class->byval_arg;
+               MonoMethodSignature *throw_sig = mono_metadata_signature_alloc (mono_get_corlib (), 2);
+               throw_sig->ret = &mono_get_void_class ()->byval_arg;
+               throw_sig->params [0] = &mono_get_int32_class ()->byval_arg;
                if (IS_LLVM_MONO_BRANCH) {
-                       icall_name = "mono_arch_llvm_throw_corlib_exception_abs";
-                       throw_sig->params [1] = &mono_defaults.int_class->byval_arg;
+                       icall_name = "llvm_throw_corlib_exception_abs_trampoline";
+                       throw_sig->params [1] = &mono_get_intptr_class ()->byval_arg;
                } else {
-                       icall_name = "mono_arch_llvm_throw_corlib_exception";
-                       throw_sig->params [1] = &mono_defaults.int32_class->byval_arg;
+                       icall_name = "llvm_throw_corlib_exception_trampoline";
+                       throw_sig->params [1] = &mono_get_int32_class ()->byval_arg;
                }
                sig = sig_to_llvm_sig (ctx, throw_sig);
 
                if (ctx->cfg->compile_aot) {
                        callee = get_plt_entry (ctx, sig, MONO_PATCH_INFO_INTERNAL_METHOD, icall_name);
                } else {
-                       callee = LLVMAddFunction (ctx->module, "llvm_throw_corlib_exception", sig_to_llvm_sig (ctx, throw_sig));
+                       callee = LLVMAddFunction (ctx->module, "llvm_throw_corlib_exception_trampoline", sig_to_llvm_sig (ctx, throw_sig));
 
                        /*
                         * Differences between the LLVM/non-LLVM throw corlib exception trampoline:
@@ -1513,10 +1614,8 @@ emit_entry_bb (EmitContext *ctx, LLVMBuilderRef builder)
                if (!MONO_TYPE_ISSTRUCT (sig->params [i]))
                        emit_volatile_store (ctx, cfg->args [i + sig->hasthis]->dreg);
 
-       if (sig->hasthis && !cfg->rgctx_var) {
-#if LLVM_CHECK_VERSION (2, 8)
-               LLVMValueRef this_alloc, md_arg;
-               int md_kind;
+       if (sig->hasthis && !cfg->rgctx_var && cfg->generic_sharing_context) {
+               LLVMValueRef this_alloc;
 
                /*
                 * The exception handling code needs the location where the this argument was
@@ -1524,21 +1623,15 @@ emit_entry_bb (EmitContext *ctx, LLVMBuilderRef builder)
                 * with the "mono.this" custom metadata to tell llvm that it needs to save its
                 * location into the LSDA.
                 */
-               // FIXME: Do this for gshared only
                this_alloc = mono_llvm_build_alloca (builder, IntPtrType (), LLVMConstInt (LLVMInt32Type (), 1, FALSE), 0, "");
                /* This volatile store will keep the alloca alive */
                mono_llvm_build_store (builder, ctx->values [cfg->args [0]->dreg], this_alloc, TRUE);
 
-               md_kind = LLVMGetMDKindID ("mono.this", strlen ("mono.this"));
-               md_arg = LLVMMDString ("this", 4);
-               LLVMSetMetadata (this_alloc, md_kind, LLVMMDNode (&md_arg, 1));
-#endif
+               set_metadata_flag (this_alloc, "mono.this");
        }
 
        if (cfg->rgctx_var) {
-#if LLVM_CHECK_VERSION (2, 8)
-               LLVMValueRef rgctx_alloc, store, md_arg;
-               int md_kind;
+               LLVMValueRef rgctx_alloc, store;
 
                /*
                 * We handle the rgctx arg similarly to the this pointer.
@@ -1548,10 +1641,7 @@ emit_entry_bb (EmitContext *ctx, LLVMBuilderRef builder)
                /* This volatile store will keep the alloca alive */
                store = mono_llvm_build_store (builder, ctx->rgctx_arg, rgctx_alloc, TRUE);
 
-               md_kind = LLVMGetMDKindID ("mono.this", strlen ("mono.this"));
-               md_arg = LLVMMDString ("this", 4);
-               LLVMSetMetadata (rgctx_alloc, md_kind, LLVMMDNode (&md_arg, 1));
-#endif
+               set_metadata_flag (rgctx_alloc, "mono.this");
        }
 
        /*
@@ -1562,10 +1652,22 @@ emit_entry_bb (EmitContext *ctx, LLVMBuilderRef builder)
                if (bb->region != -1 && (bb->flags & BB_EXCEPTION_HANDLER))
                        g_hash_table_insert (ctx->region_to_handler, GUINT_TO_POINTER (mono_get_block_region_notry (cfg, bb->region)), bb);
                if (bb->region != -1 && (bb->flags & BB_EXCEPTION_HANDLER) && bb->in_scount == 0) {
-                       LLVMValueRef val = LLVMBuildAlloca (builder, LLVMInt32Type (), "");
+                       char name [128];
+                       LLVMValueRef val;
+
+                       sprintf (name, "finally_ind_bb%d", bb->block_num);
+                       val = LLVMBuildAlloca (builder, LLVMInt32Type (), name);
                        LLVMBuildStore (builder, LLVMConstInt (LLVMInt32Type (), 0, FALSE), val);
 
                        ctx->bblocks [bb->block_num].finally_ind = val;
+
+                       /*
+                        * Create a new bblock which CALL_HANDLER can branch to, because branching to the
+                        * LLVM bblock containing the call to llvm.eh.selector causes problems for the
+                        * LLVM optimizer passes.
+                        */
+                       sprintf (name, "BB_%d_CALL_HANDLER_TARGET", bb->block_num);
+                       ctx->bblocks [bb->block_num].call_handler_target_bb = LLVMAppendBasicBlock (ctx->lmethod, name);
                }
        }
 
@@ -1597,7 +1699,7 @@ process_call (EmitContext *ctx, MonoBasicBlock *bb, LLVMBuilderRef *builder_ref,
        LLVMValueRef *args;
        LLVMCallInfo *cinfo;
        GSList *l;
-       int i;
+       int i, len;
        gboolean vretaddr;
        LLVMTypeRef llvm_sig;
        gpointer target;
@@ -1611,6 +1713,15 @@ process_call (EmitContext *ctx, MonoBasicBlock *bb, LLVMBuilderRef *builder_ref,
        if (call->rgctx_arg_reg && !IS_LLVM_MONO_BRANCH)
                LLVM_FAILURE (ctx, "rgctx reg in call");
 
+       if (call->rgctx_reg && !IS_LLVM_MONO_BRANCH) {
+               /*
+                * It might be possible to support this by creating a static rgctx trampoline, but
+                * common_call_trampoline () would patch callsites to call the trampoline, which
+                * would be incorrect if the rgctx arg is computed dynamically.
+                */
+               LLVM_FAILURE (ctx, "rgctx reg");
+       }
+
        cinfo = call->cinfo;
        if (call->rgctx_arg_reg)
                cinfo->rgctx_arg = TRUE;
@@ -1687,12 +1798,12 @@ process_call (EmitContext *ctx, MonoBasicBlock *bb, LLVMBuilderRef *builder_ref,
                                        MonoJumpInfo *abs_ji = g_hash_table_lookup (cfg->abs_patches, call->fptr);
                                        if (abs_ji) {
                                                /*
-                                                * The monitor entry/exit trampolines might have
+                                                * FIXME: Some trampolines might have
                                                 * their own calling convention on some platforms.
                                                 */
 #ifndef TARGET_AMD64
-                                               if (abs_ji->type == MONO_PATCH_INFO_MONITOR_ENTER || abs_ji->type == MONO_PATCH_INFO_MONITOR_EXIT)
-                                                       LLVM_FAILURE (ctx, "monitor enter/exit");
+                                               if (abs_ji->type == MONO_PATCH_INFO_MONITOR_ENTER || abs_ji->type == MONO_PATCH_INFO_MONITOR_EXIT || abs_ji->type == MONO_PATCH_INFO_GENERIC_CLASS_INIT)
+                                                       LLVM_FAILURE (ctx, "trampoline with own cconv");
 #endif
                                                target = mono_resolve_patch_target (cfg->method, cfg->domain, NULL, abs_ji, FALSE);
                                                LLVMAddGlobalMapping (ee, callee, target);
@@ -1711,10 +1822,6 @@ process_call (EmitContext *ctx, MonoBasicBlock *bb, LLVMBuilderRef *builder_ref,
                g_assert (ins->inst_offset % size == 0);
                index = LLVMConstInt (LLVMInt32Type (), ins->inst_offset / size, FALSE);
 
-               // FIXME: mono_arch_get_vcall_slot () can't decode the code
-               // generated by LLVM
-               //LLVM_FAILURE (ctx, "virtual call");
-
                /*
                 * When using the llvm mono branch, we can support IMT directly, otherwise
                 * we need to call a trampoline.
@@ -1749,14 +1856,20 @@ process_call (EmitContext *ctx, MonoBasicBlock *bb, LLVMBuilderRef *builder_ref,
        /* 
         * Collect and convert arguments
         */
-       args = alloca (sizeof (LLVMValueRef) * ((sig->param_count * 2) + sig->hasthis + vretaddr + call->rgctx_reg));
+       len = sizeof (LLVMValueRef) * ((sig->param_count * 2) + sig->hasthis + vretaddr + call->rgctx_reg);
+       args = alloca (len);
+       memset (args, 0, len);
        l = call->out_ireg_args;
 
        if (IS_LLVM_MONO_BRANCH) {
-               if (call->rgctx_arg_reg)
+               if (call->rgctx_arg_reg) {
+                       g_assert (values [call->rgctx_arg_reg]);
                        args [sinfo.rgctx_arg_pindex] = values [call->rgctx_arg_reg];
-               if (call->imt_arg_reg)
+               }
+               if (call->imt_arg_reg) {
+                       g_assert (values [call->imt_arg_reg]);
                        args [sinfo.imt_arg_pindex] = values [call->imt_arg_reg];
+               }
        }
 
        if (vretaddr) {
@@ -1823,10 +1936,13 @@ process_call (EmitContext *ctx, MonoBasicBlock *bb, LLVMBuilderRef *builder_ref,
        /*
         * Modify cconv and parameter attributes to pass rgctx/imt correctly.
         */
-       if (call->rgctx_arg_reg)
+#if defined(MONO_ARCH_IMT_REG) && defined(MONO_ARCH_RGCTX_REG)
+       g_assert (MONO_ARCH_IMT_REG == MONO_ARCH_RGCTX_REG);
+#endif
+       /* The two can't be used together, so use only one LLVM calling conv to pass them */
+       g_assert (!(call->rgctx_arg_reg && call->imt_arg_reg));
+       if (!sig->pinvoke)
                LLVMSetInstructionCallConv (lcall, LLVMMono1CallConv);
-       else if (call->imt_arg_reg)
-               LLVMSetInstructionCallConv (lcall, LLVMMono2CallConv);
 
        if (call->rgctx_arg_reg)
                LLVMAddInstrAttribute (lcall, 1 + sinfo.rgctx_arg_pindex, LLVMInRegAttribute);
@@ -1903,6 +2019,7 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
        if (bb->flags & BB_EXCEPTION_HANDLER) {
                LLVMTypeRef i8ptr;
                LLVMValueRef eh_selector, eh_exception, personality, args [4];
+               LLVMBasicBlockRef target_bb;
                MonoInst *exvar;
                static gint32 mapping_inited;
                static int ti_generator;
@@ -1953,10 +2070,14 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                         * Enabling this causes llc to crash:
                         * http://llvm.org/bugs/show_bug.cgi?id=6102
                         */
-                       LLVM_FAILURE (ctx, "aot+clauses");
+                       //LLVM_FAILURE (ctx, "aot+clauses");
                } else {
-                       /* exception_cb will decode this */
-                       ti = g_malloc (sizeof (gint32));
+                       /*
+                        * After the cfg mempool is freed, the type info will point to stale memory,
+                        * but this is not a problem, since we decode it once in exception_cb during
+                        * compilation.
+                        */
+                       ti = mono_mempool_alloc (cfg->mempool, sizeof (gint32));
                        *(gint32*)ti = clause_index;
 
                        type_info = LLVMAddGlobal (module, i8ptr, ti_name);
@@ -1981,6 +2102,17 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                        values [exvar->dreg] = LLVMBuildCall (builder, eh_exception, NULL, 0, "");
                        emit_volatile_store (ctx, exvar->dreg);
                }
+
+               /* Start a new bblock which CALL_HANDLER can branch to */
+               target_bb = bblocks [bb->block_num].call_handler_target_bb;
+               if (target_bb) {
+                       LLVMBuildBr (builder, target_bb);
+
+                       ctx->builder = builder = create_builder (ctx);
+                       LLVMPositionBuilderAtEnd (ctx->builder, target_bb);
+
+                       ctx->bblocks [bb->block_num].end_bblock = target_bb;
+               }
        }
 
        has_terminator = FALSE;
@@ -2131,126 +2263,83 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                case OP_COMPARE:
                case OP_ICOMPARE_IMM:
                case OP_LCOMPARE_IMM:
-               case OP_COMPARE_IMM:
-#ifdef TARGET_AMD64
-               case OP_AMD64_ICOMPARE_MEMBASE_REG:
-               case OP_AMD64_ICOMPARE_MEMBASE_IMM:
-#endif
-#ifdef TARGET_X86
-               case OP_X86_COMPARE_MEMBASE_REG:
-               case OP_X86_COMPARE_MEMBASE_IMM:
-#endif
-                       {
-                               CompRelation rel;
-                               LLVMValueRef cmp;
-
-                               if (ins->next->opcode == OP_NOP)
-                                       break;
-
-                               if (ins->next->opcode == OP_BR)
-                                       /* The comparison result is not needed */
-                                       continue;
-
-                               rel = mono_opcode_to_cond (ins->next->opcode);
-
-                               /* Used for implementing bound checks */
-#ifdef TARGET_AMD64
-                               if ((ins->opcode == OP_AMD64_ICOMPARE_MEMBASE_REG) || (ins->opcode == OP_AMD64_ICOMPARE_MEMBASE_IMM)) {
-                                       int size = 4;
-                                       LLVMValueRef index;
-                                       LLVMTypeRef t;
-
-                                       t = LLVMInt32Type ();
-
-                                       g_assert (ins->inst_offset % size == 0);
-                                       index = LLVMConstInt (LLVMInt32Type (), ins->inst_offset / size, FALSE);                                
-
-                                       lhs = LLVMBuildLoad (builder, LLVMBuildGEP (builder, convert (ctx, values [ins->inst_basereg], LLVMPointerType (t, 0)), &index, 1, ""), "");
-                               }
-                               if (ins->opcode == OP_AMD64_ICOMPARE_MEMBASE_IMM) {
-                                       lhs = convert (ctx, lhs, LLVMInt32Type ());
-                                       rhs = LLVMConstInt (LLVMInt32Type (), ins->inst_imm, FALSE);
-                               }
-                               if (ins->opcode == OP_AMD64_ICOMPARE_MEMBASE_REG)
-                                       rhs = convert (ctx, rhs, LLVMInt32Type ());
-#endif
-
-#ifdef TARGET_X86
-                               if ((ins->opcode == OP_X86_COMPARE_MEMBASE_REG) || (ins->opcode == OP_X86_COMPARE_MEMBASE_IMM)) {
-                                       int size = 4;
-                                       LLVMValueRef index;
-                                       LLVMTypeRef t;
+               case OP_COMPARE_IMM: {
+                       CompRelation rel;
+                       LLVMValueRef cmp;
 
-                                       t = LLVMInt32Type ();
+                       if (ins->next->opcode == OP_NOP)
+                               break;
 
-                                       g_assert (ins->inst_offset % size == 0);
-                                       index = LLVMConstInt (LLVMInt32Type (), ins->inst_offset / size, FALSE);                                
+                       if (ins->next->opcode == OP_BR)
+                               /* The comparison result is not needed */
+                               continue;
 
-                                       lhs = LLVMBuildLoad (builder, LLVMBuildGEP (builder, convert (ctx, values [ins->inst_basereg], LLVMPointerType (t, 0)), &index, 1, ""), "");
-                               }
-                               if (ins->opcode == OP_X86_COMPARE_MEMBASE_IMM) {
-                                       lhs = convert (ctx, lhs, LLVMInt32Type ());
-                                       rhs = LLVMConstInt (LLVMInt32Type (), ins->inst_imm, FALSE);
-                               }
-                               if (ins->opcode == OP_X86_COMPARE_MEMBASE_REG)
-                                       rhs = convert (ctx, rhs, LLVMInt32Type ());
-#endif
+                       rel = mono_opcode_to_cond (ins->next->opcode);
 
-                               if (ins->opcode == OP_ICOMPARE_IMM) {
-                                       lhs = convert (ctx, lhs, LLVMInt32Type ());
-                                       rhs = LLVMConstInt (LLVMInt32Type (), ins->inst_imm, FALSE);
-                               }
-                               if (ins->opcode == OP_LCOMPARE_IMM) {
-                                       lhs = convert (ctx, lhs, LLVMInt64Type ());
-                                       rhs = LLVMConstInt (LLVMInt64Type (), GET_LONG_IMM (ins), FALSE);
-                               }
-                               if (ins->opcode == OP_LCOMPARE) {
-                                       lhs = convert (ctx, lhs, LLVMInt64Type ());
-                                       rhs = convert (ctx, rhs, LLVMInt64Type ());
-                               }
-                               if (ins->opcode == OP_ICOMPARE) {
-                                       lhs = convert (ctx, lhs, LLVMInt32Type ());
-                                       rhs = convert (ctx, rhs, LLVMInt32Type ());
-                               }
+                       if (ins->opcode == OP_ICOMPARE_IMM) {
+                               lhs = convert (ctx, lhs, LLVMInt32Type ());
+                               rhs = LLVMConstInt (LLVMInt32Type (), ins->inst_imm, FALSE);
+                       }
+                       if (ins->opcode == OP_LCOMPARE_IMM) {
+                               lhs = convert (ctx, lhs, LLVMInt64Type ());
+                               rhs = LLVMConstInt (LLVMInt64Type (), GET_LONG_IMM (ins), FALSE);
+                       }
+                       if (ins->opcode == OP_LCOMPARE) {
+                               lhs = convert (ctx, lhs, LLVMInt64Type ());
+                               rhs = convert (ctx, rhs, LLVMInt64Type ());
+                       }
+                       if (ins->opcode == OP_ICOMPARE) {
+                               lhs = convert (ctx, lhs, LLVMInt32Type ());
+                               rhs = convert (ctx, rhs, LLVMInt32Type ());
+                       }
 
-                               if (lhs && rhs) {
-                                       if (LLVMGetTypeKind (LLVMTypeOf (lhs)) == LLVMPointerTypeKind)
-                                               rhs = convert (ctx, rhs, LLVMTypeOf (lhs));
-                                       else if (LLVMGetTypeKind (LLVMTypeOf (rhs)) == LLVMPointerTypeKind)
-                                               lhs = convert (ctx, lhs, LLVMTypeOf (rhs));
-                               }
+                       if (lhs && rhs) {
+                               if (LLVMGetTypeKind (LLVMTypeOf (lhs)) == LLVMPointerTypeKind)
+                                       rhs = convert (ctx, rhs, LLVMTypeOf (lhs));
+                               else if (LLVMGetTypeKind (LLVMTypeOf (rhs)) == LLVMPointerTypeKind)
+                                       lhs = convert (ctx, lhs, LLVMTypeOf (rhs));
+                       }
 
-                               /* We use COMPARE+SETcc/Bcc, llvm uses SETcc+br cond */
-                               if (ins->opcode == OP_FCOMPARE)
-                                       cmp = LLVMBuildFCmp (builder, fpcond_to_llvm_cond [rel], convert (ctx, lhs, LLVMDoubleType ()), convert (ctx, rhs, LLVMDoubleType ()), "");
-                               else if (ins->opcode == OP_COMPARE_IMM)
-                                       cmp = LLVMBuildICmp (builder, cond_to_llvm_cond [rel], convert (ctx, lhs, IntPtrType ()), LLVMConstInt (IntPtrType (), ins->inst_imm, FALSE), "");
-                               else if (ins->opcode == OP_COMPARE)
-                                       cmp = LLVMBuildICmp (builder, cond_to_llvm_cond [rel], convert (ctx, lhs, IntPtrType ()), convert (ctx, rhs, IntPtrType ()), "");
-                               else
-                                       cmp = LLVMBuildICmp (builder, cond_to_llvm_cond [rel], lhs, rhs, "");
-
-                               if (MONO_IS_COND_BRANCH_OP (ins->next)) {
-                                       LLVMBuildCondBr (builder, cmp, get_bb (ctx, ins->next->inst_true_bb), get_bb (ctx, ins->next->inst_false_bb));
-                                       has_terminator = TRUE;
-                               } else if (MONO_IS_SETCC (ins->next)) {
-                                       sprintf (dname_buf, "t%d", ins->next->dreg);
-                                       dname = dname_buf;
-                                       values [ins->next->dreg] = LLVMBuildZExt (builder, cmp, LLVMInt32Type (), dname);
-
-                                       /* Add stores for volatile variables */
-                                       emit_volatile_store (ctx, ins->next->dreg);
-                               } else if (MONO_IS_COND_EXC (ins->next)) {
-                                       emit_cond_system_exception (ctx, bb, ins->next->inst_p1, cmp);
-                                       CHECK_FAILURE (ctx);
-                                       builder = ctx->builder;
+                       /* We use COMPARE+SETcc/Bcc, llvm uses SETcc+br cond */
+                       if (ins->opcode == OP_FCOMPARE)
+                               cmp = LLVMBuildFCmp (builder, fpcond_to_llvm_cond [rel], convert (ctx, lhs, LLVMDoubleType ()), convert (ctx, rhs, LLVMDoubleType ()), "");
+                       else if (ins->opcode == OP_COMPARE_IMM)
+                               cmp = LLVMBuildICmp (builder, cond_to_llvm_cond [rel], convert (ctx, lhs, IntPtrType ()), LLVMConstInt (IntPtrType (), ins->inst_imm, FALSE), "");
+                       else if (ins->opcode == OP_LCOMPARE_IMM) {
+                               if (SIZEOF_REGISTER == 4 && COMPILE_LLVM (cfg))  {
+                                       /* The immediate is encoded in two fields */
+                                       guint64 l = ((guint64)(guint32)ins->inst_offset << 32) | ((guint32)ins->inst_imm);
+                                       cmp = LLVMBuildICmp (builder, cond_to_llvm_cond [rel], convert (ctx, lhs, LLVMInt64Type ()), LLVMConstInt (LLVMInt64Type (), l, FALSE), "");
                                } else {
-                                       LLVM_FAILURE (ctx, "next");
+                                       cmp = LLVMBuildICmp (builder, cond_to_llvm_cond [rel], convert (ctx, lhs, LLVMInt64Type ()), LLVMConstInt (LLVMInt64Type (), ins->inst_imm, FALSE), "");
                                }
+                       }
+                       else if (ins->opcode == OP_COMPARE)
+                               cmp = LLVMBuildICmp (builder, cond_to_llvm_cond [rel], convert (ctx, lhs, IntPtrType ()), convert (ctx, rhs, IntPtrType ()), "");
+                       else
+                               cmp = LLVMBuildICmp (builder, cond_to_llvm_cond [rel], lhs, rhs, "");
 
-                               ins = ins->next;
-                               break;
+                       if (MONO_IS_COND_BRANCH_OP (ins->next)) {
+                               LLVMBuildCondBr (builder, cmp, get_bb (ctx, ins->next->inst_true_bb), get_bb (ctx, ins->next->inst_false_bb));
+                               has_terminator = TRUE;
+                       } else if (MONO_IS_SETCC (ins->next)) {
+                               sprintf (dname_buf, "t%d", ins->next->dreg);
+                               dname = dname_buf;
+                               values [ins->next->dreg] = LLVMBuildZExt (builder, cmp, LLVMInt32Type (), dname);
+
+                               /* Add stores for volatile variables */
+                               emit_volatile_store (ctx, ins->next->dreg);
+                       } else if (MONO_IS_COND_EXC (ins->next)) {
+                               emit_cond_system_exception (ctx, bb, ins->next->inst_p1, cmp);
+                               CHECK_FAILURE (ctx);
+                               builder = ctx->builder;
+                       } else {
+                               LLVM_FAILURE (ctx, "next");
                        }
+
+                       ins = ins->next;
+                       break;
+               }
                case OP_FCEQ:
                case OP_FCLT:
                case OP_FCLT_UN:
@@ -2324,6 +2413,7 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                case OP_MOVE:
                case OP_LMOVE:
                case OP_XMOVE:
+               case OP_SETFRET:
                        g_assert (lhs);
                        values [ins->dreg] = lhs;
                        break;
@@ -2692,7 +2782,7 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                case OP_LOCALLOC: {
                        LLVMValueRef v, size;
                                
-                       size = LLVMBuildAnd (builder, LLVMBuildAdd (builder, lhs, LLVMConstInt (LLVMInt32Type (), MONO_ARCH_FRAME_ALIGNMENT - 1, FALSE), ""), LLVMConstInt (LLVMInt32Type (), ~ (MONO_ARCH_FRAME_ALIGNMENT - 1), FALSE), "");
+                       size = LLVMBuildAnd (builder, LLVMBuildAdd (builder, convert (ctx, lhs, LLVMInt32Type ()), LLVMConstInt (LLVMInt32Type (), MONO_ARCH_FRAME_ALIGNMENT - 1, FALSE), ""), LLVMConstInt (LLVMInt32Type (), ~ (MONO_ARCH_FRAME_ALIGNMENT - 1), FALSE), "");
 
                        v = mono_llvm_build_alloca (builder, LLVMInt8Type (), size, MONO_ARCH_FRAME_ALIGNMENT, "");
 
@@ -2754,6 +2844,15 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
 
                        values [ins->dreg] = emit_load (ctx, bb, &builder, size, addr, dname, is_volatile);
 
+                       if (!is_volatile && (ins->flags & MONO_INST_CONSTANT_LOAD)) {
+                               /*
+                                * These will signal LLVM that these loads do not alias any stores, and
+                                * they can't fail, allowing them to be hoisted out of loops.
+                                */
+                               set_metadata_flag (values [ins->dreg], "mono.noalias");
+                               set_metadata_flag (values [ins->dreg], "mono.nofail.load");
+                       }
+
                        if (sext)
                                values [ins->dreg] = LLVMBuildSExt (builder, values [ins->dreg], LLVMInt32Type (), dname);
                        else if (zext)
@@ -2776,6 +2875,9 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                        gboolean sext = FALSE, zext = FALSE;
                        gboolean is_volatile = (ins->flags & MONO_INST_FAULT);
 
+                       if (!values [ins->inst_destbasereg])
+                               LLVM_FAILURE (ctx, "inst_destbasereg");
+
                        t = load_store_to_llvm_type (ins->opcode, &size, &sext, &zext);
 
                        if (ins->inst_offset % size != 0) {
@@ -2892,6 +2994,10 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                        /* Might have instructions after this */
                        while (ins->next) {
                                MonoInst *next = ins->next;
+                               /* 
+                                * FIXME: If later code uses the regs defined by these instructions,
+                                * compilation will fail.
+                                */
                                MONO_DELETE_INS (bb, next);
                        }                               
                        break;
@@ -2915,17 +3021,19 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                        values [ins->dreg] = LLVMBuildCall (builder, LLVMGetNamedFunction (module, "llvm.cos.f64"), args, 1, dname);
                        break;
                }
-                       /* test_0_sqrt_nan fails with LLVM */
-                       /*
-                         case OP_SQRT: {
-                         LLVMValueRef args [1];
-
-                         args [0] = lhs;
-                         values [ins->dreg] = LLVMBuildCall (builder, LLVMGetNamedFunction (module, "llvm.sqrt.f64"), args, 1, dname);
-                         break;
-                         }
-                       */
+               case OP_SQRT: {
+                       LLVMValueRef args [1];
 
+                       /*
+                        * LLVM optimizes sqrt(nan) into undefined in
+                        * lib/Analysis/ConstantFolding.cpp
+                        * Also, sqrt(NegativeInfinity) is optimized into 0.
+                        */
+                       LLVM_FAILURE (ctx, "sqrt");
+                       args [0] = lhs;
+                       values [ins->dreg] = LLVMBuildCall (builder, LLVMGetNamedFunction (module, "llvm.sqrt.f64"), args, 1, dname);
+                       break;
+               }
                case OP_ABS: {
                        LLVMValueRef args [1];
 
@@ -2935,26 +3043,39 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                }
 
                case OP_IMIN:
-               case OP_LMIN: {
-                       LLVMValueRef v = LLVMBuildICmp (builder, LLVMIntSLE, lhs, rhs, "");
-                       values [ins->dreg] = LLVMBuildSelect (builder, v, lhs, rhs, dname);
-                       break;
-               }
+               case OP_LMIN:
                case OP_IMAX:
-               case OP_LMAX: {
-                       LLVMValueRef v = LLVMBuildICmp (builder, LLVMIntSGE, lhs, rhs, "");
-                       values [ins->dreg] = LLVMBuildSelect (builder, v, lhs, rhs, dname);
-                       break;
-               }
+               case OP_LMAX:
                case OP_IMIN_UN:
-               case OP_LMIN_UN: {
-                       LLVMValueRef v = LLVMBuildICmp (builder, LLVMIntULE, lhs, rhs, "");
-                       values [ins->dreg] = LLVMBuildSelect (builder, v, lhs, rhs, dname);
-                       break;
-               }
+               case OP_LMIN_UN:
                case OP_IMAX_UN:
                case OP_LMAX_UN: {
-                       LLVMValueRef v = LLVMBuildICmp (builder, LLVMIntUGE, lhs, rhs, "");
+                       LLVMValueRef v;
+
+                       lhs = convert (ctx, lhs, regtype_to_llvm_type (spec [MONO_INST_DEST]));
+                       rhs = convert (ctx, rhs, regtype_to_llvm_type (spec [MONO_INST_DEST]));
+
+                       switch (ins->opcode) {
+                       case OP_IMIN:
+                       case OP_LMIN:
+                               v = LLVMBuildICmp (builder, LLVMIntSLE, lhs, rhs, "");
+                               break;
+                       case OP_IMAX:
+                       case OP_LMAX:
+                               v = LLVMBuildICmp (builder, LLVMIntSGE, lhs, rhs, "");
+                               break;
+                       case OP_IMIN_UN:
+                       case OP_LMIN_UN:
+                               v = LLVMBuildICmp (builder, LLVMIntULE, lhs, rhs, "");
+                               break;
+                       case OP_IMAX_UN:
+                       case OP_LMAX_UN:
+                               v = LLVMBuildICmp (builder, LLVMIntUGE, lhs, rhs, "");
+                               break;
+                       default:
+                               g_assert_not_reached ();
+                               break;
+                       }
                        values [ins->dreg] = LLVMBuildSelect (builder, v, lhs, rhs, dname);
                        break;
                }
@@ -3023,6 +3144,11 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                case OP_MEMORY_BARRIER: {
                        LLVMValueRef args [5];
 
+#ifdef TARGET_ARM
+                       /* Not yet supported by llc on arm */
+                       LLVM_FAILURE (ctx, "memory-barrier+arm");
+#endif
+
                        for (i = 0; i < 5; ++i)
                                args [i] = LLVMConstInt (LLVMInt1Type (), TRUE, TRUE);
 
@@ -3133,6 +3259,11 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
 
                        switch (ins->opcode) {
                        case OP_STOREV_MEMBASE:
+                               if (cfg->gen_write_barriers && klass->has_references && ins->inst_destbasereg != cfg->frame_reg) {
+                                       /* FIXME: Emit write barriers like in mini_emit_stobj () */
+                                       LLVM_FAILURE (ctx, "storev_membase + write barriers");
+                                       break;
+                               }
                                if (!addresses [ins->sreg1]) {
                                        /* SIMD */
                                        g_assert (values [ins->sreg1]);
@@ -3161,6 +3292,7 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                        default:
                                g_assert_not_reached ();
                        }
+                       CHECK_FAILURE (ctx);
 
                        if (done)
                                break;
@@ -3299,9 +3431,15 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                case OP_PMIND_UN:
                case OP_PMINW_UN:
                case OP_PMINB_UN:
+               case OP_PMINW:
                case OP_PMAXD_UN:
                case OP_PMAXW_UN:
-               case OP_PMAXB_UN: {
+               case OP_PMAXB_UN:
+               case OP_PCMPEQB:
+               case OP_PCMPEQW:
+               case OP_PCMPEQD:
+               case OP_PCMPEQQ:
+               case OP_PCMPGTB: {
                        LLVMValueRef args [2];
 
                        args [0] = lhs;
@@ -3320,28 +3458,17 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                        LLVMTypeRef t;
                        gboolean zext = FALSE;
 
+                       t = simd_op_to_llvm_type (ins->opcode);
+
                        switch (ins->opcode) {
                        case OP_EXTRACT_R8:
-                               t = LLVMVectorType (LLVMDoubleType (), 2);
-                               break;
                        case OP_EXTRACT_I8:
-                               t = LLVMVectorType (LLVMInt64Type (), 2);
-                               break;
                        case OP_EXTRACT_I4:
-                               t = LLVMVectorType (LLVMInt32Type (), 4);
-                               break;
                        case OP_EXTRACT_I2:
-                               t = LLVMVectorType (LLVMInt16Type (), 8);
-                               break;
-                       case OP_EXTRACT_U2:
-                               t = LLVMVectorType (LLVMInt16Type (), 8);
-                               zext = TRUE;
-                               break;
                        case OP_EXTRACT_I1:
-                               t = LLVMVectorType (LLVMInt8Type (), 16);
                                break;
+                       case OP_EXTRACT_U2:
                        case OP_EXTRACT_U1:
-                               t = LLVMVectorType (LLVMInt8Type (), 16);
                                zext = TRUE;
                                break;
                        default:
@@ -3355,6 +3482,25 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                                values [ins->dreg] = LLVMBuildZExt (builder, values [ins->dreg], LLVMInt32Type (), "");
                        break;
                }
+
+               case OP_EXPAND_I1:
+               case OP_EXPAND_I2:
+               case OP_EXPAND_I4:
+               case OP_EXPAND_I8:
+               case OP_EXPAND_R4:
+               case OP_EXPAND_R8: {
+                       LLVMTypeRef t = simd_op_to_llvm_type (ins->opcode);
+                       LLVMValueRef mask [16], v;
+
+                       for (i = 0; i < 16; ++i)
+                               mask [i] = LLVMConstInt (LLVMInt32Type (), 0, FALSE);
+
+                       v = convert (ctx, values [ins->sreg1], LLVMGetElementType (t));
+
+                       values [ins->dreg] = LLVMBuildInsertElement (builder, LLVMConstNull (t), v, LLVMConstInt (LLVMInt32Type (), 0, FALSE), "");
+                       values [ins->dreg] = LLVMBuildShuffleVector (builder, values [ins->dreg], LLVMGetUndef (t), LLVMConstVector (mask, LLVMGetVectorSize (t)), "");
+                       break;
+               }
 #endif
 
                case OP_DUMMY_USE:
@@ -3379,9 +3525,9 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                        icall_name = rethrow ? "mono_arch_rethrow_exception" : "mono_arch_throw_exception";
 
                        if (!callee) {
-                               throw_sig = mono_metadata_signature_alloc (mono_defaults.corlib, 1);
-                               throw_sig->ret = &mono_defaults.void_class->byval_arg;
-                               throw_sig->params [0] = &mono_defaults.object_class->byval_arg;
+                               throw_sig = mono_metadata_signature_alloc (mono_get_corlib (), 1);
+                               throw_sig->ret = &mono_get_void_class ()->byval_arg;
+                               throw_sig->params [0] = &mono_get_object_class ()->byval_arg;
                                if (cfg->compile_aot) {
                                        callee = get_plt_entry (ctx, sig_to_llvm_sig (ctx, throw_sig), MONO_PATCH_INFO_INTERNAL_METHOD, icall_name);
                                } else {
@@ -3392,7 +3538,7 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                                         * LLVM doesn't push the exception argument, so we need a different
                                         * trampoline.
                                         */
-                                       LLVMAddGlobalMapping (ee, callee, resolve_patch (cfg, MONO_PATCH_INFO_INTERNAL_METHOD, rethrow ? "mono_arch_llvm_rethrow_exception" : "mono_arch_llvm_throw_exception"));
+                                       LLVMAddGlobalMapping (ee, callee, resolve_patch (cfg, MONO_PATCH_INFO_INTERNAL_METHOD, rethrow ? "llvm_rethrow_exception_trampoline" : "llvm_throw_exception_trampoline"));
 #else
                                        LLVMAddGlobalMapping (ee, callee, resolve_patch (cfg, MONO_PATCH_INFO_INTERNAL_METHOD, icall_name));
 #endif
@@ -3404,7 +3550,7 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                                else
                                        ctx->lmodule->throw = callee;
                        }
-                       arg = convert (ctx, values [ins->sreg1], type_to_llvm_type (ctx, &mono_defaults.object_class->byval_arg));
+                       arg = convert (ctx, values [ins->sreg1], type_to_llvm_type (ctx, &mono_get_object_class ()->byval_arg));
                        emit_call (ctx, bb, &builder, callee, &arg, 1);
                        break;
                }
@@ -3413,12 +3559,10 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                         * We don't 'call' handlers, but instead simply branch to them.
                         * The code generated by ENDFINALLY will branch back to us.
                         */
-                       LLVMBasicBlockRef finally_bb, noex_bb;
+                       LLVMBasicBlockRef noex_bb;
                        GSList *bb_list;
                        BBInfo *info = &bblocks [ins->inst_target_bb->block_num];
 
-                       finally_bb = get_bb (ctx, ins->inst_target_bb);
-
                        bb_list = info->call_handler_return_bbs;
 
                        /* 
@@ -3429,7 +3573,7 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                        LLVMBuildStore (builder, LLVMConstInt (LLVMInt32Type (), g_slist_length (bb_list) + 1, FALSE), lhs);
                                
                        /* Branch to the finally clause */
-                       LLVMBuildBr (builder, finally_bb);
+                       LLVMBuildBr (builder, info->call_handler_target_bb);
 
                        noex_bb = gen_bb (ctx, "CALL_HANDLER_CONT_BB");
                        info->call_handler_return_bbs = g_slist_append_mempool (cfg->mempool, info->call_handler_return_bbs, noex_bb);
@@ -3478,9 +3622,9 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
                        LLVMPositionBuilderAtEnd (ctx->builder, resume_bb);
 
                        if (ctx->cfg->compile_aot) {
-                               callee = get_plt_entry (ctx, LLVMFunctionType (LLVMVoidType (), NULL, 0, FALSE), MONO_PATCH_INFO_INTERNAL_METHOD, "mono_resume_unwind");
+                               callee = get_plt_entry (ctx, LLVMFunctionType (LLVMVoidType (), NULL, 0, FALSE), MONO_PATCH_INFO_INTERNAL_METHOD, "llvm_resume_unwind_trampoline");
                        } else {
-                               callee = LLVMGetNamedFunction (module, "mono_resume_unwind");
+                               callee = LLVMGetNamedFunction (module, "llvm_resume_unwind_trampoline");
                        }
                        LLVMBuildCall (builder, callee, NULL, 0, "");
 
@@ -3535,9 +3679,11 @@ process_bb (EmitContext *ctx, MonoBasicBlock *bb)
 void
 mono_llvm_check_method_supported (MonoCompile *cfg)
 {
+       /*
        MonoMethodHeader *header = cfg->header;
        MonoExceptionClause *clause;
        int i;
+       */
 
        if (cfg->generic_sharing_context && !IS_LLVM_MONO_BRANCH) {
                /* No way to obtain location info for this/rgctx */
@@ -3559,6 +3705,7 @@ mono_llvm_check_method_supported (MonoCompile *cfg)
                cfg->disable_llvm = TRUE;
        }
 
+#if 0
        for (i = 0; i < header->num_clauses; ++i) {
                clause = &header->clauses [i];
                
@@ -3570,6 +3717,7 @@ mono_llvm_check_method_supported (MonoCompile *cfg)
                        cfg->disable_llvm = TRUE;
                }
        }
+#endif
 
        /* FIXME: */
        if (cfg->method->dynamic) {
@@ -3604,6 +3752,7 @@ mono_llvm_emit_method (MonoCompile *cfg)
        MonoMethodHeader *header;
        MonoExceptionClause *clause;
        LLVMSigInfo sinfo;
+       char **names;
 
        /* The code below might acquire the loader lock, so use it for global locking */
        mono_loader_lock ();
@@ -3661,6 +3810,7 @@ mono_llvm_emit_method (MonoCompile *cfg)
                if (getenv ("LLVM_COUNT")) {
                        if (count == atoi (getenv ("LLVM_COUNT"))) {
                                printf ("LAST: %s\n", mono_method_full_name (cfg->method, TRUE));
+                               fflush (stdout);
                                last = TRUE;
                        }
                        if (count > atoi (getenv ("LLVM_COUNT")))
@@ -3685,12 +3835,17 @@ mono_llvm_emit_method (MonoCompile *cfg)
        method_type = sig_to_llvm_sig_full (ctx, sig, linfo, &sinfo);
        CHECK_FAILURE (ctx);
 
+       /* 
+        * This maps parameter indexes in the original signature to the indexes in
+        * the LLVM signature.
+        */
+       ctx->pindexes = sinfo.pindexes;
+
        method = LLVMAddFunction (module, method_name, method_type);
        ctx->lmethod = method;
 
 #ifdef LLVM_MONO_BRANCH
-       if (linfo->rgctx_arg)
-               LLVMSetFunctionCallConv (method, LLVMMono1CallConv);
+       LLVMSetFunctionCallConv (method, LLVMMono1CallConv);
 #endif
        LLVMSetLinkage (method, LLVMPrivateLinkage);
 
@@ -3707,11 +3862,6 @@ mono_llvm_emit_method (MonoCompile *cfg)
                        LLVM_FAILURE (ctx, "non-finally/catch clause.");
        }
 
-       /* 
-        * This maps parameter indexes in the original signature to the indexes in
-        * the LLVM signature.
-        */
-       ctx->pindexes = sinfo.pindexes;
        if (linfo->rgctx_arg) {
                ctx->rgctx_arg = LLVMGetParam (method, sinfo.rgctx_arg_pindex);
                /*
@@ -3720,16 +3870,34 @@ mono_llvm_emit_method (MonoCompile *cfg)
                 * CC_X86_64_Mono in X86CallingConv.td.
                 */
                LLVMAddAttribute (ctx->rgctx_arg, LLVMInRegAttribute);
+               LLVMSetValueName (ctx->rgctx_arg, "rgctx");
        }
-       if (cfg->vret_addr)
+       if (cfg->vret_addr) {
                values [cfg->vret_addr->dreg] = LLVMGetParam (method, sinfo.vret_arg_pindex);
-       if (sig->hasthis)
+               LLVMSetValueName (values [cfg->vret_addr->dreg], "vret");
+       }
+       if (sig->hasthis) {
                values [cfg->args [0]->dreg] = LLVMGetParam (method, sinfo.this_arg_pindex);
+               LLVMSetValueName (values [cfg->args [0]->dreg], "this");
+       }
+
+       names = g_new (char *, sig->param_count);
+       mono_method_get_param_names (cfg->method, (const char **) names);
+
        for (i = 0; i < sig->param_count; ++i) {
+               char *name;
+
                values [cfg->args [i + sig->hasthis]->dreg] = LLVMGetParam (method, sinfo.pindexes [i]);
+               if (names [i] && names [i][0] != '\0')
+                       name = g_strdup_printf ("arg_%s", names [i]);
+               else
+                       name = g_strdup_printf ("arg_%d", i);
+               LLVMSetValueName (values [cfg->args [i + sig->hasthis]->dreg], name);
+               g_free (name);
                if (linfo->args [i + sig->hasthis].storage == LLVMArgVtypeByVal)
                        LLVMAddAttribute (LLVMGetParam (method, sinfo.pindexes [i]), LLVMByValAttribute);
        }
+       g_free (names);
 
        max_block_num = 0;
        for (bb = cfg->bb_entry; bb; bb = bb->next_bb)
@@ -3749,7 +3917,20 @@ mono_llvm_emit_method (MonoCompile *cfg)
        }
 
        /*
-        * Make a first pass over the code to precreate PHI nodes.
+        * The INDIRECT flag added by OP_LDADDR inhibits optimizations, even if the LDADDR
+        * was later optimized away, so clear these flags, and add them back for the still
+        * present OP_LDADDR instructions.
+        */
+       for (i = 0; i < cfg->next_vreg; ++i) {
+               MonoInst *ins;
+
+               ins = get_vreg_to_inst (cfg, i);
+               if (ins && ins != cfg->rgctx_var)
+                       ins->flags &= ~MONO_INST_INDIRECT;
+       }
+
+       /*
+        * Make a first pass over the code to precreate PHI nodes/set INDIRECT flags.
         */
        for (bb = cfg->bb_entry; bb; bb = bb->next_bb) {
                MonoInst *ins;
@@ -3800,6 +3981,9 @@ mono_llvm_emit_method (MonoCompile *cfg)
                                }
                                break;
                                }
+                       case OP_LDADDR:
+                               ((MonoInst*)ins->inst_p0)->flags |= MONO_INST_INDIRECT;
+                               break;
                        default:
                                break;
                        }
@@ -3858,8 +4042,13 @@ mono_llvm_emit_method (MonoCompile *cfg)
 
                        g_assert (values [sreg1]);
 
-                       g_assert (LLVMTypeOf (values [sreg1]) == LLVMTypeOf (values [phi->dreg]));
-                       LLVMAddIncoming (values [phi->dreg], &values [sreg1], &in_bb, 1);
+                       if (phi->opcode == OP_VPHI) {
+                               g_assert (LLVMTypeOf (ctx->addresses [sreg1]) == LLVMTypeOf (values [phi->dreg]));
+                               LLVMAddIncoming (values [phi->dreg], &ctx->addresses [sreg1], &in_bb, 1);
+                       } else {
+                               g_assert (LLVMTypeOf (values [sreg1]) == LLVMTypeOf (values [phi->dreg]));
+                               LLVMAddIncoming (values [phi->dreg], &values [sreg1], &in_bb, 1);
+                       }
                }
        }
 
@@ -3936,6 +4125,8 @@ mono_llvm_emit_method (MonoCompile *cfg)
        g_free (ctx->vreg_types);
        g_free (ctx->vreg_cli_types);
        g_free (ctx->pindexes);
+       g_free (ctx->is_dead);
+       g_free (ctx->unreachable);
        g_free (debug_name);
        g_ptr_array_free (phi_values, TRUE);
        g_free (ctx->bblocks);
@@ -4053,7 +4244,7 @@ exception_cb (void *data)
 {
        MonoCompile *cfg;
        MonoJitExceptionInfo *ei;
-       guint32 ei_len, i;
+       guint32 ei_len, i, j, nested_len, nindex;
        gpointer *type_info;
        int this_reg, this_offset;
 
@@ -4068,8 +4259,23 @@ exception_cb (void *data)
         */
        cfg->encoded_unwind_ops = mono_unwind_decode_fde ((guint8*)data, &cfg->encoded_unwind_ops_len, NULL, &ei, &ei_len, &type_info, &this_reg, &this_offset);
 
-       cfg->llvm_ex_info = mono_mempool_alloc0 (cfg->mempool, ei_len * sizeof (MonoJitExceptionInfo));
-       cfg->llvm_ex_info_len = ei_len;
+       /* Count nested clauses */
+       nested_len = 0;
+       for (i = 0; i < ei_len; ++i) {
+               for (j = 0; j < ei_len; ++j) {
+                       gint32 cindex1 = *(gint32*)type_info [i];
+                       MonoExceptionClause *clause1 = &cfg->header->clauses [cindex1];
+                       gint32 cindex2 = *(gint32*)type_info [j];
+                       MonoExceptionClause *clause2 = &cfg->header->clauses [cindex2];
+
+                       if (cindex1 != cindex2 && clause1->try_offset >= clause2->try_offset && clause1->handler_offset <= clause2->handler_offset) {
+                               nested_len ++;
+                       }
+               }
+       }
+
+       cfg->llvm_ex_info = mono_mempool_alloc0 (cfg->mempool, (ei_len + nested_len) * sizeof (MonoJitExceptionInfo));
+       cfg->llvm_ex_info_len = ei_len + nested_len;
        memcpy (cfg->llvm_ex_info, ei, ei_len * sizeof (MonoJitExceptionInfo));
        /* Fill the rest of the information from the type info */
        for (i = 0; i < ei_len; ++i) {
@@ -4079,10 +4285,40 @@ exception_cb (void *data)
                cfg->llvm_ex_info [i].flags = clause->flags;
                cfg->llvm_ex_info [i].data.catch_class = clause->data.catch_class;
        }
+
+       /*
+        * For nested clauses, the LLVM produced exception info associates the try interval with
+        * the innermost handler, while mono expects it to be associated with all nesting clauses.
+        */
+       /* FIXME: These should be order with the normal clauses */
+       nindex = ei_len;
+       for (i = 0; i < ei_len; ++i) {
+               for (j = 0; j < ei_len; ++j) {
+                       gint32 cindex1 = *(gint32*)type_info [i];
+                       MonoExceptionClause *clause1 = &cfg->header->clauses [cindex1];
+                       gint32 cindex2 = *(gint32*)type_info [j];
+                       MonoExceptionClause *clause2 = &cfg->header->clauses [cindex2];
+
+                       if (cindex1 != cindex2 && clause1->try_offset >= clause2->try_offset && clause1->handler_offset <= clause2->handler_offset) {
+                               /* 
+                                * The try interval comes from the nested clause, everything else from the
+                                * nesting clause.
+                                */
+                               memcpy (&cfg->llvm_ex_info [nindex], &cfg->llvm_ex_info [j], sizeof (MonoJitExceptionInfo));
+                               cfg->llvm_ex_info [nindex].try_start = cfg->llvm_ex_info [i].try_start;
+                               cfg->llvm_ex_info [nindex].try_end = cfg->llvm_ex_info [i].try_end;
+                               nindex ++;
+                       }
+               }
+       }
+       g_assert (nindex == ei_len + nested_len);
        cfg->llvm_this_reg = this_reg;
        cfg->llvm_this_offset = this_offset;
 
+       /* type_info [i] is cfg mempool allocated, no need to free it */
+
        g_free (ei);
+       g_free (type_info);
 }
 
 static void
@@ -4092,26 +4328,26 @@ add_intrinsics (LLVMModuleRef module)
        {
                LLVMTypeRef memset_params [] = { LLVMPointerType (LLVMInt8Type (), 0), LLVMInt8Type (), LLVMInt32Type (), LLVMInt32Type (), LLVMInt1Type () };
 
-#if LLVM_CHECK_VERSION(2, 8)
-               memset_param_count = 5;
-               memset_func_name = "llvm.memset.p0i8.i32";
-#else
-               memset_param_count = 4;
-               memset_func_name = "llvm.memset.i32";
-#endif
+               if (LLVM_CHECK_VERSION(2, 8)) {
+                       memset_param_count = 5;
+                       memset_func_name = "llvm.memset.p0i8.i32";
+               } else {
+                       memset_param_count = 4;
+                       memset_func_name = "llvm.memset.i32";
+               }
                LLVMAddFunction (module, memset_func_name, LLVMFunctionType (LLVMVoidType (), memset_params, memset_param_count, FALSE));
        }
 
        {
                LLVMTypeRef memcpy_params [] = { LLVMPointerType (LLVMInt8Type (), 0), LLVMPointerType (LLVMInt8Type (), 0), LLVMInt32Type (), LLVMInt32Type (), LLVMInt1Type () };
 
-#if LLVM_CHECK_VERSION(2, 8)
-               memcpy_param_count = 5;
-               memcpy_func_name = "llvm.memcpy.p0i8.p0i8.i32";
-#else
-               memcpy_param_count = 4;
-               memcpy_func_name = "llvm.memcpy.i32";
-#endif
+               if (LLVM_CHECK_VERSION(2, 8)) {
+                       memcpy_param_count = 5;
+                       memcpy_func_name = "llvm.memcpy.p0i8.p0i8.i32";
+               } else {
+                       memcpy_param_count = 4;
+                       memcpy_func_name = "llvm.memcpy.i32";
+               }
 
                LLVMAddFunction (module, memcpy_func_name, LLVMFunctionType (LLVMVoidType (), memcpy_params, memcpy_param_count, FALSE));
        }
@@ -4180,25 +4416,25 @@ add_intrinsics (LLVMModuleRef module)
 
                arg_types [0] = LLVMPointerType (LLVMInt8Type (), 0);
                arg_types [1] = LLVMPointerType (LLVMInt8Type (), 0);
-#if LLVM_CHECK_VERSION(2, 8)
-               eh_selector_name = "llvm.eh.selector";
-               ret_type = LLVMInt32Type ();
-#else
-#if SIZEOF_VOID_P == 8
-               eh_selector_name = "llvm.eh.selector.i64";
-               ret_type = LLVMInt64Type ();
-#else
-               eh_selector_name = "llvm.eh.selector.i32";
-               ret_type = LLVMInt32Type ();
-#endif
-#endif
+               if (LLVM_CHECK_VERSION(2, 8)) {
+                       eh_selector_name = "llvm.eh.selector";
+                       ret_type = LLVMInt32Type ();
+               } else {
+                       if (SIZEOF_VOID_P == 8) {
+                               eh_selector_name = "llvm.eh.selector.i64";
+                               ret_type = LLVMInt64Type ();
+                       } else {
+                               eh_selector_name = "llvm.eh.selector.i32";
+                               ret_type = LLVMInt32Type ();
+                       }
+               }
                LLVMAddFunction (module, eh_selector_name, LLVMFunctionType (ret_type, arg_types, 2, TRUE));
 
                LLVMAddFunction (module, "llvm.eh.exception", LLVMFunctionType (LLVMPointerType (LLVMInt8Type (), 0), NULL, 0, FALSE));
 
                LLVMAddFunction (module, "mono_personality", LLVMFunctionType (LLVMVoidType (), NULL, 0, FALSE));
 
-               LLVMAddFunction (module, "mono_resume_unwind", LLVMFunctionType (LLVMVoidType (), NULL, 0, FALSE));
+               LLVMAddFunction (module, "llvm_resume_unwind_trampoline", LLVMFunctionType (LLVMVoidType (), NULL, 0, FALSE));
        }
 
        /* SSE intrinsics */
@@ -4208,32 +4444,42 @@ add_intrinsics (LLVMModuleRef module)
                vector_type = LLVMVectorType (LLVMInt32Type (), 4);
                arg_types [0] = vector_type;
                arg_types [1] = vector_type;
-               LLVMAddFunction (module, "llvm.x86.sse41.pminud", LLVMFunctionType (vector_type, arg_types, 2, FALSE));                                 
-               LLVMAddFunction (module, "llvm.x86.sse41.pmaxud", LLVMFunctionType (vector_type, arg_types, 2, FALSE));                                 
+               LLVMAddFunction (module, "llvm.x86.sse41.pminud", LLVMFunctionType (vector_type, arg_types, 2, FALSE));
+               LLVMAddFunction (module, "llvm.x86.sse41.pmaxud", LLVMFunctionType (vector_type, arg_types, 2, FALSE));
+               LLVMAddFunction (module, "llvm.x86.sse2.pcmpeq.d", LLVMFunctionType (vector_type, arg_types, 2, FALSE));
 
                vector_type = LLVMVectorType (LLVMInt16Type (), 8);
                arg_types [0] = vector_type;
                arg_types [1] = vector_type;
-               LLVMAddFunction (module, "llvm.x86.sse41.pminuw", LLVMFunctionType (vector_type, arg_types, 2, FALSE));                                 
-               LLVMAddFunction (module, "llvm.x86.sse41.pmaxuw", LLVMFunctionType (vector_type, arg_types, 2, FALSE));                                 
+               LLVMAddFunction (module, "llvm.x86.sse41.pminuw", LLVMFunctionType (vector_type, arg_types, 2, FALSE));
+               LLVMAddFunction (module, "llvm.x86.sse2.pmins.w", LLVMFunctionType (vector_type, arg_types, 2, FALSE));
+               LLVMAddFunction (module, "llvm.x86.sse41.pmaxuw", LLVMFunctionType (vector_type, arg_types, 2, FALSE));
+               LLVMAddFunction (module, "llvm.x86.sse2.pcmpeq.w", LLVMFunctionType (vector_type, arg_types, 2, FALSE));
 
                vector_type = LLVMVectorType (LLVMInt8Type (), 16);
                arg_types [0] = vector_type;
                arg_types [1] = vector_type;
-               LLVMAddFunction (module, "llvm.x86.sse2.pminu.b", LLVMFunctionType (vector_type, arg_types, 2, FALSE));                                 
-               LLVMAddFunction (module, "llvm.x86.sse2.pmaxu.b", LLVMFunctionType (vector_type, arg_types, 2, FALSE));                                 
+               LLVMAddFunction (module, "llvm.x86.sse2.pminu.b", LLVMFunctionType (vector_type, arg_types, 2, FALSE));
+               LLVMAddFunction (module, "llvm.x86.sse2.pmaxu.b", LLVMFunctionType (vector_type, arg_types, 2, FALSE));
+               LLVMAddFunction (module, "llvm.x86.sse2.pcmpeq.b", LLVMFunctionType (vector_type, arg_types, 2, FALSE));
+               LLVMAddFunction (module, "llvm.x86.sse2.pcmpgt.b", LLVMFunctionType (vector_type, arg_types, 2, FALSE));
+
+               vector_type = LLVMVectorType (LLVMInt64Type (), 2);
+               arg_types [0] = vector_type;
+               arg_types [1] = vector_type;
+               LLVMAddFunction (module, "llvm.x86.sse41.pcmpeqq", LLVMFunctionType (vector_type, arg_types, 2, FALSE));
 
                vector_type = LLVMVectorType (LLVMDoubleType (), 2);
                arg_types [0] = vector_type;
                arg_types [1] = vector_type;
-               LLVMAddFunction (module, "llvm.x86.sse2.min.pd", LLVMFunctionType (vector_type, arg_types, 2, FALSE));                                  
-               LLVMAddFunction (module, "llvm.x86.sse2.max.pd", LLVMFunctionType (vector_type, arg_types, 2, FALSE));                                  
+               LLVMAddFunction (module, "llvm.x86.sse2.min.pd", LLVMFunctionType (vector_type, arg_types, 2, FALSE));
+               LLVMAddFunction (module, "llvm.x86.sse2.max.pd", LLVMFunctionType (vector_type, arg_types, 2, FALSE));
 
                vector_type = LLVMVectorType (LLVMFloatType (), 4);
                arg_types [0] = vector_type;
                arg_types [1] = vector_type;
-               LLVMAddFunction (module, "llvm.x86.sse2.min.ps", LLVMFunctionType (vector_type, arg_types, 2, FALSE));                                  
-               LLVMAddFunction (module, "llvm.x86.sse2.max.ps", LLVMFunctionType (vector_type, arg_types, 2, FALSE));                                  
+               LLVMAddFunction (module, "llvm.x86.sse2.min.ps", LLVMFunctionType (vector_type, arg_types, 2, FALSE));
+               LLVMAddFunction (module, "llvm.x86.sse2.max.ps", LLVMFunctionType (vector_type, arg_types, 2, FALSE));
        }
 
        /* Load/Store intrinsics */
@@ -4268,6 +4514,8 @@ mono_llvm_init (void)
 static void
 init_jit_module (void)
 {
+       MonoJitICallInfo *info;
+
        if (jit_module_inited)
                return;
 
@@ -4286,7 +4534,9 @@ init_jit_module (void)
 
        jit_module.llvm_types = g_hash_table_new (NULL, NULL);
 
-       LLVMAddGlobalMapping (ee, LLVMGetNamedFunction (jit_module.module, "mono_resume_unwind"), mono_resume_unwind);
+       info = mono_find_jit_icall_by_name ("llvm_resume_unwind_trampoline");
+       g_assert (info);
+       LLVMAddGlobalMapping (ee, LLVMGetNamedFunction (jit_module.module, "llvm_resume_unwind_trampoline"), (void*)info->func);
 
        jit_module_inited = TRUE;
 
@@ -4301,6 +4551,11 @@ mono_llvm_cleanup (void)
 
        if (jit_module.llvm_types)
                g_hash_table_destroy (jit_module.llvm_types);
+
+       if (aot_module.module)
+               LLVMDisposeModule (aot_module.module);
+
+       LLVMContextDispose (LLVMGetGlobalContext ());
 }
 
 void
@@ -4309,6 +4564,8 @@ mono_llvm_create_aot_module (const char *got_symbol)
        /* Delete previous module */
        if (aot_module.plt_entries)
                g_hash_table_destroy (aot_module.plt_entries);
+       if (aot_module.module)
+               LLVMDisposeModule (aot_module.module);
 
        memset (&aot_module, 0, sizeof (aot_module));
 
@@ -4392,9 +4649,6 @@ mono_llvm_emit_aot_module (const char *filename, int got_size)
   - Emit LLVM IR from the mono IR using the LLVM C API.
   - The original arch specific code remains, so we can fall back to it if we run
     into something we can't handle.
-  FIXME:
-  - llvm's PrettyStackTrace class seems to register a signal handler which screws
-    up our GC. Also, it calls sigaction () a _lot_ of times instead of just once.
 */
 
 /*  
@@ -4488,9 +4742,7 @@ mono_llvm_emit_aot_module (const char *filename, int got_size)
 /* FIXME: Normalize some aspects of the mono IR to allow easier translation, like:
  *   - each bblock should end with a branch
  *   - setting the return value, making cfg->ret non-volatile
- * - merge some changes back to HEAD, to reduce the differences.
  * - avoid some transformations in the JIT which make it harder for us to generate
  *   code.
- * - fix memory leaks.
  * - use pointer types to help optimizations.
  */