[llvm] aa37342 - Reapply: Fix warning: comparison of integers of different signs

via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 8 07:19:24 PST 2022


Author: OCHyams
Date: 2022-11-08T15:18:38Z
New Revision: aa37342b3b548a75a4d8ce330f58a361a9ce22f4

URL: https://github.com/llvm/llvm-project/commit/aa37342b3b548a75a4d8ce330f58a361a9ce22f4
DIFF: https://github.com/llvm/llvm-project/commit/aa37342b3b548a75a4d8ce330f58a361a9ce22f4.diff

LOG: Reapply: Fix warning: comparison of integers of different signs

Buildbot failure:
https://lab.llvm.org/buildbot/#/builders/36/builds/26925

Review & commit:
https://reviews.llvm.org/D132224
https://reviews.llvm.org/rG171f7024cc82e8702abebdedb699d37b50574be7

Added: 
    

Modified: 
    llvm/unittests/IR/DebugInfoTest.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/unittests/IR/DebugInfoTest.cpp b/llvm/unittests/IR/DebugInfoTest.cpp
index b7570df80a71..e58b4f562e59 100644
--- a/llvm/unittests/IR/DebugInfoTest.cpp
+++ b/llvm/unittests/IR/DebugInfoTest.cpp
@@ -506,7 +506,7 @@ TEST(AssignmentTrackingTest, Utils) {
   // 4. Check that deleting works and applies only to the target function.
   at::deleteAll(&Fun1);
   // There should now only be the alloca and ret in fun1.
-  EXPECT_EQ(Fun1.begin()->size(), 2);
+  EXPECT_EQ(Fun1.begin()->size(), 2u);
   // fun2's alloca should have the same DIAssignID and remain linked to its
   // llvm.dbg.assign.
   EXPECT_EQ(Fun2ID, cast_or_null<DIAssignID>(


        


More information about the llvm-commits mailing list