[llvm] a55dec7 - [test][DebugInfo] Fix signed/unsigned comparison problem in test

James Henderson via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 14 05:41:09 PST 2020


Author: James Henderson
Date: 2020-02-14T13:40:44Z
New Revision: a55dec7d64ef128a03deae7dd9bc1ed97cd123b4

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

LOG: [test][DebugInfo] Fix signed/unsigned comparison problem in test

This caused build bot failures:
http://lab.llvm.org:8011/builders/ppc64le-lld-multistage-test/builds/8568/

Added: 
    

Modified: 
    llvm/unittests/DebugInfo/DWARF/DWARFDebugLineTest.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/unittests/DebugInfo/DWARF/DWARFDebugLineTest.cpp b/llvm/unittests/DebugInfo/DWARF/DWARFDebugLineTest.cpp
index 0552b5ad6f7b..a622c84c1f4d 100644
--- a/llvm/unittests/DebugInfo/DWARF/DWARFDebugLineTest.cpp
+++ b/llvm/unittests/DebugInfo/DWARF/DWARFDebugLineTest.cpp
@@ -670,7 +670,7 @@ TEST_F(DebugLineBasicFixture,
   ASSERT_EQ((*ExpectedLineTable)->Rows.size(), 3u);
   EXPECT_EQ((*ExpectedLineTable)->Sequences.size(), 1u);
   // Show that the set address opcode is ignored in this case.
-  EXPECT_EQ((*ExpectedLineTable)->Rows[0].Address.Address, 0);
+  EXPECT_EQ((*ExpectedLineTable)->Rows[0].Address.Address, 0u);
 }
 
 TEST_F(DebugLineBasicFixture, ErrorForAddressSizeGreaterThanByteSize) {
@@ -731,7 +731,7 @@ TEST_F(DebugLineBasicFixture, ErrorForUnsupportedAddressSizeDefinedInHeader) {
   ASSERT_EQ((*ExpectedLineTable)->Rows.size(), 3u);
   EXPECT_EQ((*ExpectedLineTable)->Sequences.size(), 1u);
   // Show that the set address opcode is ignored in this case.
-  EXPECT_EQ((*ExpectedLineTable)->Rows[0].Address.Address, 0);
+  EXPECT_EQ((*ExpectedLineTable)->Rows[0].Address.Address, 0u);
 }
 
 TEST_F(DebugLineBasicFixture, CallbackUsedForUnterminatedSequence) {


        


More information about the llvm-commits mailing list