8147918: Rename develop_log_is_enabled() to log_develop_is_enabled()
Reviewed-by: dholmes, brutisso
This commit is contained in:
parent
3cda485fe5
commit
4bfa63dd34
@ -414,7 +414,7 @@ void ParScanThreadStateSet::print_taskqueue_stats_hdr(outputStream* const st) {
|
||||
}
|
||||
|
||||
void ParScanThreadStateSet::print_taskqueue_stats() {
|
||||
if (!develop_log_is_enabled(Trace, gc, task, stats)) {
|
||||
if (!log_develop_is_enabled(Trace, gc, task, stats)) {
|
||||
return;
|
||||
}
|
||||
LogHandle(gc, task, stats) log;
|
||||
|
@ -3141,7 +3141,7 @@ void G1CollectedHeap::print_taskqueue_stats_hdr(outputStream* const st) {
|
||||
}
|
||||
|
||||
void G1CollectedHeap::print_taskqueue_stats() const {
|
||||
if (!develop_log_is_enabled(Trace, gc, task, stats)) {
|
||||
if (!log_develop_is_enabled(Trace, gc, task, stats)) {
|
||||
return;
|
||||
}
|
||||
LogHandle(gc, task, stats) log;
|
||||
|
@ -195,7 +195,7 @@ const char* PSParallelCompact::space_names[] = {
|
||||
};
|
||||
|
||||
void PSParallelCompact::print_region_ranges() {
|
||||
if (!develop_log_is_enabled(Trace, gc, compaction, phases)) {
|
||||
if (!log_develop_is_enabled(Trace, gc, compaction, phases)) {
|
||||
return;
|
||||
}
|
||||
LogHandle(gc, compaction, phases) log;
|
||||
@ -265,7 +265,7 @@ void
|
||||
print_generic_summary_data(ParallelCompactData& summary_data,
|
||||
SpaceInfo* space_info)
|
||||
{
|
||||
if (!develop_log_is_enabled(Trace, gc, compaction, phases)) {
|
||||
if (!log_develop_is_enabled(Trace, gc, compaction, phases)) {
|
||||
return;
|
||||
}
|
||||
|
||||
@ -360,7 +360,7 @@ print_initial_summary_data(ParallelCompactData& summary_data,
|
||||
void
|
||||
print_initial_summary_data(ParallelCompactData& summary_data,
|
||||
SpaceInfo* space_info) {
|
||||
if (!develop_log_is_enabled(Trace, gc, compaction, phases)) {
|
||||
if (!log_develop_is_enabled(Trace, gc, compaction, phases)) {
|
||||
return;
|
||||
}
|
||||
|
||||
@ -641,7 +641,7 @@ ParallelCompactData::summarize_split_space(size_t src_region,
|
||||
*target_next = split_destination + partial_obj_size;
|
||||
HeapWord* const source_next = region_to_addr(split_region) + partial_obj_size;
|
||||
|
||||
if (develop_log_is_enabled(Trace, gc, compaction, phases)) {
|
||||
if (log_develop_is_enabled(Trace, gc, compaction, phases)) {
|
||||
const char * split_type = partial_obj_size == 0 ? "easy" : "hard";
|
||||
log_develop_trace(gc, compaction, phases)("%s split: src=" PTR_FORMAT " src_c=" SIZE_FORMAT " pos=" SIZE_FORMAT,
|
||||
split_type, p2i(source_next), split_region, partial_obj_size);
|
||||
@ -1535,7 +1535,7 @@ PSParallelCompact::summarize_space(SpaceId id, bool maximum_compaction)
|
||||
}
|
||||
}
|
||||
|
||||
if (develop_log_is_enabled(Trace, gc, compaction, phases)) {
|
||||
if (log_develop_is_enabled(Trace, gc, compaction, phases)) {
|
||||
const size_t region_size = ParallelCompactData::RegionSize;
|
||||
HeapWord* const dense_prefix_end = _space_info[id].dense_prefix();
|
||||
const size_t dp_region = _summary_data.addr_to_region_idx(dense_prefix_end);
|
||||
@ -2190,7 +2190,7 @@ private:
|
||||
bool _enabled;
|
||||
size_t _total_regions;
|
||||
public:
|
||||
FillableRegionLogger() : _next_index(0), _total_regions(0), _enabled(develop_log_is_enabled(Trace, gc, compaction)) { }
|
||||
FillableRegionLogger() : _next_index(0), _total_regions(0), _enabled(log_develop_is_enabled(Trace, gc, compaction)) { }
|
||||
~FillableRegionLogger() {
|
||||
log.trace(SIZE_FORMAT " initially fillable regions", _total_regions);
|
||||
}
|
||||
@ -2381,7 +2381,7 @@ void PSParallelCompact::enqueue_region_stealing_tasks(
|
||||
// region.
|
||||
void PSParallelCompact::write_block_fill_histogram()
|
||||
{
|
||||
if (!develop_log_is_enabled(Trace, gc, compaction)) {
|
||||
if (!log_develop_is_enabled(Trace, gc, compaction)) {
|
||||
return;
|
||||
}
|
||||
|
||||
|
@ -130,7 +130,7 @@ static const char* const pm_stats_hdr[] = {
|
||||
|
||||
void
|
||||
PSPromotionManager::print_taskqueue_stats() {
|
||||
if (!develop_log_is_enabled(Trace, gc, task, stats)) {
|
||||
if (!log_develop_is_enabled(Trace, gc, task, stats)) {
|
||||
return;
|
||||
}
|
||||
LogHandle(gc, task, stats) log;
|
||||
|
@ -284,7 +284,7 @@ inline void PSPromotionManager::copy_and_push_safe_barrier(T* p) {
|
||||
|
||||
// This code must come after the CAS test, or it will print incorrect
|
||||
// information.
|
||||
if (develop_log_is_enabled(Trace, gc, scavenge) && o->is_forwarded()) {
|
||||
if (log_develop_is_enabled(Trace, gc, scavenge) && o->is_forwarded()) {
|
||||
log_develop_trace(gc, scavenge)("{%s %s " PTR_FORMAT " -> " PTR_FORMAT " (%d)}",
|
||||
"forwarding",
|
||||
new_obj->klass()->internal_name(), p2i((void *)o), p2i((void *)new_obj), new_obj->size());
|
||||
|
@ -763,7 +763,7 @@ void InterpreterRuntime::resolve_invoke(JavaThread* thread, Bytecodes::Code byte
|
||||
if (cp_cache_entry->is_resolved(bytecode)) return;
|
||||
|
||||
if (bytecode == Bytecodes::_invokeinterface) {
|
||||
if (develop_log_is_enabled(Trace, itables)) {
|
||||
if (log_develop_is_enabled(Trace, itables)) {
|
||||
ResourceMark rm(thread);
|
||||
log_develop_trace(itables)("Resolving: klass: %s to method: %s",
|
||||
info.resolved_klass()->name()->as_C_string(),
|
||||
|
@ -816,7 +816,7 @@ methodHandle LinkResolver::resolve_interface_method(const LinkInfo& link_info,
|
||||
THROW_MSG_NULL(vmSymbols::java_lang_IncompatibleClassChangeError(), buf);
|
||||
}
|
||||
|
||||
if (develop_log_is_enabled(Trace, itables)) {
|
||||
if (log_develop_is_enabled(Trace, itables)) {
|
||||
trace_method_resolution("invokeinterface resolved method: caller-class",
|
||||
link_info.current_klass(), resolved_klass,
|
||||
resolved_method, true);
|
||||
@ -1064,7 +1064,7 @@ methodHandle LinkResolver::linktime_resolve_special_method(const LinkInfo& link_
|
||||
THROW_MSG_NULL(vmSymbols::java_lang_IncompatibleClassChangeError(), buf);
|
||||
}
|
||||
|
||||
if (develop_log_is_enabled(Trace, itables)) {
|
||||
if (log_develop_is_enabled(Trace, itables)) {
|
||||
trace_method_resolution("invokespecial resolved method: caller-class:",
|
||||
current_klass, resolved_klass, resolved_method, true);
|
||||
}
|
||||
@ -1135,7 +1135,7 @@ void LinkResolver::runtime_resolve_special_method(CallInfo& result,
|
||||
sel_method->signature()));
|
||||
}
|
||||
|
||||
if (develop_log_is_enabled(Trace, itables)) {
|
||||
if (log_develop_is_enabled(Trace, itables)) {
|
||||
trace_method_resolution("invokespecial selected method: resolved-class:",
|
||||
resolved_klass, resolved_klass, sel_method, true);
|
||||
}
|
||||
@ -1188,7 +1188,7 @@ methodHandle LinkResolver::linktime_resolve_virtual_method(const LinkInfo& link_
|
||||
THROW_MSG_NULL(vmSymbols::java_lang_IncompatibleClassChangeError(), buf);
|
||||
}
|
||||
|
||||
if (develop_log_is_enabled(Trace, vtables)) {
|
||||
if (log_develop_is_enabled(Trace, vtables)) {
|
||||
trace_method_resolution("invokevirtual resolved method: caller-class:",
|
||||
current_klass, resolved_klass, resolved_method, false);
|
||||
}
|
||||
@ -1264,7 +1264,7 @@ void LinkResolver::runtime_resolve_virtual_method(CallInfo& result,
|
||||
selected_method->signature()));
|
||||
}
|
||||
|
||||
if (develop_log_is_enabled(Trace, vtables)) {
|
||||
if (log_develop_is_enabled(Trace, vtables)) {
|
||||
trace_method_resolution("invokevirtual selected method: receiver-class:",
|
||||
recv_klass, resolved_klass, selected_method,
|
||||
false, vtable_index);
|
||||
@ -1363,7 +1363,7 @@ void LinkResolver::runtime_resolve_interface_method(CallInfo& result,
|
||||
sel_method->signature()));
|
||||
}
|
||||
|
||||
if (develop_log_is_enabled(Trace, itables)) {
|
||||
if (log_develop_is_enabled(Trace, itables)) {
|
||||
trace_method_resolution("invokeinterface selected method: receiver-class",
|
||||
recv_klass, resolved_klass, sel_method, true);
|
||||
}
|
||||
|
@ -57,13 +57,13 @@
|
||||
#define log_develop_info(...) (!log_is_enabled(Info, __VA_ARGS__)) ? (void)0 : Log<LOG_TAGS(__VA_ARGS__)>::write<LogLevel::Info>
|
||||
#define log_develop_debug(...) (!log_is_enabled(Debug, __VA_ARGS__)) ? (void)0 : Log<LOG_TAGS(__VA_ARGS__)>::write<LogLevel::Debug>
|
||||
#define log_develop_trace(...) (!log_is_enabled(Trace, __VA_ARGS__)) ? (void)0 : Log<LOG_TAGS(__VA_ARGS__)>::write<LogLevel::Trace>
|
||||
#define develop_log_is_enabled(level, ...) log_is_enabled(level, __VA_ARGS__)
|
||||
#define log_develop_is_enabled(level, ...) log_is_enabled(level, __VA_ARGS__)
|
||||
#else
|
||||
#define DUMMY_ARGUMENT_CONSUMER(...)
|
||||
#define log_develop_info(...) DUMMY_ARGUMENT_CONSUMER
|
||||
#define log_develop_debug(...) DUMMY_ARGUMENT_CONSUMER
|
||||
#define log_develop_trace(...) DUMMY_ARGUMENT_CONSUMER
|
||||
#define develop_log_is_enabled(...) false
|
||||
#define log_develop_is_enabled(...) false
|
||||
#endif
|
||||
|
||||
// Convenience macro to test if the logging is enabled on the specified level for given tags.
|
||||
|
@ -1234,7 +1234,7 @@ void VirtualSpaceList::link_vs(VirtualSpaceNode* new_entry) {
|
||||
#ifdef ASSERT
|
||||
new_entry->mangle();
|
||||
#endif
|
||||
if (develop_log_is_enabled(Trace, gc, metaspace)) {
|
||||
if (log_develop_is_enabled(Trace, gc, metaspace)) {
|
||||
LogHandle(gc, metaspace) log;
|
||||
VirtualSpaceNode* vsl = current_virtual_space();
|
||||
ResourceMark rm;
|
||||
@ -3051,7 +3051,7 @@ void Metaspace::allocate_metaspace_compressed_klass_ptrs(char* requested_addr, a
|
||||
|
||||
initialize_class_space(metaspace_rs);
|
||||
|
||||
if (develop_log_is_enabled(Trace, gc, metaspace)) {
|
||||
if (log_develop_is_enabled(Trace, gc, metaspace)) {
|
||||
LogHandle(gc, metaspace) log;
|
||||
ResourceMark rm;
|
||||
print_compressed_class_space(log.trace_stream(), requested_addr);
|
||||
|
@ -135,7 +135,7 @@ int klassVtable::initialize_from_super(KlassHandle super) {
|
||||
superVtable->verify(tty, true);
|
||||
#endif
|
||||
superVtable->copy_vtable_to(table());
|
||||
if (develop_log_is_enabled(Trace, vtables)) {
|
||||
if (log_develop_is_enabled(Trace, vtables)) {
|
||||
ResourceMark rm;
|
||||
log_develop_trace(vtables)("copy vtable from %s to %s size %d",
|
||||
super->internal_name(), klass()->internal_name(),
|
||||
@ -272,7 +272,7 @@ InstanceKlass* klassVtable::find_transitive_override(InstanceKlass* initialsuper
|
||||
assert(super_method->name() == name && super_method->signature() == signature, "vtable entry name/sig mismatch");
|
||||
#endif
|
||||
if (supersuperklass->is_override(super_method, target_loader, target_classname, THREAD)) {
|
||||
if (develop_log_is_enabled(Trace, vtables)) {
|
||||
if (log_develop_is_enabled(Trace, vtables)) {
|
||||
ResourceMark rm(THREAD);
|
||||
outputStream* logst = LogHandle(vtables)::trace_stream();
|
||||
char* sig = target_method()->name_and_sig_as_C_string();
|
||||
@ -303,7 +303,7 @@ static void log_vtables(int i, bool overrides, methodHandle target_method,
|
||||
KlassHandle target_klass, Method* super_method,
|
||||
Thread* thread) {
|
||||
#ifndef PRODUCT
|
||||
if (develop_log_is_enabled(Trace, vtables)) {
|
||||
if (log_develop_is_enabled(Trace, vtables)) {
|
||||
ResourceMark rm(thread);
|
||||
outputStream* logst = LogHandle(vtables)::trace_stream();
|
||||
char* sig = target_method()->name_and_sig_as_C_string();
|
||||
@ -491,7 +491,7 @@ bool klassVtable::update_inherited_vtable(InstanceKlass* klass, methodHandle tar
|
||||
}
|
||||
|
||||
void klassVtable::put_method_at(Method* m, int index) {
|
||||
if (develop_log_is_enabled(Trace, vtables)) {
|
||||
if (log_develop_is_enabled(Trace, vtables)) {
|
||||
ResourceMark rm;
|
||||
outputStream* logst = LogHandle(vtables)::trace_stream();
|
||||
const char* sig = (m != NULL) ? m->name_and_sig_as_C_string() : "<NULL>";
|
||||
@ -818,7 +818,7 @@ int klassVtable::fill_in_mirandas(int initialized) {
|
||||
get_mirandas(&mirandas, NULL, ik()->super(), ik()->methods(),
|
||||
ik()->default_methods(), ik()->local_interfaces());
|
||||
for (int i = 0; i < mirandas.length(); i++) {
|
||||
if (develop_log_is_enabled(Trace, vtables)) {
|
||||
if (log_develop_is_enabled(Trace, vtables)) {
|
||||
Method* meth = mirandas.at(i);
|
||||
ResourceMark rm(Thread::current());
|
||||
outputStream* logst = LogHandle(vtables)::trace_stream();
|
||||
@ -1043,7 +1043,7 @@ int klassItable::assign_itable_indices_for_interface(Klass* klass) {
|
||||
if (interface_method_needs_itable_index(m)) {
|
||||
assert(!m->is_final_method(), "no final interface methods");
|
||||
// If m is already assigned a vtable index, do not disturb it.
|
||||
if (develop_log_is_enabled(Trace, itables)) {
|
||||
if (log_develop_is_enabled(Trace, itables)) {
|
||||
ResourceMark rm;
|
||||
outputStream* logst = LogHandle(itables)::trace_stream();
|
||||
assert(m != NULL, "methods can never be null");
|
||||
@ -1158,7 +1158,7 @@ void klassItable::initialize_itable_for_interface(int method_table_offset, Klass
|
||||
int ime_num = m->itable_index();
|
||||
assert(ime_num < ime_count, "oob");
|
||||
itableOffsetEntry::method_entry(_klass(), method_table_offset)[ime_num].initialize(target());
|
||||
if (develop_log_is_enabled(Trace, itables)) {
|
||||
if (log_develop_is_enabled(Trace, itables)) {
|
||||
ResourceMark rm(THREAD);
|
||||
if (target() != NULL) {
|
||||
outputStream* logst = LogHandle(itables)::trace_stream();
|
||||
|
Loading…
x
Reference in New Issue
Block a user