src/hotspot/share/runtime/fieldDescriptor.cpp
author coleenp
Wed, 13 Nov 2019 08:23:23 -0500
changeset 59056 15936b142f86
parent 58273 08a5148e7c4e
permissions -rw-r--r--
8233913: Remove implicit conversion from Method* to methodHandle Summary: Fix call sites to use existing THREAD local or pass down THREAD local for shallower callsites. Make linkResolver methods return Method* for caller to handleize if needed. Reviewed-by: iklam, thartmann, hseigel
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
54807
33fe50b6d707 8223626: move print() functions to cpp files
coleenp
parents: 54786
diff changeset
     2
 * Copyright (c) 1997, 2019, Oracle and/or its affiliates. All rights reserved.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     4
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
489c9b5090e2 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
489c9b5090e2 Initial load
duke
parents:
diff changeset
     7
 * published by the Free Software Foundation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     8
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
489c9b5090e2 Initial load
duke
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    13
 * accompanied this code).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    14
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
489c9b5090e2 Initial load
duke
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 2332
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 2332
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 2332
diff changeset
    21
 * questions.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    22
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    23
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
    24
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    25
#include "precompiled.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    26
#include "classfile/systemDictionary.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    27
#include "classfile/vmSymbols.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    28
#include "memory/resourceArea.hpp"
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13101
diff changeset
    29
#include "oops/annotations.hpp"
53838
c8c9bd65c198 8219229: Make ConstantPool::tag_at and release_tag_at_put inlineable
redestad
parents: 51467
diff changeset
    30
#include "oops/constantPool.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    31
#include "oops/instanceKlass.hpp"
29081
c61eb4914428 8072911: Remove includes of oop.inline.hpp from .hpp files
stefank
parents: 27680
diff changeset
    32
#include "oops/oop.inline.hpp"
59056
15936b142f86 8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents: 58273
diff changeset
    33
#include "oops/fieldStreams.inline.hpp"
51467
12997ebbc0d8 8209647: constantPoolHandle::constantPoolHandle(ConstantPool*) when precompiled header is disabled
iklam
parents: 51268
diff changeset
    34
#include "runtime/fieldDescriptor.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    35
#include "runtime/handles.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    36
#include "runtime/signature.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    37
489c9b5090e2 Initial load
duke
parents:
diff changeset
    38
489c9b5090e2 Initial load
duke
parents:
diff changeset
    39
oop fieldDescriptor::loader() const {
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 13728
diff changeset
    40
  return _cp->pool_holder()->class_loader();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    41
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    42
12772
d317e5e08194 7168280: Eliminate the generic signature index slot from field array for field without generic signature.
jiangli
parents: 10546
diff changeset
    43
Symbol* fieldDescriptor::generic_signature() const {
13101
67539edd246d 7177409: Perf regression in JVM_GetClassDeclaredFields after generic signature changes.
jiangli
parents: 12772
diff changeset
    44
  if (!has_generic_signature()) {
67539edd246d 7177409: Perf regression in JVM_GetClassDeclaredFields after generic signature changes.
jiangli
parents: 12772
diff changeset
    45
    return NULL;
67539edd246d 7177409: Perf regression in JVM_GetClassDeclaredFields after generic signature changes.
jiangli
parents: 12772
diff changeset
    46
  }
67539edd246d 7177409: Perf regression in JVM_GetClassDeclaredFields after generic signature changes.
jiangli
parents: 12772
diff changeset
    47
12772
d317e5e08194 7168280: Eliminate the generic signature index slot from field array for field without generic signature.
jiangli
parents: 10546
diff changeset
    48
  int idx = 0;
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 13728
diff changeset
    49
  InstanceKlass* ik = field_holder();
12772
d317e5e08194 7168280: Eliminate the generic signature index slot from field array for field without generic signature.
jiangli
parents: 10546
diff changeset
    50
  for (AllFieldStream fs(ik); !fs.done(); fs.next()) {
d317e5e08194 7168280: Eliminate the generic signature index slot from field array for field without generic signature.
jiangli
parents: 10546
diff changeset
    51
    if (idx == _index) {
d317e5e08194 7168280: Eliminate the generic signature index slot from field array for field without generic signature.
jiangli
parents: 10546
diff changeset
    52
      return fs.generic_signature();
d317e5e08194 7168280: Eliminate the generic signature index slot from field array for field without generic signature.
jiangli
parents: 10546
diff changeset
    53
    } else {
d317e5e08194 7168280: Eliminate the generic signature index slot from field array for field without generic signature.
jiangli
parents: 10546
diff changeset
    54
      idx ++;
d317e5e08194 7168280: Eliminate the generic signature index slot from field array for field without generic signature.
jiangli
parents: 10546
diff changeset
    55
    }
d317e5e08194 7168280: Eliminate the generic signature index slot from field array for field without generic signature.
jiangli
parents: 10546
diff changeset
    56
  }
d317e5e08194 7168280: Eliminate the generic signature index slot from field array for field without generic signature.
jiangli
parents: 10546
diff changeset
    57
  assert(false, "should never happen");
d317e5e08194 7168280: Eliminate the generic signature index slot from field array for field without generic signature.
jiangli
parents: 10546
diff changeset
    58
  return NULL;
d317e5e08194 7168280: Eliminate the generic signature index slot from field array for field without generic signature.
jiangli
parents: 10546
diff changeset
    59
}
d317e5e08194 7168280: Eliminate the generic signature index slot from field array for field without generic signature.
jiangli
parents: 10546
diff changeset
    60
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13101
diff changeset
    61
AnnotationArray* fieldDescriptor::annotations() const {
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 13728
diff changeset
    62
  InstanceKlass* ik = field_holder();
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13101
diff changeset
    63
  Array<AnnotationArray*>* md = ik->fields_annotations();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
  if (md == NULL)
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
    return NULL;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13101
diff changeset
    66
  return md->at(index());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
15097
9db149412e0e 8004823: Add VM support for type annotation reflection
stefank
parents: 14391
diff changeset
    69
AnnotationArray* fieldDescriptor::type_annotations() const {
9db149412e0e 8004823: Add VM support for type annotation reflection
stefank
parents: 14391
diff changeset
    70
  InstanceKlass* ik = field_holder();
15601
df8faef6efaf 8007320: NPG: move method annotations
coleenp
parents: 15097
diff changeset
    71
  Array<AnnotationArray*>* type_annos = ik->fields_type_annotations();
15097
9db149412e0e 8004823: Add VM support for type annotation reflection
stefank
parents: 14391
diff changeset
    72
  if (type_annos == NULL)
9db149412e0e 8004823: Add VM support for type annotation reflection
stefank
parents: 14391
diff changeset
    73
    return NULL;
15601
df8faef6efaf 8007320: NPG: move method annotations
coleenp
parents: 15097
diff changeset
    74
  return type_annos->at(index());
15097
9db149412e0e 8004823: Add VM support for type annotation reflection
stefank
parents: 14391
diff changeset
    75
}
9db149412e0e 8004823: Add VM support for type annotation reflection
stefank
parents: 14391
diff changeset
    76
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
constantTag fieldDescriptor::initial_value_tag() const {
10546
e79347eebbc5 7086585: make Java field injection more flexible
never
parents: 7397
diff changeset
    78
  return constants()->tag_at(initial_value_index());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
jint fieldDescriptor::int_initial_value() const {
10546
e79347eebbc5 7086585: make Java field injection more flexible
never
parents: 7397
diff changeset
    82
  return constants()->int_at(initial_value_index());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
jlong fieldDescriptor::long_initial_value() const {
10546
e79347eebbc5 7086585: make Java field injection more flexible
never
parents: 7397
diff changeset
    86
  return constants()->long_at(initial_value_index());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
jfloat fieldDescriptor::float_initial_value() const {
10546
e79347eebbc5 7086585: make Java field injection more flexible
never
parents: 7397
diff changeset
    90
  return constants()->float_at(initial_value_index());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
jdouble fieldDescriptor::double_initial_value() const {
10546
e79347eebbc5 7086585: make Java field injection more flexible
never
parents: 7397
diff changeset
    94
  return constants()->double_at(initial_value_index());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
oop fieldDescriptor::string_initial_value(TRAPS) const {
27680
8ecc0871c18e 8064811: Use THREAD instead of CHECK_NULL in return statements
stefank
parents: 20017
diff changeset
    98
  return constants()->uncached_string_at(initial_value_index(), THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 15601
diff changeset
   101
void fieldDescriptor::reinitialize(InstanceKlass* ik, int index) {
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 15601
diff changeset
   102
  if (_cp.is_null() || field_holder() != ik) {
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 15601
diff changeset
   103
    _cp = constantPoolHandle(Thread::current(), ik->constants());
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 15601
diff changeset
   104
    // _cp should now reference ik's constant pool; i.e., ik is now field_holder.
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 15601
diff changeset
   105
    assert(field_holder() == ik, "must be already initialized to this class");
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 15601
diff changeset
   106
  }
10546
e79347eebbc5 7086585: make Java field injection more flexible
never
parents: 7397
diff changeset
   107
  FieldInfo* f = ik->field(index);
e79347eebbc5 7086585: make Java field injection more flexible
never
parents: 7397
diff changeset
   108
  assert(!f->is_internal(), "regular Java fields only");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
10546
e79347eebbc5 7086585: make Java field injection more flexible
never
parents: 7397
diff changeset
   110
  _access_flags = accessFlags_from(f->access_flags());
e79347eebbc5 7086585: make Java field injection more flexible
never
parents: 7397
diff changeset
   111
  guarantee(f->name_index() != 0 && f->signature_index() != 0, "bad constant pool index for fieldDescriptor");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
  _index = index;
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 15601
diff changeset
   113
  verify();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 15601
diff changeset
   118
void fieldDescriptor::verify() const {
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 15601
diff changeset
   119
  if (_cp.is_null()) {
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 15601
diff changeset
   120
    assert(_index == badInt, "constructor must be called");  // see constructor
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 15601
diff changeset
   121
  } else {
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 15601
diff changeset
   122
    assert(_index >= 0, "good index");
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 15601
diff changeset
   123
    assert(_index < field_holder()->java_fields_count(), "oob");
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 15601
diff changeset
   124
  }
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 15601
diff changeset
   125
}
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 15601
diff changeset
   126
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
void fieldDescriptor::print_on(outputStream* st) const {
10546
e79347eebbc5 7086585: make Java field injection more flexible
never
parents: 7397
diff changeset
   128
  access_flags().print_on(st);
e79347eebbc5 7086585: make Java field injection more flexible
never
parents: 7397
diff changeset
   129
  name()->print_value_on(st);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
  st->print(" ");
10546
e79347eebbc5 7086585: make Java field injection more flexible
never
parents: 7397
diff changeset
   131
  signature()->print_value_on(st);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
  st->print(" @%d ", offset());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
  if (WizardMode && has_initial_value()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
    st->print("(initval ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
    constantTag t = initial_value_tag();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
    if (t.is_int()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
      st->print("int %d)", int_initial_value());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
    } else if (t.is_long()){
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
      st->print_jlong(long_initial_value());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
    } else if (t.is_float()){
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
      st->print("float %f)", float_initial_value());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
    } else if (t.is_double()){
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
      st->print("double %lf)", double_initial_value());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
54807
33fe50b6d707 8223626: move print() functions to cpp files
coleenp
parents: 54786
diff changeset
   148
void fieldDescriptor::print() const { print_on(tty); }
33fe50b6d707 8223626: move print() functions to cpp files
coleenp
parents: 54786
diff changeset
   149
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
void fieldDescriptor::print_on_for(outputStream* st, oop obj) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   151
  print_on(st);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   152
  BasicType ft = field_type();
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 1
diff changeset
   153
  jint as_int = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
  switch (ft) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
    case T_BYTE:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   156
      as_int = (jint)obj->byte_field(offset());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   157
      st->print(" %d", obj->byte_field(offset()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   158
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   159
    case T_CHAR:
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 1
diff changeset
   160
      as_int = (jint)obj->char_field(offset());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   161
      {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   162
        jchar c = obj->char_field(offset());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
        as_int = c;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   164
        st->print(" %c %d", isprint(c) ? c : ' ', c);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
    case T_DOUBLE:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
      st->print(" %lf", obj->double_field(offset()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   169
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   170
    case T_FLOAT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
      as_int = obj->int_field(offset());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
      st->print(" %f", obj->float_field(offset()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
    case T_INT:
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 1
diff changeset
   175
      as_int = obj->int_field(offset());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   176
      st->print(" %d", obj->int_field(offset()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   177
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
    case T_LONG:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   179
      st->print(" ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   180
      st->print_jlong(obj->long_field(offset()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
    case T_SHORT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   183
      as_int = obj->short_field(offset());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   184
      st->print(" %d", obj->short_field(offset()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   185
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   186
    case T_BOOLEAN:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   187
      as_int = obj->bool_field(offset());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   188
      st->print(" %s", obj->bool_field(offset()) ? "true" : "false");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   190
    case T_ARRAY:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   191
      st->print(" ");
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 1
diff changeset
   192
      NOT_LP64(as_int = obj->int_field(offset()));
51268
4ac20e5f96ce 8202171: Some oopDesc functions compare this with NULL
hseigel
parents: 49359
diff changeset
   193
      if (obj->obj_field(offset()) != NULL) {
4ac20e5f96ce 8202171: Some oopDesc functions compare this with NULL
hseigel
parents: 49359
diff changeset
   194
        obj->obj_field(offset())->print_value_on(st);
4ac20e5f96ce 8202171: Some oopDesc functions compare this with NULL
hseigel
parents: 49359
diff changeset
   195
      } else {
4ac20e5f96ce 8202171: Some oopDesc functions compare this with NULL
hseigel
parents: 49359
diff changeset
   196
        st->print_cr("NULL");
4ac20e5f96ce 8202171: Some oopDesc functions compare this with NULL
hseigel
parents: 49359
diff changeset
   197
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
    case T_OBJECT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
      st->print(" ");
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 1
diff changeset
   201
      NOT_LP64(as_int = obj->int_field(offset()));
51268
4ac20e5f96ce 8202171: Some oopDesc functions compare this with NULL
hseigel
parents: 49359
diff changeset
   202
      if (obj->obj_field(offset()) != NULL) {
4ac20e5f96ce 8202171: Some oopDesc functions compare this with NULL
hseigel
parents: 49359
diff changeset
   203
        obj->obj_field(offset())->print_value_on(st);
4ac20e5f96ce 8202171: Some oopDesc functions compare this with NULL
hseigel
parents: 49359
diff changeset
   204
      } else {
4ac20e5f96ce 8202171: Some oopDesc functions compare this with NULL
hseigel
parents: 49359
diff changeset
   205
        st->print_cr("NULL");
4ac20e5f96ce 8202171: Some oopDesc functions compare this with NULL
hseigel
parents: 49359
diff changeset
   206
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
    default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
      ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   210
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
  // Print a hint as to the underlying integer representation. This can be wrong for
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
  // pointers on an LP64 machine
48972
e9b007212236 8197857: fieldDescriptor prints incorrect 32-bit representation of compressed oops
iklam
parents: 47216
diff changeset
   214
#ifdef _LP64
58273
08a5148e7c4e 8230505: Replace JVM type comparisons to T_OBJECT and T_ARRAY with call to is_reference_type
lfoltan
parents: 54807
diff changeset
   215
  if (is_reference_type(ft) && UseCompressedOops) {
48972
e9b007212236 8197857: fieldDescriptor prints incorrect 32-bit representation of compressed oops
iklam
parents: 47216
diff changeset
   216
    st->print(" (%x)", obj->int_field(offset()));
e9b007212236 8197857: fieldDescriptor prints incorrect 32-bit representation of compressed oops
iklam
parents: 47216
diff changeset
   217
  }
e9b007212236 8197857: fieldDescriptor prints incorrect 32-bit representation of compressed oops
iklam
parents: 47216
diff changeset
   218
  else // <- intended
e9b007212236 8197857: fieldDescriptor prints incorrect 32-bit representation of compressed oops
iklam
parents: 47216
diff changeset
   219
#endif
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 1
diff changeset
   220
  if (ft == T_LONG || ft == T_DOUBLE LP64_ONLY(|| !is_java_primitive(ft)) ) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
    st->print(" (%x %x)", obj->int_field(offset()), obj->int_field(offset()+sizeof(jint)));
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 1
diff changeset
   222
  } else if (as_int < 0 || as_int > 9) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   223
    st->print(" (%x)", as_int);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   224
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   225
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   226
489c9b5090e2 Initial load
duke
parents:
diff changeset
   227
#endif /* PRODUCT */