[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