Skip to content

Commit 57fa508

Browse files
committed
Fix build break for [CGData] Outlined Hash Tree (#89792)
1 parent 8a41327 commit 57fa508

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

llvm/unittests/CodeGenData/OutlinedHashTreeTest.cpp

+8-8
Original file line numberDiff line numberDiff line change
@@ -18,19 +18,19 @@ TEST(OutlinedHashTreeTest, Empty) {
1818
OutlinedHashTree HashTree;
1919
EXPECT_TRUE(HashTree.empty());
2020
// The header node is always present.
21-
EXPECT_EQ(HashTree.size(), 1);
22-
EXPECT_EQ(HashTree.depth(), 0);
21+
EXPECT_EQ(HashTree.size(), 1u);
22+
EXPECT_EQ(HashTree.depth(), 0u);
2323
}
2424

2525
TEST(OutlinedHashTreeTest, Insert) {
2626
OutlinedHashTree HashTree;
2727
HashTree.insert({{1, 2, 3}, 1});
2828
// The node count is 4 (including the root node).
29-
EXPECT_EQ(HashTree.size(), 4);
29+
EXPECT_EQ(HashTree.size(), 4u);
3030
// The terminal count is 1.
31-
EXPECT_EQ(HashTree.size(/*GetTerminalCountOnly=*/true), 1);
31+
EXPECT_EQ(HashTree.size(/*GetTerminalCountOnly=*/true), 1u);
3232
// The depth is 3.
33-
EXPECT_EQ(HashTree.depth(), 3);
33+
EXPECT_EQ(HashTree.depth(), 3u);
3434

3535
HashTree.clear();
3636
EXPECT_TRUE(HashTree.empty());
@@ -39,7 +39,7 @@ TEST(OutlinedHashTreeTest, Insert) {
3939
HashTree.insert({{1, 2, 4}, 2});
4040
// The nodes of 1 and 2 are shared with the same prefix.
4141
// The nodes are root, 1, 2, 3 and 4, whose counts are 5.
42-
EXPECT_EQ(HashTree.size(), 5);
42+
EXPECT_EQ(HashTree.size(), 5u);
4343
}
4444

4545
TEST(OutlinedHashTreeTest, Find) {
@@ -48,10 +48,10 @@ TEST(OutlinedHashTreeTest, Find) {
4848
HashTree.insert({{1, 2, 3}, 2});
4949

5050
// The node count does not change as the same sequences are added.
51-
EXPECT_EQ(HashTree.size(), 4);
51+
EXPECT_EQ(HashTree.size(), 4u);
5252
// The terminal counts are accumulated from two same sequences.
5353
EXPECT_TRUE(HashTree.find({1, 2, 3}));
54-
EXPECT_EQ(HashTree.find({1, 2, 3}).value(), 3);
54+
EXPECT_EQ(HashTree.find({1, 2, 3}).value(), 3u);
5555
EXPECT_FALSE(HashTree.find({1, 2}));
5656
}
5757

0 commit comments

Comments
 (0)