Skip to content

Commit bdd5fa0

Browse files
ellertaaronj0
authored andcommitted
[test] Improve struct size check for 32-bit and MSVC double alignment
1 parent 180869d commit bdd5fa0

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

unittests/CppInterOp/ScopeReflectionTest.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,8 @@ TEST(ScopeReflectionTest, SizeOf) {
158158
EXPECT_EQ(Cpp::SizeOf(Decls[4]), (size_t)0);
159159
EXPECT_EQ(Cpp::SizeOf(Decls[5]), (size_t)1);
160160
EXPECT_EQ(Cpp::SizeOf(Decls[6]), (size_t)4);
161-
EXPECT_EQ(Cpp::SizeOf(Decls[7]), (size_t)16);
161+
struct B {short a; double b;};
162+
EXPECT_EQ(Cpp::SizeOf(Decls[7]), sizeof(B));
162163
}
163164

164165

unittests/CppInterOp/TypeReflectionTest.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,8 @@ TEST(TypeReflectionTest, GetSizeOfType) {
7878
EXPECT_EQ(Cpp::GetSizeOfType(Cpp::GetVariableType(Decls[1])), 1);
7979
EXPECT_EQ(Cpp::GetSizeOfType(Cpp::GetVariableType(Decls[2])), 4);
8080
EXPECT_EQ(Cpp::GetSizeOfType(Cpp::GetVariableType(Decls[3])), 8);
81-
EXPECT_EQ(Cpp::GetSizeOfType(Cpp::GetVariableType(Decls[4])), 16);
81+
struct B {int a; double b;};
82+
EXPECT_EQ(Cpp::GetSizeOfType(Cpp::GetVariableType(Decls[4])), sizeof(B));
8283
EXPECT_EQ(Cpp::GetSizeOfType(Cpp::GetTypeFromScope(Decls[5])), 0);
8384
EXPECT_EQ(Cpp::GetSizeOfType(Cpp::GetVariableType(Decls[6])),
8485
sizeof(intptr_t));

0 commit comments

Comments
 (0)