Skip to content
Permalink
Browse files
8253008: Remove develop flags TraceLongCompiles/LongCompileThreshold
Reviewed-by: shade, dholmes, adityam
  • Loading branch information
robehn committed Sep 11, 2020
1 parent c7062dc commit 8777ded1238ad195c6b1b63f54fa51984f8c95e4
Showing with 1 addition and 23 deletions.
  1. +0 −7 src/hotspot/share/runtime/globals.hpp
  2. +1 −16 src/hotspot/share/runtime/vmThread.cpp
@@ -263,9 +263,6 @@ const intx ObjectAlignmentInBytes = 8;
develop(bool, TraceRelocator, false, \
"Trace the bytecode relocator") \
\
develop(bool, TraceLongCompiles, false, \
"Print out every time compilation is longer than " \
"a given threshold") \
\
product(bool, SafepointALot, false, DIAGNOSTIC, \
"Generate a lot of safepoints. This works with " \
@@ -1933,10 +1930,6 @@ const intx ObjectAlignmentInBytes = 8;
develop(intx, MinOopMapAllocation, 8, \
"Minimum number of OopMap entries in an OopMapSet") \
\
/* Background Compilation */ \
develop(intx, LongCompileThreshold, 50, \
"Used with +TraceLongCompiles") \
\
/* recompilation */ \
product_pd(intx, CompileThreshold, \
"number of interpreted method invocations before (re-)compiling") \
@@ -484,22 +484,7 @@ void VMThread::loop() {

} else { // not a safepoint operation
log_debug(vmthread)("Evaluating non-safepoint VM operation: %s", _cur_vm_operation->name());
if (TraceLongCompiles) {
elapsedTimer t;
t.start();
evaluate_operation(_cur_vm_operation);
t.stop();
double secs = t.seconds();
if (secs * 1e3 > LongCompileThreshold) {
// XXX - _cur_vm_operation should not be accessed after
// the completed count has been incremented; the waiting
// thread may have already freed this memory.
tty->print_cr("vm %s: %3.7f secs]", _cur_vm_operation->name(), secs);
}
} else {
evaluate_operation(_cur_vm_operation);
}

evaluate_operation(_cur_vm_operation);
_cur_vm_operation = NULL;
}
}

1 comment on commit 8777ded

@bridgekeeper

This comment has been minimized.

Copy link

@bridgekeeper bridgekeeper bot commented on 8777ded Sep 11, 2020

Please sign in to comment.