RFR (sh/8): Remove to wrong handlings of Shenandoah LRB in escape analysis

Roman Kennke rkennke at redhat.com
Tue Nov 26 16:30:58 UTC 2019


Before I forget (because it is not tracked), here is the backport of a
recent fix that I did in 11. It has baked there for 2 weeks now.

ok to push?

Thanks,
Roman

# HG changeset patch
# User rkennke
# Date 1574785684 -3600
#      Tue Nov 26 17:28:04 2019 +0100
# Node ID b9601b57b3c413454c93cbee75739bc75fee6f20
# Parent  e90e85d7d6bfcfaaeb89d38e8d8b9fae2eed4c73
[backport] Remove to wrong handlings of Shenandoah LRB in escape analysis

diff -r e90e85d7d6bf -r b9601b57b3c4 src/share/vm/opto/escape.cpp
--- a/src/share/vm/opto/escape.cpp      Thu Nov 14 22:35:14 2019 +0100
+++ b/src/share/vm/opto/escape.cpp      Tue Nov 26 17:28:04 2019 +0100
@@ -2991,7 +2991,6 @@
                n->is_CheckCastPP() ||
                n->is_EncodeP() ||
                n->is_DecodeN() ||
-               n->Opcode() == Op_ShenandoahLoadReferenceBarrier ||
                (n->is_ConstraintCast() && n->Opcode() == Op_CastPP)) {
       if (visited.test_set(n->_idx)) {
         assert(n->is_Phi(), "loops only through Phi's");
@@ -3062,7 +3061,6 @@
                  use->is_CheckCastPP() ||
                  use->is_EncodeNarrowPtr() ||
                  use->is_DecodeNarrowPtr() ||
-                 use->Opcode() == Op_ShenandoahLoadReferenceBarrier ||
                  (use->is_ConstraintCast() && use->Opcode() ==
Op_CastPP)) {
         alloc_worklist.append_if_missing(use);
 #ifdef ASSERT



More information about the shenandoah-dev mailing list