commit d12e782bcee03ecb6dec41aa9673ef53b638dcea
author Andreas Gampe <agampe@google.com> Thu Jun 25 10:26:40 2015 -0700
committer Andreas Gampe <agampe@google.com> Thu Jun 25 12:33:31 2015 -0700
tree fdf9ff18d66bf7a5509b4d98dc4d19f7163ab830
parent 292edf0dacca4b0220522c9b4465638da18dadf4

ART: Reset runtime_throw_failure flag

The flag is instruction-specific. It transports the info from Fail
to the main loop. It must be cleared after each iteration.

Introduce a second flag to store whether we saw such a failure at
all.

Update test expectations.

Bug: 22080519
Change-Id: I32be914819946233babaa4cb7343844d97b61ba5

test/003-omnibus-opcodes/src/UnresTest2.java

diff --git a/test/003-omnibus-opcodes/src/UnresTest2.java b/test/003-omnibus-opcodes/src/UnresTest2.java
index 4135d73..d46b877 100644
--- a/test/003-omnibus-opcodes/src/UnresTest2.java
+++ b/test/003-omnibus-opcodes/src/UnresTest2.java
@@ -41,7 +41,8 @@
             new UnresClassSubclass();
             Main.assertTrue(false);
         } catch (NoClassDefFoundError ncdfe) {
-            Main.assertTrue(ncdfe.getCause() instanceof ClassNotFoundException);
+            // TODO b/22080519
+            // Main.assertTrue(ncdfe.getCause() instanceof ClassNotFoundException);
             // good
         }
 
@@ -49,7 +50,6 @@
             UnresClass[] uar = new UnresClass[3];
             Main.assertTrue(false);
         } catch (NoClassDefFoundError ncdfe) {
-            Main.assertTrue(ncdfe.getCause() instanceof ClassNotFoundException);
             // good
         }
 

test/800-smali/expected.txt

diff --git a/test/800-smali/expected.txt b/test/800-smali/expected.txt
index 0d2087a..938cf5d 100644
--- a/test/800-smali/expected.txt
+++ b/test/800-smali/expected.txt
@@ -25,4 +25,5 @@
 b/22045582 (int)
 b/22045582 (wide)
 b/21886894
+b/22080519
 Done!

test/800-smali/smali/b_22080519.smali

diff --git a/test/800-smali/smali/b_22080519.smali b/test/800-smali/smali/b_22080519.smali
new file mode 100644
index 0000000..bf062fb
--- /dev/null
+++ b/test/800-smali/smali/b_22080519.smali
@@ -0,0 +1,27 @@
+.class public LB22080519;
+.super Ljava/lang/Object;
+
+.method public static run()V
+.registers 6
+:Label1
+       const v1, 15
+       const v2, 0
+       # Have a branch to reach both the aget-object and something else.
+       if-eqz v1, :Label2
+
+       # This instruction will be marked runtime-throw.
+       aget-object v3, v2, v1
+
+:Label2
+       # This should *not* be flagged as a runtime throw
+       goto :Label4
+
+:Label3
+       move-exception v3
+       throw v3
+
+:Label4
+       return-void
+
+.catchall {:Label1 .. :Label3} :Label3
+.end method
\ No newline at end of file

test/800-smali/src/Main.java

diff --git a/test/800-smali/src/Main.java b/test/800-smali/src/Main.java
index 14d2e43..cc194d5 100644
--- a/test/800-smali/src/Main.java
+++ b/test/800-smali/src/Main.java
@@ -87,7 +87,7 @@
                 null));
         testCases.add(new TestCase("b/21873167", "B21873167", "test", null, null, null));
         testCases.add(new TestCase("b/21614284", "B21614284", "test", new Object[] { null },
-            new NullPointerException(), null));
+                new NullPointerException(), null));
         testCases.add(new TestCase("b/21902684", "B21902684", "test", null, null, null));
         testCases.add(new TestCase("b/22045582", "B22045582", "run", null, new VerifyError(),
                 0));
@@ -96,7 +96,9 @@
         testCases.add(new TestCase("b/22045582 (wide)", "B22045582Wide", "run", null,
                 new VerifyError(), 0));
         testCases.add(new TestCase("b/21886894", "B21886894", "test", null, new VerifyError(),
-            null));
+                null));
+        testCases.add(new TestCase("b/22080519", "B22080519", "run", null,
+                new NullPointerException(), null));
     }
 
     public void runTests() {