# HG changeset patch # User kvn # Date 1265238202 28800 # Node ID da88c27a92412ad54bcb66d11d3a4ac122eb1ddd # Parent eee57ea6d9108423be6e67bef13b0fb26e2deca9 6923043: failed nightly tests which use -XX:+PrintCompilation -Xcomp -XX:CompileOnly Summary: Print "made not compilable" line only for deoptimizations. Reviewed-by: never diff -r eee57ea6d910 -r da88c27a9241 hotspot/src/share/vm/compiler/compileBroker.cpp --- a/hotspot/src/share/vm/compiler/compileBroker.cpp Wed Feb 03 12:28:30 2010 -0800 +++ b/hotspot/src/share/vm/compiler/compileBroker.cpp Wed Feb 03 15:03:22 2010 -0800 @@ -1,5 +1,5 @@ /* - * Copyright 1999-2009 Sun Microsystems, Inc. All Rights Reserved. + * Copyright 1999-2010 Sun Microsystems, Inc. All Rights Reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it @@ -1132,7 +1132,7 @@ // the specified level if (is_native && (!CICompileNatives || !compiler(comp_level)->supports_native())) { - method->set_not_compilable(); + method->set_not_compilable_quietly(); return true; } @@ -1156,7 +1156,7 @@ method->print_short_name(tty); tty->cr(); } - method->set_not_compilable(); + method->set_not_compilable_quietly(); } return false; @@ -1189,7 +1189,7 @@ } // Method was not in the appropriate compilation range. - method->set_not_compilable(); + method->set_not_compilable_quietly(); return 0; } @@ -1590,10 +1590,10 @@ if (is_osr) { method->set_not_osr_compilable(); } else { - method->set_not_compilable(); + method->set_not_compilable_quietly(); } } else if (compilable == ciEnv::MethodCompilable_not_at_tier) { - method->set_not_compilable(task->comp_level()); + method->set_not_compilable_quietly(task->comp_level()); } // Note that the queued_for_compilation bits are cleared without diff -r eee57ea6d910 -r da88c27a9241 hotspot/src/share/vm/oops/methodOop.cpp --- a/hotspot/src/share/vm/oops/methodOop.cpp Wed Feb 03 12:28:30 2010 -0800 +++ b/hotspot/src/share/vm/oops/methodOop.cpp Wed Feb 03 15:03:22 2010 -0800 @@ -587,8 +587,8 @@ } // call this when compiler finds that this method is not compilable -void methodOopDesc::set_not_compilable(int comp_level) { - if (PrintCompilation) { +void methodOopDesc::set_not_compilable(int comp_level, bool report) { + if (PrintCompilation && report) { ttyLocker ttyl; tty->print("made not compilable "); this->print_short_name(tty); diff -r eee57ea6d910 -r da88c27a9241 hotspot/src/share/vm/oops/methodOop.hpp --- a/hotspot/src/share/vm/oops/methodOop.hpp Wed Feb 03 12:28:30 2010 -0800 +++ b/hotspot/src/share/vm/oops/methodOop.hpp Wed Feb 03 15:03:22 2010 -0800 @@ -596,7 +596,10 @@ // whether it is not compilable for another reason like having a // breakpoint set in it. bool is_not_compilable(int comp_level = CompLevel_highest_tier) const; - void set_not_compilable(int comp_level = CompLevel_highest_tier); + void set_not_compilable(int comp_level = CompLevel_highest_tier, bool report = true); + void set_not_compilable_quietly(int comp_level = CompLevel_highest_tier) { + set_not_compilable(comp_level, false); + } bool is_not_osr_compilable() const { return is_not_compilable() || access_flags().is_not_osr_compilable(); } void set_not_osr_compilable() { _access_flags.set_not_osr_compilable(); }