[llvm] 004eb2c - GSYMTest::TestLineTable - check all LT1+LT2 comparisons.

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Sat Nov 2 13:13:15 PDT 2019


Author: Simon Pilgrim
Date: 2019-11-02T20:12:58Z
New Revision: 004eb2c8627d484cadb2a7268fdbfd35cddc986c

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

LOG: GSYMTest::TestLineTable - check all LT1+LT2 comparisons.

PVS Studio was warning about "LT2 < LT2" but really we should be testing all permutations of LT1 and LT2.

Added: 
    

Modified: 
    llvm/unittests/DebugInfo/GSYM/GSYMTest.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/unittests/DebugInfo/GSYM/GSYMTest.cpp b/llvm/unittests/DebugInfo/GSYM/GSYMTest.cpp
index 90d8594eec6e..9fd8edf6a107 100644
--- a/llvm/unittests/DebugInfo/GSYM/GSYMTest.cpp
+++ b/llvm/unittests/DebugInfo/GSYM/GSYMTest.cpp
@@ -853,7 +853,9 @@ TEST(GSYMTest, TestLineTable) {
   // Test that two empty line tables are equal and neither are less than
   // each other.
   ASSERT_EQ(LT1, LT2);
+  ASSERT_FALSE(LT1 < LT1);
   ASSERT_FALSE(LT1 < LT2);
+  ASSERT_FALSE(LT2 < LT1);
   ASSERT_FALSE(LT2 < LT2);
 
   // Test that a line table with less number of line entries is less than a


        


More information about the llvm-commits mailing list