Don't consider types with empty base classes blittable. They have a different managed...

* Don't consider types with empty base classes blittable. They have a different managed and native size.

* Add test.

* Update src/coreclr/vm/classlayoutinfo.cpp

diff --git a/src/coreclr/vm/classlayoutinfo.cpp b/src/coreclr/vm/classlayoutinfo.cpp
index e5f9345..bdef0e4 100644 (file)
--- a/src/coreclr/vm/classlayoutinfo.cpp
+++ b/src/coreclr/vm/classlayoutinfo.cpp
@@ -653,8 +653,12 @@ VOID EEClassLayoutInfo::CollectLayoutFieldMetadataThrowing(
         DEBUGARG(szName)
         );
 
-    // Type is blittable only if parent is also blittable
-    isBlittable = isBlittable && (fHasNonTrivialParent ? pParentMT->IsBlittable() : TRUE);
+    // Type is blittable only if parent is also blittable and is not empty.
+    if (isBlittable && fHasNonTrivialParent)
+    {
+        isBlittable = pParentMT->IsBlittable()  // Check parent
+            && (!pParentLayoutInfo || !pParentLayoutInfo->IsZeroSized()); // Ensure non-zero size
+    }
     pEEClassLayoutInfoOut->SetIsBlittable(isBlittable);
 
     S_UINT32 cbSortArraySize = S_UINT32(cTotalFields) * S_UINT32(sizeof(LayoutRawFieldInfo*));

diff --git a/src/libraries/System.Runtime.InteropServices/tests/System/Runtime/InteropServices/Marshal/SizeOfTests.cs b/src/libraries/System.Runtime.InteropServices/tests/System/Runtime/InteropServices/Marshal/SizeOfTests.cs
index 5d866c7..6a009a0 100644 (file)
--- a/src/libraries/System.Runtime.InteropServices/tests/System/Runtime/InteropServices/Marshal/SizeOfTests.cs
+++ b/src/libraries/System.Runtime.InteropServices/tests/System/Runtime/InteropServices/Marshal/SizeOfTests.cs
@@ -59,6 +59,12 @@ namespace System.Runtime.InteropServices.Tests
             Assert.Equal(8, Marshal.SizeOf<TestStructWithVector64>());
         }
 
+        [Fact]
+        public void SizeOf_TypeWithEmptyBase_ReturnsExpected()
+        {
+            Assert.Equal(4, Marshal.SizeOf<DerivedClass>());
+        }
+
         public static IEnumerable<object[]> SizeOf_InvalidType_TestData()
         {
             yield return new object[] { typeof(int).MakeByRefType(), null };
@@ -136,5 +142,16 @@ namespace System.Runtime.InteropServices.Tests
         {
             public System.Runtime.Intrinsics.Vector64<double> v;
         }
+
+        [StructLayout(LayoutKind.Sequential)]
+        public class EmptyClass
+        {
+        }
+
+        [StructLayout(LayoutKind.Sequential)]
+        public class DerivedClass : EmptyClass
+        {
+            public int i;
+        }
     }
 }