[llvm] 57fa508 - Fix build break for [CGData] Outlined Hash Tree (#89792)
Kyungwoo Lee via llvm-commits
llvm-commits at lists.llvm.org
Sat Jul 6 18:14:09 PDT 2024
Author: Kyungwoo Lee
Date: 2024-07-06T18:13:46-07:00
New Revision: 57fa5084036d10824f6f64e3bcc481db8fa73c24
URL: https://github.com/llvm/llvm-project/commit/57fa5084036d10824f6f64e3bcc481db8fa73c24
DIFF: https://github.com/llvm/llvm-project/commit/57fa5084036d10824f6f64e3bcc481db8fa73c24.diff
LOG: Fix build break for [CGData] Outlined Hash Tree (#89792)
Added:
Modified:
llvm/unittests/CodeGenData/OutlinedHashTreeTest.cpp
Removed:
################################################################################
diff --git a/llvm/unittests/CodeGenData/OutlinedHashTreeTest.cpp b/llvm/unittests/CodeGenData/OutlinedHashTreeTest.cpp
index 4ab5566cf817b..637ab3cd08c1c 100644
--- a/llvm/unittests/CodeGenData/OutlinedHashTreeTest.cpp
+++ b/llvm/unittests/CodeGenData/OutlinedHashTreeTest.cpp
@@ -18,19 +18,19 @@ TEST(OutlinedHashTreeTest, Empty) {
OutlinedHashTree HashTree;
EXPECT_TRUE(HashTree.empty());
// The header node is always present.
- EXPECT_EQ(HashTree.size(), 1);
- EXPECT_EQ(HashTree.depth(), 0);
+ EXPECT_EQ(HashTree.size(), 1u);
+ EXPECT_EQ(HashTree.depth(), 0u);
}
TEST(OutlinedHashTreeTest, Insert) {
OutlinedHashTree HashTree;
HashTree.insert({{1, 2, 3}, 1});
// The node count is 4 (including the root node).
- EXPECT_EQ(HashTree.size(), 4);
+ EXPECT_EQ(HashTree.size(), 4u);
// The terminal count is 1.
- EXPECT_EQ(HashTree.size(/*GetTerminalCountOnly=*/true), 1);
+ EXPECT_EQ(HashTree.size(/*GetTerminalCountOnly=*/true), 1u);
// The depth is 3.
- EXPECT_EQ(HashTree.depth(), 3);
+ EXPECT_EQ(HashTree.depth(), 3u);
HashTree.clear();
EXPECT_TRUE(HashTree.empty());
@@ -39,7 +39,7 @@ TEST(OutlinedHashTreeTest, Insert) {
HashTree.insert({{1, 2, 4}, 2});
// The nodes of 1 and 2 are shared with the same prefix.
// The nodes are root, 1, 2, 3 and 4, whose counts are 5.
- EXPECT_EQ(HashTree.size(), 5);
+ EXPECT_EQ(HashTree.size(), 5u);
}
TEST(OutlinedHashTreeTest, Find) {
@@ -48,10 +48,10 @@ TEST(OutlinedHashTreeTest, Find) {
HashTree.insert({{1, 2, 3}, 2});
// The node count does not change as the same sequences are added.
- EXPECT_EQ(HashTree.size(), 4);
+ EXPECT_EQ(HashTree.size(), 4u);
// The terminal counts are accumulated from two same sequences.
EXPECT_TRUE(HashTree.find({1, 2, 3}));
- EXPECT_EQ(HashTree.find({1, 2, 3}).value(), 3);
+ EXPECT_EQ(HashTree.find({1, 2, 3}).value(), 3u);
EXPECT_FALSE(HashTree.find({1, 2}));
}
More information about the llvm-commits
mailing list