8355614: [lworld] SIGSEGV in InlineKlass::write_value_to_addr

Co-authored-by: Frederic Parain <[email protected]>

Author: TobiHartmann

src/hotspot/share/classfile/fieldLayoutBuilder.cpp

@@ -103,6 +103,7 @@ LayoutRawBlock::LayoutRawBlock(Kind kind, int size) :
   _prev_block(nullptr),
   _inline_klass(nullptr),
   _block_kind(kind),
+  _layout_kind(LayoutKind::UNKNOWN),
   _offset(-1),
   _alignment(1),
   _size(size),
@@ -118,6 +119,7 @@ LayoutRawBlock::LayoutRawBlock(int index, Kind kind, int size, int alignment) :
  _prev_block(nullptr),
  _inline_klass(nullptr),
  _block_kind(kind),
+ _layout_kind(LayoutKind::UNKNOWN),
  _offset(-1),
  _alignment(alignment),
  _size(size),

src/hotspot/share/classfile/fieldLayoutBuilder.hpp

@@ -77,7 +77,6 @@ class LayoutRawBlock : public ResourceObj {
   int _alignment;
   int _size;
   int _field_index;
-  int _null_marker_offset;
 
  public:
   LayoutRawBlock(Kind kind, int size);
@@ -110,8 +109,6 @@ class LayoutRawBlock : public ResourceObj {
     return _inline_klass;
   }
   void set_inline_klass(InlineKlass* inline_klass) { _inline_klass = inline_klass; }
-  void set_null_marker_offset(int offset) { _null_marker_offset = offset; }
-  int null_marker_offset() const { return _null_marker_offset; }
 
   LayoutKind layout_kind() const { return _layout_kind; }
   void set_layout_kind(LayoutKind kind) { _layout_kind = kind; }

src/hotspot/share/oops/resolvedFieldEntry.cpp

@@ -44,6 +44,7 @@ void ResolvedFieldEntry::print_on(outputStream* st) const {
   st->print_cr(" - Is Volatile: %d", is_volatile());
   st->print_cr(" - Is Flat: %d", is_flat());
   st->print_cr(" - Is Null Free Inline Type: %d", is_null_free_inline_type());
+  st->print_cr(" - Has null marker: %d", has_null_marker());
   st->print_cr(" - Get Bytecode: %s", Bytecodes::name((Bytecodes::Code)get_code()));
   st->print_cr(" - Put Bytecode: %s", Bytecodes::name((Bytecodes::Code)put_code()));
 }

test/hotspot/jtreg/compiler/valhalla/inlinetypes/TestValueConstruction.java

@@ -1171,23 +1171,23 @@ public MyValue12() {
         }
     }
 
-    static abstract value class MyAbsract13b {
-        MyAbsract13b() {
+    static abstract value class MyAbstract13b {
+        MyAbstract13b() {
             checkDeopt(4);
             super();
             checkDeopt(5);
         }
     }
 
-    static abstract value class MyAbsract13a extends MyAbsract13b {
-        MyAbsract13a() {
+    static abstract value class MyAbstract13a extends MyAbstract13b {
+        MyAbstract13a() {
             checkDeopt(2);
             super();
             checkDeopt(3);
         }
     }
 
-    static value class MyValue13 extends MyAbsract13a {
+    static value class MyValue13 extends MyAbstract13a {
         public MyValue13() {
             checkDeopt(0);
             super();