[DebugInfo] Define base function on DWARFDie reverse iterators
authorJonas Devlieghere <jonas@devlieghere.com>
Tue, 30 Oct 2018 18:25:28 +0000 (18:25 +0000)
committerJonas Devlieghere <jonas@devlieghere.com>
Tue, 30 Oct 2018 18:25:28 +0000 (18:25 +0000)
This defines member function base on the specialization of
std::reverse_iterator for DWARFDie::iterator as required by C++
[reverse.iter.conv].

This fixes unit test DWARFDebugInfoTest.cpp under EXPENSIVE_CHECKS which
currently can't be built due to GNU C++ Library calling this member
function in debug mode.

This fixes https://llvm.org/PR38785

Patch by: Eugene Sharygin

Differential revision: https://reviews.llvm.org/D53792

llvm-svn: 345621

llvm/include/llvm/DebugInfo/DWARF/DWARFDie.h
llvm/unittests/DebugInfo/DWARF/DWARFDebugInfoTest.cpp

index c77034f..baa47c2 100644 (file)
@@ -404,6 +404,10 @@ public:
       Die = Die.getPreviousSibling();
   }
 
+  llvm::DWARFDie::iterator base() const {
+    return llvm::DWARFDie::iterator(AtEnd ? Die : Die.getSibling());
+  }
+
   reverse_iterator<llvm::DWARFDie::iterator> &operator++() {
     assert(!AtEnd && "Incrementing rend");
     llvm::DWARFDie D = Die.getPreviousSibling();
index 1be0363..ffbde2d 100644 (file)
@@ -1227,6 +1227,10 @@ TEST(DWARFDebugInfo, TestRelations) {
   EXPECT_THAT(std::vector<DWARFDie>(A.rbegin(), A.rend()),
               testing::ElementsAre(D, C, B));
 
+  // Make sure conversion from reverse iterator works as expected.
+  EXPECT_EQ(A.rbegin().base(), A.end());
+  EXPECT_EQ(A.rend().base(), A.begin());
+
   // Make sure iterator is bidirectional.
   {
     auto Begin = A.begin();