* Removed all Id tags.
[cacao.git] / src / vm / jit / mips / patcher.c
index 30258f9453a2915ace4418fc3ccc4b72ed463ea5..703ee87695d84a25916d608eb541315712b6d1aa 100644 (file)
@@ -1,6 +1,6 @@
 /* src/vm/jit/mips/patcher.c - MIPS code patching functions
 
-   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
-
-   Authors: Christian Thalinger
-
-   $Id: patcher.c 5930 2006-11-06 21:12:52Z twisti $
-
 */
 
 
 #include "config.h"
 
 #include <assert.h>
+#include <stdint.h>
 
 #include "vm/types.h"
 
 #include "vm/jit/mips/codegen.h"
 
 #include "mm/memory.h"
+
 #include "native/native.h"
+
 #include "vm/builtin.h"
-#include "vm/class.h"
 #include "vm/exceptions.h"
-#include "vm/field.h"
 #include "vm/initialize.h"
-#include "vm/options.h"
-#include "vm/resolve.h"
-#include "vm/references.h"
-#include "vm/jit/asmpart.h"
-#include "vm/jit/patcher.h"
 
+#include "vm/jit/asmpart.h"
+#include "vm/jit/md.h"
+#include "vm/jit/patcher-common.h"
 
-/* patcher_wrapper *************************************************************
+#include "vmcore/class.h"
+#include "vmcore/field.h"
+#include "vmcore/options.h"
+#include "vm/resolve.h"
+#include "vmcore/references.h"
 
-   Wrapper for all patchers.  It also creates the stackframe info
-   structure.
 
-   If the return value of the patcher function is false, it gets the
-   exception object, clears the exception pointer and returns the
-   exception.
+#define PATCH_BACK_ORIGINAL_MCODE \
+       *((u4 *) pr->mpc) = (u4) pr->mcode; \
+       md_icacheflush((u1 *) pr->mpc, PATCHER_CALL_SIZE);
 
-*******************************************************************************/
 
-java_objectheader *patcher_wrapper(u1 *sp, u1 *pv, u1 *ra)
-{
-       stackframeinfo     sfi;
-       u1                *xpc;
-       java_objectheader *o;
-       u4                 mcode[5];
-       functionptr        f;
-       bool               result;
-       java_objectheader *e;
+/* patcher_initialize_class ****************************************************
 
-       /* define the patcher function */
+   Initalizes a given classinfo pointer.  This function does not patch
+   any data.
 
-       bool (*patcher_function)(u1 *);
+*******************************************************************************/
 
-       assert(pv != NULL);
+bool patcher_initialize_class(patchref_t *pr)
+{
+       classinfo *c;
 
        /* get stuff from the stack */
 
-       xpc = (u1 *)                *((ptrint *) (sp + 7 * 8));
-       o   = (java_objectheader *) *((ptrint *) (sp + 6 * 8));
-       f   = (functionptr)         *((ptrint *) (sp + 0 * 8));
-
-       /* store PV into the patcher function position */
-
-       *((ptrint *) (sp + 0 * 8)) = (ptrint) pv;
-
-       /* cast the passed function to a patcher function */
-
-       patcher_function = (bool (*)(u1 *)) (ptrint) f;
+       c = (classinfo *) pr->ref;
 
-       /* enter a monitor on the patching position */
-
-       PATCHER_MONITORENTER;
-
-       /* create the stackframeinfo */
-
-       stacktrace_create_extern_stackframeinfo(&sfi, pv, sp + 8 * 8, ra, xpc);
-
-       /* call the proper patcher function */
-
-       result = (patcher_function)(sp);
-
-       /* remove the stackframeinfo */
+       /* check if the class is initialized */
 
-       stacktrace_remove_stackframeinfo(&sfi);
+       if (!(c->state & CLASS_INITIALIZED))
+               if (!initialize_class(c))
+                       return false;
 
-       /* check for return value and exit accordingly */
+       PATCH_BACK_ORIGINAL_MCODE;
 
-       if (result == false) {
-               e = exceptions_get_and_clear_exception();
+       return true;
+}
 
-               PATCHER_MONITOREXIT;
 
-               return e;
-       }
+/* patcher_resolve_class *****************************************************
 
-       /* patch back original code */
+   Initalizes a given classinfo pointer.  This function does not patch
+   any data.
 
-       mcode[0] = *((u4 *) (sp + 3 * 8 + 0));
-       mcode[1] = *((u4 *) (sp + 3 * 8 + 4));
+*******************************************************************************/
 
-       mcode[2] = *((u4 *) (sp + 4 * 8 + 0));
-       mcode[3] = *((u4 *) (sp + 4 * 8 + 4));
-       mcode[4] = *((u4 *) (sp + 5 * 8 + 0));
+#ifdef ENABLE_VERIFIER
+bool patcher_resolve_class(patchref_t *pr)
+{
+       unresolved_class *uc;
 
-       *((u4 *) (xpc + 0 * 4)) = mcode[0];
-       *((u4 *) (xpc + 1 * 4)) = mcode[1];
+       /* get stuff from the stack */
 
-       *((u4 *) (xpc + 2 * 4)) = mcode[2];
-       *((u4 *) (xpc + 3 * 4)) = mcode[3];
-       *((u4 *) (xpc + 4 * 4)) = mcode[4];
+       uc = (unresolved_class *) pr->ref;
 
-       /* synchronize instruction cache */
+       /* resolve the class and check subtype constraints */
 
-       md_icacheflush(xpc, PATCHER_CALL_SIZE);
+       if (!resolve_class_eager_no_access_check(uc))
+               return false;
 
-       PATCHER_MARK_PATCHED_MONITOREXIT;
+       PATCH_BACK_ORIGINAL_MCODE;
 
-       return NULL;
+       return true;
 }
+#endif /* ENABLE_VERIFIER */
 
 
 /* patcher_get_putstatic *******************************************************
@@ -156,18 +123,16 @@ java_objectheader *patcher_wrapper(u1 *sp, u1 *pv, u1 *ra)
 
 *******************************************************************************/
 
-bool patcher_get_putstatic(u1 *sp)
+bool patcher_get_putstatic(patchref_t *pr)
 {
        unresolved_field *uf;
-       s4                disp;
-       u1               *pv;
+       u1               *datap;
        fieldinfo        *fi;
 
        /* get stuff from the stack */
 
-       uf   = (unresolved_field *) *((ptrint *) (sp + 2 * 8));
-       disp =                      *((s4 *)     (sp + 1 * 8));
-       pv   = (u1 *)               *((ptrint *) (sp + 0 * 8));
+       uf    = (unresolved_field *) pr->ref;
+       datap = (u1 *)               pr->datap;
 
        /* get the fieldinfo */
 
@@ -180,13 +145,15 @@ bool patcher_get_putstatic(u1 *sp)
                if (!initialize_class(fi->class))
                        return false;
 
+       PATCH_BACK_ORIGINAL_MCODE;
+
        /* patch the field value's address */
 
-       *((ptrint *) (pv + disp)) = (ptrint) &(fi->value);
+       *((intptr_t *) datap) = (intptr_t) fi->value;
 
        /* synchronize data cache */
 
-       md_dcacheflush(pv + disp, SIZEOF_VOID_P);
+       md_dcacheflush(datap, SIZEOF_VOID_P);
 
        return true;
 }
@@ -201,7 +168,7 @@ bool patcher_get_putstatic(u1 *sp)
 
 *******************************************************************************/
 
-bool patcher_get_putfield(u1 *sp)
+bool patcher_get_putfield(patchref_t *pr)
 {
        u1               *ra;
        unresolved_field *uf;
@@ -209,183 +176,220 @@ bool patcher_get_putfield(u1 *sp)
 
        /* get stuff from the stack */
 
-       ra = (u1 *)               *((ptrint *) (sp + 7 * 8));
-       uf = (unresolved_field *) *((ptrint *) (sp + 2 * 8));
+       ra = (u1 *)               pr->mpc;
+       uf = (unresolved_field *) pr->ref;
 
        /* get the fieldinfo */
 
        if (!(fi = resolve_field_eager(uf)))
                return false;
 
+       PATCH_BACK_ORIGINAL_MCODE;
+
        /* if we show disassembly, we have to skip the nop's */
 
-       if (opt_shownops) {
+       if (opt_shownops)
                ra = ra + PATCHER_CALL_SIZE;
 
 #if SIZEOF_VOID_P == 4
-               if (IS_LNG_TYPE(fi->type)) {
+       if (IS_LNG_TYPE(fi->type)) {
 # if WORDS_BIGENDIAN == 1
-                       /* ATTENTION: order of these instructions depend on M_LLD_INTERN */
-                       *((u4 *) (ra + 0 * 4)) |= (s2) ((fi->offset + 0) & 0x0000ffff);
-                       *((u4 *) (ra + 1 * 4)) |= (s2) ((fi->offset + 4) & 0x0000ffff);
+               /* ATTENTION: order of these instructions depend on M_LLD_INTERN */
+               *((u4 *) (ra + 0 * 4)) |= (s2) ((fi->offset + 0) & 0x0000ffff);
+               *((u4 *) (ra + 1 * 4)) |= (s2) ((fi->offset + 4) & 0x0000ffff);
 # else
-                       /* ATTENTION: order of these instructions depend on M_LLD_INTERN */
-                       *((u4 *) (ra + 0 * 4)) |= (s2) ((fi->offset + 4) & 0x0000ffff);
-                       *((u4 *) (ra + 1 * 4)) |= (s2) ((fi->offset + 0) & 0x0000ffff);
+               /* ATTENTION: order of these instructions depend on M_LLD_INTERN */
+               *((u4 *) (ra + 0 * 4)) |= (s2) ((fi->offset + 4) & 0x0000ffff);
+               *((u4 *) (ra + 1 * 4)) |= (s2) ((fi->offset + 0) & 0x0000ffff);
 # endif
-               }
-               else
+       }
+       else
 #endif
-                       *((u4 *) (ra + 0 * 4)) |= (s2) (fi->offset & 0x0000ffff);
+               *((u4 *) (ra + 0 * 4)) |= (s2) (fi->offset & 0x0000ffff);
 
-               /* synchronize instruction cache */
+       /* synchronize instruction cache */
 
-               md_icacheflush(ra, 2 * 4);
-       }
-       else {
-#if SIZEOF_VOID_P == 4
-               if (IS_LNG_TYPE(fi->type)) {
-# if WORDS_BIGENDIAN == 1
-                       /* ATTENTION: order of these instructions depend on M_LLD_INTERN */
-                       *((u4 *) (sp + 3 * 8 + 0)) |= (s2) ((fi->offset + 0) & 0x0000ffff);
-                       *((u4 *) (sp + 3 * 8 + 4)) |= (s2) ((fi->offset + 4) & 0x0000ffff);
-# else
-                       /* ATTENTION: order of these instructions depend on M_LLD_INTERN */
-                       *((u4 *) (sp + 3 * 8 + 0)) |= (s2) ((fi->offset + 4) & 0x0000ffff);
-                       *((u4 *) (sp + 3 * 8 + 4)) |= (s2) ((fi->offset + 0) & 0x0000ffff);
-# endif
-               }
-               else
-#endif
-                       *((u4 *) (sp + 3 * 8 + 0)) |= (s2) (fi->offset & 0x0000ffff);
-       }
+       md_icacheflush(ra, 2 * 4);
 
        return true;
 }
 
 
-/* patcher_aconst **************************************************************
+/* patcher_resolve_classref_to_classinfo ***************************************
 
-   Machine code:
+   ACONST:
 
    <patched call postition>
    dfc4ff98    ld       a0,-104(s8)
 
+   MULTIANEWARRAY:
+
+   <patched call position>
+   dfc5ff90    ld       a1,-112(s8)
+   03a03025    move     a2,sp
+   dfd9ff88    ld       t9,-120(s8)
+   0320f809    jalr     t9
+   00000000    nop
+
+   ARRAYCHECKCAST:
+
+   <patched call position>
+   dfc5ffc0    ld       a1,-64(s8)
+   dfd9ffb8    ld       t9,-72(s8)
+   0320f809    jalr     t9
+   00000000    nop
+
 *******************************************************************************/
 
-bool patcher_aconst(u1 *sp)
+bool patcher_resolve_classref_to_classinfo(patchref_t *pr)
 {
        constant_classref *cr;
-       s4                 disp;
-       u1                *pv;
+       u1                *datap;
        classinfo         *c;
 
        /* get stuff from the stack */
 
-       cr   = (constant_classref *) *((ptrint *) (sp + 2 * 8));
-       disp =                       *((s4 *)     (sp + 1 * 8));
-       pv   = (u1 *)                *((ptrint *) (sp + 0 * 8));
+       cr    = (constant_classref *) pr->ref;
+       datap = (u1 *)                pr->datap;
 
        /* get the classinfo */
 
        if (!(c = resolve_classref_eager(cr)))
                return false;
 
+       PATCH_BACK_ORIGINAL_MCODE;
+
        /* patch the classinfo pointer */
 
-       *((ptrint *) (pv + disp)) = (ptrint) c;
+       *((intptr_t *) datap) = (intptr_t) c;
 
        /* synchronize data cache */
 
-       md_dcacheflush(pv + disp, SIZEOF_VOID_P);
+       md_dcacheflush(datap, SIZEOF_VOID_P);
 
        return true;
 }
 
 
-/* patcher_builtin_multianewarray **********************************************
+/* patcher_resolve_classref_to_vftbl *******************************************
 
-   Machine code:
+   CHECKCAST (class):
+   INSTANCEOF (class):
 
    <patched call position>
-   dfc5ff90    ld       a1,-112(s8)
-   03a03025    move     a2,sp
-   dfd9ff88    ld       t9,-120(s8)
-   0320f809    jalr     t9
-   00000000    nop
+   dd030000    ld       v1,0(a4)
+   dfd9ff18    ld       t9,-232(s8)
 
 *******************************************************************************/
 
-bool patcher_builtin_multianewarray(u1 *sp)
+bool patcher_resolve_classref_to_vftbl(patchref_t *pr)
 {
        constant_classref *cr;
-       s4                 disp;
-       u1                *pv;
+       u1                *datap;
        classinfo         *c;
 
        /* get stuff from the stack */
 
-       cr   = (constant_classref *) *((ptrint *) (sp + 2 * 8));
-       disp =                       *((s4 *)     (sp + 1 * 8));
-       pv   = (u1 *)                *((ptrint *) (sp + 0 * 8));
+       cr    = (constant_classref *) pr->ref;
+       datap = (u1 *)                pr->datap;
 
-       /* get the classinfo */
+       /* get the fieldinfo */
 
        if (!(c = resolve_classref_eager(cr)))
                return false;
 
-       /* patch the classinfo pointer */
+       PATCH_BACK_ORIGINAL_MCODE;
+
+       /* patch super class' vftbl */
 
-       *((ptrint *) (pv + disp)) = (ptrint) c;
+       *((intptr_t *) datap) = (intptr_t) c->vftbl;
 
        /* synchronize data cache */
 
-       md_dcacheflush(pv + disp, SIZEOF_VOID_P);
+       md_dcacheflush(datap, SIZEOF_VOID_P);
 
        return true;
 }
 
 
-/* patcher_builtin_arraycheckcast **********************************************
+/* patcher_resolve_classref_to_flags *******************************************
 
-   Machine code:
+   CHECKCAST/INSTANCEOF:
 
    <patched call position>
-   dfc5ffc0    ld       a1,-64(s8)
-   dfd9ffb8    ld       t9,-72(s8)
-   0320f809    jalr     t9
+   8fc3ff24    lw       v1,-220(s8)
+   30630200    andi     v1,v1,512
+   1060000d    beq      v1,zero,0x000000001051824c
    00000000    nop
 
 *******************************************************************************/
 
-bool patcher_builtin_arraycheckcast(u1 *sp)
+bool patcher_resolve_classref_to_flags(patchref_t *pr)
 {
        constant_classref *cr;
-       s4                 disp;
-       u1                *pv;
+       u1                *datap;
        classinfo         *c;
 
        /* get stuff from the stack */
 
-       cr   = (constant_classref *) *((ptrint *) (sp + 2 * 8));
-       disp =                       *((s4 *)     (sp + 1 * 8));
-       pv   = (u1 *)                *((ptrint *) (sp + 0 * 8));
+       cr    = (constant_classref *) pr->ref;
+       datap = (u1 *)                pr->datap;
 
-       /* get the classinfo */
+       /* get the fieldinfo */
 
        if (!(c = resolve_classref_eager(cr)))
                return false;
 
-       /* patch the classinfo pointer */
+       PATCH_BACK_ORIGINAL_MCODE;
+
+       /* patch class flags */
+
+       *((int32_t *) datap) = (int32_t) c->flags;
+
+       /* synchronize data cache */
+
+       md_dcacheflush(datap, sizeof(int32_t));
+
+       return true;
+}
+
+
+/* patcher_resolve_native ******************************************************
+
+   XXX
+
+*******************************************************************************/
+
+#if !defined(WITH_STATIC_CLASSPATH)
+bool patcher_resolve_native_function(patchref_t *pr)
+{
+       methodinfo  *m;
+       u1          *datap;
+       functionptr  f;
+
+       /* get stuff from the stack */
+
+       m     = (methodinfo *) pr->ref;
+       datap = (u1 *)         pr->datap;
+
+       /* resolve native function */
 
-       *((ptrint *) (pv + disp)) = (ptrint) c;
+       if (!(f = native_resolve_function(m)))
+               return false;
+
+       PATCH_BACK_ORIGINAL_MCODE;
+
+       /* patch native function pointer */
+
+       *((ptrint *) datap) = (ptrint) f;
 
        /* synchronize data cache */
 
-       md_dcacheflush(pv + disp, SIZEOF_VOID_P);
+       md_dcacheflush(datap, SIZEOF_VOID_P);
 
        return true;
 }
+#endif /* !defined(WITH_STATIC_CLASSPATH) */
 
 
 /* patcher_invokestatic_special ************************************************
@@ -399,31 +403,31 @@ bool patcher_builtin_arraycheckcast(u1 *sp)
 
 ******************************************************************************/
 
-bool patcher_invokestatic_special(u1 *sp)
+bool patcher_invokestatic_special(patchref_t *pr)
 {
        unresolved_method *um;
-       s4                 disp;
-       u1                *pv;
+       u1                *datap;
        methodinfo        *m;
 
        /* get stuff from the stack */
 
-       um   = (unresolved_method *) *((ptrint *) (sp + 2 * 8));
-       disp =                       *((s4 *)     (sp + 1 * 8));
-       pv   = (u1 *)                *((ptrint *) (sp + 0 * 8));
+       um    = (unresolved_method *) pr->ref;
+       datap = (u1 *)                pr->datap;
 
        /* get the fieldinfo */
 
        if (!(m = resolve_method_eager(um)))
                return false;
 
+       PATCH_BACK_ORIGINAL_MCODE;
+
        /* patch stubroutine */
 
-       *((ptrint *) (pv + disp)) = (ptrint) m->stubroutine;
+       *((ptrint *) datap) = (ptrint) m->stubroutine;
 
        /* synchronize data cache */
 
-       md_dcacheflush(pv + disp, SIZEOF_VOID_P);
+       md_dcacheflush(datap, SIZEOF_VOID_P);
 
        return true;
 }
@@ -441,7 +445,7 @@ bool patcher_invokestatic_special(u1 *sp)
 
 *******************************************************************************/
 
-bool patcher_invokevirtual(u1 *sp)
+bool patcher_invokevirtual(patchref_t *pr)
 {
        u1                *ra;
        unresolved_method *um;
@@ -449,36 +453,30 @@ bool patcher_invokevirtual(u1 *sp)
 
        /* get stuff from the stack */
 
-       ra = (u1 *)                *((ptrint *) (sp + 7 * 8));
-       um = (unresolved_method *) *((ptrint *) (sp + 2 * 8));
+       ra = (u1 *)                pr->mpc;
+       um = (unresolved_method *) pr->ref;
 
        /* get the fieldinfo */
 
        if (!(m = resolve_method_eager(um)))
                return false;
 
+       PATCH_BACK_ORIGINAL_MCODE;
+
        /* if we show disassembly, we have to skip the nop's */
 
-       if (opt_shownops) {
+       if (opt_shownops)
                ra = ra + PATCHER_CALL_SIZE;
 
-               /* patch vftbl index */
+       /* patch vftbl index */
 
-               *((s4 *) (ra + 1 * 4)) |=
-                       (s4) ((OFFSET(vftbl_t, table[0]) +
-                                  sizeof(methodptr) * m->vftblindex) & 0x0000ffff);
+       *((s4 *) (ra + 1 * 4)) |=
+               (s4) ((OFFSET(vftbl_t, table[0]) +
+                          sizeof(methodptr) * m->vftblindex) & 0x0000ffff);
 
-               /* synchronize instruction cache */
+       /* synchronize instruction cache */
 
-               md_icacheflush(ra + 1 * 4, 1 * 4);
-       }
-       else {
-               /* patch vftbl index */
-
-               *((s4 *) (sp + 3 * 8 + 4)) |=
-                       (s4) ((OFFSET(vftbl_t, table[0]) +
-                                  sizeof(methodptr) * m->vftblindex) & 0x0000ffff);
-       }
+       md_icacheflush(ra + 1 * 4, 1 * 4);
 
        return true;
 }
@@ -497,7 +495,7 @@ bool patcher_invokevirtual(u1 *sp)
 
 *******************************************************************************/
 
-bool patcher_invokeinterface(u1 *sp)
+bool patcher_invokeinterface(patchref_t *pr)
 {
        u1                *ra;
        unresolved_method *um;
@@ -505,94 +503,41 @@ bool patcher_invokeinterface(u1 *sp)
 
        /* get stuff from the stack */
 
-       ra = (u1 *)                *((ptrint *) (sp + 7 * 8));
-       um = (unresolved_method *) *((ptrint *) (sp + 2 * 8));
+       ra = (u1 *)                pr->mpc;
+       um = (unresolved_method *) pr->ref;
 
        /* get the fieldinfo */
 
        if (!(m = resolve_method_eager(um)))
                return false;
 
+       PATCH_BACK_ORIGINAL_MCODE;
+
        /* if we show disassembly, we have to skip the nop's */
 
-       if (opt_shownops) {
+       if (opt_shownops)
                ra = ra + PATCHER_CALL_SIZE;
 
-               /* patch interfacetable index */
-
-               *((s4 *) (ra + 1 * 4)) |=
-                       (s4) ((OFFSET(vftbl_t, interfacetable[0]) -
-                                  sizeof(methodptr*) * m->class->index) & 0x0000ffff);
-
-               /* patch method offset */
-
-               *((s4 *) (ra + 2 * 4)) |=
-                       (s4) ((sizeof(methodptr) * (m - m->class->methods)) & 0x0000ffff);
-
-               /* synchronize instruction cache */
+       /* patch interfacetable index */
 
-               md_icacheflush(ra + 1 * 4, 2 * 4);
-       }
-       else {
-               /* patch interfacetable index */
-
-               *((s4 *) (sp + 3 * 8 + 4)) |=
-                       (s4) ((OFFSET(vftbl_t, interfacetable[0]) -
-                                  sizeof(methodptr*) * m->class->index) & 0x0000ffff);
+       *((s4 *) (ra + 1 * 4)) |=
+               (s4) ((OFFSET(vftbl_t, interfacetable[0]) -
+                          sizeof(methodptr*) * m->class->index) & 0x0000ffff);
 
-               /* patch method offset */
+       /* patch method offset */
 
-               *((s4 *) (sp + 4 * 8 + 0)) |=
-                       (s4) ((sizeof(methodptr) * (m - m->class->methods)) & 0x0000ffff);
-       }
+       *((s4 *) (ra + 2 * 4)) |=
+               (s4) ((sizeof(methodptr) * (m - m->class->methods)) & 0x0000ffff);
 
-       return true;
-}
-
-
-/* patcher_checkcast_instanceof_flags ******************************************
-
-   Machine code:
-
-   <patched call position>
-   8fc3ff24    lw       v1,-220(s8)
-   30630200    andi     v1,v1,512
-   1060000d    beq      v1,zero,0x000000001051824c
-   00000000    nop
-
-*******************************************************************************/
-
-bool patcher_checkcast_instanceof_flags(u1 *sp)
-{
-       constant_classref *cr;
-       s4                 disp;
-       u1                *pv;
-       classinfo         *c;
-
-       /* get stuff from the stack */
-
-       cr   = (constant_classref *) *((ptrint *) (sp + 2 * 8));
-       disp =                       *((s4 *)     (sp + 1 * 8));
-       pv   = (u1 *)                *((ptrint *) (sp + 0 * 8));
-
-       /* get the fieldinfo */
-
-       if (!(c = resolve_classref_eager(cr)))
-               return false;
-
-       /* patch class flags */
-
-       *((s4 *) (pv + disp)) = (s4) c->flags;
-
-       /* synchronize data cache */
+       /* synchronize instruction cache */
 
-       md_dcacheflush(pv + disp, sizeof(s4));
+       md_icacheflush(ra + 1 * 4, 2 * 4);
 
        return true;
 }
 
 
-/* patcher_checkcast_instanceof_interface **************************************
+/* patcher_checkcast_interface *************************************************
 
    Machine code:
 
@@ -606,7 +551,7 @@ bool patcher_checkcast_instanceof_flags(u1 *sp)
 
 *******************************************************************************/
 
-bool patcher_checkcast_interface(u1 *sp)
+bool patcher_checkcast_interface(patchref_t *pr)
 {
        u1                *ra;
        constant_classref *cr;
@@ -614,46 +559,33 @@ bool patcher_checkcast_interface(u1 *sp)
 
        /* get stuff from the stack */
 
-       ra = (u1 *)                *((ptrint *) (sp + 7 * 8));
-       cr = (constant_classref *) *((ptrint *) (sp + 2 * 8));
+       ra = (u1 *)                pr->mpc;
+       cr = (constant_classref *) pr->ref;
 
        /* get the fieldinfo */
 
        if (!(c = resolve_classref_eager(cr)))
                return false;
 
+       PATCH_BACK_ORIGINAL_MCODE;
+
        /* if we show disassembly, we have to skip the nop's */
 
-       if (opt_shownops) {
+       if (opt_shownops)
                ra = ra + PATCHER_CALL_SIZE;
 
-               /* patch super class index */
+       /* patch super class index */
 
-               *((s4 *) (ra + 2 * 4)) |= (s4) (-(c->index) & 0x0000ffff);
-               /*      *((s4 *) (ra + 5 * 4)) |= (s4) ((OFFSET(vftbl_t, interfacetable[0]) - */
-               /*                                                                       c->index * sizeof(methodptr*)) & 0x0000ffff); */
-               *((s4 *) (ra + 10 * 4)) |=
-                       (s4) ((OFFSET(vftbl_t, interfacetable[0]) -
-                                  c->index * sizeof(methodptr*)) & 0x0000ffff);
+       *((s4 *) (ra + 2 * 4)) |= (s4) (-(c->index) & 0x0000ffff);
+       /*      *((s4 *) (ra + 5 * 4)) |= (s4) ((OFFSET(vftbl_t, interfacetable[0]) - */
+       /*                                                                       c->index * sizeof(methodptr*)) & 0x0000ffff); */
+       *((s4 *) (ra + 6 * 4)) |=
+               (s4) ((OFFSET(vftbl_t, interfacetable[0]) -
+                          c->index * sizeof(methodptr*)) & 0x0000ffff);
 
-               /* synchronize instruction cache */
-
-               md_icacheflush(ra + 2 * 4, 7 * 4);
-       }
-       else {
-               /* patch super class index */
-
-               *((s4 *) (sp + 2 * 4)) |= (s4) (-(c->index) & 0x0000ffff);
-               /*      *((s4 *) (ra + 5 * 4)) |= (s4) ((OFFSET(vftbl_t, interfacetable[0]) - */
-               /*                                                                       c->index * sizeof(methodptr*)) & 0x0000ffff); */
-               *((s4 *) (ra + 10 * 4)) |=
-                       (s4) ((OFFSET(vftbl_t, interfacetable[0]) -
-                                  c->index * sizeof(methodptr*)) & 0x0000ffff);
+       /* synchronize instruction cache */
 
-               /* synchronize instruction cache */
-
-               md_icacheflush(ra + 10 * 4, 1 * 4);
-       }
+       md_icacheflush(ra + 2 * 4, 5 * 4);
 
        return true;
 }
@@ -673,7 +605,7 @@ bool patcher_checkcast_interface(u1 *sp)
 
 *******************************************************************************/
 
-bool patcher_instanceof_interface(u1 *sp)
+bool patcher_instanceof_interface(patchref_t *pr)
 {
        u1                *ra;
        constant_classref *cr;
@@ -681,175 +613,34 @@ bool patcher_instanceof_interface(u1 *sp)
 
        /* get stuff from the stack */
 
-       ra = (u1 *)                *((ptrint *) (sp + 7 * 8));
-       cr = (constant_classref *) *((ptrint *) (sp + 2 * 8));
+       ra = (u1 *)                pr->mpc;
+       cr = (constant_classref *) pr->ref;
 
        /* get the fieldinfo */
 
        if (!(c = resolve_classref_eager(cr)))
                return false;
 
+       PATCH_BACK_ORIGINAL_MCODE;
+
        /* if we show disassembly, we have to skip the nop's */
 
-       if (opt_shownops) {
+       if (opt_shownops)
                ra = ra + PATCHER_CALL_SIZE;
 
-               /* patch super class index */
-
-               *((s4 *) (ra + 2 * 4)) |= (s4) (-(c->index) & 0x0000ffff);
-               *((s4 *) (ra + 5 * 4)) |=
-                       (s4) ((OFFSET(vftbl_t, interfacetable[0]) -
-                                  c->index * sizeof(methodptr*)) & 0x0000ffff);
-
-               /* synchronize instruction cache */
-
-               md_icacheflush(ra + 2 * 4, 4 * 4);
-       }
-       else {
-               /* patch super class index */
-
-               *((s4 *) (sp + 4 * 8 + 0)) |= (s4) (-(c->index) & 0x0000ffff);
-               *((s4 *) (ra + 5 * 4)) |=
-                       (s4) ((OFFSET(vftbl_t, interfacetable[0]) -
-                                  c->index * sizeof(methodptr*)) & 0x0000ffff);
-
-               /* synchronize instruction cache */
-
-               md_icacheflush(ra + 5 * 4, 1 * 4);
-       }
-
-       return true;
-}
-
-
-/* patcher_checkcast_instanceof_class ******************************************
-
-   Machine code:
-
-   <patched call position>
-   dd030000    ld       v1,0(a4)
-   dfd9ff18    ld       t9,-232(s8)
-
-*******************************************************************************/
-
-bool patcher_checkcast_instanceof_class(u1 *sp)
-{
-       constant_classref *cr;
-       s4                 disp;
-       u1                *pv;
-       classinfo         *c;
-
-       /* get stuff from the stack */
-
-       cr   = (constant_classref *) *((ptrint *) (sp + 2 * 8));
-       disp =                       *((s4 *)     (sp + 1 * 8));
-       pv   = (u1 *)                *((ptrint *) (sp + 0 * 8));
-
-       /* get the fieldinfo */
-
-       if (!(c = resolve_classref_eager(cr)))
-               return false;
-
-       /* patch super class' vftbl */
-
-       *((ptrint *) (pv + disp)) = (ptrint) c->vftbl;
-
-       /* synchronize data cache */
+       /* patch super class index */
 
-       md_dcacheflush(pv + disp, SIZEOF_VOID_P);
+       *((s4 *) (ra + 2 * 4)) |= (s4) (-(c->index) & 0x0000ffff);
+       *((s4 *) (ra + 5 * 4)) |=
+               (s4) ((OFFSET(vftbl_t, interfacetable[0]) -
+                          c->index * sizeof(methodptr*)) & 0x0000ffff);
 
-       return true;
-}
-
-
-/* patcher_clinit **************************************************************
-
-   No special machine code.
-
-*******************************************************************************/
-
-bool patcher_clinit(u1 *sp)
-{
-       classinfo *c;
-
-       /* get stuff from the stack */
-
-       c = (classinfo *) *((ptrint *) (sp + 2 * 8));
-
-       /* check if the class is initialized */
-
-       if (!(c->state & CLASS_INITIALIZED))
-               if (!initialize_class(c))
-                       return false;
-
-       return true;
-}
-
-
-/* patcher_athrow_areturn ******************************************************
-
-   Machine code:
-
-   <patched call position>
-
-*******************************************************************************/
-
-#ifdef ENABLE_VERIFIER
-bool patcher_athrow_areturn(u1 *sp)
-{
-       unresolved_class *uc;
-       classinfo        *c;
-
-       /* get stuff from the stack */
-
-       uc = (unresolved_class *) *((ptrint *) (sp + 2 * 8));
-
-       /* resolve the class */
-
-       if (!resolve_class(uc, resolveEager, false, &c))
-               return false;
-
-       return true;
-}
-#endif /* ENABLE_VERIFIER */
-
-
-/* patcher_resolve_native ******************************************************
-
-   XXX
-
-*******************************************************************************/
-
-#if !defined(WITH_STATIC_CLASSPATH)
-bool patcher_resolve_native(u1 *sp)
-{
-       methodinfo  *m;
-       s4           disp;
-       u1          *pv;
-       functionptr  f;
-
-       /* get stuff from the stack */
-
-       m    = (methodinfo *) *((ptrint *) (sp + 2 * 8));
-       disp =                *((s4 *)     (sp + 1 * 8));
-       pv   = (u1 *)         *((ptrint *) (sp + 0 * 8));
-
-       /* resolve native function */
-
-       if (!(f = native_resolve_function(m)))
-               return false;
-
-       /* patch native function pointer */
-
-       *((ptrint *) (pv + disp)) = (ptrint) f;
-
-       /* synchronize data cache */
+       /* synchronize instruction cache */
 
-       md_dcacheflush(pv + disp, SIZEOF_VOID_P);
+       md_icacheflush(ra + 2 * 4, 4 * 4);
 
        return true;
 }
-#endif /* !defined(WITH_STATIC_CLASSPATH) */
 
 
 /*