* Removed all Id tags.
[cacao.git] / src / vm / jit / parse.c
index 90e4abff9eeeb43df9a98c86df11208a9eac0cee..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 4685 2006-03-23 02:06:50Z edwin $
-
 */
 
 
+#include "config.h"
+
 #include <assert.h>
 #include <string.h>
 
-#include "config.h"
-
 #include "vm/types.h"
 
 #include "mm/memory.h"
+
 #include "native/native.h"
+
+#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/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 exceptiontable * fillextable(methodinfo *m, 
-                                                                       exceptiontable *extable, 
-                                                                       exceptiontable *raw_extable, 
-                                                               int exceptiontablelength, 
-                                                                       int *block_count)
+static basicblock *parse_bytecodeindex_to_basicblock(jitdata *jd,
+                                                                                                        parsedata_t *pd,
+                                                                                                        s4 bcindex)
 {
-       int b_count, p, src, insertBlock;
+       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 bool parse_mark_exception_boundaries(jitdata *jd, parsedata_t *pd)
+{
+       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;
+
+       rex = m->rawexceptiontable;
+
+       for (i = 0; i < len; ++i, ++rex) {
 
-       b_count = *block_count;
+               /* the start of the handled region becomes a basic block start */
 
-       for (src = exceptiontablelength-1; src >=0; src--) {
-               p = raw_extable[src].startpc;
-               extable->startpc = p;
-               bound_check(p);
-               block_insert(p);
+               bcindex = rex->startpc;
+               CHECK_BYTECODE_INDEX(bcindex);
+               MARK_BASICBLOCK(pd, bcindex);
                
-               p = raw_extable[src].endpc; /* see JVM Spec 4.7.3 */
-               if (p <= raw_extable[src].startpc) {
-                       *exceptionptr = new_verifyerror(m,
-                               "Invalid exception handler range");
-                       return NULL;
+               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 (bcindex <= rex->startpc) {
+                       exceptions_throw_verifyerror(m, "Invalid exception handler range");
+                       return false;
                }
+#endif
+               
+               /* 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  */
+
+               bcindex = rex->handlerpc;
+               CHECK_BYTECODE_INDEX(bcindex);
+               MARK_BASICBLOCK(pd, bcindex);
+       }
+
+       /* everything ok */
+
+       return true;
+
+#if defined(ENABLE_VERIFIER)
+throw_invalid_bytecode_index:
+       exceptions_throw_verifyerror(m,
+                                                                "Illegal bytecode index in exception table");
+       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
+
+*******************************************************************************/
 
-               if (p >m->jcodelength) {
-                       *exceptionptr = new_verifyerror(m,
-                               "Invalid exception handler end is after code end");
-                       return NULL;
+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;
                }
 
-               if (p<m->jcodelength) insertBlock=1; else insertBlock=0;
-               extable->endpc = p;
-               bound_check_exclusive(p);
-               /* if (p < m->jcodelength) block_insert(p); */
-        if (insertBlock) 
-                       block_insert(p);
-
-               p = raw_extable[src].handlerpc;
-               extable->handlerpc = p;
-               bound_check(p);
-               block_insert(p);
-
-               extable->catchtype  = raw_extable[src].catchtype;
-               extable->next = NULL;
-               extable->down = &extable[1];
-               extable--;
+               ex->catchtype = rex->catchtype;
+               ex->next = NULL;   /* set by loop analysis */
+               ex->down = ex + 1; /* link to next exception entry */
        }
 
-       *block_count = b_count;
-       return extable; /*&extable[i];*/  /* return the next free xtable* */
+       /* 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.
+
+*******************************************************************************/
 
-methodinfo *parse(methodinfo *m, codegendata *cd)
+/*** macro for checking the length of the bytecode ***/
+
+#if defined(ENABLE_VERIFIER)
+#define CHECK_END_OF_BYTECODE(neededlength) \
+       do { \
+               if ((neededlength) > m->jcodelength) \
+                       goto throw_unexpected_end_of_bytecode; \
+       } while (0)
+#else /* !ENABLE_VERIFIER */
+#define CHECK_END_OF_BYTECODE(neededlength)
+#endif /* ENABLE_VERIFIER */
+
+bool parse(jitdata *jd)
 {
-       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 */
-       int gp;                     /* global java instruction counter    */
-
-       int firstlocal = 0;         /* first local variable of method     */
-       exceptiontable* nextex;     /* points next free entry in extable  */
-       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    *fmi;
+       methoddesc         *md;
+       unresolved_method  *um;
+       unresolved_field   *uf;
+
+       resolve_result_t    result;
+       u2                  lineindex = 0;
+       u2                  currentline = 0;
+       u2                  linepcchange = 0;
+       u4                  flags;
+       basicblock         *bptr;
+
+       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;
+
+       /* get required compiler data */
+
+       m = jd->m;
+
+       /* allocate buffers for local variable renaming */
+
+       local_map = DMNEW(int, m->maxlocals * 5);
+
+       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;
+       }
 
-       u2 lineindex = 0;
-       u2 currentline = 0;
-       u2 linepcchange = 0;
-
-       u2 skipBasicBlockChange;
+       /* initialize the parse data structures */
+  
+       parse_setup(jd, &pd);
+  
+       /* initialize local variables */
+  
+       iptr     = pd.instructions;
+       ircount  = 0;
+       bbcount  = 0;
+       blockend = false;
+       iswide   = false;
 
-       /* allocate instruction array and block index table */
-       
-       /* 1 additional for end ipc  */
-       m->basicblockindex = DMNEW(s4, m->jcodelength + 1);
-       memset(m->basicblockindex, 0, sizeof(s4) * (m->jcodelength + 1));
+       /* mark basic block boundaries for exception table */
 
-       instructionstart = DMNEW(u1, m->jcodelength + 1);
-       memset(instructionstart, 0, sizeof(u1) * (m->jcodelength + 1));
+       if (!parse_mark_exception_boundaries(jd, &pd))
+               return false;
 
-       /* 1 additional for TRACEBUILTIN and 4 for MONITORENTER/EXIT */
-       /* additional MONITOREXITS are reached by branches which are 3 bytes */
-       
-       iptr = m->instructions = DMNEW(instruction, m->jcodelength + 5);
+       /* initialize stack element counter */
 
-       /* Zero the intermediate instructions array so we don't have any
-        * invalid pointers in it if we cannot finish analyse_stack(). */
+       s_count = 1 + m->rawexceptiontablelength;
 
-       memset(iptr, 0, sizeof(instruction) * (m->jcodelength + 5));
-       
-       /* compute branch targets of exception table */
-
-       nextex = fillextable(m, 
-         &(cd->exceptiontable[cd->exceptiontablelength-1]), m->exceptiontable, 
-               m->exceptiontablelength, &b_count);
-       if (!nextex)
-               return NULL;
-       s_count = 1 + m->exceptiontablelength; /* initialize stack element counter   */
-
-#if defined(USE_THREADS)
-       if (checksync && (m->flags & ACC_SYNCHRONIZED)) {
-               m->isleafmethod = false;
-       }                       
-#endif
+       /* setup line number info */
 
-       /* scan all java instructions */
        currentline = 0;
        linepcchange = 0;
 
        if (m->linenumbercount == 0) {
                lineindex = 0;
-       } 
+       }
        else {
                linepcchange = m->linenumbers[0].start_pc;
        }
 
-       skipBasicBlockChange=0;
-       for (p = 0, gp = 0; p < m->jcodelength; gp += (nextp - p), p = nextp) {
-         
-               /* mark this position as a valid instruction start */
-               if (!iswide) {
-                       instructionstart[gp] = 1;
-                       if (linepcchange==p) {
-                               if (m->linenumbercount > lineindex) {
-                                       currentline = m->linenumbers[lineindex].line_number;
-                                       lineindex++;
-                                       if (lineindex < m->linenumbercount)
-                                               linepcchange = m->linenumbers[lineindex].start_pc;
+       /*** LOOP OVER ALL BYTECODE INSTRUCTIONS **********************************/
+
+       for (bcindex = 0; bcindex < m->jcodelength; bcindex = nextbc) {
+
+               /* mark this position as a valid bytecode instruction start */
+
+               pd.bytecodestart[bcindex] = 1;
+
+               /* change the current line number, if necessary */
+
+               /* XXX rewrite this using pointer arithmetic */
+
+               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 == bcindex)
+                                               goto next_linenumber;
                                }
                        }
                }
 
-               /* fetch next opcode  */
-
-               opcode = code_get_u1(p, m);
+fetch_opcode:
+               /* fetch next opcode  */        
 
-               if (!skipBasicBlockChange) {
-                       m->basicblockindex[gp] |= (ipc << 1); /*store intermed cnt*/
-               } 
-               else 
-                       skipBasicBlockChange = 0;
+               opcode = SUCK_BE_U1(m->jcode + bcindex);
 
-               /* some compilers put a JAVA_NOP after a blockend instruction */
+               /* If the previous instruction was a block-end instruction,
+                  mark the current bytecode instruction as basic-block
+                  starting instruction. */
 
-               if ((opcode != JAVA_NOP) && (blockend == true)) {
-                       /* start new block */
+               /* NOTE: Some compilers put a JAVA_NOP after a blockend
+                  instruction. */
 
-                       block_insert(gp);
+               if (blockend && (opcode != JAVA_NOP)) {
+                       MARK_BASICBLOCK(&pd, bcindex);
                        blockend = false;
                }
 
-               nextp = p + jcommandsize[opcode];   /* compute next instruction start */
+               /* 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 (nextp > m->jcodelength) {
-                       *exceptionptr = new_verifyerror(m,
-                                       "Unexpected end of bytecode");
-                       return NULL;
+               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);
                }
 
-               s_count += stackreq[opcode];            /* compute stack element count    */
+               /* store intermediate instruction count (bit 0 mark block starts) */
+
+               pd.bytecodemap[bcindex] = ircount;
+
+               /* compute next instruction start */
+
+               nextbc = bcindex + jcommandsize[opcode];
+
+               CHECK_END_OF_BYTECODE(nextbc);
+
+               /* add stack elements produced by this instruction */
+
+               s_count += stackreq[opcode];
+
+               /* 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:
                        break;
 
-                       /* pushing constants onto the stack p */
+               /* pushing constants onto the stack ***********************************/
 
                case JAVA_BIPUSH:
-                       LOADCONST_I(code_get_s1(p+1,m));
+                       OP_LOADCONST_I(SUCK_BE_S1(m->jcode + bcindex + 1));
                        break;
 
                case JAVA_SIPUSH:
-                       LOADCONST_I(code_get_s2(p+1,m));
+                       OP_LOADCONST_I(SUCK_BE_S2(m->jcode + bcindex + 1));
                        break;
 
                case JAVA_LDC1:
-                       i = code_get_u1(p + 1, m);
+                       i = SUCK_BE_U1(m->jcode + bcindex + 1);
                        goto pushconstantitem;
 
                case JAVA_LDC2:
                case JAVA_LDC2W:
-                       i = code_get_u2(p + 1, m);
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
 
                pushconstantitem:
 
+#if defined(ENABLE_VERIFIER)
                        if (i >= m->class->cpcount) {
-                               *exceptionptr = new_verifyerror(m,
+                               exceptions_throw_verifyerror(m,
                                        "Attempt to access constant outside range");
-                               return NULL;
+                               return false;
                        }
+#endif
 
                        switch (m->class->cptags[i]) {
                        case CONSTANT_Integer:
-                               LOADCONST_I(((constant_integer *) (m->class->cpinfos[i]))->value);
+                               OP_LOADCONST_I(((constant_integer *) (m->class->cpinfos[i]))->value);
                                break;
                        case CONSTANT_Long:
-                               LOADCONST_L(((constant_long *) (m->class->cpinfos[i]))->value);
+                               OP_LOADCONST_L(((constant_long *) (m->class->cpinfos[i]))->value);
                                break;
                        case CONSTANT_Float:
-                               LOADCONST_F(((constant_float *) (m->class->cpinfos[i]))->value);
+                               OP_LOADCONST_F(((constant_float *) (m->class->cpinfos[i]))->value);
                                break;
                        case CONSTANT_Double:
-                               LOADCONST_D(((constant_double *) (m->class->cpinfos[i]))->value);
+                               OP_LOADCONST_D(((constant_double *) (m->class->cpinfos[i]))->value);
                                break;
                        case CONSTANT_String:
-                               LOADCONST_A(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))
-                                       return NULL;
+                               if (!resolve_classref(m, cr, resolveLazy, true, true, &c))
+                                       return false;
 
                                /* if not resolved, c == NULL */
 
-                               LOADCONST_A_CLASS(c, cr);
+                               OP_LOADCONST_CLASSINFO_OR_CLASSREF_CHECK(c, cr);
+
                                break;
+
+#if defined(ENABLE_VERIFIER)
                        default:
-                               *exceptionptr = new_verifyerror(m,
+                               exceptions_throw_verifyerror(m,
                                                "Invalid constant type to push");
-                               return NULL;
+                               return false;
+#endif
                        }
                        break;
 
                case JAVA_ACONST_NULL:
-                       LOADCONST_A(NULL);
+                       OP_LOADCONST_NULL();
                        break;
 
                case JAVA_ICONST_M1:
@@ -322,162 +618,243 @@ methodinfo *parse(methodinfo *m, codegendata *cd)
                case JAVA_ICONST_3:
                case JAVA_ICONST_4:
                case JAVA_ICONST_5:
-                       LOADCONST_I(opcode - JAVA_ICONST_0);
+                       OP_LOADCONST_I(opcode - JAVA_ICONST_0);
                        break;
 
                case JAVA_LCONST_0:
                case JAVA_LCONST_1:
-                       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:
-                       LOADCONST_F(opcode - JAVA_FCONST_0);
+                       OP_LOADCONST_F(opcode - JAVA_FCONST_0);
                        break;
 
                case JAVA_DCONST_0:
                case JAVA_DCONST_1:
-                       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;
 
-                       /* loading variables onto the stack */
+               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 *********************************/
 
                case JAVA_ILOAD:
-               case JAVA_LLOAD:
                case JAVA_FLOAD:
-               case JAVA_DLOAD:
                case JAVA_ALOAD:
-                       if (!iswide) {
-                               i = code_get_u1(p + 1,m);
-                       } 
+                       if (iswide == false) {
+                               i = SUCK_BE_U1(m->jcode + bcindex + 1);
+                       }
+                       else {
+                               i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                               nextbc = bcindex + 3;
+                               iswide = false;
+                       }
+                       OP_LOAD_ONEWORD(opcode, i, opcode - JAVA_ILOAD);
+                       break;
+
+               case JAVA_LLOAD:
+               case JAVA_DLOAD:
+                       if (iswide == false) {
+                               i = SUCK_BE_U1(m->jcode + bcindex + 1);
+                       }
                        else {
-                               i = code_get_u2(p + 1,m);
-                               nextp = p + 3;
+                               i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                               nextbc = bcindex + 3;
                                iswide = false;
                        }
-                       OP1LOAD(opcode, i + firstlocal);
+                       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:
-                       OP1LOAD(ICMD_ILOAD, opcode - JAVA_ILOAD_0 + firstlocal);
+                       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:
-                       OP1LOAD(ICMD_LLOAD, opcode - JAVA_LLOAD_0 + firstlocal);
+                       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:
-                       OP1LOAD(ICMD_FLOAD, opcode - JAVA_FLOAD_0 + firstlocal);
+                       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:
-                       OP1LOAD(ICMD_DLOAD, opcode - JAVA_DLOAD_0 + firstlocal);
+                       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:
-                       OP1LOAD(ICMD_ALOAD, opcode - JAVA_ALOAD_0 + firstlocal);
+                       OP_LOAD_ONEWORD(ICMD_ALOAD, opcode - JAVA_ALOAD_0, TYPE_ADR);
                        break;
 
-                       /* storing stack values into local variables */
-
                case JAVA_ISTORE:
-               case JAVA_LSTORE:
                case JAVA_FSTORE:
-               case JAVA_DSTORE:
                case JAVA_ASTORE:
-                       if (!iswide) {
-                               i = code_get_u1(p + 1,m);
-                       } 
+                       if (iswide == false) {
+                               i = SUCK_BE_U1(m->jcode + bcindex + 1);
+                       }
                        else {
-                               i = code_get_u2(p + 1,m);
+                               i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                               nextbc = bcindex + 3;
                                iswide = false;
-                               nextp = p + 3;
                        }
-                       OP1STORE(opcode, i + firstlocal);
+                       OP_STORE_ONEWORD(opcode, i, opcode - JAVA_ISTORE);
+                       break;
+
+               case JAVA_LSTORE:
+               case JAVA_DSTORE:
+                       if (iswide == false) {
+                               i = SUCK_BE_U1(m->jcode + bcindex + 1);
+                       }
+                       else {
+                               i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                               nextbc = bcindex + 3;
+                               iswide = false;
+                       }
+                       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:
-                       OP1STORE(ICMD_ISTORE, opcode - JAVA_ISTORE_0 + firstlocal);
+                       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:
-                       OP1STORE(ICMD_LSTORE, opcode - JAVA_LSTORE_0 + firstlocal);
+                       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:
-                       OP1STORE(ICMD_FSTORE, opcode - JAVA_FSTORE_0 + firstlocal);
+                       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:
-                       OP1STORE(ICMD_DSTORE, opcode - JAVA_DSTORE_0 + firstlocal);
+                       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:
-                       OP1STORE(ICMD_ASTORE, opcode - JAVA_ASTORE_0 + firstlocal);
+                       OP_STORE_ONEWORD(ICMD_ASTORE, opcode - JAVA_ASTORE_0, TYPE_ADR);
                        break;
 
                case JAVA_IINC:
                        {
                                int v;
-                               
-                               if (!iswide) {
-                                       i = code_get_u1(p + 1,m);
-                                       v = code_get_s1(p + 2,m);
 
-                               } 
+                               if (iswide == false) {
+                                       i = SUCK_BE_U1(m->jcode + bcindex + 1);
+                                       v = SUCK_BE_S1(m->jcode + bcindex + 2);
+
+                               }
                                else {
-                                       i = code_get_u2(p + 1,m);
-                                       v = code_get_s2(p + 3,m);
+                                       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 + firstlocal);
-                               OP2I(opcode, i + firstlocal, v);
+                               INDEX_ONEWORD(i);
+                               LOCALTYPE_USED(i, TYPE_INT);
+                               OP_LOCALINDEX_I(opcode, i, v);
                        }
                        break;
 
-                       /* wider index for loading, storing and incrementing */
+               /* wider index for loading, storing and incrementing ******************/
 
                case JAVA_WIDE:
+                       bcindex++;
                        iswide = true;
-                       nextp = p + 1;
-                       break;
+                       goto fetch_opcode;
 
                /* managing arrays ****************************************************/
 
                case JAVA_NEWARRAY:
-                       switch (code_get_s1(p + 1, m)) {
+                       switch (SUCK_BE_S1(m->jcode + bcindex + 1)) {
                        case 4:
                                bte = builtintable_get_internal(BUILTIN_newarray_boolean);
                                break;
@@ -502,50 +879,54 @@ methodinfo *parse(methodinfo *m, codegendata *cd)
                        case 11:
                                bte = builtintable_get_internal(BUILTIN_newarray_long);
                                break;
+#if defined(ENABLE_VERIFIER)
                        default:
-                               *exceptionptr = new_verifyerror(m,
-                                               "Invalid array-type to create");
-                               return NULL;
+                               exceptions_throw_verifyerror(m, "Invalid array-type to create");
+                               return false;
+#endif
                        }
-                       BUILTIN(bte, true, NULL, currentline);
+                       OP_BUILTIN_CHECK_EXCEPTION(bte);
                        break;
 
                case JAVA_ANEWARRAY:
-                       i = code_get_u2(p + 1, m);
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
                        compr = (constant_classref *) class_getconstant(m->class, i, CONSTANT_Class);
-                       if (!compr)
-                               return NULL;
+                       if (compr == NULL)
+                               return false;
 
                        if (!(cr = class_get_classref_multiarray_of(1, compr)))
-                               return NULL;
+                               return false;
 
                        if (!resolve_classref(m, cr, resolveLazy, true, true, &c))
-                               return NULL;
+                               return false;
 
-                       LOADCONST_A_BUILTIN(c, cr);
+                       INSTRUCTIONS_CHECK(2);
+                       OP_LOADCONST_CLASSINFO_OR_CLASSREF_NOCHECK(c, cr);
                        bte = builtintable_get_internal(BUILTIN_newarray);
-                       BUILTIN(bte, true, NULL, currentline);
+                       OP_BUILTIN_CHECK_EXCEPTION(bte);
                        s_count++;
                        break;
 
                case JAVA_MULTIANEWARRAY:
-                       m->isleafmethod = false;
-                       i = code_get_u2(p + 1, m);
-                       {
-                               s4 v = code_get_u1(p + 3, m);
-
-                               cr = (constant_classref *) class_getconstant(m->class, i, CONSTANT_Class);
-                               if (!cr)
-                                       return NULL;
-
-                               if (!resolve_classref(m, cr, resolveLazy, true, true, &c))
-                                       return NULL;
-
-                               /* if unresolved, c == NULL */
-                               OP2AT(opcode, v, c, cr, currentline);
-                       }
+                       jd->isleafmethod = false;
+                       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 ******************************************/
+
                case JAVA_IFEQ:
                case JAVA_IFLT:
                case JAVA_IFLE:
@@ -563,35 +944,48 @@ methodinfo *parse(methodinfo *m, codegendata *cd)
                case JAVA_IF_ACMPEQ:
                case JAVA_IF_ACMPNE:
                case JAVA_GOTO:
-               case JAVA_JSR:
-                       i = p + code_get_s2(p + 1,m);
-                       bound_check(i);
-                       block_insert(i);
+                       i = bcindex + SUCK_BE_S2(m->jcode + bcindex + 1);
+                       CHECK_BYTECODE_INDEX(i);
+                       MARK_BASICBLOCK(&pd, i);
                        blockend = true;
-                       OP1(opcode, i);
+                       OP_INSINDEX(opcode, i);
                        break;
 
                case JAVA_GOTO_W:
-               case JAVA_JSR_W:
-                       i = p + code_get_s4(p + 1,m);
-                       bound_check(i);
-                       block_insert(i);
+                       i = bcindex + SUCK_BE_S4(m->jcode + bcindex + 1);
+                       CHECK_BYTECODE_INDEX(i);
+                       MARK_BASICBLOCK(&pd, i);
+                       blockend = true;
+                       OP_INSINDEX(ICMD_GOTO, i);
+                       break;
+
+               case JAVA_JSR:
+                       i = bcindex + SUCK_BE_S2(m->jcode + bcindex + 1);
+jsr_tail:
+                       CHECK_BYTECODE_INDEX(i);
+                       MARK_BASICBLOCK(&pd, i);
                        blockend = true;
-                       OP1(opcode, i);
+                       OP_PREPARE_ZEROFLAGS(JAVA_JSR);
+                       iptr->sx.s23.s3.jsrtarget.insindex = i;
+                       PINC;
                        break;
 
+               case JAVA_JSR_W:
+                       i = bcindex + SUCK_BE_S4(m->jcode + bcindex + 1);
+                       goto jsr_tail;
+
                case JAVA_RET:
-                       if (!iswide) {
-                               i = code_get_u1(p + 1,m);
-                       } 
+                       if (iswide == false) {
+                               i = SUCK_BE_U1(m->jcode + bcindex + 1);
+                       }
                        else {
-                               i = code_get_u2(p + 1,m);
-                               nextp = p + 3;
+                               i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                               nextbc = bcindex + 3;
                                iswide = false;
                        }
                        blockend = true;
-                               
-                       OP1LOAD(opcode, i + firstlocal);
+
+                       OP_LOAD_ONEWORD(opcode, i, TYPE_ADR);
                        break;
 
                case JAVA_IRETURN:
@@ -601,90 +995,84 @@ methodinfo *parse(methodinfo *m, codegendata *cd)
                case JAVA_ARETURN:
                case JAVA_RETURN:
                        blockend = true;
-                       /* zero val.a so no patcher is inserted */
-                       /* the type checker may set this later  */
-                       iptr->val.a = NULL;
+                       /* XXX ARETURN will need a flag in the typechecker */
                        OP(opcode);
                        break;
 
                case JAVA_ATHROW:
                        blockend = true;
-                       /* zero val.a so no patcher is inserted */
-                       /* the type checker may set this later  */
-                       iptr->val.a = NULL;
+                       /* XXX ATHROW will need a flag in the typechecker */
                        OP(opcode);
                        break;
-                               
+
 
                /* table jumps ********************************************************/
 
                case JAVA_LOOKUPSWITCH:
                        {
                                s4 num, j;
-                               s4 *tablep;
-                               s4 prevvalue=0;
-
+                               lookup_target_t *lookup;
+#if defined(ENABLE_VERIFIER)
+                               s4 prevvalue = 0;
+#endif
                                blockend = true;
-                               nextp = ALIGN((p + 1), 4);
-
-                               if (nextp + 8 > m->jcodelength) {
-                                       *exceptionptr = new_verifyerror(m,
-                                                       "Unexpected end of bytecode");
-                                       return NULL;
-                               }
+                               nextbc = MEMORY_ALIGN((bcindex + 1), 4);
 
-                               tablep = (s4 *) (m->jcode + nextp);
+                               CHECK_END_OF_BYTECODE(nextbc + 8);
 
-                               OP2A(opcode, 0, tablep, currentline);
+                               OP_PREPARE_ZEROFLAGS(opcode);
 
                                /* default target */
 
-                               j =  p + code_get_s4(nextp, m);
-                               *tablep = j;     /* restore for little endian */
-                               tablep++;
-                               nextp += 4;
-                               bound_check(j);
-                               block_insert(j);
+                               j = bcindex + SUCK_BE_S4(m->jcode + nextbc);
+                               iptr->sx.s23.s3.lookupdefault.insindex = j;
+                               nextbc += 4;
+                               CHECK_BYTECODE_INDEX(j);
+                               MARK_BASICBLOCK(&pd, j);
 
                                /* number of pairs */
 
-                               num = code_get_u4(nextp, m);
-                               *tablep = num;
-                               tablep++;
-                               nextp += 4;
+                               num = SUCK_BE_U4(m->jcode + nextbc);
+                               iptr->sx.s23.s2.lookupcount = num;
+                               nextbc += 4;
 
-                               if (nextp + 8 * num > m->jcodelength) {
-                                       *exceptionptr = new_verifyerror(m,
-                                               "Unexpected end of bytecode");
-                                       return NULL;
-                               }
+                               /* allocate the intermediate code table */
+
+                               lookup = DMNEW(lookup_target_t, num);
+                               iptr->dst.lookup = lookup;
+
+                               /* iterate over the lookup table */
+
+                               CHECK_END_OF_BYTECODE(nextbc + 8 * num);
 
                                for (i = 0; i < num; i++) {
                                        /* value */
 
-                                       j = code_get_s4(nextp, m);
-                                       *tablep = j; /* restore for little endian */
-                                       tablep++;
-                                       nextp += 4;
+                                       j = SUCK_BE_S4(m->jcode + nextbc);
+                                       lookup->value = j;
 
+                                       nextbc += 4;
+
+#if defined(ENABLE_VERIFIER)
                                        /* check if the lookup table is sorted correctly */
-                                       
+
                                        if (i && (j <= prevvalue)) {
-                                               *exceptionptr = new_verifyerror(m, "Unsorted lookup switch");
-                                               return NULL;
+                                               exceptions_throw_verifyerror(m, "Unsorted lookup switch");
+                                               return false;
                                        }
                                        prevvalue = j;
-
+#endif
                                        /* target */
 
-                                       j = p + code_get_s4(nextp,m);
-                                       *tablep = j; /* restore for little endian */
-                                       tablep++;
-                                       nextp += 4;
-                                       bound_check(j);
-                                       block_insert(j);
+                                       j = bcindex + SUCK_BE_S4(m->jcode + nextbc);
+                                       lookup->target.insindex = j;
+                                       lookup++;
+                                       nextbc += 4;
+                                       CHECK_BYTECODE_INDEX(j);
+                                       MARK_BASICBLOCK(&pd, j);
                                }
 
+                               PINC;
                                break;
                        }
 
@@ -692,66 +1080,66 @@ methodinfo *parse(methodinfo *m, codegendata *cd)
                case JAVA_TABLESWITCH:
                        {
                                s4 num, j;
-                               s4 *tablep;
+                               s4 deftarget;
+                               branch_target_t *table;
 
                                blockend = true;
-                               nextp = ALIGN((p + 1), 4);
-                               if (nextp + 12 > m->jcodelength) {
-                                       *exceptionptr = new_verifyerror(m,
-                                               "Unexpected end of bytecode");
-                                       return NULL;
-                               }
+                               nextbc = MEMORY_ALIGN((bcindex + 1), 4);
 
-                               tablep = (s4 *) (m->jcode + nextp);
+                               CHECK_END_OF_BYTECODE(nextbc + 12);
 
-                               OP2A(opcode, 0, tablep, currentline);
+                               OP_PREPARE_ZEROFLAGS(opcode);
 
                                /* default target */
 
-                               j = p + code_get_s4(nextp, m);
-                               *tablep = j;     /* restore for little endian */
-                               tablep++;
-                               nextp += 4;
-                               bound_check(j);
-                               block_insert(j);
+                               deftarget = bcindex + SUCK_BE_S4(m->jcode + nextbc);
+                               nextbc += 4;
+                               CHECK_BYTECODE_INDEX(deftarget);
+                               MARK_BASICBLOCK(&pd, deftarget);
 
                                /* lower bound */
 
-                               j = code_get_s4(nextp, m);
-                               *tablep = j;     /* restore for little endian */
-                               tablep++;
-                               nextp += 4;
+                               j = SUCK_BE_S4(m->jcode + nextbc);
+                               iptr->sx.s23.s2.tablelow = j;
+                               nextbc += 4;
 
                                /* upper bound */
 
-                               num = code_get_s4(nextp, m);
-                               *tablep = num;   /* restore for little endian */
-                               tablep++;
-                               nextp += 4;
+                               num = SUCK_BE_S4(m->jcode + nextbc);
+                               iptr->sx.s23.s3.tablehigh = num;
+                               nextbc += 4;
+
+                               /* calculate the number of table entries */
 
-                               num -= j;  /* difference of upper - lower */
+                               num = num - j + 1;
 
-                               if (num < 0) {
-                                       *exceptionptr = new_verifyerror(m,
+#if defined(ENABLE_VERIFIER)
+                               if (num < 1) {
+                                       exceptions_throw_verifyerror(m,
                                                        "invalid TABLESWITCH: upper bound < lower bound");
-                                       return NULL;
+                                       return false;
                                }
+#endif
+                               /* create the intermediate code table */
+                               /* the first entry is the default target */
 
-                               if (nextp + 4 * (num + 1) > m->jcodelength) {
-                                       *exceptionptr = new_verifyerror(m,
-                                               "Unexpected end of bytecode");
-                                       return NULL;
-                               }
+                               table = DMNEW(branch_target_t, 1 + num);
+                               iptr->dst.table = table;
+                               (table++)->insindex = deftarget;
+
+                               /* iterate over the target table */
+
+                               CHECK_END_OF_BYTECODE(nextbc + 4 * num);
 
-                               for (i = 0; i <= num; i++) {
-                                       j = p + code_get_s4(nextp,m);
-                                       *tablep = j; /* restore for little endian */
-                                       tablep++;
-                                       nextp += 4;
-                                       bound_check(j);
-                                       block_insert(j);
+                               for (i = 0; i < num; i++) {
+                                       j = bcindex + SUCK_BE_S4(m->jcode + nextbc);
+                                       (table++)->insindex = j;
+                                       nextbc += 4;
+                                       CHECK_BYTECODE_INDEX(j);
+                                       MARK_BASICBLOCK(&pd, j);
                                }
 
+                               PINC;
                                break;
                        }
 
@@ -760,329 +1148,297 @@ methodinfo *parse(methodinfo *m, codegendata *cd)
 
                case JAVA_AASTORE:
                        OP(opcode);
-                       m->isleafmethod = false;
+                       jd->isleafmethod = false;
                        break;
 
                case JAVA_GETSTATIC:
                case JAVA_PUTSTATIC:
                case JAVA_GETFIELD:
                case JAVA_PUTFIELD:
-                       {
-                               constant_FMIref  *fr;
-                               unresolved_field *uf;
-                               fieldinfo        *fi;
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                       fmi = class_getconstant(m->class, i, CONSTANT_Fieldref);
 
-                               i = code_get_u2(p + 1, m);
-                               fr = class_getconstant(m->class, i,
-                                                                          CONSTANT_Fieldref);
-                               if (!fr)
-                                       return NULL;
+                       if (fmi == NULL)
+                               return false;
 
-                               OP2A_NOINC(opcode, fr->parseddesc.fd->type, fr, currentline);
+                       OP_PREPARE_ZEROFLAGS(opcode);
+                       iptr->sx.s23.s3.fmiref = fmi;
 
-                               if (!(uf = create_unresolved_field(m->class,
-                                                                                                  m,
-                                                                                                  iptr)))
-                                       return NULL;
+                       /* only with -noverify, otherwise the typechecker does this */
 
-                               /* store unresolved_field pointer */
+#if defined(ENABLE_VERIFIER)
+                       if (!JITDATA_HAS_FLAG_VERIFY(jd)) {
+#endif
+                               result = resolve_field_lazy(m, fmi);
 
-                               iptr->target = uf;
+                               if (result == resolveFailed)
+                                       return false;
 
-                               /* only with -noverify, otherwise the typechecker does this */
+                               if (result != resolveSucceeded) {
+                                       uf = resolve_create_unresolved_field(m->class, m, iptr);
 
-                               if (!opt_verify) {
-                                       if (!resolve_field(uf, resolveLazy, &fi))
-                                               return NULL;
+                                       if (uf == NULL)
+                                               return false;
 
-                                       iptr->val.a = fi;
+                                       /* store the unresolved_field pointer */
 
-                               } 
-                               else {
-                                       iptr->val.a = NULL;
+                                       iptr->sx.s23.s3.uf = uf;
+                                       iptr->flags.bits |= INS_FLAG_UNRESOLVED;
                                }
-                               PINC;
+#if defined(ENABLE_VERIFIER)
                        }
+#endif
+                       PINC;
                        break;
 
 
                /* method invocation **************************************************/
 
                case JAVA_INVOKESTATIC:
-                       i = code_get_u2(p + 1, m);
-                       {
-                               constant_FMIref   *mr;
-                               methoddesc        *md;
-                               unresolved_method *um;
-                               methodinfo        *mi;
-
-                               m->isleafmethod = false;
-
-                               mr = class_getconstant(m->class, i,
-                                                                          CONSTANT_Methodref);
-                               if (!mr)
-                                       return NULL;
+                       OP_PREPARE_ZEROFLAGS(opcode);
 
-                               md = mr->parseddesc.md;
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                       fmi = class_getconstant(m->class, i, CONSTANT_Methodref);
 
-                               if (!md->params)
-                                       if (!descriptor_params_from_paramtypes(md, ACC_STATIC))
-                                               return NULL;
+                       if (fmi == NULL)
+                               return false;
 
-                               OP2A_NOINC(opcode, 0, mr, currentline);
+                       md = fmi->parseddesc.md;
 
-                               um = create_unresolved_method(m->class,
-                                                                                         m, iptr);
+                       if (md->params == NULL)
+                               if (!descriptor_params_from_paramtypes(md, ACC_STATIC))
+                                       return false;
 
-                               if (!um)
-                                       return NULL;
-
-                               /* store the unresolved_method pointer */
-
-                               iptr->target = um;
-
-                               /* only with -noverify, otherwise the typechecker does this */
-
-                               if (!opt_verify) {
-                                       if (!resolve_method(um, resolveLazy, &mi))
-                                               return NULL;
-
-                                       iptr->val.a = mi;
-                               }
-                               else {
-                                       iptr->val.a = NULL;
-                               }
-                               PINC;
-                       }
-                       break;
+                       goto invoke_method;
 
                case JAVA_INVOKESPECIAL:
-               case JAVA_INVOKEVIRTUAL:
-                       {
-                               constant_FMIref   *mr;
-                               methoddesc        *md;
-                               unresolved_method *um;
-                               methodinfo        *mi;
+                       OP_PREPARE_FLAGS(opcode, INS_FLAG_CHECK);
 
-                               m->isleafmethod = false;
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                       fmi = class_getconstant(m->class, i, CONSTANT_Methodref);
 
-                               i = code_get_u2(p + 1, m);
-                               mr = class_getconstant(m->class, i,
-                                                                          CONSTANT_Methodref);
-                               if (!mr)
-                                       return NULL;
+                       goto invoke_nonstatic_method;
 
-                               md = mr->parseddesc.md;
-
-                               if (!md->params)
-                                       if (!descriptor_params_from_paramtypes(md, 0))
-                                               return NULL;
-                               
-                               OP2A_NOINC(opcode, 0, mr, currentline);
+               case JAVA_INVOKEINTERFACE:
+                       OP_PREPARE_ZEROFLAGS(opcode);
 
-                               um = create_unresolved_method(m->class,
-                                                                                         m, iptr);
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                       fmi = class_getconstant(m->class, i, CONSTANT_InterfaceMethodref);
 
-                               if (!um)
-                                       return NULL;
+                       goto invoke_nonstatic_method;
 
-                               /* store the unresolved_method* */
+               case JAVA_INVOKEVIRTUAL:
+                       OP_PREPARE_ZEROFLAGS(opcode);
 
-                               iptr->target = um;
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                       fmi = class_getconstant(m->class, i, CONSTANT_Methodref);
 
-                               /* only with -noverify, otherwise the typechecker does this */
+invoke_nonstatic_method:
+                       if (fmi == NULL)
+                               return false;
 
-                               if (!opt_verify) {
-                                       if (!resolve_method(um, resolveLazy, &mi))
-                                               return NULL;
+                       md = fmi->parseddesc.md;
 
-                                       iptr->val.a = mi;
-                               }
-                               else {
-                                       iptr->val.a = NULL;
-                               }
-                               PINC;
-                       }
-                       break;
+                       if (md->params == NULL)
+                               if (!descriptor_params_from_paramtypes(md, 0))
+                                       return false;
 
-               case JAVA_INVOKEINTERFACE:
-                       i = code_get_u2(p + 1, m);
-                       {
-                               constant_FMIref   *mr;
-                               methoddesc        *md;
-                               unresolved_method *um;
-                               methodinfo        *mi;
-                               
-                               m->isleafmethod = false;
+invoke_method:
+                       jd->isleafmethod = false;
 
-                               mr = class_getconstant(m->class, i,
-                                                                          CONSTANT_InterfaceMethodref);
-                               if (!mr)
-                                       return NULL;
+                       iptr->sx.s23.s3.fmiref = fmi;
 
-                               md = mr->parseddesc.md;
+                       /* only with -noverify, otherwise the typechecker does this */
 
-                               if (!md->params)
-                                       if (!descriptor_params_from_paramtypes(md, 0))
-                                               return NULL;
+#if defined(ENABLE_VERIFIER)
+                       if (!JITDATA_HAS_FLAG_VERIFY(jd)) {
+#endif
+                               result = resolve_method_lazy(m, fmi, 
+                                                                                        (opcode == JAVA_INVOKESPECIAL));
 
-                               OP2A_NOINC(opcode, 0, mr, currentline);
+                               if (result == resolveFailed)
+                                       return false;
 
-                               um = create_unresolved_method(m->class,
-                                                                                         m, iptr);
+                               if (result == resolveSucceeded) {
+                                       methodinfo *mi = iptr->sx.s23.s3.fmiref->p.method;
 
-                               if (!um)
-                                       return NULL;
+                                       /* if this call is monomorphic, turn it into an
+                                          INVOKESPECIAL */
 
-                               /* store the unresolved_method* */
+                                       assert(IS_FMIREF_RESOLVED(iptr->sx.s23.s3.fmiref));
 
-                               iptr->target = um;
+                                       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));
 
-                               /* only with -noverify, otherwise the typechecker does this */
+                                       if (um == NULL)
+                                               return false;
 
-                               if (!opt_verify) {
-                                       if (!resolve_method(um, resolveLazy, &mi))
-                                               return NULL;
+                                       /* store the unresolved_method pointer */
 
-                                       iptr->val.a = mi;
-                               }
-                               else {
-                                       iptr->val.a = NULL;
+                                       iptr->sx.s23.s3.um = um;
+                                       iptr->flags.bits |= INS_FLAG_UNRESOLVED;
                                }
-                               PINC;
+#if defined(ENABLE_VERIFIER)
                        }
+#endif
+                       PINC;
                        break;
 
-               /* miscellaneous object operations ************************************/
+               /* instructions taking class arguments ********************************/
 
                case JAVA_NEW:
-                       i = code_get_u2(p + 1, m);
-                       cr = (constant_classref *) class_getconstant(m->class, i, CONSTANT_Class);
-                       if (!cr)
-                               return NULL;
+                       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 NULL;
+                       if (!resolve_classref(m, cr, resolveLazy, true, true, &c))
+                               return false;
 
-                       LOADCONST_A_BUILTIN(c, cr);
+                       INSTRUCTIONS_CHECK(2);
+                       OP_LOADCONST_CLASSINFO_OR_CLASSREF_NOCHECK(c, cr);
                        bte = builtintable_get_internal(BUILTIN_new);
-                       BUILTIN(bte, true, NULL, currentline);
+                       OP_BUILTIN_CHECK_EXCEPTION(bte);
                        s_count++;
                        break;
 
                case JAVA_CHECKCAST:
-                       i = code_get_u2(p + 1, m);
-                       cr = (constant_classref *) class_getconstant(m->class, i, CONSTANT_Class);
-                       if (!cr)
-                               return NULL;
+                       i = SUCK_BE_U2(m->jcode + bcindex + 1);
+                       cr = class_getconstant(m->class, i, CONSTANT_Class);
 
-                       if (!resolve_classref(m, cr, resolveLazy, true,
-                                                                 true, &c))
-                               return NULL;
+                       if (cr == NULL)
+                               return false;
+
+                       if (!resolve_classref(m, cr, resolveLazy, true, true, &c))
+                               return false;
 
                        if (cr->name->text[0] == '[') {
                                /* array type cast-check */
-                               OP2AT(opcode, 0, c, cr, currentline);
-                               m->isleafmethod = false;
-
-                       } 
+                               flags = INS_FLAG_CHECK | INS_FLAG_ARRAY;
+                               jd->isleafmethod = false;
+                       }
                        else {
                                /* object type cast-check */
-                               OP2AT(opcode, 1, c, cr, currentline);
+                               flags = INS_FLAG_CHECK;
                        }
+                       OP_S3_CLASSINFO_OR_CLASSREF(opcode, c, cr, flags);
                        break;
 
                case JAVA_INSTANCEOF:
-                       i = code_get_u2(p + 1,m);
-                       cr = (constant_classref *) class_getconstant(m->class, i, CONSTANT_Class);
-                       if (!cr)
-                               return NULL;
+                       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 NULL;
+                               return false;
 
                        if (cr->name->text[0] == '[') {
                                /* array type cast-check */
-                               LOADCONST_A_BUILTIN(c, cr);
+                               INSTRUCTIONS_CHECK(2);
+                               OP_LOADCONST_CLASSINFO_OR_CLASSREF_NOCHECK(c, cr);
                                bte = builtintable_get_internal(BUILTIN_arrayinstanceof);
-                               BUILTIN(bte, false, NULL, currentline);
+                               OP_BUILTIN_NO_EXCEPTION(bte);
                                s_count++;
-
-                       } 
+                       }
                        else {
                                /* object type cast-check */
-                               OP2AT(opcode, 1, c, cr, currentline);
+                               OP_S3_CLASSINFO_OR_CLASSREF(opcode, c, cr, 0 /* flags*/);
                        }
                        break;
 
+               /* synchronization instructions ***************************************/
+
                case JAVA_MONITORENTER:
-#if defined(USE_THREADS)
+#if defined(ENABLE_THREADS)
                        if (checksync) {
-                               OP(ICMD_CHECKNULL);
-                               bte = builtintable_get_internal(BUILTIN_monitorenter);
-                               BUILTIN(bte, false, NULL, currentline);
-                       } 
+                               bte = builtintable_get_internal(LOCK_monitor_enter);
+                               OP_BUILTIN_CHECK_EXCEPTION(bte);
+                       }
                        else
 #endif
-                               {
-                                       OP(ICMD_CHECKNULL);
-                                       OP(ICMD_POP);
-                               }
+                       {
+                               OP_CHECK_EXCEPTION(ICMD_CHECKNULL);
+                               OP(ICMD_POP);
+                       }
                        break;
 
                case JAVA_MONITOREXIT:
-#if defined(USE_THREADS)
+#if defined(ENABLE_THREADS)
                        if (checksync) {
-                               bte = builtintable_get_internal(BUILTIN_monitorexit);
-                               BUILTIN(bte, false, NULL, currentline);
-                       } 
+                               bte = builtintable_get_internal(LOCK_monitor_exit);
+                               OP_BUILTIN_CHECK_EXCEPTION(bte);
+                       }
                        else
 #endif
-                               {
-                                       OP(ICMD_POP);
-                               }
+                       {
+                               OP_CHECK_EXCEPTION(ICMD_CHECKNULL);
+                               OP(ICMD_POP);
+                       }
                        break;
 
-               /* any other basic operation ******************************************/
+               /* arithmetic instructions that may become builtin functions **********/
 
                case JAVA_IDIV:
 #if !SUPPORT_DIVISION
                        bte = builtintable_get_internal(BUILTIN_idiv);
-                       OP2A(opcode, bte->md->paramcount, bte, currentline);
-                       m->isleafmethod = false;
+                       OP_BUILTIN_ARITHMETIC(opcode, bte);
 #else
+# 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);
-                       OP2A(opcode, bte->md->paramcount, bte, currentline);
-                       m->isleafmethod = false;
+                       OP_BUILTIN_ARITHMETIC(opcode, bte);
 #else
+# 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);
-                       OP2A(opcode, bte->md->paramcount, bte, currentline);
-                       m->isleafmethod = false;
+                       OP_BUILTIN_ARITHMETIC(opcode, bte);
 #else
+# 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);
-                       OP2A(opcode, bte->md->paramcount, bte, currentline);
-                       m->isleafmethod = false;
+                       OP_BUILTIN_ARITHMETIC(opcode, bte);
 #else
+# if SUPPORT_HARDWARE_DIVIDE_BY_ZERO
                        OP(opcode);
+# else
+                       OP_CHECK_EXCEPTION(opcode);
+# endif
 #endif
                        break;
 
@@ -1091,7 +1447,7 @@ methodinfo *parse(methodinfo *m, codegendata *cd)
                        OP(opcode);
 #else
                        bte = builtintable_get_internal(BUILTIN_frem);
-                       BUILTIN(bte, false, NULL, currentline);
+                       OP_BUILTIN_NO_EXCEPTION(bte);
 #endif
                        break;
 
@@ -1100,7 +1456,7 @@ methodinfo *parse(methodinfo *m, codegendata *cd)
                        OP(opcode);
 #else
                        bte = builtintable_get_internal(BUILTIN_drem);
-                       BUILTIN(bte, false, NULL, currentline);
+                       OP_BUILTIN_NO_EXCEPTION(bte);
 #endif
                        break;
 
@@ -1108,58 +1464,61 @@ methodinfo *parse(methodinfo *m, codegendata *cd)
 #if defined(__ALPHA__)
                        if (!opt_noieee) {
                                bte = builtintable_get_internal(BUILTIN_f2i);
-                               BUILTIN(bte, false, NULL, currentline);
-                       } 
+                               OP_BUILTIN_NO_EXCEPTION(bte);
+                       }
                        else
 #endif
-                               {
-                                       OP(opcode);
-                               }
+                       {
+                               OP(opcode);
+                       }
                        break;
 
                case JAVA_F2L:
 #if defined(__ALPHA__)
                        if (!opt_noieee) {
                                bte = builtintable_get_internal(BUILTIN_f2l);
-                               BUILTIN(bte, false, NULL, currentline);
-                       } 
-                       else 
+                               OP_BUILTIN_NO_EXCEPTION(bte);
+                       }
+                       else
 #endif
-                               {
-                                       OP(opcode);
-                               }
+                       {
+                               OP(opcode);
+                       }
                        break;
 
                case JAVA_D2I:
 #if defined(__ALPHA__)
                        if (!opt_noieee) {
                                bte = builtintable_get_internal(BUILTIN_d2i);
-                               BUILTIN(bte, false, NULL, currentline);
-                       } 
+                               OP_BUILTIN_NO_EXCEPTION(bte);
+                       }
                        else
 #endif
-                               {
-                                       OP(opcode);
-                               }
+                       {
+                               OP(opcode);
+                       }
                        break;
 
                case JAVA_D2L:
 #if defined(__ALPHA__)
                        if (!opt_noieee) {
                                bte = builtintable_get_internal(BUILTIN_d2l);
-                               BUILTIN(bte, false, NULL, currentline);
-                       } 
+                               OP_BUILTIN_NO_EXCEPTION(bte);
+                       }
                        else
 #endif
-                               {
-                                       OP(opcode);
-                               }
+                       {
+                               OP(opcode);
+                       }
                        break;
 
+               /* invalid opcodes ****************************************************/
+
+                       /* check for invalid opcodes if the verifier is enabled */
+#if defined(ENABLE_VERIFIER)
                case JAVA_BREAKPOINT:
-                       *exceptionptr =
-                               new_verifyerror(m, "Quick instructions shouldn't appear yet.");
-                       return NULL;
+                       exceptions_throw_verifyerror(m, "Quick instructions shouldn't appear, yet.");
+                       return false;
 
                case 186: /* unused opcode */
                case 203:
@@ -1215,158 +1574,334 @@ methodinfo *parse(methodinfo *m, codegendata *cd)
                case 253:
                case 254:
                case 255:
-                       *exceptionptr =
-                               new_verifyerror(m,"Illegal opcode %d at instr %d\n",
-                                                                 opcode, ipc);
-                       return NULL;
+                       exceptions_throw_verifyerror(m, "Illegal opcode %d at instr %d\n",
+                                                                                opcode, ircount);
+                       return false;
                        break;
+#endif /* defined(ENABLE_VERIFIER) */
+
+               /* opcodes that don't require translation *****************************/
 
                default:
+                       /* straight-forward translation to ICMD */
                        OP(opcode);
                        break;
-                               
+
                } /* end switch */
 
+               /* verifier checks ****************************************************/
+
+#if defined(ENABLE_VERIFIER)
                /* If WIDE was used correctly, iswide should have been reset by now. */
-               if (iswide && opcode != JAVA_WIDE) {
-                       *exceptionptr = new_verifyerror(m,
+               if (iswide) {
+                       exceptions_throw_verifyerror(m,
                                        "Illegal instruction: WIDE before incompatible opcode");
-                       return NULL;
+                       return false;
                }
+#endif /* defined(ENABLE_VERIFIER) */
 
        } /* end for */
 
-       if (p != m->jcodelength) {
-               printf("p (%d) != m->jcodelength (%d)\n",p,m->jcodelength);
-               *exceptionptr = new_verifyerror(m,
+       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(ircount <= pd.instructionslength);
+       assert(ircount == (iptr - pd.instructions));
+
+       /*** verifier checks ******************************************************/
+
+#if defined(ENABLE_VERIFIER)
+       if (bcindex != m->jcodelength) {
+               exceptions_throw_verifyerror(m,
                                "Command-sequence crosses code-boundary");
-               return NULL;
+               return false;
        }
 
        if (!blockend) {
-               *exceptionptr = new_verifyerror(m, "Falling off the end of the code");
-               return NULL;
+               exceptions_throw_verifyerror(m, "Falling off the end of the code");
+               return false;
        }
+#endif /* defined(ENABLE_VERIFIER) */
 
-       /* adjust block count if target 0 is not first intermediate instruction */
+       /*** setup the methodinfo, allocate stack and basic blocks ****************/
 
-       if (!m->basicblockindex[0] || (m->basicblockindex[0] > 1))
-               b_count++;
+       /* identify basic blocks */
 
-       /* copy local to method variables */
+       /* check if first instruction is a branch target */
 
-       m->instructioncount = ipc;
-       m->basicblockcount = b_count;
-       m->stackcount = s_count + m->basicblockcount * m->maxstack;
+       if (pd.basicblockstart[0] == 1) {
+               jd->branchtoentry = true;
+       }
+       else {
+               /* first instruction always starts a basic block */
 
-       /* allocate stack table */
+               iptr = pd.instructions;
+
+               iptr->flags.bits |= INS_FLAG_BASICBLOCK;
+       }
 
-       m->stack = DMNEW(stackelement, m->stackcount);
+       /* Iterate over all bytecode instructions and set missing
+          basic-block starts in IR instructions. */
 
-       {
-               basicblock *bptr;
+       for (bcindex = 0; bcindex < m->jcodelength; bcindex++) {
+               /* Does the current bytecode instruction start a basic
+                  block? */
 
-               bptr = m->basicblocks = DMNEW(basicblock, b_count + 1);    /* one more for end ipc */
+               if (pd.basicblockstart[bcindex] == 1) {
+#if defined(ENABLE_VERIFIER)
+                       /* Check if this bytecode basic-block start at the
+                          beginning of a bytecode instruction. */
 
-               b_count = 0;
-               m->c_debug_nr = 0;
-       
-               /* additional block if target 0 is not first intermediate 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. */
 
-               if (!m->basicblockindex[0] || (m->basicblockindex[0] > 1)) {
-                       BASICBLOCK_INIT(bptr,m);
+                       irindex = pd.bytecodemap[bcindex];
+                       iptr    = pd.instructions + irindex;
+
+                       iptr->flags.bits |= INS_FLAG_BASICBLOCK;
+               }
+       }
+
+       /* IR instruction index to basic-block index mapping */
+
+       pd.instructionmap = DMNEW(s4, ircount);
+       MZERO(pd.instructionmap, s4, ircount);
+
+       /* Iterate over all IR instructions and count the basic blocks. */
+
+       iptr = pd.instructions;
+
+       bbcount = 0;
+
+       for (i = 0; i < ircount; i++, iptr++) {
+               if (INSTRUCTION_STARTS_BASICBLOCK(iptr)) {
+                       /* store the basic-block number in the IR instruction
+                          map */
+
+                       pd.instructionmap[i] = bbcount;
+
+                       /* post-increment the basic-block count */
+
+                       bbcount++;
+               }
+       }
+
+       /* Allocate basic block array (one more for end ipc). */
+
+       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 */
+
+               if (INSTRUCTION_STARTS_BASICBLOCK(iptr)) {
+                       /* intialize the basic block */
+
+                       BASICBLOCK_INIT(bptr, m);
+
+                       bptr->iinstr = iptr;
+
+                       if (bbcount > 0) {
+                               bptr[-1].icount = bptr->iinstr - bptr[-1].iinstr;
+                       }
 
-                       bptr->iinstr = m->instructions;
                        /* bptr->icount is set when the next block is allocated */
 
+                       bptr->nr = bbcount++;
                        bptr++;
-                       b_count++;
                        bptr[-1].next = bptr;
                }
 
-               /* allocate blocks */
+               /* 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;
 
-               for (p = 0; p < m->jcodelength; p++) { 
-                       if (m->basicblockindex[p] & 1) {
-                               /* Check if this block starts at the beginning of an          */
-                               /* instruction.                                               */
+               case ICMD_TABLESWITCH:
+                       table = iptr->dst.table;
 
-                               if (!instructionstart[p]) {
-                                       *exceptionptr = new_verifyerror(m,
-                                               "Branch into middle of instruction");
-                                       return NULL;
-                               }
+                       BYTECODEINDEX_TO_BASICBLOCK(*table);
+                       table++;
+
+                       j = iptr->sx.s23.s3.tablehigh - iptr->sx.s23.s2.tablelow + 1;
 
-                               /* allocate the block */
+                       while (--j >= 0) {
+                               BYTECODEINDEX_TO_BASICBLOCK(*table);
+                               table++;
+                       }
+                       break;
 
-                               BASICBLOCK_INIT(bptr,m);
+               case ICMD_LOOKUPSWITCH:
+                       BYTECODEINDEX_TO_BASICBLOCK(iptr->sx.s23.s3.lookupdefault);
 
-                               bptr->iinstr = m->instructions + (m->basicblockindex[p] >> 1);
-                               if (b_count) {
-                                       bptr[-1].icount = bptr->iinstr - bptr[-1].iinstr;
-                               }
-                               /* bptr->icount is set when the next block is allocated */
+                       lookup = iptr->dst.lookup;
 
-                               m->basicblockindex[p] = b_count;
+                       j = iptr->sx.s23.s2.lookupcount;
 
-                               bptr++;
-                               b_count++;
-                               bptr[-1].next = bptr;
+                       while (--j >= 0) {
+                               BYTECODEINDEX_TO_BASICBLOCK(lookup->target);
+                               lookup++;
                        }
+                       break;
                }
+       }
 
-               /* set instruction count of last real block */
+       /* set instruction count of last real block */
 
-               if (b_count) {
-                       bptr[-1].icount = (m->instructions + m->instructioncount) - bptr[-1].iinstr;
-               }
+       if (bbcount > 0) {
+               bptr[-1].icount = (pd.instructions + ircount) - bptr[-1].iinstr;
+       }
 
-               /* allocate additional block at end */
+       /* allocate additional block at end */
 
-               BASICBLOCK_INIT(bptr,m);
-               
-               bptr->instack = bptr->outstack = NULL;
-               bptr->indepth = bptr->outdepth = 0;
-               bptr->iinstr = NULL;
-               bptr->icount = 0;
-               bptr->next = NULL;
+       BASICBLOCK_INIT(bptr, m);
+       bptr->nr = bbcount;
 
-               /* set basicblock pointers in exception table */
+       /* 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;
+
+       /* store the local map */
+
+       jd->local_map = local_map;
+
+       /* calculate local variable renaming */
+
+       {
+               s4 nlocals = 0;
+               s4 i;
+               s4 *mapptr;
+
+               mapptr = local_map;
+
+               /* 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;
                }
-               
-               for (i = 0; i < cd->exceptiontablelength; ++i) {
-                       p = cd->exceptiontable[i].startpc;
-                       cd->exceptiontable[i].start = m->basicblocks + m->basicblockindex[p];
 
-                       p = cd->exceptiontable[i].endpc;
-                       cd->exceptiontable[i].end = (p == m->jcodelength) ? (m->basicblocks + m->basicblockcount /*+ 1*/) : (m->basicblocks + m->basicblockindex[p]);
+               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;
 
-                       p = cd->exceptiontable[i].handlerpc;
-                       cd->exceptiontable[i].handler = m->basicblocks + m->basicblockindex[p];
-           }
+               /* reserve extra variables needed by stack analyse */
 
-               /* 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 = m->basicblocks + m->basicblockindex[p];
+               jd->varcount += STACK_EXTRA_VARS;
+               jd->vartop   += STACK_EXTRA_VARS;
 
-                       p = m->exceptiontable[i].endpc;
-                       m->exceptiontable[i].end = (p == m->jcodelength) ? (m->basicblocks + m->basicblockcount /*+ 1*/) : (m->basicblocks + m->basicblockindex[p]);
+               /* The verifier needs space for saving invars in some cases and */
+               /* extra variables.                                             */
 
-                       p = m->exceptiontable[i].handlerpc;
-                       m->exceptiontable[i].handler = m->basicblocks + m->basicblockindex[p];
-           }
+#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;
        }
 
-       /* just return methodinfo* to signal everything was ok */
+       /* 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 */
+
+       return true;
+
+       /*** goto labels for throwing verifier exceptions *************************/
+
+#if defined(ENABLE_VERIFIER)
+
+throw_unexpected_end_of_bytecode:
+       exceptions_throw_verifyerror(m, "Unexpected end of bytecode");
+       return false;
+
+throw_invalid_bytecode_index:
+       exceptions_throw_verifyerror(m, "Illegal target of branch instruction");
+       return false;
+
+throw_illegal_local_variable_number:
+       exceptions_throw_verifyerror(m, "Illegal local variable number");
+       return false;
 
-       return m;
+#endif /* ENABLE_VERIFIER */
 }