changeset 24120:23201e47b032

Clean up some useless product printing from PrintDeoptimizationDetails
author Tom Rodriguez <tom.rodriguez@oracle.com>
date Tue, 04 Apr 2017 12:31:03 -0700
parents c7f35230907d
children 813c28963a9e
files src/share/vm/runtime/deoptimization.cpp src/share/vm/runtime/vframeArray.cpp
diffstat 2 files changed, 6 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/share/vm/runtime/deoptimization.cpp	Tue Apr 04 09:36:50 2017 -0700
+++ b/src/share/vm/runtime/deoptimization.cpp	Tue Apr 04 12:31:03 2017 -0700
@@ -314,7 +314,7 @@
   ScopeDesc* trap_scope = chunk->at(0)->scope();
   Handle exceptionObject;
   if (trap_scope->rethrow_exception()) {
-    if (PrintDeoptimizationDetails) {
+    if (PrintDeoptimizationDetails || TraceDeoptimization) {
       tty->print_cr("Exception to be rethrown in the interpreter for method %s::%s at bci %d", trap_scope->method()->method_holder()->name()->as_C_string(), trap_scope->method()->name()->as_C_string(), trap_scope->bci());
     }
     GrowableArray<ScopeValue*>* expressions = trap_scope->expressions();
@@ -1081,9 +1081,11 @@
     if (obj.is_null()) {
       continue;
     }
+#ifndef PRODUCT
     if (PrintDeoptimizationDetails) {
       tty->print_cr("reassign fields for object of type %s!", k->name()->as_C_string());
     }
+#endif
 
     if (k->oop_is_instance()) {
       InstanceKlass* ik = InstanceKlass::cast(k());
@@ -1330,7 +1332,7 @@
     assert(nm != NULL, "only nmethods can deopt");
 
     ttyLocker ttyl;
-    xtty->begin_head("deoptimized thread='" UINTX_FORMAT "' pc='" UINTX_FORMAT "'", thread->osthread()->thread_id(), fr.pc());
+    xtty->begin_head("deoptimized thread='" UINTX_FORMAT "' pc='" INTPTR_FORMAT "'", thread->osthread()->thread_id(), fr.pc());
     nm->log_identity(xtty);
     xtty->end_head();
     for (ScopeDesc* sd = nm->scope_desc_at(fr.pc()); ; sd = sd->sender()) {
--- a/src/share/vm/runtime/vframeArray.cpp	Tue Apr 04 09:36:50 2017 -0700
+++ b/src/share/vm/runtime/vframeArray.cpp	Tue Apr 04 12:31:03 2017 -0700
@@ -316,9 +316,11 @@
     }
   }
 
+#ifndef PRODUCT
   if (PrintDeoptimizationDetails) {
     tty->print_cr("Expressions size: %d", expressions()->size());
   }
+#endif
 
   // Unpack expression stack
   // If this is an intermediate frame (i.e. not top frame) then this