commit fc519dc8f4444f6d93806ec15ce7445b322070fd
author Bill Buzbee <buzbee@google.com> Sat Mar 06 23:30:57 2010 -0800
committer Bill Buzbee <buzbee@google.com> Sun Mar 07 08:24:00 2010 -0800
tree 7128bbc741b5125cbe85300e1cd264ab3aca6bde
parent 164d1279b67ec13061e473cb453b1ff24189e0e0

Jit: Make most Jit compile failures non-fatal; just abort offending translation

Issue 2175597 Jit compile failures should abort translation, but not the VM

Added new dvmCompileAbort() to replace uses of dvmAbort() when something goes
wrong during the compliation of a trace.  In that case, we'll abort the translation
and set it's head to the interpret-only "translation".

vm/compiler/codegen/arm/RallocUtil.c

diff --git a/vm/compiler/codegen/arm/RallocUtil.c b/vm/compiler/codegen/arm/RallocUtil.c
index 131df2c..6d83e73 100644
--- a/vm/compiler/codegen/arm/RallocUtil.c
+++ b/vm/compiler/codegen/arm/RallocUtil.c
@@ -123,7 +123,7 @@
         }
     }
     LOGE("Tried to get info on a non-existant temp: r%d",reg);
-    dvmAbort();  // FIXME: abort translation intead of vm
+    dvmCompilerAbort(cUnit);
     return NULL;
 }
 
@@ -249,7 +249,7 @@
     }
     if (required) {
         LOGE("No free temp registers");
-        dvmAbort();  // FIXME: abort translation instead of vm
+        dvmCompilerAbort(cUnit);
     }
     return -1;  // No register available
 }
@@ -298,7 +298,7 @@
         next += 2;
     }
     LOGE("No free temp registers");
-    dvmAbort(); // FIXME: abort translation instead of vm
+    dvmCompilerAbort(cUnit);
     return -1;
 }
 
@@ -359,7 +359,7 @@
             break;
         default:
             LOGE("Invalid register type");
-            dvmAbort();    //FIXME: abort translation instead of vm
+            dvmCompilerAbort(cUnit);
     }
     return res;
 }
@@ -386,7 +386,7 @@
         }
     }
     LOGE("Tried to free a non-existant temp: r%d",reg);
-    dvmAbort();  // FIXME: abort translation instead of vm
+    dvmCompilerAbort(cUnit);
 }
 
 /*
@@ -460,7 +460,7 @@
         }
     }
     LOGE("Tried to lock a non-existant temp: r%d",reg);
-    dvmAbort();  // FIXME: abort translation instead of vm
+    dvmCompilerAbort(cUnit);
 }
 
 static void lockArgRegs(CompilationUnit *cUnit)