commit ae4631e1003f16edaa92a44ffae54a739275c129
author Daco Harkes <dacoharkes@google.com> Wed Feb 03 10:21:08 2021 +0000
committer commit-bot@chromium.org <commit-bot@chromium.org> Wed Feb 03 10:21:08 2021 +0000
tree f1c64c1e48370148d0a0c78b57dab1f29e2b7750
parent f306f4c4d41c8f2af2a3f3f30880703d43a8c5af

[vm/ffi] Fix test `sizeOf` struct on arm32

Bug introduced in https://dart-review.googlesource.com/c/sdk/+/182262.

See documentation on alignment in the following places:
- pkg/vm/lib/transformations/ffi.dart
- runtime/vm/compiler/ffi/abi.cc
- runtime/vm/compiler/ffi/native_type.cc

Closes: https://github.com/dart-lang/sdk/issues/44835

Change-Id: I114dd4286e559bb230e1ee27ae5cfd8a0b0d2927
Cq-Include-Trybots: luci.dart.try:vm-precomp-ffi-qemu-linux-release-arm-try,vm-ffi-android-debug-arm-try
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/182501
Commit-Queue: Daco Harkes <dacoharkes@google.com>
Reviewed-by: Clement Skau <cskau@google.com>

tests/ffi/sizeof_test.dart

diff --git a/tests/ffi/sizeof_test.dart b/tests/ffi/sizeof_test.dart
index 4fd42fd..35cb36c 100644
--- a/tests/ffi/sizeof_test.dart
+++ b/tests/ffi/sizeof_test.dart
@@ -16,9 +16,10 @@
 void main() async {
   if (is32Bit) {
     Expect.equals(4, sizeOf<Pointer>());
-    Expect.equals(20, sizeOf<Coordinate>());
-  }
-  if (is64Bit) {
+    // Struct is 20 bytes on ia32 and arm32-iOS, but 24 bytes on arm32-Android
+    // and arm32-Linux due to alignment.
+    Expect.isTrue(20 == sizeOf<Coordinate>() || 24 == sizeOf<Coordinate>());
+  } else if (is64Bit) {
     Expect.equals(8, sizeOf<Pointer>());
     Expect.equals(24, sizeOf<Coordinate>());
   }

tests/ffi_2/sizeof_test.dart

diff --git a/tests/ffi_2/sizeof_test.dart b/tests/ffi_2/sizeof_test.dart
index 4fd42fd..35cb36c 100644
--- a/tests/ffi_2/sizeof_test.dart
+++ b/tests/ffi_2/sizeof_test.dart
@@ -16,9 +16,10 @@
 void main() async {
   if (is32Bit) {
     Expect.equals(4, sizeOf<Pointer>());
-    Expect.equals(20, sizeOf<Coordinate>());
-  }
-  if (is64Bit) {
+    // Struct is 20 bytes on ia32 and arm32-iOS, but 24 bytes on arm32-Android
+    // and arm32-Linux due to alignment.
+    Expect.isTrue(20 == sizeOf<Coordinate>() || 24 == sizeOf<Coordinate>());
+  } else if (is64Bit) {
     Expect.equals(8, sizeOf<Pointer>());
     Expect.equals(24, sizeOf<Coordinate>());
   }