8296912: C2: CreateExNode::Identity fails with assert(i < _max) faile…

…d: oob: i=1, _max=1 Reviewed-by: rrich Backport-of: 502fa3eeea849cfcc50436602be1654695ef4e26
openjdk · Feb 8, 2023 · 9d7bb60 · 9d7bb60 · openjdk-notifier · Feb 8, 2023
1 parent 65180f1
commit 9d7bb60
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 5 deletions.
diff --git a/src/hotspot/share/opto/cfgnode.cpp b/src/hotspot/share/opto/cfgnode.cpp
@@ -2439,9 +2439,8 @@ Node* CreateExNode::Identity(PhaseGVN* phase) {
   // exception oop through.
   CallNode *call = in(1)->in(0)->as_Call();
 
-  return ( in(0)->is_CatchProj() && in(0)->in(0)->in(1) == in(1) )
-    ? this
-    : call->in(TypeFunc::Parms);
+  return (in(0)->is_CatchProj() && in(0)->in(0)->is_Catch() &&
+          in(0)->in(0)->in(1) == in(1)) ? this : call->in(TypeFunc::Parms);
 }
 
 //=============================================================================

diff --git a/test/hotspot/jtreg/compiler/c2/TestDeadDataLoop.java b/test/hotspot/jtreg/compiler/c2/TestDeadDataLoop.java
@@ -23,11 +23,11 @@
 
 /*
  * @test
- * @bug 8284358
+ * @bug 8284358 8296912
  * @summary An unreachable loop is not removed, leading to a broken graph.
  * @requires vm.compiler2.enabled
  * @run main/othervm -Xcomp -XX:-TieredCompilation -XX:+UnlockDiagnosticVMOptions
- *                   -XX:CompileCommand=compileonly,*TestDeadDataLoop::test* -XX:CompileCommand=dontinline,*TestDeadDataLoop::notInlined
+ *                   -XX:CompileCommand=compileonly,*TestDeadDataLoop::test* -XX:CompileCommand=dontinline,*TestDeadDataLoop::notInlined*
  *                   compiler.c2.TestDeadDataLoop
  */
 
@@ -210,10 +210,29 @@ static void test10() {
         }
     }
 
+    static long l;
+
+    static void test11(boolean never) {
+        float f = 1;
+        boolean b;
+        for (int i = 0; i < 5; ++i) {
+            b = (never || l < 0);
+            l = notInlined2();
+            if (!never) {
+                f += i;
+            }
+        }
+        l += f;
+    }
+
     public static boolean notInlined() {
         return false;
     }
 
+    public static int notInlined2() {
+        return 42;
+    }
+
     public static void main(String[] args) {
         // Make sure classes are initialized
         Integer i = 42;
@@ -228,6 +247,7 @@ public static void main(String[] args) {
         test8();
         test9();
         test10();
+        test11(false);
     }
 }