8239852: java/util/concurrent tests fail with -XX:+VerifyGraphEdges: assert(!VerifyGraphEdges) failed: verification should have failed

Fri, 28 Feb 2020 15:33:44 +0100

author
chagedorn
date
Fri, 28 Feb 2020 15:33:44 +0100
changeset 9914
d99632e69372
parent 9913
bd77de4e5e2b
child 9915
26ffadc256e1

8239852: java/util/concurrent tests fail with -XX:+VerifyGraphEdges: assert(!VerifyGraphEdges) failed: verification should have failed
Summary: Remove an assertion which was too strong for some valid IRs when running with -XX:+VerifyGraphEdges
Reviewed-by: neliasso, thartmann

src/share/vm/opto/gcm.cpp file | annotate | diff | comparison | revisions
     1.1 --- a/src/share/vm/opto/gcm.cpp	Mon Dec 02 16:38:34 2019 -0800
     1.2 +++ b/src/share/vm/opto/gcm.cpp	Fri Feb 28 15:33:44 2020 +0100
     1.3 @@ -1307,8 +1307,6 @@
     1.4    // ( visited.Clear() called in schedule_late()->Node_Backward_Iterator() )
     1.5    schedule_late(visited, stack);
     1.6    if (C->failing()) {
     1.7 -    // schedule_late fails only when graph is incorrect.
     1.8 -    assert(!VerifyGraphEdges, "verification should have failed");
     1.9      return;
    1.10    }
    1.11  

mercurial