* Removed all Id tags.
[cacao.git] / src / vm / jit / parse.c
index 03f9c94fbb6a7da4fdfcc757e72a6ad0aec69b98..068dc65afed4e81e1d102e31ffbf3505206554c9 100644 (file)
@@ -1,6 +1,6 @@
 /* src/vm/jit/parse.c - parser for JavaVM to intermediate code translation
 
-   Copyright (C) 1996-2005, 2006 R. Grafl, A. Krall, C. Kruegel,
+   Copyright (C) 1996-2005, 2006, 2007 R. Grafl, A. Krall, C. Kruegel,
    C. Oates, R. Obermaisser, M. Platter, M. Probst, S. Ring,
    E. Steiner, C. Thalinger, D. Thuernbeck, P. Tomsich, C. Ullrich,
    J. Wenninger, Institut f. Computersprachen - TU Wien
    Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
    02110-1301, USA.
 
-   Contact: cacao@cacaojvm.org
-
-   Author: Andreas Krall
-
-   Changes: Carolyn Oates
-            Edwin Steiner
-            Joseph Wenninger
-            Christian Thalinger
-
-   $Id: parse.c 5332 2006-09-05 19:38:28Z twisti $
-
 */
 
 
 #include "vm/types.h"
 
 #include "mm/memory.h"
+
 #include "native/native.h"
 
-#if defined(ENABLE_THREADS)
-# include "threads/native/lock.h"
-#endif
+#include "threads/lock-common.h"
 
 #include "toolbox/logging.h"
+
 #include "vm/builtin.h"
 #include "vm/exceptions.h"
 #include "vm/global.h"
-#include "vm/linker.h"
-#include "vm/loader.h"
-#include "vm/resolve.h"
-#include "vm/options.h"
-#include "vm/statistics.h"
 #include "vm/stringlocal.h"
-#include "vm/suck.h"
+
 #include "vm/jit/asmpart.h"
 #include "vm/jit/jit.h"
 #include "vm/jit/parse.h"
-#include "vm/jit/patcher.h"
 #include "vm/jit/loop/loop.h"
 
-/*******************************************************************************
+#include "vmcore/linker.h"
+#include "vmcore/loader.h"
+#include "vmcore/options.h"
+#include "vm/resolve.h"
 
-       function 'parse' scans the JavaVM code and generates intermediate code
+#if defined(ENABLE_STATISTICS)
+# include "vmcore/statistics.h"
+#endif
 
-       During parsing the block index table is used to store at bit pos 0
-       a flag which marks basic block starts and at position 1 to 31 the
-       intermediate instruction index. After parsing the block index table
-       is scanned, for marked positions a block is generated and the block
-       number is stored in the block index table.
+#include "vmcore/suck.h"
+
+#define INSTRUCTIONS_INCREMENT  5  /* number of additional instructions to    */
+                                   /* allocate if space runs out              */
+
+
+/* local macros ***************************************************************/
+
+#define BYTECODEINDEX_TO_BASICBLOCK(dst) \
+    do { \
+        (dst).block = \
+            parse_bytecodeindex_to_basicblock(jd, &pd, (dst).insindex); \
+    } while (0)
+
+
+/* parserdata_t ***************************************************************/
+
+typedef struct parsedata_t parsedata_t;
+
+struct parsedata_t {
+       u1          *bytecodestart;         /* start of bytecode instructions     */
+       u1          *basicblockstart;       /* start of bytecode basic-blocks     */
+
+       s4          *bytecodemap;           /* bytecode to IR mapping             */
+       
+       instruction *instructions;          /* instruction array                  */
+       s4           instructionslength;    /* length of the instruction array    */
+
+       s4          *instructionmap;        /* IR to basic-block mapping          */
+};
+
+
+/* parse_setup *****************************************************************
+
+   Fills the passed parsedata_t structure.
+
+*******************************************************************************/
+
+static void parse_setup(jitdata *jd, parsedata_t *pd)
+{
+       methodinfo *m;
+
+       /* get required compiler data */
+
+       m = jd->m;
+
+       /* bytecode start array */
+
+       pd->bytecodestart = DMNEW(u1, m->jcodelength + 1);
+       MZERO(pd->bytecodestart, u1, m->jcodelength + 1);
+
+       /* bytecode basic-block start array */
+
+       pd->basicblockstart = DMNEW(u1, m->jcodelength + 1);
+       MZERO(pd->basicblockstart, u1, m->jcodelength + 1);
+
+       /* bytecode instruction index to IR instruction mapping */
+
+       pd->bytecodemap = DMNEW(s4, m->jcodelength + 1);
+       MSET(pd->bytecodemap, -1, s4, m->jcodelength + 1);
+
+       /* allocate the instruction array */
+
+       pd->instructionslength = m->jcodelength + 1;
+       pd->instructions = DMNEW(instruction, pd->instructionslength);
+
+       /* Zero the intermediate instructions array so we don't have any
+          invalid pointers in it if we cannot finish stack_analyse(). */
+
+       MZERO(pd->instructions, instruction, pd->instructionslength);
+
+       /* The instructionmap is allocated later when we know the count of
+          instructions. */
+
+       pd->instructionmap = NULL;
+}
+
+
+/* parse_realloc_instructions **************************************************
+
+   Reallocate the instructions array so there is room for at least N 
+   additional instructions.
+
+   RETURN VALUE:
+       the new value for iptr
+
+*******************************************************************************/
+
+static instruction *parse_realloc_instructions(parsedata_t *pd, s4 icount, s4 n)
+{
+       /* increase the size of the instruction array */
+
+       pd->instructionslength += (n + INSTRUCTIONS_INCREMENT);
+
+       /* reallocate the array */
+
+       pd->instructions = DMREALLOC(pd->instructions, instruction, icount,
+                                                                pd->instructionslength);
+       MZERO(pd->instructions + icount, instruction,
+                 (pd->instructionslength - icount));
+
+       /* return the iptr */
+
+       return pd->instructions + icount;
+}
+
+
+/* parse_bytecodeindex_to_basicblock *******************************************
+
+   Resolves a bytecode index to the corresponding basic block.
+
+*******************************************************************************/
+
+static basicblock *parse_bytecodeindex_to_basicblock(jitdata *jd,
+                                                                                                        parsedata_t *pd,
+                                                                                                        s4 bcindex)
+{
+       s4          irindex;
+       basicblock *bb;
+
+       irindex = pd->bytecodemap[bcindex];
+       bb      = jd->basicblocks + pd->instructionmap[irindex];
+
+       return bb;
+}
+
+
+/* parse_mark_exception_boundaries *********************************************
+
+   Mark exception handlers and the boundaries of the handled regions as
+   basic block boundaries.
+
+   IN:
+       jd...............current jitdata
+
+   RETURN VALUE:
+       true.............everything ok
+          false............an exception has been thrown
 
 *******************************************************************************/
 
-static exceptiontable * new_fillextable(
-                                                                       jitdata *jd,
-                                                                       methodinfo *m, 
-                                                                       exceptiontable *extable, 
-                                                                       exceptiontable *raw_extable, 
-                                                               int exceptiontablelength, 
-                                                                       int *block_count)
+static bool parse_mark_exception_boundaries(jitdata *jd, parsedata_t *pd)
 {
-       int b_count, p, src;
+       s4                   bcindex;
+       s4                   i;
+       s4                   len;
+       raw_exception_entry *rex;
+       methodinfo          *m;
+
+       m = jd->m;
        
-       if (exceptiontablelength == 0) 
-               return extable;
+       len = m->rawexceptiontablelength;
+
+       if (len == 0)
+               return true;
 
-       b_count = *block_count;
+       rex = m->rawexceptiontable;
+
+       for (i = 0; i < len; ++i, ++rex) {
 
-       for (src = exceptiontablelength-1; src >=0; src--) {
                /* the start of the handled region becomes a basic block start */
-               p = raw_extable[src].startpc;
-               CHECK_BYTECODE_INDEX(p);
-               extable->startpc = p;
-               NEW_MARK_BASICBLOCK(p);
+
+               bcindex = rex->startpc;
+               CHECK_BYTECODE_INDEX(bcindex);
+               MARK_BASICBLOCK(pd, bcindex);
                
-               p = raw_extable[src].endpc; /* see JVM Spec 4.7.3 */
-               CHECK_BYTECODE_INDEX_EXCLUSIVE(p);
+               bcindex = rex->endpc; /* see JVM Spec 4.7.3 */
+               CHECK_BYTECODE_INDEX_EXCLUSIVE(bcindex);
+
+               /* check that the range is valid */
 
 #if defined(ENABLE_VERIFIER)
-               if (p <= raw_extable[src].startpc) {
-                       exceptions_throw_verifyerror(m,
-                               "Invalid exception handler range");
-                       return NULL;
+               if (bcindex <= rex->startpc) {
+                       exceptions_throw_verifyerror(m, "Invalid exception handler range");
+                       return false;
                }
 #endif
-               extable->endpc = p;
                
-               /* end of handled region becomes a basic block boundary  */
-               /* (If it is the bytecode end, we'll use the special     */
-               /* end block that is created anyway.)                    */
-               if (p < m->jcodelength) 
-                       NEW_MARK_BASICBLOCK(p);
+               /* End of handled region becomes a basic block boundary (if it
+                  is the bytecode end, we'll use the special end block that
+                  is created anyway). */
+
+               if (bcindex < m->jcodelength)
+                       MARK_BASICBLOCK(pd, bcindex);
+               else
+                       jd->branchtoend = true;
 
                /* the start of the handler becomes a basic block start  */
-               p = raw_extable[src].handlerpc;
-               CHECK_BYTECODE_INDEX(p);
-               extable->handlerpc = p;
-               NEW_MARK_BASICBLOCK(p);
-
-               extable->catchtype = raw_extable[src].catchtype;
-               extable->next = NULL;
-               extable->down = &extable[1];
-               extable--;
+
+               bcindex = rex->handlerpc;
+               CHECK_BYTECODE_INDEX(bcindex);
+               MARK_BASICBLOCK(pd, bcindex);
        }
 
-       *block_count = b_count;
-       
        /* everything ok */
-       return extable;
+
+       return true;
 
 #if defined(ENABLE_VERIFIER)
 throw_invalid_bytecode_index:
        exceptions_throw_verifyerror(m,
                                                                 "Illegal bytecode index in exception table");
-       return NULL;
+       return false;
 #endif
 }
 
+
+/* parse_resolve_exception_table ***********************************************
+
+   Enter the exception handlers and their ranges, resolved to basicblock *s,
+   in the jitdata.
+
+   IN:
+       jd...............current jitdata
+
+   RETURN VALUE:
+          true.............everything ok
+          false............an exception has been thrown
+
+*******************************************************************************/
+
+static bool parse_resolve_exception_table(jitdata *jd, parsedata_t *pd)
+{
+       methodinfo          *m;
+       raw_exception_entry *rex;
+       exception_entry     *ex;
+       s4                   i;
+       s4                   len;
+       classinfo           *exclass;
+
+       m = jd->m;
+
+       len = m->rawexceptiontablelength;
+
+       /* common case: no handler entries */
+
+       if (len == 0)
+               return true;
+
+       /* allocate the exception table */
+
+       jd->exceptiontablelength = len;
+       jd->exceptiontable = DMNEW(exception_entry, len + 1); /* XXX why +1? */
+
+       /* copy and resolve the entries */
+
+       ex = jd->exceptiontable;
+       rex = m->rawexceptiontable;
+
+       for (i = 0; i < len; ++i, ++rex, ++ex) {
+               /* resolve instruction indices to basic blocks */
+
+               ex->start   = parse_bytecodeindex_to_basicblock(jd, pd, rex->startpc);
+               ex->end     = parse_bytecodeindex_to_basicblock(jd, pd, rex->endpc);
+               ex->handler = parse_bytecodeindex_to_basicblock(jd, pd, rex->handlerpc);
+
+               /* lazily resolve the catchtype */
+
+               if (rex->catchtype.any != NULL) {
+                       if (!resolve_classref_or_classinfo(m,
+                                                                                          rex->catchtype,
+                                                                                          resolveLazy, true, false,
+                                                                                          &exclass))
+                               return false;
+
+                       /* if resolved, enter the result of resolution in the table */
+
+                       if (exclass != NULL)
+                               rex->catchtype.cls = exclass;
+               }
+
+               ex->catchtype = rex->catchtype;
+               ex->next = NULL;   /* set by loop analysis */
+               ex->down = ex + 1; /* link to next exception entry */
+       }
+
+       /* terminate the ->down linked list */
+
+       assert(ex != jd->exceptiontable);
+       ex[-1].down = NULL;
+
+       return true;
+}
+
+
+/*******************************************************************************
+
+       function 'parse' scans the JavaVM code and generates intermediate code
+
+       During parsing the block index table is used to store at bit pos 0
+       a flag which marks basic block starts and at position 1 to 31 the
+       intermediate instruction index. After parsing the block index table
+       is scanned, for marked positions a block is generated and the block
+       number is stored in the block index table.
+
+*******************************************************************************/
+
 /*** macro for checking the length of the bytecode ***/
 
 #if defined(ENABLE_VERIFIER)
@@ -156,29 +368,34 @@ throw_invalid_bytecode_index:
 #define CHECK_END_OF_BYTECODE(neededlength)
 #endif /* ENABLE_VERIFIER */
 
-bool new_parse(jitdata *jd)
+bool parse(jitdata *jd)
 {
-       methodinfo  *m;             /* method being parsed                      */
-       codeinfo    *code;
-       codegendata *cd;
-       int  p;                     /* java instruction counter                 */
-       int  nextp;                 /* start of next java instruction           */
-       int  opcode;                /* java opcode                              */
-       int  i;                     /* temporary for different uses (ctrs)      */
-       int  ipc = 0;               /* intermediate instruction counter         */
-       int  b_count = 0;           /* basic block counter                      */
-       int  s_count = 0;           /* stack element counter                    */
-       bool blockend = false;      /* true if basic block end has been reached */
-       bool iswide = false;        /* true if last instruction was a wide      */
-       instruction *iptr;          /* current ptr into instruction array       */
-       u1 *instructionstart;       /* 1 for pcs which are valid instr. starts  */
+       methodinfo  *m;                     /* method being parsed                */
+       parsedata_t  pd;
+       instruction *iptr;                  /* current ptr into instruction array */
+
+       s4           bcindex;               /* bytecode instruction index         */
+       s4           nextbc;                /* start of next bytecode instruction */
+       s4           opcode;                /* bytecode instruction opcode        */
+
+       s4           irindex;               /* IR instruction index               */
+       s4           ircount;               /* IR instruction count               */
+
+       s4           bbcount;               /* basic block count                  */
+
+       int  s_count = 0;             /* stack element counter                    */
+       bool blockend;                /* true if basic block end has been reached */
+       bool iswide;                  /* true if last instruction was a wide      */
+
        constant_classref  *cr;
        constant_classref  *compr;
        classinfo          *c;
        builtintable_entry *bte;
-       constant_FMIref    *mr;
+       constant_FMIref    *fmi;
        methoddesc         *md;
        unresolved_method  *um;
+       unresolved_field   *uf;
+
        resolve_result_t    result;
        u2                  lineindex = 0;
        u2                  currentline = 0;
@@ -186,43 +403,49 @@ bool new_parse(jitdata *jd)
        u4                  flags;
        basicblock         *bptr;
 
-       /* get required compiler data */
-
-       m    = jd->m;
-       code = jd->code;
-       cd   = jd->cd;
-
-       /* allocate instruction array and block index table */
+       int                *local_map; /* local pointer to renaming map           */
+                                      /* is assigned to rd->local_map at the end */
+       branch_target_t *table;
+       lookup_target_t *lookup;
+       s4               i;
+       s4               j;
 
-       /* 1 additional for end ipc  */
-       jd->new_basicblockindex = DMNEW(s4, m->jcodelength + 1);
-       memset(jd->new_basicblockindex, 0, sizeof(s4) * (m->jcodelength + 1));
+       /* get required compiler data */
 
-       instructionstart = DMNEW(u1, m->jcodelength + 1);
-       memset(instructionstart, 0, sizeof(u1) * (m->jcodelength + 1));
+       m = jd->m;
 
-       /* IMPORTANT: We assume that parsing creates at most one instruction per */
-       /*            byte of original bytecode!                                 */
+       /* allocate buffers for local variable renaming */
 
-       iptr = jd->new_instructions = DMNEW(instruction, m->jcodelength);
+       local_map = DMNEW(int, m->maxlocals * 5);
 
-       /* compute branch targets of exception table */
+       for (i = 0; i < m->maxlocals; i++) {
+               local_map[i * 5 + 0] = 0;
+               local_map[i * 5 + 1] = 0;
+               local_map[i * 5 + 2] = 0;
+               local_map[i * 5 + 3] = 0;
+               local_map[i * 5 + 4] = 0;
+       }
 
-       if (!new_fillextable(jd, m,
-                       &(cd->exceptiontable[cd->exceptiontablelength-1]),
-                       m->exceptiontable,
-                       m->exceptiontablelength,
-                       &b_count))
-       {
+       /* initialize the parse data structures */
+  
+       parse_setup(jd, &pd);
+  
+       /* initialize local variables */
+  
+       iptr     = pd.instructions;
+       ircount  = 0;
+       bbcount  = 0;
+       blockend = false;
+       iswide   = false;
+
+       /* mark basic block boundaries for exception table */
+
+       if (!parse_mark_exception_boundaries(jd, &pd))
                return false;
-       }
 
-       s_count = 1 + m->exceptiontablelength; /* initialize stack element counter   */
+       /* initialize stack element counter */
 
-#if defined(ENABLE_THREADS)
-       if (checksync && (m->flags & ACC_SYNCHRONIZED))
-               jd->isleafmethod = false;
-#endif
+       s_count = 1 + m->rawexceptiontablelength;
 
        /* setup line number info */
 
@@ -238,58 +461,82 @@ bool new_parse(jitdata *jd)
 
        /*** LOOP OVER ALL BYTECODE INSTRUCTIONS **********************************/
 
-       for (p = 0; p < m->jcodelength; p = nextp) {
+       for (bcindex = 0; bcindex < m->jcodelength; bcindex = nextbc) {
 
-               /* mark this position as a valid instruction start */
+               /* mark this position as a valid bytecode instruction start */
 
-               instructionstart[p] = 1;
+               pd.bytecodestart[bcindex] = 1;
 
                /* change the current line number, if necessary */
 
                /* XXX rewrite this using pointer arithmetic */
 
-               if (linepcchange == p) {
+               if (linepcchange == bcindex) {
                        if (m->linenumbercount > lineindex) {
 next_linenumber:
                                currentline = m->linenumbers[lineindex].line_number;
                                lineindex++;
                                if (lineindex < m->linenumbercount) {
                                        linepcchange = m->linenumbers[lineindex].start_pc;
-                                       if (linepcchange == p)
+                                       if (linepcchange == bcindex)
                                                goto next_linenumber;
                                }
                        }
                }
 
-               /* fetch next opcode  */
 fetch_opcode:
-               opcode = SUCK_BE_U1(m->jcode + p);
+               /* fetch next opcode  */        
 
-               /* store intermediate instruction count (bit 0 mark block starts) */
+               opcode = SUCK_BE_U1(m->jcode + bcindex);
 
-               jd->new_basicblockindex[p] |= (ipc << 1);
+               /* If the previous instruction was a block-end instruction,
+                  mark the current bytecode instruction as basic-block
+                  starting instruction. */
 
-               /* some compilers put a JAVA_NOP after a blockend instruction */
+               /* NOTE: Some compilers put a JAVA_NOP after a blockend
+                  instruction. */
 
                if (blockend && (opcode != JAVA_NOP)) {
-                       /* start new block */
-
-                       NEW_MARK_BASICBLOCK(p);
+                       MARK_BASICBLOCK(&pd, bcindex);
                        blockend = false;
                }
 
+               /* If the current bytecode instruction was marked as
+                  basic-block starting instruction before (e.g. blockend,
+                  forward-branch target), mark the current IR instruction
+                  too. */
+
+               if (pd.basicblockstart[bcindex] != 0) {
+                       /* We need a NOP as last instruction in each basic block
+                          for basic block reordering (may be replaced with a GOTO
+                          later). */
+
+                       INSTRUCTIONS_CHECK(1);
+                       OP(ICMD_NOP);
+               }
+
+               /* store intermediate instruction count (bit 0 mark block starts) */
+
+               pd.bytecodemap[bcindex] = ircount;
+
                /* compute next instruction start */
 
-               nextp = p + jcommandsize[opcode];
+               nextbc = bcindex + jcommandsize[opcode];
 
-               CHECK_END_OF_BYTECODE(nextp);
+               CHECK_END_OF_BYTECODE(nextbc);
 
                /* add stack elements produced by this instruction */
 
                s_count += stackreq[opcode];
 
-               /* translate this bytecode instruction */
+               /* We check here for the space of 1 instruction in the
+                  instruction array.  If an opcode is converted to more than
+                  1 instruction, this is checked in the corresponding
+                  case. */
 
+               INSTRUCTIONS_CHECK(1);
+
+               /* translate this bytecode instruction */
                switch (opcode) {
 
                case JAVA_NOP:
@@ -298,20 +545,20 @@ fetch_opcode:
                /* pushing constants onto the stack ***********************************/
 
                case JAVA_BIPUSH:
-                       NEW_OP_LOADCONST_I(SUCK_BE_S1(m->jcode + p + 1));
+                       OP_LOADCONST_I(SUCK_BE_S1(m->jcode + bcindex + 1));
                        break;
 
                case JAVA_SIPUSH:
-                       NEW_OP_LOADCONST_I(SUCK_BE_S2(m->jcode + p + 1));
+                       OP_LOADCONST_I(SUCK_BE_S2(m->jcode + bcindex + 1));
                        break;
 
                case JAVA_LDC1:
-                       i = SUCK_BE_U1(m->jcode + p + 1);
+                       i = SUCK_BE_U1(m->jcode + bcindex + 1);
                        goto pushconstantitem;
 
                case JAVA_LDC2:
                case JAVA_LDC2W:
-                       i = SUCK_BE_U2(m->jcode + p + 1);
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
 
                pushconstantitem:
 
@@ -325,30 +572,29 @@ fetch_opcode:
 
                        switch (m->class->cptags[i]) {
                        case CONSTANT_Integer:
-                               NEW_OP_LOADCONST_I(((constant_integer *) (m->class->cpinfos[i]))->value);
+                               OP_LOADCONST_I(((constant_integer *) (m->class->cpinfos[i]))->value);
                                break;
                        case CONSTANT_Long:
-                               NEW_OP_LOADCONST_L(((constant_long *) (m->class->cpinfos[i]))->value);
+                               OP_LOADCONST_L(((constant_long *) (m->class->cpinfos[i]))->value);
                                break;
                        case CONSTANT_Float:
-                               NEW_OP_LOADCONST_F(((constant_float *) (m->class->cpinfos[i]))->value);
+                               OP_LOADCONST_F(((constant_float *) (m->class->cpinfos[i]))->value);
                                break;
                        case CONSTANT_Double:
-                               NEW_OP_LOADCONST_D(((constant_double *) (m->class->cpinfos[i]))->value);
+                               OP_LOADCONST_D(((constant_double *) (m->class->cpinfos[i]))->value);
                                break;
                        case CONSTANT_String:
-                               NEW_OP_LOADCONST_STRING(literalstring_new((utf *) (m->class->cpinfos[i])));
+                               OP_LOADCONST_STRING(literalstring_new((utf *) (m->class->cpinfos[i])));
                                break;
                        case CONSTANT_Class:
                                cr = (constant_classref *) (m->class->cpinfos[i]);
 
-                               if (!resolve_classref(m, cr, resolveLazy, true,
-                                                                         true, &c))
+                               if (!resolve_classref(m, cr, resolveLazy, true, true, &c))
                                        return false;
 
                                /* if not resolved, c == NULL */
 
-                               NEW_OP_LOADCONST_CLASSINFO_OR_CLASSREF(c, cr, 0 /* no extra flags */);
+                               OP_LOADCONST_CLASSINFO_OR_CLASSREF_CHECK(c, cr);
 
                                break;
 
@@ -362,7 +608,7 @@ fetch_opcode:
                        break;
 
                case JAVA_ACONST_NULL:
-                       NEW_OP_LOADCONST_NULL();
+                       OP_LOADCONST_NULL();
                        break;
 
                case JAVA_ICONST_M1:
@@ -372,23 +618,83 @@ fetch_opcode:
                case JAVA_ICONST_3:
                case JAVA_ICONST_4:
                case JAVA_ICONST_5:
-                       NEW_OP_LOADCONST_I(opcode - JAVA_ICONST_0);
+                       OP_LOADCONST_I(opcode - JAVA_ICONST_0);
                        break;
 
                case JAVA_LCONST_0:
                case JAVA_LCONST_1:
-                       NEW_OP_LOADCONST_L(opcode - JAVA_LCONST_0);
+                       OP_LOADCONST_L(opcode - JAVA_LCONST_0);
                        break;
 
                case JAVA_FCONST_0:
                case JAVA_FCONST_1:
                case JAVA_FCONST_2:
-                       NEW_OP_LOADCONST_F(opcode - JAVA_FCONST_0);
+                       OP_LOADCONST_F(opcode - JAVA_FCONST_0);
                        break;
 
                case JAVA_DCONST_0:
                case JAVA_DCONST_1:
-                       NEW_OP_LOADCONST_D(opcode - JAVA_DCONST_0);
+                       OP_LOADCONST_D(opcode - JAVA_DCONST_0);
+                       break;
+
+               /* stack operations ***************************************************/
+
+               /* We need space for additional ICMDs so we can translate these       */
+               /* instructions to sequences of ICMD_COPY and ICMD_MOVE instructions. */
+
+               case JAVA_DUP_X1:
+                       INSTRUCTIONS_CHECK(4);
+                       OP(opcode);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       break;
+
+               case JAVA_DUP_X2:
+                       INSTRUCTIONS_CHECK(6);
+                       OP(opcode);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       break;
+
+               case JAVA_DUP2:
+                       INSTRUCTIONS_CHECK(2);
+                       OP(opcode);
+                       OP(ICMD_NOP);
+                       break;
+
+               case JAVA_DUP2_X1:
+                       INSTRUCTIONS_CHECK(7);
+                       OP(opcode);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       break;
+
+               case JAVA_DUP2_X2:
+                       INSTRUCTIONS_CHECK(9);
+                       OP(opcode);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
+                       break;
+
+               case JAVA_SWAP:
+                       INSTRUCTIONS_CHECK(3);
+                       OP(opcode);
+                       OP(ICMD_NOP);
+                       OP(ICMD_NOP);
                        break;
 
                /* local variable access instructions *********************************/
@@ -397,124 +703,124 @@ fetch_opcode:
                case JAVA_FLOAD:
                case JAVA_ALOAD:
                        if (iswide == false) {
-                               i = SUCK_BE_U1(m->jcode + p + 1);
+                               i = SUCK_BE_U1(m->jcode + bcindex + 1);
                        }
                        else {
-                               i = SUCK_BE_U2(m->jcode + p + 1);
-                               nextp = p + 3;
+                               i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                               nextbc = bcindex + 3;
                                iswide = false;
                        }
-                       NEW_OP_LOAD_ONEWORD(opcode, i);
+                       OP_LOAD_ONEWORD(opcode, i, opcode - JAVA_ILOAD);
                        break;
 
                case JAVA_LLOAD:
                case JAVA_DLOAD:
                        if (iswide == false) {
-                               i = SUCK_BE_U1(m->jcode + p + 1);
+                               i = SUCK_BE_U1(m->jcode + bcindex + 1);
                        }
                        else {
-                               i = SUCK_BE_U2(m->jcode + p + 1);
-                               nextp = p + 3;
+                               i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                               nextbc = bcindex + 3;
                                iswide = false;
                        }
-                       NEW_OP_LOAD_TWOWORD(opcode, i);
+                       OP_LOAD_TWOWORD(opcode, i, opcode - JAVA_ILOAD);
                        break;
 
                case JAVA_ILOAD_0:
                case JAVA_ILOAD_1:
                case JAVA_ILOAD_2:
                case JAVA_ILOAD_3:
-                       NEW_OP_LOAD_ONEWORD(ICMD_ILOAD, opcode - JAVA_ILOAD_0);
+                       OP_LOAD_ONEWORD(ICMD_ILOAD, opcode - JAVA_ILOAD_0, TYPE_INT);
                        break;
 
                case JAVA_LLOAD_0:
                case JAVA_LLOAD_1:
                case JAVA_LLOAD_2:
                case JAVA_LLOAD_3:
-                       NEW_OP_LOAD_TWOWORD(ICMD_LLOAD, opcode - JAVA_LLOAD_0);
+                       OP_LOAD_TWOWORD(ICMD_LLOAD, opcode - JAVA_LLOAD_0, TYPE_LNG);
                        break;
 
                case JAVA_FLOAD_0:
                case JAVA_FLOAD_1:
                case JAVA_FLOAD_2:
                case JAVA_FLOAD_3:
-                       NEW_OP_LOAD_ONEWORD(ICMD_FLOAD, opcode - JAVA_FLOAD_0);
+                       OP_LOAD_ONEWORD(ICMD_FLOAD, opcode - JAVA_FLOAD_0, TYPE_FLT);
                        break;
 
                case JAVA_DLOAD_0:
                case JAVA_DLOAD_1:
                case JAVA_DLOAD_2:
                case JAVA_DLOAD_3:
-                       NEW_OP_LOAD_TWOWORD(ICMD_DLOAD, opcode - JAVA_DLOAD_0);
+                       OP_LOAD_TWOWORD(ICMD_DLOAD, opcode - JAVA_DLOAD_0, TYPE_DBL);
                        break;
 
                case JAVA_ALOAD_0:
                case JAVA_ALOAD_1:
                case JAVA_ALOAD_2:
                case JAVA_ALOAD_3:
-                       NEW_OP_LOAD_ONEWORD(ICMD_ALOAD, opcode - JAVA_ALOAD_0);
+                       OP_LOAD_ONEWORD(ICMD_ALOAD, opcode - JAVA_ALOAD_0, TYPE_ADR);
                        break;
 
                case JAVA_ISTORE:
                case JAVA_FSTORE:
                case JAVA_ASTORE:
                        if (iswide == false) {
-                               i = SUCK_BE_U1(m->jcode + p + 1);
+                               i = SUCK_BE_U1(m->jcode + bcindex + 1);
                        }
                        else {
-                               i = SUCK_BE_U2(m->jcode + p + 1);
+                               i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                               nextbc = bcindex + 3;
                                iswide = false;
-                               nextp = p + 3;
                        }
-                       NEW_OP_STORE_ONEWORD(opcode, i);
+                       OP_STORE_ONEWORD(opcode, i, opcode - JAVA_ISTORE);
                        break;
 
                case JAVA_LSTORE:
                case JAVA_DSTORE:
                        if (iswide == false) {
-                               i = SUCK_BE_U1(m->jcode + p + 1);
+                               i = SUCK_BE_U1(m->jcode + bcindex + 1);
                        }
                        else {
-                               i = SUCK_BE_U2(m->jcode + p + 1);
+                               i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                               nextbc = bcindex + 3;
                                iswide = false;
-                               nextp = p + 3;
                        }
-                       NEW_OP_STORE_TWOWORD(opcode, i);
+                       OP_STORE_TWOWORD(opcode, i, opcode - JAVA_ISTORE);
                        break;
 
                case JAVA_ISTORE_0:
                case JAVA_ISTORE_1:
                case JAVA_ISTORE_2:
                case JAVA_ISTORE_3:
-                       NEW_OP_STORE_ONEWORD(ICMD_ISTORE, opcode - JAVA_ISTORE_0);
+                       OP_STORE_ONEWORD(ICMD_ISTORE, opcode - JAVA_ISTORE_0, TYPE_INT);
                        break;
 
                case JAVA_LSTORE_0:
                case JAVA_LSTORE_1:
                case JAVA_LSTORE_2:
                case JAVA_LSTORE_3:
-                       NEW_OP_STORE_TWOWORD(ICMD_LSTORE, opcode - JAVA_LSTORE_0);
+                       OP_STORE_TWOWORD(ICMD_LSTORE, opcode - JAVA_LSTORE_0, TYPE_LNG);
                        break;
 
                case JAVA_FSTORE_0:
                case JAVA_FSTORE_1:
                case JAVA_FSTORE_2:
                case JAVA_FSTORE_3:
-                       NEW_OP_STORE_ONEWORD(ICMD_FSTORE, opcode - JAVA_FSTORE_0);
+                       OP_STORE_ONEWORD(ICMD_FSTORE, opcode - JAVA_FSTORE_0, TYPE_FLT);
                        break;
 
                case JAVA_DSTORE_0:
                case JAVA_DSTORE_1:
                case JAVA_DSTORE_2:
                case JAVA_DSTORE_3:
-                       NEW_OP_STORE_TWOWORD(ICMD_DSTORE, opcode - JAVA_DSTORE_0);
+                       OP_STORE_TWOWORD(ICMD_DSTORE, opcode - JAVA_DSTORE_0, TYPE_DBL);
                        break;
 
                case JAVA_ASTORE_0:
                case JAVA_ASTORE_1:
                case JAVA_ASTORE_2:
                case JAVA_ASTORE_3:
-                       NEW_OP_STORE_ONEWORD(ICMD_ASTORE, opcode - JAVA_ASTORE_0);
+                       OP_STORE_ONEWORD(ICMD_ASTORE, opcode - JAVA_ASTORE_0, TYPE_ADR);
                        break;
 
                case JAVA_IINC:
@@ -522,32 +828,33 @@ fetch_opcode:
                                int v;
 
                                if (iswide == false) {
-                                       i = SUCK_BE_U1(m->jcode + p + 1);
-                                       v = SUCK_BE_S1(m->jcode + p + 2);
+                                       i = SUCK_BE_U1(m->jcode + bcindex + 1);
+                                       v = SUCK_BE_S1(m->jcode + bcindex + 2);
 
                                }
                                else {
-                                       i = SUCK_BE_U2(m->jcode + p + 1);
-                                       v = SUCK_BE_S2(m->jcode + p + 3);
+                                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                                       v = SUCK_BE_S2(m->jcode + bcindex + 3);
+                                       nextbc = bcindex + 5;
                                        iswide = false;
-                                       nextp = p + 5;
                                }
                                INDEX_ONEWORD(i);
-                               NEW_OP_LOCALINDEX_I(opcode, i, v);
+                               LOCALTYPE_USED(i, TYPE_INT);
+                               OP_LOCALINDEX_I(opcode, i, v);
                        }
                        break;
 
                /* wider index for loading, storing and incrementing ******************/
 
                case JAVA_WIDE:
+                       bcindex++;
                        iswide = true;
-                       p++;
                        goto fetch_opcode;
 
                /* managing arrays ****************************************************/
 
                case JAVA_NEWARRAY:
-                       switch (SUCK_BE_S1(m->jcode + p + 1)) {
+                       switch (SUCK_BE_S1(m->jcode + bcindex + 1)) {
                        case 4:
                                bte = builtintable_get_internal(BUILTIN_newarray_boolean);
                                break;
@@ -578,13 +885,13 @@ fetch_opcode:
                                return false;
 #endif
                        }
-                       NEW_OP_BUILTIN_CHECK_EXCEPTION(bte);
+                       OP_BUILTIN_CHECK_EXCEPTION(bte);
                        break;
 
                case JAVA_ANEWARRAY:
-                       i = SUCK_BE_U2(m->jcode + p + 1);
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
                        compr = (constant_classref *) class_getconstant(m->class, i, CONSTANT_Class);
-                       if (!compr)
+                       if (compr == NULL)
                                return false;
 
                        if (!(cr = class_get_classref_multiarray_of(1, compr)))
@@ -593,30 +900,29 @@ fetch_opcode:
                        if (!resolve_classref(m, cr, resolveLazy, true, true, &c))
                                return false;
 
-                       NEW_OP_LOADCONST_CLASSINFO_OR_CLASSREF(c, cr, INS_FLAG_NOCHECK);
+                       INSTRUCTIONS_CHECK(2);
+                       OP_LOADCONST_CLASSINFO_OR_CLASSREF_NOCHECK(c, cr);
                        bte = builtintable_get_internal(BUILTIN_newarray);
-                       NEW_OP_BUILTIN_CHECK_EXCEPTION(bte);
+                       OP_BUILTIN_CHECK_EXCEPTION(bte);
                        s_count++;
                        break;
 
                case JAVA_MULTIANEWARRAY:
                        jd->isleafmethod = false;
-                       i = SUCK_BE_U2(m->jcode + p + 1);
-                       {
-                               s4 v = SUCK_BE_U1(m->jcode + p + 3);
-
-                               cr = (constant_classref *) class_getconstant(m->class, i, CONSTANT_Class);
-                               if (!cr)
-                                       return false;
-
-                               if (!resolve_classref(m, cr, resolveLazy, true, true, &c))
-                                       return false;
-
-                               /* if unresolved, c == NULL */
-
-                               iptr->s1.argcount = v;
-                               NEW_OP_S3_CLASSINFO_OR_CLASSREF(opcode, c, cr, 0 /* flags */);
-                       }
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                       j = SUCK_BE_U1(m->jcode + bcindex + 3);
+  
+                       cr = (constant_classref *) class_getconstant(m->class, i, CONSTANT_Class);
+                       if (cr == NULL)
+                               return false;
+  
+                       if (!resolve_classref(m, cr, resolveLazy, true, true, &c))
+                               return false;
+  
+                       /* if unresolved, c == NULL */
+  
+                       iptr->s1.argcount = j;
+                       OP_S3_CLASSINFO_OR_CLASSREF(opcode, c, cr, INS_FLAG_CHECK);
                        break;
 
                /* control flow instructions ******************************************/
@@ -638,48 +944,48 @@ fetch_opcode:
                case JAVA_IF_ACMPEQ:
                case JAVA_IF_ACMPNE:
                case JAVA_GOTO:
-                       i = p + SUCK_BE_S2(m->jcode + p + 1);
+                       i = bcindex + SUCK_BE_S2(m->jcode + bcindex + 1);
                        CHECK_BYTECODE_INDEX(i);
-                       NEW_MARK_BASICBLOCK(i);
+                       MARK_BASICBLOCK(&pd, i);
                        blockend = true;
-                       NEW_OP_INSINDEX(opcode, i);
+                       OP_INSINDEX(opcode, i);
                        break;
 
                case JAVA_GOTO_W:
-                       i = p + SUCK_BE_S4(m->jcode + p + 1);
+                       i = bcindex + SUCK_BE_S4(m->jcode + bcindex + 1);
                        CHECK_BYTECODE_INDEX(i);
-                       NEW_MARK_BASICBLOCK(i);
+                       MARK_BASICBLOCK(&pd, i);
                        blockend = true;
-                       NEW_OP_INSINDEX(opcode, i);
+                       OP_INSINDEX(ICMD_GOTO, i);
                        break;
 
                case JAVA_JSR:
-                       i = p + SUCK_BE_S2(m->jcode + p + 1);
+                       i = bcindex + SUCK_BE_S2(m->jcode + bcindex + 1);
 jsr_tail:
                        CHECK_BYTECODE_INDEX(i);
-                       NEW_MARK_BASICBLOCK(i);
+                       MARK_BASICBLOCK(&pd, i);
                        blockend = true;
-                       NEW_OP_PREPARE_ZEROFLAGS(JAVA_JSR);
+                       OP_PREPARE_ZEROFLAGS(JAVA_JSR);
                        iptr->sx.s23.s3.jsrtarget.insindex = i;
                        PINC;
                        break;
 
                case JAVA_JSR_W:
-                       i = p + SUCK_BE_S4(m->jcode + p + 1);
+                       i = bcindex + SUCK_BE_S4(m->jcode + bcindex + 1);
                        goto jsr_tail;
 
                case JAVA_RET:
                        if (iswide == false) {
-                               i = SUCK_BE_U1(m->jcode + p + 1);
+                               i = SUCK_BE_U1(m->jcode + bcindex + 1);
                        }
                        else {
-                               i = SUCK_BE_U2(m->jcode + p + 1);
-                               nextp = p + 3;
+                               i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                               nextbc = bcindex + 3;
                                iswide = false;
                        }
                        blockend = true;
 
-                       NEW_OP_LOAD_ONEWORD(opcode, i);
+                       OP_LOAD_ONEWORD(opcode, i, TYPE_ADR);
                        break;
 
                case JAVA_IRETURN:
@@ -690,13 +996,13 @@ jsr_tail:
                case JAVA_RETURN:
                        blockend = true;
                        /* XXX ARETURN will need a flag in the typechecker */
-                       NEW_OP(opcode);
+                       OP(opcode);
                        break;
 
                case JAVA_ATHROW:
                        blockend = true;
                        /* XXX ATHROW will need a flag in the typechecker */
-                       NEW_OP(opcode);
+                       OP(opcode);
                        break;
 
 
@@ -710,25 +1016,25 @@ jsr_tail:
                                s4 prevvalue = 0;
 #endif
                                blockend = true;
-                               nextp = ALIGN((p + 1), 4);
+                               nextbc = MEMORY_ALIGN((bcindex + 1), 4);
 
-                               CHECK_END_OF_BYTECODE(nextp + 8);
+                               CHECK_END_OF_BYTECODE(nextbc + 8);
 
-                               NEW_OP_PREPARE_ZEROFLAGS(opcode);
+                               OP_PREPARE_ZEROFLAGS(opcode);
 
                                /* default target */
 
-                               j = p + SUCK_BE_S4(m->jcode + nextp);
+                               j = bcindex + SUCK_BE_S4(m->jcode + nextbc);
                                iptr->sx.s23.s3.lookupdefault.insindex = j;
-                               nextp += 4;
+                               nextbc += 4;
                                CHECK_BYTECODE_INDEX(j);
-                               NEW_MARK_BASICBLOCK(j);
+                               MARK_BASICBLOCK(&pd, j);
 
                                /* number of pairs */
 
-                               num = SUCK_BE_U4(m->jcode + nextp);
+                               num = SUCK_BE_U4(m->jcode + nextbc);
                                iptr->sx.s23.s2.lookupcount = num;
-                               nextp += 4;
+                               nextbc += 4;
 
                                /* allocate the intermediate code table */
 
@@ -737,15 +1043,15 @@ jsr_tail:
 
                                /* iterate over the lookup table */
 
-                               CHECK_END_OF_BYTECODE(nextp + 8 * num);
+                               CHECK_END_OF_BYTECODE(nextbc + 8 * num);
 
                                for (i = 0; i < num; i++) {
                                        /* value */
 
-                                       j = SUCK_BE_S4(m->jcode + nextp);
+                                       j = SUCK_BE_S4(m->jcode + nextbc);
                                        lookup->value = j;
 
-                                       nextp += 4;
+                                       nextbc += 4;
 
 #if defined(ENABLE_VERIFIER)
                                        /* check if the lookup table is sorted correctly */
@@ -758,12 +1064,12 @@ jsr_tail:
 #endif
                                        /* target */
 
-                                       j = p + SUCK_BE_S4(m->jcode + nextp);
+                                       j = bcindex + SUCK_BE_S4(m->jcode + nextbc);
                                        lookup->target.insindex = j;
                                        lookup++;
-                                       nextp += 4;
+                                       nextbc += 4;
                                        CHECK_BYTECODE_INDEX(j);
-                                       NEW_MARK_BASICBLOCK(j);
+                                       MARK_BASICBLOCK(&pd, j);
                                }
 
                                PINC;
@@ -778,30 +1084,30 @@ jsr_tail:
                                branch_target_t *table;
 
                                blockend = true;
-                               nextp = ALIGN((p + 1), 4);
+                               nextbc = MEMORY_ALIGN((bcindex + 1), 4);
 
-                               CHECK_END_OF_BYTECODE(nextp + 12);
+                               CHECK_END_OF_BYTECODE(nextbc + 12);
 
-                               NEW_OP_PREPARE_ZEROFLAGS(opcode);
+                               OP_PREPARE_ZEROFLAGS(opcode);
 
                                /* default target */
 
-                               deftarget = p + SUCK_BE_S4(m->jcode + nextp);
-                               nextp += 4;
+                               deftarget = bcindex + SUCK_BE_S4(m->jcode + nextbc);
+                               nextbc += 4;
                                CHECK_BYTECODE_INDEX(deftarget);
-                               NEW_MARK_BASICBLOCK(deftarget);
+                               MARK_BASICBLOCK(&pd, deftarget);
 
                                /* lower bound */
 
-                               j = SUCK_BE_S4(m->jcode + nextp);
+                               j = SUCK_BE_S4(m->jcode + nextbc);
                                iptr->sx.s23.s2.tablelow = j;
-                               nextp += 4;
+                               nextbc += 4;
 
                                /* upper bound */
 
-                               num = SUCK_BE_S4(m->jcode + nextp);
+                               num = SUCK_BE_S4(m->jcode + nextbc);
                                iptr->sx.s23.s3.tablehigh = num;
-                               nextp += 4;
+                               nextbc += 4;
 
                                /* calculate the number of table entries */
 
@@ -817,20 +1123,20 @@ jsr_tail:
                                /* create the intermediate code table */
                                /* the first entry is the default target */
 
-                               table = MNEW(branch_target_t, 1 + num);
+                               table = DMNEW(branch_target_t, 1 + num);
                                iptr->dst.table = table;
                                (table++)->insindex = deftarget;
 
                                /* iterate over the target table */
 
-                               CHECK_END_OF_BYTECODE(nextp + 4 * num);
+                               CHECK_END_OF_BYTECODE(nextbc + 4 * num);
 
                                for (i = 0; i < num; i++) {
-                                       j = p + SUCK_BE_S4(m->jcode + nextp);
+                                       j = bcindex + SUCK_BE_S4(m->jcode + nextbc);
                                        (table++)->insindex = j;
-                                       nextp += 4;
+                                       nextbc += 4;
                                        CHECK_BYTECODE_INDEX(j);
-                                       NEW_MARK_BASICBLOCK(j);
+                                       MARK_BASICBLOCK(&pd, j);
                                }
 
                                PINC;
@@ -841,7 +1147,7 @@ jsr_tail:
                /* load and store of object fields ************************************/
 
                case JAVA_AASTORE:
-                       NEW_OP(opcode);
+                       OP(opcode);
                        jd->isleafmethod = false;
                        break;
 
@@ -849,110 +1155,137 @@ jsr_tail:
                case JAVA_PUTSTATIC:
                case JAVA_GETFIELD:
                case JAVA_PUTFIELD:
-                       {
-                               constant_FMIref  *fr;
-                               unresolved_field *uf;
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                       fmi = class_getconstant(m->class, i, CONSTANT_Fieldref);
 
-                               i = SUCK_BE_U2(m->jcode + p + 1);
-                               fr = class_getconstant(m->class, i, CONSTANT_Fieldref);
-                               if (!fr)
-                                       return false;
+                       if (fmi == NULL)
+                               return false;
 
-                               NEW_OP_PREPARE_ZEROFLAGS(opcode);
-                               iptr->sx.s23.s3.fmiref = fr;
+                       OP_PREPARE_ZEROFLAGS(opcode);
+                       iptr->sx.s23.s3.fmiref = fmi;
 
-                               /* only with -noverify, otherwise the typechecker does this */
+                       /* only with -noverify, otherwise the typechecker does this */
 
 #if defined(ENABLE_VERIFIER)
-                               if (!JITDATA_HAS_FLAG_VERIFY(jd)) {
+                       if (!JITDATA_HAS_FLAG_VERIFY(jd)) {
 #endif
-                                       result = new_resolve_field_lazy(iptr, m);
-                                       if (result == resolveFailed)
-                                               return false;
+                               result = resolve_field_lazy(m, fmi);
 
-                                       if (result != resolveSucceeded) {
-                                               uf = new_create_unresolved_field(m->class, m, iptr);
+                               if (result == resolveFailed)
+                                       return false;
 
-                                               if (uf == NULL)
-                                                       return false;
+                               if (result != resolveSucceeded) {
+                                       uf = resolve_create_unresolved_field(m->class, m, iptr);
 
-                                               /* store the unresolved_field pointer */
+                                       if (uf == NULL)
+                                               return false;
 
-                                               iptr->sx.s23.s3.uf = uf;
-                                               iptr->flags.bits = INS_FLAG_UNRESOLVED;
-                                       }
-#if defined(ENABLE_VERIFIER)
+                                       /* store the unresolved_field pointer */
+
+                                       iptr->sx.s23.s3.uf = uf;
+                                       iptr->flags.bits |= INS_FLAG_UNRESOLVED;
                                }
-#endif
-                               PINC;
+#if defined(ENABLE_VERIFIER)
                        }
+#endif
+                       PINC;
                        break;
 
 
                /* method invocation **************************************************/
 
                case JAVA_INVOKESTATIC:
-                       i = SUCK_BE_U2(m->jcode + p + 1);
-                       mr = class_getconstant(m->class, i, CONSTANT_Methodref);
-                       if (!mr)
+                       OP_PREPARE_ZEROFLAGS(opcode);
+
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                       fmi = class_getconstant(m->class, i, CONSTANT_Methodref);
+
+                       if (fmi == NULL)
                                return false;
 
-                       md = mr->parseddesc.md;
+                       md = fmi->parseddesc.md;
 
-                       if (!md->params)
+                       if (md->params == NULL)
                                if (!descriptor_params_from_paramtypes(md, ACC_STATIC))
                                        return false;
 
                        goto invoke_method;
 
+               case JAVA_INVOKESPECIAL:
+                       OP_PREPARE_FLAGS(opcode, INS_FLAG_CHECK);
+
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                       fmi = class_getconstant(m->class, i, CONSTANT_Methodref);
+
+                       goto invoke_nonstatic_method;
+
                case JAVA_INVOKEINTERFACE:
-                       i = SUCK_BE_U2(m->jcode + p + 1);
+                       OP_PREPARE_ZEROFLAGS(opcode);
 
-                       mr = class_getconstant(m->class, i,
-                                       CONSTANT_InterfaceMethodref);
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                       fmi = class_getconstant(m->class, i, CONSTANT_InterfaceMethodref);
 
                        goto invoke_nonstatic_method;
 
-               case JAVA_INVOKESPECIAL:
                case JAVA_INVOKEVIRTUAL:
-                       i = SUCK_BE_U2(m->jcode + p + 1);
-                       mr = class_getconstant(m->class, i, CONSTANT_Methodref);
+                       OP_PREPARE_ZEROFLAGS(opcode);
+
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                       fmi = class_getconstant(m->class, i, CONSTANT_Methodref);
 
 invoke_nonstatic_method:
-                       if (!mr)
+                       if (fmi == NULL)
                                return false;
 
-                       md = mr->parseddesc.md;
+                       md = fmi->parseddesc.md;
 
-                       if (!md->params)
+                       if (md->params == NULL)
                                if (!descriptor_params_from_paramtypes(md, 0))
                                        return false;
 
 invoke_method:
                        jd->isleafmethod = false;
 
-                       NEW_OP_PREPARE_ZEROFLAGS(opcode);
-                       iptr->sx.s23.s3.fmiref = mr;
+                       iptr->sx.s23.s3.fmiref = fmi;
 
                        /* only with -noverify, otherwise the typechecker does this */
 
 #if defined(ENABLE_VERIFIER)
                        if (!JITDATA_HAS_FLAG_VERIFY(jd)) {
 #endif
-                               result = new_resolve_method_lazy(iptr, m);
+                               result = resolve_method_lazy(m, fmi, 
+                                                                                        (opcode == JAVA_INVOKESPECIAL));
+
                                if (result == resolveFailed)
                                        return false;
 
-                               if (result != resolveSucceeded) {
-                                       um = new_create_unresolved_method(m->class, m, iptr);
+                               if (result == resolveSucceeded) {
+                                       methodinfo *mi = iptr->sx.s23.s3.fmiref->p.method;
+
+                                       /* if this call is monomorphic, turn it into an
+                                          INVOKESPECIAL */
+
+                                       assert(IS_FMIREF_RESOLVED(iptr->sx.s23.s3.fmiref));
+
+                                       if ((iptr->opc == ICMD_INVOKEVIRTUAL)
+                                               && (mi->flags & (ACC_FINAL | ACC_PRIVATE)))
+                                       {
+                                               iptr->opc         = ICMD_INVOKESPECIAL;
+                                               iptr->flags.bits |= INS_FLAG_CHECK;
+                                       }
+                               }
+                               else {
+                                       um = resolve_create_unresolved_method(m->class, m, fmi,
+                                                       (opcode == JAVA_INVOKESTATIC),
+                                                       (opcode == JAVA_INVOKESPECIAL));
 
-                                       if (!um)
+                                       if (um == NULL)
                                                return false;
 
                                        /* store the unresolved_method pointer */
 
                                        iptr->sx.s23.s3.um = um;
-                                       iptr->flags.bits = INS_FLAG_UNRESOLVED;
+                                       iptr->flags.bits |= INS_FLAG_UNRESOLVED;
                                }
 #if defined(ENABLE_VERIFIER)
                        }
@@ -963,24 +1296,27 @@ invoke_method:
                /* instructions taking class arguments ********************************/
 
                case JAVA_NEW:
-                       i = SUCK_BE_U2(m->jcode + p + 1);
-                       cr = (constant_classref *) class_getconstant(m->class, i, CONSTANT_Class);
-                       if (!cr)
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                       cr = class_getconstant(m->class, i, CONSTANT_Class);
+
+                       if (cr == NULL)
                                return false;
 
                        if (!resolve_classref(m, cr, resolveLazy, true, true, &c))
                                return false;
 
-                       NEW_OP_LOADCONST_CLASSINFO_OR_CLASSREF(c, cr, INS_FLAG_NOCHECK);
+                       INSTRUCTIONS_CHECK(2);
+                       OP_LOADCONST_CLASSINFO_OR_CLASSREF_NOCHECK(c, cr);
                        bte = builtintable_get_internal(BUILTIN_new);
-                       NEW_OP_BUILTIN_CHECK_EXCEPTION(bte);
+                       OP_BUILTIN_CHECK_EXCEPTION(bte);
                        s_count++;
                        break;
 
                case JAVA_CHECKCAST:
-                       i = SUCK_BE_U2(m->jcode + p + 1);
-                       cr = (constant_classref *) class_getconstant(m->class, i, CONSTANT_Class);
-                       if (!cr)
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                       cr = class_getconstant(m->class, i, CONSTANT_Class);
+
+                       if (cr == NULL)
                                return false;
 
                        if (!resolve_classref(m, cr, resolveLazy, true, true, &c))
@@ -988,20 +1324,21 @@ invoke_method:
 
                        if (cr->name->text[0] == '[') {
                                /* array type cast-check */
-                               flags = INS_FLAG_ARRAY;
+                               flags = INS_FLAG_CHECK | INS_FLAG_ARRAY;
                                jd->isleafmethod = false;
                        }
                        else {
                                /* object type cast-check */
-                               flags = 0;
+                               flags = INS_FLAG_CHECK;
                        }
-                       NEW_OP_S3_CLASSINFO_OR_CLASSREF(opcode, c, cr, flags);
+                       OP_S3_CLASSINFO_OR_CLASSREF(opcode, c, cr, flags);
                        break;
 
                case JAVA_INSTANCEOF:
-                       i = SUCK_BE_U2(m->jcode + p + 1);
-                       cr = (constant_classref *) class_getconstant(m->class, i, CONSTANT_Class);
-                       if (!cr)
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                       cr = class_getconstant(m->class, i, CONSTANT_Class);
+
+                       if (cr == NULL)
                                return false;
 
                        if (!resolve_classref(m, cr, resolveLazy, true, true, &c))
@@ -1009,14 +1346,15 @@ invoke_method:
 
                        if (cr->name->text[0] == '[') {
                                /* array type cast-check */
-                               NEW_OP_LOADCONST_CLASSINFO_OR_CLASSREF(c, cr, INS_FLAG_NOCHECK);
+                               INSTRUCTIONS_CHECK(2);
+                               OP_LOADCONST_CLASSINFO_OR_CLASSREF_NOCHECK(c, cr);
                                bte = builtintable_get_internal(BUILTIN_arrayinstanceof);
-                               NEW_OP_BUILTIN_NO_EXCEPTION(bte);
+                               OP_BUILTIN_NO_EXCEPTION(bte);
                                s_count++;
                        }
                        else {
                                /* object type cast-check */
-                               NEW_OP_S3_CLASSINFO_OR_CLASSREF(opcode, c, cr, 0 /* flags*/);
+                               OP_S3_CLASSINFO_OR_CLASSREF(opcode, c, cr, 0 /* flags*/);
                        }
                        break;
 
@@ -1025,28 +1363,28 @@ invoke_method:
                case JAVA_MONITORENTER:
 #if defined(ENABLE_THREADS)
                        if (checksync) {
-                               /* XXX null check */
                                bte = builtintable_get_internal(LOCK_monitor_enter);
-                               NEW_OP_BUILTIN_NO_EXCEPTION(bte);
+                               OP_BUILTIN_CHECK_EXCEPTION(bte);
                        }
                        else
 #endif
                        {
-                               NEW_OP(ICMD_CHECKNULL_POP);
+                               OP_CHECK_EXCEPTION(ICMD_CHECKNULL);
+                               OP(ICMD_POP);
                        }
                        break;
 
                case JAVA_MONITOREXIT:
 #if defined(ENABLE_THREADS)
                        if (checksync) {
-                               /* XXX null check */
                                bte = builtintable_get_internal(LOCK_monitor_exit);
-                               NEW_OP_BUILTIN_CHECK_EXCEPTION(bte);
+                               OP_BUILTIN_CHECK_EXCEPTION(bte);
                        }
                        else
 #endif
                        {
-                               NEW_OP(ICMD_CHECKNULL_POP);
+                               OP_CHECK_EXCEPTION(ICMD_CHECKNULL);
+                               OP(ICMD_POP);
                        }
                        break;
 
@@ -1055,54 +1393,70 @@ invoke_method:
                case JAVA_IDIV:
 #if !SUPPORT_DIVISION
                        bte = builtintable_get_internal(BUILTIN_idiv);
-                       NEW_OP_BUILTIN_ARITHMETIC(opcode, bte);
+                       OP_BUILTIN_ARITHMETIC(opcode, bte);
 #else
-                       NEW_OP(opcode);
+# if SUPPORT_HARDWARE_DIVIDE_BY_ZERO
+                       OP(opcode);
+# else
+                       OP_CHECK_EXCEPTION(opcode);
+# endif
 #endif
                        break;
 
                case JAVA_IREM:
 #if !SUPPORT_DIVISION
                        bte = builtintable_get_internal(BUILTIN_irem);
-                       NEW_OP_BUILTIN_ARITHMETIC(opcode, bte);
+                       OP_BUILTIN_ARITHMETIC(opcode, bte);
 #else
-                       NEW_OP(opcode);
+# if SUPPORT_HARDWARE_DIVIDE_BY_ZERO
+                       OP(opcode);
+# else
+                       OP_CHECK_EXCEPTION(opcode);
+# endif
 #endif
                        break;
 
                case JAVA_LDIV:
 #if !(SUPPORT_DIVISION && SUPPORT_LONG && SUPPORT_LONG_DIV)
                        bte = builtintable_get_internal(BUILTIN_ldiv);
-                       NEW_OP_BUILTIN_ARITHMETIC(opcode, bte);
+                       OP_BUILTIN_ARITHMETIC(opcode, bte);
 #else
-                       NEW_OP(opcode);
+# if SUPPORT_HARDWARE_DIVIDE_BY_ZERO
+                       OP(opcode);
+# else
+                       OP_CHECK_EXCEPTION(opcode);
+# endif
 #endif
                        break;
 
                case JAVA_LREM:
 #if !(SUPPORT_DIVISION && SUPPORT_LONG && SUPPORT_LONG_DIV)
                        bte = builtintable_get_internal(BUILTIN_lrem);
-                       NEW_OP_BUILTIN_ARITHMETIC(opcode, bte);
+                       OP_BUILTIN_ARITHMETIC(opcode, bte);
 #else
-                       NEW_OP(opcode);
+# if SUPPORT_HARDWARE_DIVIDE_BY_ZERO
+                       OP(opcode);
+# else
+                       OP_CHECK_EXCEPTION(opcode);
+# endif
 #endif
                        break;
 
                case JAVA_FREM:
 #if defined(__I386__)
-                       NEW_OP(opcode);
+                       OP(opcode);
 #else
                        bte = builtintable_get_internal(BUILTIN_frem);
-                       NEW_OP_BUILTIN_NO_EXCEPTION(bte);
+                       OP_BUILTIN_NO_EXCEPTION(bte);
 #endif
                        break;
 
                case JAVA_DREM:
 #if defined(__I386__)
-                       NEW_OP(opcode);
+                       OP(opcode);
 #else
                        bte = builtintable_get_internal(BUILTIN_drem);
-                       NEW_OP_BUILTIN_NO_EXCEPTION(bte);
+                       OP_BUILTIN_NO_EXCEPTION(bte);
 #endif
                        break;
 
@@ -1110,12 +1464,12 @@ invoke_method:
 #if defined(__ALPHA__)
                        if (!opt_noieee) {
                                bte = builtintable_get_internal(BUILTIN_f2i);
-                               NEW_OP_BUILTIN_NO_EXCEPTION(bte);
+                               OP_BUILTIN_NO_EXCEPTION(bte);
                        }
                        else
 #endif
                        {
-                               NEW_OP(opcode);
+                               OP(opcode);
                        }
                        break;
 
@@ -1123,12 +1477,12 @@ invoke_method:
 #if defined(__ALPHA__)
                        if (!opt_noieee) {
                                bte = builtintable_get_internal(BUILTIN_f2l);
-                               NEW_OP_BUILTIN_NO_EXCEPTION(bte);
+                               OP_BUILTIN_NO_EXCEPTION(bte);
                        }
                        else
 #endif
                        {
-                               NEW_OP(opcode);
+                               OP(opcode);
                        }
                        break;
 
@@ -1136,12 +1490,12 @@ invoke_method:
 #if defined(__ALPHA__)
                        if (!opt_noieee) {
                                bte = builtintable_get_internal(BUILTIN_d2i);
-                               NEW_OP_BUILTIN_NO_EXCEPTION(bte);
+                               OP_BUILTIN_NO_EXCEPTION(bte);
                        }
                        else
 #endif
                        {
-                               NEW_OP(opcode);
+                               OP(opcode);
                        }
                        break;
 
@@ -1149,12 +1503,12 @@ invoke_method:
 #if defined(__ALPHA__)
                        if (!opt_noieee) {
                                bte = builtintable_get_internal(BUILTIN_d2l);
-                               NEW_OP_BUILTIN_NO_EXCEPTION(bte);
+                               OP_BUILTIN_NO_EXCEPTION(bte);
                        }
                        else
 #endif
                        {
-                               NEW_OP(opcode);
+                               OP(opcode);
                        }
                        break;
 
@@ -1221,7 +1575,7 @@ invoke_method:
                case 254:
                case 255:
                        exceptions_throw_verifyerror(m, "Illegal opcode %d at instr %d\n",
-                                                                                opcode, ipc);
+                                                                                opcode, ircount);
                        return false;
                        break;
 #endif /* defined(ENABLE_VERIFIER) */
@@ -1230,7 +1584,7 @@ invoke_method:
 
                default:
                        /* straight-forward translation to ICMD */
-                       NEW_OP(opcode);
+                       OP(opcode);
                        break;
 
                } /* end switch */
@@ -1248,17 +1602,24 @@ invoke_method:
 
        } /* end for */
 
+       if (JITDATA_HAS_FLAG_REORDER(jd)) {
+               /* add a NOP to the last basic block */
+
+               INSTRUCTIONS_CHECK(1);
+               OP(ICMD_NOP);
+       }
+
        /*** END OF LOOP **********************************************************/
 
        /* assert that we did not write more ICMDs than allocated */
 
-       assert(ipc == (iptr - jd->new_instructions));
-       assert(ipc <= m->jcodelength);
+       assert(ircount <= pd.instructionslength);
+       assert(ircount == (iptr - pd.instructions));
 
        /*** verifier checks ******************************************************/
 
 #if defined(ENABLE_VERIFIER)
-       if (p != m->jcodelength) {
+       if (bcindex != m->jcodelength) {
                exceptions_throw_verifyerror(m,
                                "Command-sequence crosses code-boundary");
                return false;
@@ -1272,117 +1633,253 @@ invoke_method:
 
        /*** setup the methodinfo, allocate stack and basic blocks ****************/
 
-       /* adjust block count if target 0 is not first intermediate instruction */
+       /* identify basic blocks */
+
+       /* check if first instruction is a branch target */
 
-       if (!jd->new_basicblockindex[0] || (jd->new_basicblockindex[0] > 1))
-               b_count++;
+       if (pd.basicblockstart[0] == 1) {
+               jd->branchtoentry = true;
+       }
+       else {
+               /* first instruction always starts a basic block */
 
-       /* copy local to method variables */
+               iptr = pd.instructions;
 
-       jd->new_instructioncount = ipc;
-       jd->new_basicblockcount = b_count;
-       jd->new_stackcount = s_count + jd->new_basicblockcount * m->maxstack; /* in-stacks */
+               iptr->flags.bits |= INS_FLAG_BASICBLOCK;
+       }
 
-       /* allocate stack table */
+       /* Iterate over all bytecode instructions and set missing
+          basic-block starts in IR instructions. */
+
+       for (bcindex = 0; bcindex < m->jcodelength; bcindex++) {
+               /* Does the current bytecode instruction start a basic
+                  block? */
+
+               if (pd.basicblockstart[bcindex] == 1) {
+#if defined(ENABLE_VERIFIER)
+                       /* Check if this bytecode basic-block start at the
+                          beginning of a bytecode instruction. */
+
+                       if (pd.bytecodestart[bcindex] == 0) {
+                               exceptions_throw_verifyerror(m,
+                                                                                "Branch into middle of instruction");
+                               return false;
+                       }
+#endif
+
+                       /* Get the IR instruction mapped to the bytecode
+                          instruction and set the basic block flag. */
 
-       jd->new_stack = DMNEW(stackelement, jd->new_stackcount);
+                       irindex = pd.bytecodemap[bcindex];
+                       iptr    = pd.instructions + irindex;
+
+                       iptr->flags.bits |= INS_FLAG_BASICBLOCK;
+               }
+       }
 
-       /* build basic block list */
+       /* IR instruction index to basic-block index mapping */
 
-       bptr = jd->new_basicblocks = DMNEW(basicblock, b_count + 1);    /* one more for end ipc */
+       pd.instructionmap = DMNEW(s4, ircount);
+       MZERO(pd.instructionmap, s4, ircount);
 
-       /* zero out all basic block structures */
+       /* Iterate over all IR instructions and count the basic blocks. */
 
-       MZERO(bptr, basicblock, b_count + 1);
+       iptr = pd.instructions;
 
-       b_count = 0;
-       jd->new_c_debug_nr = 0;
+       bbcount = 0;
 
-       /* additional block if target 0 is not first intermediate instruction */
+       for (i = 0; i < ircount; i++, iptr++) {
+               if (INSTRUCTION_STARTS_BASICBLOCK(iptr)) {
+                       /* store the basic-block number in the IR instruction
+                          map */
 
-       if (!jd->new_basicblockindex[0] || (jd->new_basicblockindex[0] > 1)) {
-               BASICBLOCK_INIT(bptr, m);
+                       pd.instructionmap[i] = bbcount;
 
-               bptr->iinstr = jd->new_instructions;
-               /* bptr->icount is set when the next block is allocated */
+                       /* post-increment the basic-block count */
 
-               bptr++;
-               b_count++;
-               bptr[-1].next = bptr;
+                       bbcount++;
+               }
        }
 
-       /* allocate blocks */
+       /* Allocate basic block array (one more for end ipc). */
 
-       for (p = 0; p < m->jcodelength; p++) {
-               if (jd->new_basicblockindex[p] & 1) {
-                       /* Check if this block starts at the beginning of an          */
-                       /* instruction.                                               */
-#if defined(ENABLE_VERIFIER)
-                       if (!instructionstart[p]) {
-                               exceptions_throw_verifyerror(m,
-                                               "Branch into middle of instruction");
-                               return false;
-                       }
-#endif
+       jd->basicblocks = DMNEW(basicblock, bbcount + 1);
+       MZERO(jd->basicblocks, basicblock, bbcount + 1);
+
+       /* Now iterate again over all IR instructions and initialize the
+          basic block structures and, in the same loop, resolve the
+          branch-target instruction indices to basic blocks. */
+
+       iptr = pd.instructions;
+       bptr = jd->basicblocks;
+
+       bbcount = 0;
+
+       for (i = 0; i < ircount; i++, iptr++) {
+               /* check for basic block */
 
-                       /* allocate the block */
+               if (INSTRUCTION_STARTS_BASICBLOCK(iptr)) {
+                       /* intialize the basic block */
 
                        BASICBLOCK_INIT(bptr, m);
 
-                       bptr->iinstr = jd->new_instructions + (jd->new_basicblockindex[p] >> 1);
-                       if (b_count) {
+                       bptr->iinstr = iptr;
+
+                       if (bbcount > 0) {
                                bptr[-1].icount = bptr->iinstr - bptr[-1].iinstr;
                        }
-                       /* bptr->icount is set when the next block is allocated */
 
-                       jd->new_basicblockindex[p] = b_count;
+                       /* bptr->icount is set when the next block is allocated */
 
+                       bptr->nr = bbcount++;
                        bptr++;
-                       b_count++;
                        bptr[-1].next = bptr;
                }
+
+               /* resolve instruction indices to basic blocks */
+
+               switch (iptr->opc) {
+               case JAVA_IFEQ:
+               case JAVA_IFLT:
+               case JAVA_IFLE:
+               case JAVA_IFNE:
+               case JAVA_IFGT:
+               case JAVA_IFGE:
+               case JAVA_IFNULL:
+               case JAVA_IFNONNULL:
+               case JAVA_IF_ICMPEQ:
+               case JAVA_IF_ICMPNE:
+               case JAVA_IF_ICMPLT:
+               case JAVA_IF_ICMPGT:
+               case JAVA_IF_ICMPLE:
+               case JAVA_IF_ICMPGE:
+               case JAVA_IF_ACMPEQ:
+               case JAVA_IF_ACMPNE:
+               case JAVA_GOTO:
+                       BYTECODEINDEX_TO_BASICBLOCK(iptr->dst);
+                       break;
+
+               case ICMD_JSR:
+                       BYTECODEINDEX_TO_BASICBLOCK(iptr->sx.s23.s3.jsrtarget);
+                       break;
+
+               case ICMD_TABLESWITCH:
+                       table = iptr->dst.table;
+
+                       BYTECODEINDEX_TO_BASICBLOCK(*table);
+                       table++;
+
+                       j = iptr->sx.s23.s3.tablehigh - iptr->sx.s23.s2.tablelow + 1;
+
+                       while (--j >= 0) {
+                               BYTECODEINDEX_TO_BASICBLOCK(*table);
+                               table++;
+                       }
+                       break;
+
+               case ICMD_LOOKUPSWITCH:
+                       BYTECODEINDEX_TO_BASICBLOCK(iptr->sx.s23.s3.lookupdefault);
+
+                       lookup = iptr->dst.lookup;
+
+                       j = iptr->sx.s23.s2.lookupcount;
+
+                       while (--j >= 0) {
+                               BYTECODEINDEX_TO_BASICBLOCK(lookup->target);
+                               lookup++;
+                       }
+                       break;
+               }
        }
 
        /* set instruction count of last real block */
 
-       if (b_count) {
-               bptr[-1].icount = (jd->new_instructions + jd->new_instructioncount) - bptr[-1].iinstr;
+       if (bbcount > 0) {
+               bptr[-1].icount = (pd.instructions + ircount) - bptr[-1].iinstr;
        }
 
        /* allocate additional block at end */
 
-       BASICBLOCK_INIT(bptr,m);
+       BASICBLOCK_INIT(bptr, m);
+       bptr->nr = bbcount;
 
        /* set basicblock pointers in exception table */
 
-       if (cd->exceptiontablelength > 0) {
-               cd->exceptiontable[cd->exceptiontablelength - 1].down = NULL;
-       }
+       if (!parse_resolve_exception_table(jd, &pd))
+               return false;
 
-       for (i = 0; i < cd->exceptiontablelength; ++i) {
-               p = cd->exceptiontable[i].startpc;
-               cd->exceptiontable[i].start = jd->new_basicblocks + jd->new_basicblockindex[p];
+       /* store the local map */
 
-               p = cd->exceptiontable[i].endpc;
-               cd->exceptiontable[i].end = (p == m->jcodelength) ? (jd->new_basicblocks + jd->new_basicblockcount /*+ 1*/) : (jd->new_basicblocks + jd->new_basicblockindex[p]);
+       jd->local_map = local_map;
 
-               p = cd->exceptiontable[i].handlerpc;
-               cd->exceptiontable[i].handler = jd->new_basicblocks + jd->new_basicblockindex[p];
-       }
+       /* calculate local variable renaming */
 
-       /* XXX activate this if you want to try inlining */
-#if 0
-       for (i = 0; i < m->exceptiontablelength; ++i) {
-               p = m->exceptiontable[i].startpc;
-               m->exceptiontable[i].start = jd->new_basicblocks + jd->new_basicblockindex[p];
+       {
+               s4 nlocals = 0;
+               s4 i;
+               s4 *mapptr;
 
-               p = m->exceptiontable[i].endpc;
-               m->exceptiontable[i].end = (p == m->jcodelength) ? (jd->new_basicblocks + jd->new_basicblockcount /*+ 1*/) : (jd->new_basicblocks + jd->new_basicblockindex[p]);
+               mapptr = local_map;
 
-               p = m->exceptiontable[i].handlerpc;
-               m->exceptiontable[i].handler = jd->new_basicblocks + jd->new_basicblockindex[p];
-       }
+               /* iterate over local_map[0..m->maxlocals*5-1] and allocate a unique */
+               /* variable index for each _used_ (javaindex,type) pair.             */
+               /* (local_map[javaindex*5+type] = cacaoindex)                        */
+               /* Unused (javaindex,type) pairs are marked with UNUSED.             */
+
+               for (i = 0; i < (m->maxlocals * 5); i++, mapptr++) {
+                       if (*mapptr)
+                               *mapptr = nlocals++;
+                       else
+                               *mapptr = UNUSED;
+               }
+
+               jd->localcount = nlocals;
+
+               /* calculate the (maximum) number of variables needed */
+
+               jd->varcount = 
+                         nlocals                                      /* local variables */
+                       + bbcount * m->maxstack                                 /* invars */
+                       + s_count;         /* variables created within blocks (non-invar) */
+
+               /* reserve the first indices for local variables */
+
+               jd->vartop = nlocals;
+
+               /* reserve extra variables needed by stack analyse */
+
+               jd->varcount += STACK_EXTRA_VARS;
+               jd->vartop   += STACK_EXTRA_VARS;
+
+               /* The verifier needs space for saving invars in some cases and */
+               /* extra variables.                                             */
+
+#if defined(ENABLE_VERIFIER)
+               jd->varcount += VERIFIER_EXTRA_LOCALS + VERIFIER_EXTRA_VARS + m->maxstack;
+               jd->vartop   += VERIFIER_EXTRA_LOCALS + VERIFIER_EXTRA_VARS + m->maxstack;
 #endif
+               /* allocate and initialize the variable array */
+
+               jd->var = DMNEW(varinfo, jd->varcount);
+               MZERO(jd->var, varinfo, jd->varcount);
+
+               /* set types of all locals in jd->var */
+
+               for (mapptr = local_map, i = 0; i < (m->maxlocals * 5); i++, mapptr++)
+                       if (*mapptr != UNUSED)
+                               VAR(*mapptr)->type = i%5;
+       }
+
+       /* assign local variables to method variables */
+
+       jd->instructions     = pd.instructions;
+       jd->instructioncount = ircount;
+       jd->basicblockcount  = bbcount;
+       jd->stackcount       = s_count + bbcount * m->maxstack; /* in-stacks */
+
+       /* allocate stack table */
+
+       jd->stack = DMNEW(stackelement, jd->stackcount);
 
        /* everything's ok */