[llvm] 035dd1d - [ADT] Fix a warning

Kazu Hirata via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 21 08:50:01 PDT 2025


Author: Kazu Hirata
Date: 2025-08-21T08:49:56-07:00
New Revision: 035dd1d85427f0d7701a6faf3e72681a7eddce5c

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

LOG: [ADT] Fix a warning

This patch fixes:

  third-party/unittest/googletest/include/gtest/gtest.h:1379:11:
  error: comparison of integers of different signs: 'const unsigned
  long' and 'const int' [-Werror,-Wsign-compare]

Added: 
    

Modified: 
    llvm/unittests/ADT/APIntTest.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/unittests/ADT/APIntTest.cpp b/llvm/unittests/ADT/APIntTest.cpp
index 5b010729ae3da..116693c873f30 100644
--- a/llvm/unittests/ADT/APIntTest.cpp
+++ b/llvm/unittests/ADT/APIntTest.cpp
@@ -3747,18 +3747,19 @@ TEST(APIntTest, TryExt) {
 TEST(APIntTest, Fshl) {
   EXPECT_EQ(
       APIntOps::fshl(APInt(8, 0), APInt(8, 255), APInt(8, 8)).getZExtValue(),
-      0);
+      0U);
   EXPECT_EQ(
       APIntOps::fshl(APInt(8, 255), APInt(8, 0), APInt(8, 8)).getZExtValue(),
-      255);
+      255U);
   EXPECT_EQ(
       APIntOps::fshl(APInt(8, 255), APInt(8, 0), APInt(8, 15)).getZExtValue(),
-      128);
+      128U);
   EXPECT_EQ(
       APIntOps::fshl(APInt(8, 15), APInt(8, 15), APInt(8, 11)).getZExtValue(),
-      120);
+      120U);
   EXPECT_EQ(
-      APIntOps::fshl(APInt(8, 2), APInt(8, 1), APInt(8, 3)).getZExtValue(), 16);
+      APIntOps::fshl(APInt(8, 2), APInt(8, 1), APInt(8, 3)).getZExtValue(),
+      16U);
   EXPECT_EQ(
       APIntOps::fshl(APInt(8, 2), APInt(8, 1), APInt(8, 1)).getZExtValue(),
       APIntOps::fshl(APInt(8, 2), APInt(8, 1), APInt(8, 9)).getZExtValue());
@@ -3785,18 +3786,19 @@ TEST(APIntTest, Fshl) {
 TEST(APIntTest, Fshr) {
   EXPECT_EQ(
       APIntOps::fshr(APInt(8, 0), APInt(8, 255), APInt(8, 8)).getZExtValue(),
-      255);
+      255U);
   EXPECT_EQ(
       APIntOps::fshr(APInt(8, 255), APInt(8, 0), APInt(8, 8)).getZExtValue(),
-      0);
+      0U);
   EXPECT_EQ(
       APIntOps::fshr(APInt(8, 255), APInt(8, 0), APInt(8, 15)).getZExtValue(),
-      254);
+      254U);
   EXPECT_EQ(
       APIntOps::fshr(APInt(8, 15), APInt(8, 15), APInt(8, 11)).getZExtValue(),
-      225);
+      225U);
   EXPECT_EQ(
-      APIntOps::fshr(APInt(8, 1), APInt(8, 2), APInt(8, 3)).getZExtValue(), 32);
+      APIntOps::fshr(APInt(8, 1), APInt(8, 2), APInt(8, 3)).getZExtValue(),
+      32U);
   EXPECT_EQ(
       APIntOps::fshr(APInt(8, 1), APInt(8, 2), APInt(8, 1)).getZExtValue(),
       APIntOps::fshr(APInt(8, 1), APInt(8, 2), APInt(8, 9)).getZExtValue());


        


More information about the llvm-commits mailing list