Fix -Wsign-compare in OperationSupportTest.cpp
authorAlex Zinenko <zinenko@google.com>
Tue, 21 May 2019 14:07:17 +0000 (07:07 -0700)
committerMehdi Amini <joker.eph@gmail.com>
Sun, 2 Jun 2019 02:54:13 +0000 (19:54 -0700)
    A couple of warnings was produced when compiling this test due to comaprisons
    with a signed literal.  Used unsigned literals instead.

--

PiperOrigin-RevId: 249242970

mlir/unittests/IR/OperationSupportTest.cpp

index 801b9f5..cb0801e 100644 (file)
@@ -50,11 +50,11 @@ TEST(OperandStorageTest, NonResizable) {
 
   // The same number of operands is okay.
   user->setOperands(operand);
-  EXPECT_EQ(user->getNumOperands(), 1);
+  EXPECT_EQ(user->getNumOperands(), 1u);
 
   // Removing is okay.
   user->setOperands(llvm::None);
-  EXPECT_EQ(user->getNumOperands(), 0);
+  EXPECT_EQ(user->getNumOperands(), 0u);
 
   // Destroy the operations.
   user->destroy();
@@ -99,15 +99,15 @@ TEST(OperandStorageTest, Resizable) {
 
   // The same number of operands is okay.
   user->setOperands(operand);
-  EXPECT_EQ(user->getNumOperands(), 1);
+  EXPECT_EQ(user->getNumOperands(), 1u);
 
   // Removing is okay.
   user->setOperands(llvm::None);
-  EXPECT_EQ(user->getNumOperands(), 0);
+  EXPECT_EQ(user->getNumOperands(), 0u);
 
   // Adding more operands is okay.
   user->setOperands({operand, operand, operand});
-  EXPECT_EQ(user->getNumOperands(), 3);
+  EXPECT_EQ(user->getNumOperands(), 3u);
 
   // Destroy the operations.
   user->destroy();