diff options
author | David Blaikie <dblaikie@gmail.com> | 2021-10-18 19:20:21 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2021-10-18 19:20:21 +0000 |
commit | 1f850056d361c8529329041eeb3fc1b50290366c (patch) | |
tree | 92f9ef0bf5641c49dd630f0434f1c3924fa12b09 | |
parent | 406cc10c0b7743d2fe1f1f1e2383359150a87eab (diff) | |
parent | 1cb5745a89a7c3970e3827636867b73c93438ce5 (diff) | |
download | gwp_asan-1f850056d361c8529329041eeb3fc1b50290366c.tar.gz |
Fix a few warnings (signed/unsigned comparison in gtest, and missing field initializers) am: 6841f6e058 am: 6b63e40153 am: 1cb5745a89
Original change: https://android-review.googlesource.com/c/platform/external/gwp_asan/+/1860473
Change-Id: If4fe32be627206b37f1b0b56227bd21f3cfc7922
-rw-r--r-- | gwp_asan/tests/alignment.cpp | 42 |
1 files changed, 21 insertions, 21 deletions
diff --git a/gwp_asan/tests/alignment.cpp b/gwp_asan/tests/alignment.cpp index 5f24a9a..6d1e912 100644 --- a/gwp_asan/tests/alignment.cpp +++ b/gwp_asan/tests/alignment.cpp @@ -34,81 +34,81 @@ public: // numerics of the testing. TEST(AlignmentTest, LeftAlignedAllocs) { // Alignment < Page Size. - EXPECT_EQ(0x4000, AlignmentTestGPA::alignUp( + EXPECT_EQ(0x4000u, AlignmentTestGPA::alignUp( /* Ptr */ 0x4000, /* Alignment */ 0x1)); // Alignment == Page Size. - EXPECT_EQ(0x4000, AlignmentTestGPA::alignUp( + EXPECT_EQ(0x4000u, AlignmentTestGPA::alignUp( /* Ptr */ 0x4000, /* Alignment */ 0x1000)); // Alignment > Page Size. - EXPECT_EQ(0x4000, AlignmentTestGPA::alignUp( + EXPECT_EQ(0x4000u, AlignmentTestGPA::alignUp( /* Ptr */ 0x4000, /* Alignment */ 0x4000)); } TEST(AlignmentTest, SingleByteAllocs) { // Alignment < Page Size. - EXPECT_EQ(0x1, + EXPECT_EQ(0x1u, AlignmentTestGPA::getRequiredBackingSize( /* Size */ 0x1, /* Alignment */ 0x1, /* PageSize */ 0x1000)); - EXPECT_EQ(0x7fff, AlignmentTestGPA::alignDown( + EXPECT_EQ(0x7fffu, AlignmentTestGPA::alignDown( /* Ptr */ 0x8000 - 0x1, /* Alignment */ 0x1)); // Alignment == Page Size. - EXPECT_EQ(0x1, + EXPECT_EQ(0x1u, AlignmentTestGPA::getRequiredBackingSize( /* Size */ 0x1, /* Alignment */ 0x1000, /* PageSize */ 0x1000)); - EXPECT_EQ(0x7000, AlignmentTestGPA::alignDown( + EXPECT_EQ(0x7000u, AlignmentTestGPA::alignDown( /* Ptr */ 0x8000 - 0x1, /* Alignment */ 0x1000)); // Alignment > Page Size. - EXPECT_EQ(0x3001, + EXPECT_EQ(0x3001u, AlignmentTestGPA::getRequiredBackingSize( /* Size */ 0x1, /* Alignment */ 0x4000, /* PageSize */ 0x1000)); - EXPECT_EQ(0x4000, AlignmentTestGPA::alignDown( + EXPECT_EQ(0x4000u, AlignmentTestGPA::alignDown( /* Ptr */ 0x8000 - 0x1, /* Alignment */ 0x4000)); } TEST(AlignmentTest, PageSizedAllocs) { // Alignment < Page Size. - EXPECT_EQ(0x1000, + EXPECT_EQ(0x1000u, AlignmentTestGPA::getRequiredBackingSize( /* Size */ 0x1000, /* Alignment */ 0x1, /* PageSize */ 0x1000)); - EXPECT_EQ(0x7000, AlignmentTestGPA::alignDown( + EXPECT_EQ(0x7000u, AlignmentTestGPA::alignDown( /* Ptr */ 0x8000 - 0x1000, /* Alignment */ 0x1)); // Alignment == Page Size. - EXPECT_EQ(0x1000, AlignmentTestGPA::getRequiredBackingSize( + EXPECT_EQ(0x1000u, AlignmentTestGPA::getRequiredBackingSize( /* Size */ 0x1000, /* Alignment */ 0x1000, /* PageSize */ 0x1000)); - EXPECT_EQ(0x7000, AlignmentTestGPA::alignDown( + EXPECT_EQ(0x7000u, AlignmentTestGPA::alignDown( /* Ptr */ 0x8000 - 0x1000, /* Alignment */ 0x1000)); // Alignment > Page Size. - EXPECT_EQ(0x4000, AlignmentTestGPA::getRequiredBackingSize( + EXPECT_EQ(0x4000u, AlignmentTestGPA::getRequiredBackingSize( /* Size */ 0x1000, /* Alignment */ 0x4000, /* PageSize */ 0x1000)); - EXPECT_EQ(0x4000, AlignmentTestGPA::alignDown( + EXPECT_EQ(0x4000u, AlignmentTestGPA::alignDown( /* Ptr */ 0x8000 - 0x1000, /* Alignment */ 0x4000)); } TEST(AlignmentTest, MoreThanPageAllocs) { // Alignment < Page Size. - EXPECT_EQ(0x2fff, + EXPECT_EQ(0x2fffu, AlignmentTestGPA::getRequiredBackingSize( /* Size */ 0x2fff, /* Alignment */ 0x1, /* PageSize */ 0x1000)); - EXPECT_EQ(0x5001, AlignmentTestGPA::alignDown( + EXPECT_EQ(0x5001u, AlignmentTestGPA::alignDown( /* Ptr */ 0x8000 - 0x2fff, /* Alignment */ 0x1)); // Alignment == Page Size. - EXPECT_EQ(0x2fff, AlignmentTestGPA::getRequiredBackingSize( + EXPECT_EQ(0x2fffu, AlignmentTestGPA::getRequiredBackingSize( /* Size */ 0x2fff, /* Alignment */ 0x1000, /* PageSize */ 0x1000)); - EXPECT_EQ(0x5000, AlignmentTestGPA::alignDown( + EXPECT_EQ(0x5000u, AlignmentTestGPA::alignDown( /* Ptr */ 0x8000 - 0x2fff, /* Alignment */ 0x1000)); // Alignment > Page Size. - EXPECT_EQ(0x5fff, AlignmentTestGPA::getRequiredBackingSize( + EXPECT_EQ(0x5fffu, AlignmentTestGPA::getRequiredBackingSize( /* Size */ 0x2fff, /* Alignment */ 0x4000, /* PageSize */ 0x1000)); - EXPECT_EQ(0x4000, AlignmentTestGPA::alignDown( + EXPECT_EQ(0x4000u, AlignmentTestGPA::alignDown( /* Ptr */ 0x8000 - 0x2fff, /* Alignment */ 0x4000)); } |