commit c2b486fe6e49555c3a5df0702d04c20b6d6a33c4
author Dan Bornstein <danfuzz@android.com> Fri Nov 12 16:07:16 2010 -0800
committer Dan Bornstein <danfuzz@android.com> Fri Nov 12 16:07:16 2010 -0800
tree 8c3da0dac593a4ed8ee907ca6933f235a73a4037
parent 71318a1c1337fe5cd7a189d13df8cda913336af7

Get rid of kInstrUnconditional.

Replace its use with an inline function dexIsGoto(flags), which uses
the existing flag bits kInstrCanBranch and kInstrCanContinue.

Change-Id: I84f533f619434ad0ad2831c147b50f6dc79b9928

vm/compiler/Frontend.c

diff --git a/vm/compiler/Frontend.c b/vm/compiler/Frontend.c
index 7d28b36..3c1e837 100644
--- a/vm/compiler/Frontend.c
+++ b/vm/compiler/Frontend.c
@@ -820,7 +820,7 @@
             /* For unconditional branches, request a hot chaining cell */
             } else {
 #if !defined(WITH_SELF_VERIFICATION)
-                newBB = dvmCompilerNewBB(flags & kInstrUnconditional ?
+                newBB = dvmCompilerNewBB(dexIsGoto(flags) ?
                                                   kChainingCellHot :
                                                   kChainingCellNormal);
                 newBB->startOffset = targetOffset;
@@ -830,7 +830,7 @@
                     targetOffset <= curBB->lastMIRInsn->offset) {
                     newBB = dvmCompilerNewBB(kChainingCellBackwardBranch);
                 } else {
-                    newBB = dvmCompilerNewBB(flags & kInstrUnconditional ?
+                    newBB = dvmCompilerNewBB(dexIsGoto(flags) ?
                                                       kChainingCellHot :
                                                       kChainingCellNormal);
                 }