8205499: C1 temporary code buffers are not removed with -XX:+UseDynamicNumberOfCompilerThreads
Deallocate C1 code buffers in the compiler thread destructor.
Reviewed-by: neliasso, kvn, mdoerr
diff --git a/src/hotspot/share/compiler/compileBroker.cpp b/src/hotspot/share/compiler/compileBroker.cpp
index 91c19dd..c41df2b 100644
--- a/src/hotspot/share/compiler/compileBroker.cpp
+++ b/src/hotspot/share/compiler/compileBroker.cpp
@@ -1638,12 +1638,6 @@
* out to be a problem.
*/
void CompileBroker::shutdown_compiler_runtime(AbstractCompiler* comp, CompilerThread* thread) {
- // Free buffer blob, if allocated
- if (thread->get_buffer_blob() != NULL) {
- MutexLockerEx mu(CodeCache_lock, Mutex::_no_safepoint_check_flag);
- CodeCache::free(thread->get_buffer_blob());
- }
-
if (comp->should_perform_shutdown()) {
// There are two reasons for shutting down the compiler
// 1) compiler runtime initialization failed
diff --git a/src/hotspot/share/runtime/thread.cpp b/src/hotspot/share/runtime/thread.cpp
index b080881..ed391ce 100644
--- a/src/hotspot/share/runtime/thread.cpp
+++ b/src/hotspot/share/runtime/thread.cpp
@@ -3298,6 +3298,11 @@
}
CompilerThread::~CompilerThread() {
+ // Free buffer blob, if allocated
+ if (get_buffer_blob() != NULL) {
+ MutexLockerEx mu(CodeCache_lock, Mutex::_no_safepoint_check_flag);
+ CodeCache::free(get_buffer_blob());
+ }
// Delete objects which were allocated on heap.
delete _counters;
}
