6923043: failed nightly tests which use -XX:+PrintCompilation -Xcomp -XX:CompileOnly
authorkvn
Wed, 03 Feb 2010 15:03:22 -0800
changeset 4756 da88c27a9241
parent 4755 eee57ea6d910
child 4758 5911cc4cb361
6923043: failed nightly tests which use -XX:+PrintCompilation -Xcomp -XX:CompileOnly Summary: Print "made not compilable" line only for deoptimizations. Reviewed-by: never
hotspot/src/share/vm/compiler/compileBroker.cpp
hotspot/src/share/vm/oops/methodOop.cpp
hotspot/src/share/vm/oops/methodOop.hpp
--- 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
--- 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);
--- 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(); }