truffle: src/share/vm/code/nmethod.cpp comparison

comparison src/share/vm/code/nmethod.cpp @ 20804:7848fc12602b

Merge with jdk8u40-b25

author	Gilles Duboscq <gilles.m.duboscq@oracle.com>
date	Tue, 07 Apr 2015 14:58:49 +0200
parents	f8147c931ce4 b12a2a9b05ca
children	eb21f2944d7d

comparison

equal deleted inserted replaced

-:84105dcdb05b
+:7848fc12602b
 #include "compiler/disassembler.hpp"
 #include "interpreter/bytecode.hpp"
 #include "oops/methodData.hpp"
 #include "prims/jvmtiRedefineClassesTrace.hpp"
 #include "prims/jvmtiImpl.hpp"
+#include "runtime/orderAccess.inline.hpp"
 #include "runtime/sharedRuntime.hpp"
 #include "runtime/sweeper.hpp"
 #include "utilities/dtrace.hpp"
 #include "utilities/events.hpp"
 #include "utilities/xmlstream.hpp"
 #ifdef GRAAL
 #include "graal/graalJavaAccess.hpp"
 #endif
 PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC
+unsigned char nmethod::_global_unloading_clock = 0;
 #ifdef DTRACE_ENABLED
 // Only bother with this argument setup if dtrace is available
 new_entry->set_next(exception_cache());
 }
 set_exception_cache(new_entry);
 }
-void nmethod::remove_from_exception_cache(ExceptionCache* ec) {
+void nmethod::clean_exception_cache(BoolObjectClosure* is_alive) {
 ExceptionCache* prev = NULL;
 ExceptionCache* curr = exception_cache();
-assert(curr != NULL, "nothing to remove");
-// find the previous and next entry of ec
+while (curr != NULL) {
-while (curr != ec) {
+ExceptionCache* next = curr->next();
-prev = curr;
-curr = curr->next();
+Klass* ex_klass = curr->exception_type();
-assert(curr != NULL, "ExceptionCache not found");
+if (ex_klass != NULL && !ex_klass->is_loader_alive(is_alive)) {
-}
+if (prev == NULL) {
-// now: curr == ec
+set_exception_cache(next);
-ExceptionCache* next = curr->next();
+} else {
-if (prev == NULL) {
+prev->set_next(next);
-set_exception_cache(next);
+}
-} else {
+delete curr;
-prev->set_next(next);
+// prev stays the same.
-}
+} else {
-delete curr;
+prev = curr;
 }
+curr = next;
+}
+}
 // public method for accessing the exception cache
 // These are the public access methods.
 address nmethod::handler_for_exception_and_pc(Handle exception, address pc) {
 // We never grab a lock to read the exception cache, so we may
 }
 // Fill in default values for various flag fields
 void nmethod::init_defaults() {
 _state                      = in_use;
+_unloading_clock            = 0;
 _marked_for_reclamation     = 0;
 _has_flushed_dependencies   = 0;
 _has_unsafe_access          = 0;
 _has_method_handle_invokes  = 0;
 _lazy_critical_native       = 0;
 #endif
 _oops_do_mark_link       = NULL;
 _jmethod_id              = NULL;
 _osr_link                = NULL;
-_scavenge_root_link      = NULL;
+if (UseG1GC) {
+_unloading_next        = NULL;
+} else {
+_scavenge_root_link    = NULL;
+}
 _scavenge_root_state     = 0;
 _compiler                = NULL;
 #if INCLUDE_RTM_OPT
 _rtm_state               = NoRTM;
 #endif
 _exception_cache         = NULL;
 _pc_desc_cache.reset_to(NULL);
 _hotness_counter         = NMethodSweeper::hotness_counter_reset_val();
 code_buffer->copy_values_to(this);
-if (ScavengeRootsInCode && detect_scavenge_root_oops()) {
+if (ScavengeRootsInCode) {
-CodeCache::add_scavenge_root_nmethod(this);
+if (detect_scavenge_root_oops()) {
+CodeCache::add_scavenge_root_nmethod(this);
+}
 Universe::heap()->register_nmethod(this);
 }
 debug_only(verify_scavenge_root_oops());
 CodeCache::commit(this);
 }
 _exception_cache         = NULL;
 _pc_desc_cache.reset_to(NULL);
 _hotness_counter         = NMethodSweeper::hotness_counter_reset_val();
 code_buffer->copy_values_to(this);
-if (ScavengeRootsInCode && detect_scavenge_root_oops()) {
+if (ScavengeRootsInCode) {
-CodeCache::add_scavenge_root_nmethod(this);
+if (detect_scavenge_root_oops()) {
+CodeCache::add_scavenge_root_nmethod(this);
+}
 Universe::heap()->register_nmethod(this);
 }
 DEBUG_ONLY(verify_scavenge_root_oops();)
 CodeCache::commit(this);
 }
 // Copy contents of ScopeDescRecorder to nmethod
 code_buffer->copy_values_to(this);
 debug_info->copy_to(this);
 dependencies->copy_to(this);
-if (ScavengeRootsInCode && detect_scavenge_root_oops()) {
+if (ScavengeRootsInCode) {
-CodeCache::add_scavenge_root_nmethod(this);
+if (detect_scavenge_root_oops()) {
+CodeCache::add_scavenge_root_nmethod(this);
+}
 Universe::heap()->register_nmethod(this);
 }
 debug_only(verify_scavenge_root_oops());
 CodeCache::commit(this);
 RelocIterator iter(this, low_boundary);
 while(iter.next()) {
 switch(iter.type()) {
 case relocInfo::virtual_call_type:
 case relocInfo::opt_virtual_call_type: {
-CompiledIC *ic = CompiledIC_at(iter.reloc());
+CompiledIC *ic = CompiledIC_at(&iter);
 // Ok, to lookup references to zombies here
 CodeBlob *cb = CodeCache::find_blob_unsafe(ic->ic_destination());
 if( cb != NULL && cb->is_nmethod() ) {
 nmethod* nm = (nmethod*)cb;
 // Clean inline caches pointing to both zombie and not_entrant methods
 }
 }
 }
 }
+void nmethod::verify_clean_inline_caches() {
+assert_locked_or_safepoint(CompiledIC_lock);
+// If the method is not entrant or zombie then a JMP is plastered over the
+// first few bytes.  If an oop in the old code was there, that oop
+// should not get GC'd.  Skip the first few bytes of oops on
+// not-entrant methods.
+address low_boundary = verified_entry_point();
+if (!is_in_use()) {
+low_boundary += NativeJump::instruction_size;
+// %%% Note:  On SPARC we patch only a 4-byte trap, not a full NativeJump.
+// This means that the low_boundary is going to be a little too high.
+// This shouldn't matter, since oops of non-entrant methods are never used.
+// In fact, why are we bothering to look at oops in a non-entrant method??
+}
+ResourceMark rm;
+RelocIterator iter(this, low_boundary);
+while(iter.next()) {
+switch(iter.type()) {
+case relocInfo::virtual_call_type:
+case relocInfo::opt_virtual_call_type: {
+CompiledIC *ic = CompiledIC_at(&iter);
+// Ok, to lookup references to zombies here
+CodeBlob *cb = CodeCache::find_blob_unsafe(ic->ic_destination());
+if( cb != NULL && cb->is_nmethod() ) {
+nmethod* nm = (nmethod*)cb;
+// Verify that inline caches pointing to both zombie and not_entrant methods are clean
+if (!nm->is_in_use() || (nm->method()->code() != nm)) {
+assert(ic->is_clean(), "IC should be clean");
+}
+}
+break;
+}
+case relocInfo::static_call_type: {
+CompiledStaticCall *csc = compiledStaticCall_at(iter.reloc());
+CodeBlob *cb = CodeCache::find_blob_unsafe(csc->destination());
+if( cb != NULL && cb->is_nmethod() ) {
+nmethod* nm = (nmethod*)cb;
+// Verify that inline caches pointing to both zombie and not_entrant methods are clean
+if (!nm->is_in_use() || (nm->method()->code() != nm)) {
+assert(csc->is_clean(), "IC should be clean");
+}
+}
+break;
+}
+}
+}
+}
+int nmethod::verify_icholder_relocations() {
+int count = 0;
+RelocIterator iter(this);
+while(iter.next()) {
+if (iter.type() == relocInfo::virtual_call_type) {
+if (CompiledIC::is_icholder_call_site(iter.virtual_call_reloc())) {
+CompiledIC *ic = CompiledIC_at(&iter);
+if (TraceCompiledIC) {
+tty->print("noticed icholder " INTPTR_FORMAT " ", p2i(ic->cached_icholder()));
+ic->print();
+}
+assert(ic->cached_icholder() != NULL, "must be non-NULL");
+count++;
+}
+}
+}
+return count;
+}
 // This is a private interface with the sweeper.
 void nmethod::mark_as_seen_on_stack() {
 assert(is_alive(), "Must be an alive method");
 // Set the traversal mark to ensure that the sweeper does 2
 // cleaning passes before moving to zombie.
 if (mdo == NULL)  return;
 // There is a benign race here.  See comments in methodData.hpp.
 mdo->inc_decompile_count();
 }
+void nmethod::increase_unloading_clock() {
+_global_unloading_clock++;
+if (_global_unloading_clock == 0) {
+// _nmethods are allocated with _unloading_clock == 0,
+// so 0 is never used as a clock value.
+_global_unloading_clock = 1;
+}
+}
+void nmethod::set_unloading_clock(unsigned char unloading_clock) {
+OrderAccess::release_store((volatile jubyte*)&_unloading_clock, unloading_clock);
+}
+unsigned char nmethod::unloading_clock() {
+return (unsigned char)OrderAccess::load_acquire((volatile jubyte*)&_unloading_clock);
+}
 void nmethod::make_unloaded(BoolObjectClosure* is_alive, oop cause) {
 post_compiled_method_unload();
 // Since this nmethod is being unloaded, make sure that dependencies
 // Transitioning directly from live to unloaded -- so
 // we need to force a cache clean-up; remember this
 // for later on.
 CodeCache::set_needs_cache_clean(true);
 }
+// Unregister must be done before the state change
+Universe::heap()->unregister_nmethod(this);
 _state = unloaded;
 // Log the unloading.
 log_state_change();
 // attempt to report the event in the unlikely scenario where the
 // event is enabled at the time the nmethod is made a zombie.
 set_unload_reported();
 }
+void static clean_ic_if_metadata_is_dead(CompiledIC *ic, BoolObjectClosure *is_alive, bool mark_on_stack) {
+if (ic->is_icholder_call()) {
+// The only exception is compiledICHolder oops which may
+// yet be marked below. (We check this further below).
+CompiledICHolder* cichk_oop = ic->cached_icholder();
+if (mark_on_stack) {
+Metadata::mark_on_stack(cichk_oop->holder_method());
+Metadata::mark_on_stack(cichk_oop->holder_klass());
+}
+if (cichk_oop->holder_method()->method_holder()->is_loader_alive(is_alive) &&
+cichk_oop->holder_klass()->is_loader_alive(is_alive)) {
+return;
+}
+} else {
+Metadata* ic_oop = ic->cached_metadata();
+if (ic_oop != NULL) {
+if (mark_on_stack) {
+Metadata::mark_on_stack(ic_oop);
+}
+if (ic_oop->is_klass()) {
+if (((Klass*)ic_oop)->is_loader_alive(is_alive)) {
+return;
+}
+} else if (ic_oop->is_method()) {
+if (((Method*)ic_oop)->method_holder()->is_loader_alive(is_alive)) {
+return;
+}
+} else {
+ShouldNotReachHere();
+}
+}
+}
+ic->set_to_clean();
+}
 // This is called at the end of the strong tracing/marking phase of a
 // GC to unload an nmethod if it contains otherwise unreachable
 // oops.
 void nmethod::do_unloading(BoolObjectClosure* is_alive, bool unloading_occurred) {
 }
 }
 #endif
 // Exception cache
-ExceptionCache* ec = exception_cache();
+clean_exception_cache(is_alive);
-while (ec != NULL) {
-Klass* ex_klass = ec->exception_type();
-ExceptionCache* next_ec = ec->next();
-if (ex_klass != NULL && !ex_klass->is_loader_alive(is_alive)) {
-remove_from_exception_cache(ec);
-}
-ec = next_ec;
-}
 // If class unloading occurred we first iterate over all inline caches and
 // clear ICs where the cached oop is referring to an unloaded klass or method.
 // The remaining live cached oops will be traversed in the relocInfo::oop_type
 // iteration below.
 if (unloading_occurred) {
 RelocIterator iter(this, low_boundary);
 while(iter.next()) {
 if (iter.type() == relocInfo::virtual_call_type) {
-CompiledIC *ic = CompiledIC_at(iter.reloc());
+CompiledIC *ic = CompiledIC_at(&iter);
-if (ic->is_icholder_call()) {
+clean_ic_if_metadata_is_dead(ic, is_alive, false);
-// The only exception is compiledICHolder oops which may
-// yet be marked below. (We check this further below).
-CompiledICHolder* cichk_oop = ic->cached_icholder();
-if (cichk_oop->holder_method()->method_holder()->is_loader_alive(is_alive) &&
-cichk_oop->holder_klass()->is_loader_alive(is_alive)) {
-continue;
-}
-} else {
-Metadata* ic_oop = ic->cached_metadata();
-if (ic_oop != NULL) {
-if (ic_oop->is_klass()) {
-if (((Klass*)ic_oop)->is_loader_alive(is_alive)) {
-continue;
-}
-} else if (ic_oop->is_method()) {
-if (((Method*)ic_oop)->method_holder()->is_loader_alive(is_alive)) {
-continue;
-}
-} else {
-ShouldNotReachHere();
-}
-}
-}
-ic->set_to_clean();
 }
 }
 }
 // Compiled code
 // Ensure that all metadata is still alive
 verify_metadata_loaders(low_boundary, is_alive);
 }
+template <class CompiledICorStaticCall>
+static bool clean_if_nmethod_is_unloaded(CompiledICorStaticCall *ic, address addr, BoolObjectClosure *is_alive, nmethod* from) {
+// Ok, to lookup references to zombies here
+CodeBlob *cb = CodeCache::find_blob_unsafe(addr);
+if (cb != NULL && cb->is_nmethod()) {
+nmethod* nm = (nmethod*)cb;
+if (nm->unloading_clock() != nmethod::global_unloading_clock()) {
+// The nmethod has not been processed yet.
+return true;
+}
+// Clean inline caches pointing to both zombie and not_entrant methods
+if (!nm->is_in_use() || (nm->method()->code() != nm)) {
+ic->set_to_clean();
+assert(ic->is_clean(), err_msg("nmethod " PTR_FORMAT "not clean %s", from, from->method()->name_and_sig_as_C_string()));
+}
+}
+return false;
+}
+static bool clean_if_nmethod_is_unloaded(CompiledIC *ic, BoolObjectClosure *is_alive, nmethod* from) {
+return clean_if_nmethod_is_unloaded(ic, ic->ic_destination(), is_alive, from);
+}
+static bool clean_if_nmethod_is_unloaded(CompiledStaticCall *csc, BoolObjectClosure *is_alive, nmethod* from) {
+return clean_if_nmethod_is_unloaded(csc, csc->destination(), is_alive, from);
+}
+bool nmethod::unload_if_dead_at(RelocIterator* iter_at_oop, BoolObjectClosure *is_alive, bool unloading_occurred) {
+assert(iter_at_oop->type() == relocInfo::oop_type, "Wrong relocation type");
+oop_Relocation* r = iter_at_oop->oop_reloc();
+// Traverse those oops directly embedded in the code.
+// Other oops (oop_index>0) are seen as part of scopes_oops.
+assert(1 == (r->oop_is_immediate()) +
+(r->oop_addr() >= oops_begin() && r->oop_addr() < oops_end()),
+"oop must be found in exactly one place");
+if (r->oop_is_immediate() && r->oop_value() != NULL) {
+// Unload this nmethod if the oop is dead.
+if (can_unload(is_alive, r->oop_addr(), unloading_occurred)) {
+return true;;
+}
+}
+return false;
+}
+void nmethod::mark_metadata_on_stack_at(RelocIterator* iter_at_metadata) {
+assert(iter_at_metadata->type() == relocInfo::metadata_type, "Wrong relocation type");
+metadata_Relocation* r = iter_at_metadata->metadata_reloc();
+// In this metadata, we must only follow those metadatas directly embedded in
+// the code.  Other metadatas (oop_index>0) are seen as part of
+// the metadata section below.
+assert(1 == (r->metadata_is_immediate()) +
+(r->metadata_addr() >= metadata_begin() && r->metadata_addr() < metadata_end()),
+"metadata must be found in exactly one place");
+if (r->metadata_is_immediate() && r->metadata_value() != NULL) {
+Metadata* md = r->metadata_value();
+if (md != _method) Metadata::mark_on_stack(md);
+}
+}
+void nmethod::mark_metadata_on_stack_non_relocs() {
+// Visit the metadata section
+for (Metadata** p = metadata_begin(); p < metadata_end(); p++) {
+if (*p == Universe::non_oop_word() || *p == NULL)  continue;  // skip non-oops
+Metadata* md = *p;
+Metadata::mark_on_stack(md);
+}
+// Visit metadata not embedded in the other places.
+if (_method != NULL) Metadata::mark_on_stack(_method);
+}
+bool nmethod::do_unloading_parallel(BoolObjectClosure* is_alive, bool unloading_occurred) {
+ResourceMark rm;
+// Make sure the oop's ready to receive visitors
+assert(!is_zombie() && !is_unloaded(),
+"should not call follow on zombie or unloaded nmethod");
+// If the method is not entrant then a JMP is plastered over the
+// first few bytes.  If an oop in the old code was there, that oop
+// should not get GC'd.  Skip the first few bytes of oops on
+// not-entrant methods.
+address low_boundary = verified_entry_point();
+if (is_not_entrant()) {
+low_boundary += NativeJump::instruction_size;
+// %%% Note:  On SPARC we patch only a 4-byte trap, not a full NativeJump.
+// (See comment above.)
+}
+// The RedefineClasses() API can cause the class unloading invariant
+// to no longer be true. See jvmtiExport.hpp for details.
+// Also, leave a debugging breadcrumb in local flag.
+bool a_class_was_redefined = JvmtiExport::has_redefined_a_class();
+if (a_class_was_redefined) {
+// This set of the unloading_occurred flag is done before the
+// call to post_compiled_method_unload() so that the unloading
+// of this nmethod is reported.
+unloading_occurred = true;
+}
+// When class redefinition is used all metadata in the CodeCache has to be recorded,
+// so that unused "previous versions" can be purged. Since walking the CodeCache can
+// be expensive, the "mark on stack" is piggy-backed on this parallel unloading code.
+bool mark_metadata_on_stack = a_class_was_redefined;
+// Exception cache
+clean_exception_cache(is_alive);
+bool is_unloaded = false;
+bool postponed = false;
+RelocIterator iter(this, low_boundary);
+while(iter.next()) {
+switch (iter.type()) {
+case relocInfo::virtual_call_type:
+if (unloading_occurred) {
+// If class unloading occurred we first iterate over all inline caches and
+// clear ICs where the cached oop is referring to an unloaded klass or method.
+clean_ic_if_metadata_is_dead(CompiledIC_at(&iter), is_alive, mark_metadata_on_stack);
+}
+postponed |= clean_if_nmethod_is_unloaded(CompiledIC_at(&iter), is_alive, this);
+break;
+case relocInfo::opt_virtual_call_type:
+postponed |= clean_if_nmethod_is_unloaded(CompiledIC_at(&iter), is_alive, this);
+break;
+case relocInfo::static_call_type:
+postponed |= clean_if_nmethod_is_unloaded(compiledStaticCall_at(iter.reloc()), is_alive, this);
+break;
+case relocInfo::oop_type:
+if (!is_unloaded) {
+is_unloaded = unload_if_dead_at(&iter, is_alive, unloading_occurred);
+}
+break;
+case relocInfo::metadata_type:
+if (mark_metadata_on_stack) {
+mark_metadata_on_stack_at(&iter);
+}
+}
+}
+if (mark_metadata_on_stack) {
+mark_metadata_on_stack_non_relocs();
+}
+if (is_unloaded) {
+return postponed;
+}
+// Scopes
+for (oop* p = oops_begin(); p < oops_end(); p++) {
+if (*p == Universe::non_oop_word())  continue;  // skip non-oops
+if (can_unload(is_alive, p, unloading_occurred)) {
+is_unloaded = true;
+break;
+}
+}
+if (is_unloaded) {
+return postponed;
+}
+// Ensure that all metadata is still alive
+verify_metadata_loaders(low_boundary, is_alive);
+return postponed;
+}
+void nmethod::do_unloading_parallel_postponed(BoolObjectClosure* is_alive, bool unloading_occurred) {
+ResourceMark rm;
+// Make sure the oop's ready to receive visitors
+assert(!is_zombie(),
+"should not call follow on zombie nmethod");
+// If the method is not entrant then a JMP is plastered over the
+// first few bytes.  If an oop in the old code was there, that oop
+// should not get GC'd.  Skip the first few bytes of oops on
+// not-entrant methods.
+address low_boundary = verified_entry_point();
+if (is_not_entrant()) {
+low_boundary += NativeJump::instruction_size;
+// %%% Note:  On SPARC we patch only a 4-byte trap, not a full NativeJump.
+// (See comment above.)
+}
+RelocIterator iter(this, low_boundary);
+while(iter.next()) {
+switch (iter.type()) {
+case relocInfo::virtual_call_type:
+clean_if_nmethod_is_unloaded(CompiledIC_at(&iter), is_alive, this);
+break;
+case relocInfo::opt_virtual_call_type:
+clean_if_nmethod_is_unloaded(CompiledIC_at(&iter), is_alive, this);
+break;
+case relocInfo::static_call_type:
+clean_if_nmethod_is_unloaded(compiledStaticCall_at(iter.reloc()), is_alive, this);
+break;
+}
+}
+}
 #ifdef ASSERT
 class CheckClass : AllStatic {
 static BoolObjectClosure* _is_alive;
 // static_stub_Relocations may have dangling references to
 // Method*s so trim them out here.  Otherwise it looks like
 // compiled code is maintaining a link to dead metadata.
 address static_call_addr = NULL;
 if (iter.type() == relocInfo::opt_virtual_call_type) {
-CompiledIC* cic = CompiledIC_at(iter.reloc());
+CompiledIC* cic = CompiledIC_at(&iter);
 if (!cic->is_call_to_interpreted()) {
 static_call_addr = iter.addr();
 }
 } else if (iter.type() == relocInfo::static_call_type) {
 CompiledStaticCall* csc = compiledStaticCall_at(iter.reloc());
 // Visit all immediate references that are embedded in the instruction stream.
 RelocIterator iter(this, low_boundary);
 while (iter.next()) {
 if (iter.type() == relocInfo::metadata_type ) {
 metadata_Relocation* r = iter.metadata_reloc();
-// In this lmetadata, we must only follow those metadatas directly embedded in
+// In this metadata, we must only follow those metadatas directly embedded in
 // the code.  Other metadatas (oop_index>0) are seen as part of
 // the metadata section below.
 assert(1 == (r->metadata_is_immediate()) +
 (r->metadata_addr() >= metadata_begin() && r->metadata_addr() < metadata_end()),
 "metadata must be found in exactly one place");
 Metadata* md = r->metadata_value();
 f(md);
 }
 } else if (iter.type() == relocInfo::virtual_call_type) {
 // Check compiledIC holders associated with this nmethod
-CompiledIC *ic = CompiledIC_at(iter.reloc());
+CompiledIC *ic = CompiledIC_at(&iter);
 if (ic->is_icholder_call()) {
 CompiledICHolder* cichk = ic->cached_icholder();
 f(cichk->holder_method());
 f(cichk->holder_klass());
 } else {
 if (*p == Universe::non_oop_word() || *p == NULL)  continue;  // skip non-oops
 Metadata* md = *p;
 f(md);
 }
-// Call function Method*, not embedded in these other places.
+// Visit metadata not embedded in the other places.
 if (_method != NULL) f(_method);
 }
 void nmethod::oops_do(OopClosure* f, bool allow_zombie) {
 // make sure the oops ready to receive visitors
 nmethod* cur = _oops_do_mark_nmethods;
 while (cur != NMETHOD_SENTINEL) {
 assert(cur != NULL, "not NULL-terminated");
 nmethod* next = cur->_oops_do_mark_link;
 cur->_oops_do_mark_link = NULL;
-cur->fix_oop_relocations();
+cur->verify_oop_relocations();
 NOT_PRODUCT(if (TraceScavenge)  cur->print_on(tty, "oops_do, unmark"));
 cur = next;
 }
 void* required = _oops_do_mark_nmethods;
 void* observed = Atomic::cmpxchg_ptr(NULL, &_oops_do_mark_nmethods, required);
 }
 virtual void do_oop(narrowOop* p) { ShouldNotReachHere(); }
 };
 void nmethod::verify_scavenge_root_oops() {
+if (UseG1GC) {
+return;
+}
 if (!on_scavenge_root_list()) {
 // Actually look inside, to verify the claim that it's clean.
 DebugScavengeRoot debug_scavenge_root(this);
 oops_do(&debug_scavenge_root);
 if (!debug_scavenge_root.ok())
 while (iter.next()) {
 switch (iter.type()) {
 case relocInfo::virtual_call_type:
 case relocInfo::opt_virtual_call_type: {
 VerifyMutexLocker mc(CompiledIC_lock);
-CompiledIC_at(iter.reloc())->print();
+CompiledIC_at(&iter)->print();
 break;
 }
 case relocInfo::static_call_type:
 st->print_cr("Static call at " INTPTR_FORMAT, iter.reloc()->addr());
 compiledStaticCall_at(iter.reloc())->print();

Mercurial > hg > truffle

comparison src/share/vm/code/nmethod.cpp @ 20804:7848fc12602b