# HG changeset patch # User dlong # Date 1536178240 25200 # Node ID 8f594f75e0547d4ca16649cb3501659e3155e81b # Parent b7b69bc93a2de60108633e1241a9c9b6519d88cc 8187078: -XX:+VerifyOops finds numerous problems when running JPRT Reviewed-by: kvn diff -r b7b69bc93a2d -r 8f594f75e054 src/hotspot/share/c1/c1_LIRGenerator.cpp --- a/src/hotspot/share/c1/c1_LIRGenerator.cpp Wed Sep 05 10:39:16 2018 -0700 +++ b/src/hotspot/share/c1/c1_LIRGenerator.cpp Wed Sep 05 13:10:40 2018 -0700 @@ -3218,7 +3218,7 @@ void LIRGenerator::do_ProfileCall(ProfileCall* x) { // Need recv in a temporary register so it interferes with the other temporaries LIR_Opr recv = LIR_OprFact::illegalOpr; - LIR_Opr mdo = new_register(T_OBJECT); + LIR_Opr mdo = new_register(T_METADATA); // tmp is used to hold the counters on SPARC LIR_Opr tmp = new_pointer_register(); diff -r b7b69bc93a2d -r 8f594f75e054 src/hotspot/share/gc/shared/c1/barrierSetC1.cpp --- a/src/hotspot/share/gc/shared/c1/barrierSetC1.cpp Wed Sep 05 10:39:16 2018 -0700 +++ b/src/hotspot/share/gc/shared/c1/barrierSetC1.cpp Wed Sep 05 13:10:40 2018 -0700 @@ -321,7 +321,7 @@ __ cmp(lir_cond_equal, base.result(), LIR_OprFact::oopConst(NULL)); __ branch(lir_cond_equal, T_OBJECT, cont->label()); } - LIR_Opr src_klass = gen->new_register(T_OBJECT); + LIR_Opr src_klass = gen->new_register(T_METADATA); if (gen_type_check) { // We have determined that offset == referent_offset && src != null. // if (src->_klass->_reference_type == REF_NONE) -> continue diff -r b7b69bc93a2d -r 8f594f75e054 src/hotspot/share/runtime/java.cpp --- a/src/hotspot/share/runtime/java.cpp Wed Sep 05 10:39:16 2018 -0700 +++ b/src/hotspot/share/runtime/java.cpp Wed Sep 05 13:10:40 2018 -0700 @@ -333,7 +333,7 @@ klassVtable::print_statistics(); klassItable::print_statistics(); } - if (VerifyOops) { + if (VerifyOops && Verbose) { tty->print_cr("+VerifyOops count: %d", StubRoutines::verify_oop_count()); }