[llvm] [llvm][unittests][NFC] Fix warning from -Wsign-compare (PR #68613)

Paul Kirth via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 9 10:27:46 PDT 2023


https://github.com/ilovepi created https://github.com/llvm/llvm-project/pull/68613

None

>From bd12a6862941054858d57154e7c1bd41ef0d29fc Mon Sep 17 00:00:00 2001
From: Paul Kirth <paulkirth at google.com>
Date: Mon, 9 Oct 2023 10:22:27 -0700
Subject: [PATCH] [llvm][unittests][NFC] Fix warning from -Wsign-compare

---
 llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp b/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp
index c56b11d3c5faeab..9be5b1fc521c2d5 100644
--- a/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp
+++ b/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp
@@ -5547,7 +5547,7 @@ TEST_F(OpenMPIRBuilderTest, CreateTask) {
   // Verify that the data argument is used only once, and that too in the load
   // instruction that is then used for accessing shared data.
   Value *DataPtr = OutlinedFn->getArg(1);
-  EXPECT_EQ(DataPtr->getNumUses(), 1);
+  EXPECT_EQ(DataPtr->getNumUses(), 1U);
   EXPECT_TRUE(isa<LoadInst>(DataPtr->uses().begin()->getUser()));
   Value *Data = DataPtr->uses().begin()->getUser();
   EXPECT_TRUE(all_of(Data->uses(), [](Use &U) {
@@ -5604,7 +5604,7 @@ TEST_F(OpenMPIRBuilderTest, CreateTaskNoArgs) {
           ->user_back());
   Function *OutlinedFn = dyn_cast<Function>(TaskAllocCall->getArgOperand(5));
   ASSERT_NE(OutlinedFn, nullptr);
-  ASSERT_EQ(OutlinedFn->arg_size(), 1);
+  ASSERT_EQ(OutlinedFn->arg_size(), 1UL);
 }
 
 TEST_F(OpenMPIRBuilderTest, CreateTaskUntied) {



More information about the llvm-commits mailing list