src/hotspot/share/oops/valueKlass.hpp

@@ -141,11 +141,11 @@
 
   void set_alignment(int alignment) {
     *(int*)adr_alignment() = alignment;
   }
 
-  int get_first_field_offset() const {
+  int first_field_offset() const {
     int offset = *(int*)adr_first_field_offset();
     assert(offset != 0, "Must be initialized before use");
     return *(int*)adr_first_field_offset();
   }
 

@@ -159,16 +159,17 @@
 
   void set_exact_size_in_bytes(int exact_size) {
     *(int*)adr_exact_size_in_bytes() = exact_size;
   }
 
+  int first_field_offset_old();
+
  private:
-  int collect_fields(GrowableArray<SigEntry>* sig, int base_off = 0) const;
+  int collect_fields(GrowableArray<SigEntry>* sig, int base_off = 0);
 
   void cleanup_blobs();
 
-  int first_field_offset_old() const;
 
  protected:
   // Returns the array class for the n'th dimension
   Klass* array_klass_impl(ArrayStorageProperties storage_props, bool or_null, int n, TRAPS);
 

@@ -204,20 +205,11 @@
 
   // allocate_instance() allocates a stand alone value in the Java heap
   instanceOop allocate_instance(TRAPS);
 
   // minimum number of bytes occupied by nonstatic fields, HeapWord aligned or pow2
-  int raw_value_byte_size() const;
-
-  int first_field_offset() const {
-    // To be refactored/simplified once old layout code and UseNewLayout are removed
-    if (UseNewLayout) {
-      return get_first_field_offset();
-    } else {
-      return first_field_offset_old();
-    }
-  };
+  int raw_value_byte_size();
 
   address data_for_oop(oop o) const {
     return ((address) (void*) o) + first_field_offset();
   }