# HG changeset patch
# User kvn
# Date 1279904866 25200
# Node ID e5dfb3ccb88b269ef208a83995793d444c8502b3
# Parent  7139e81efd2d62c4b615f4d90349fd4dd5f82a90
6969569: assert(is_static() && is_constant()) failed: illegal call to constant_value()
Summary: Add missing is_static guard.
Reviewed-by: twisti

diff -r 7139e81efd2d -r e5dfb3ccb88b src/share/vm/ci/ciField.cpp
--- a/src/share/vm/ci/ciField.cpp	Thu Jul 22 15:29:22 2010 -0700
+++ b/src/share/vm/ci/ciField.cpp	Fri Jul 23 10:07:46 2010 -0700
@@ -339,7 +339,7 @@
   if (_type != NULL) _type->print_name();
   else               tty->print("(reference)");
   tty->print(" is_constant=%s", bool_to_str(_is_constant));
-  if (_is_constant) {
+  if (_is_constant && is_static()) {
     tty->print(" constant_value=");
     _constant_value.print();
   }
diff -r 7139e81efd2d -r e5dfb3ccb88b src/share/vm/opto/macro.cpp
--- a/src/share/vm/opto/macro.cpp	Thu Jul 22 15:29:22 2010 -0700
+++ b/src/share/vm/opto/macro.cpp	Fri Jul 23 10:07:46 2010 -0700
@@ -720,7 +720,7 @@
       if (basic_elem_type == T_OBJECT || basic_elem_type == T_ARRAY) {
         if (!elem_type->is_loaded()) {
           field_type = TypeInstPtr::BOTTOM;
-        } else if (field != NULL && field->is_constant()) {
+        } else if (field != NULL && field->is_constant() && field->is_static()) {
           // This can happen if the constant oop is non-perm.
           ciObject* con = field->constant_value().as_object();
           // Do not "join" in the previous type; it doesn't add value,