# HG changeset patch # User zgu # Date 1590756085 14400 # Fri May 29 08:41:25 2020 -0400 # Node ID 711d1e835158e13dc67fd1a696c99cbad78cfc45 # Parent f67f90f4e8e29ea85a8acce35ade848584f67b71 8246075: Missing logging in nmethod::oops_do_marking_epilogue() on early return path diff --git a/src/hotspot/share/code/nmethod.cpp b/src/hotspot/share/code/nmethod.cpp --- a/src/hotspot/share/code/nmethod.cpp +++ b/src/hotspot/share/code/nmethod.cpp @@ -2004,23 +2004,22 @@ nmethod* next = _oops_do_mark_nmethods; _oops_do_mark_nmethods = NULL; - if (next == NULL) { - return; + if (next != NULL) { + nmethod* cur; + do { + cur = next; + next = extract_nmethod(cur->_oops_do_mark_link); + cur->_oops_do_mark_link = NULL; + DEBUG_ONLY(cur->verify_oop_relocations()); + + LogTarget(Trace, gc, nmethod) lt; + if (lt.is_enabled()) { + LogStream ls(lt); + CompileTask::print(&ls, cur, "oops_do, unmark", /*short_form:*/ true); + } + // End if self-loop has been detected. + } while (cur != next); } - nmethod* cur; - do { - cur = next; - next = extract_nmethod(cur->_oops_do_mark_link); - cur->_oops_do_mark_link = NULL; - DEBUG_ONLY(cur->verify_oop_relocations()); - - LogTarget(Trace, gc, nmethod) lt; - if (lt.is_enabled()) { - LogStream ls(lt); - CompileTask::print(&ls, cur, "oops_do, unmark", /*short_form:*/ true); - } - // End if self-loop has been detected. - } while (cur != next); log_trace(gc, nmethod)("oops_do_marking_epilogue"); }