[Lldb-commits] [lldb] r336403 - Remove a bunch more references to _LIBCPP_INLINE_VISIBILITY

Jim Ingham via lldb-commits lldb-commits at lists.llvm.org
Thu Jul 5 17:16:21 PDT 2018


Author: jingham
Date: Thu Jul  5 17:16:21 2018
New Revision: 336403

URL: http://llvm.org/viewvc/llvm-project?rev=336403&view=rev
Log:
Remove a bunch more references to _LIBCPP_INLINE_VISIBILITY
and adjust the tests that needed it to set their breakpoints more robustly.

<rdar://problem/41867390>

Modified:
    lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/iterator/main.cpp
    lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/list/TestDataFormatterLibcxxList.py
    lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/list/loop/main.cpp
    lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/list/main.cpp
    lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/multiset/main.cpp
    lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/set/main.cpp
    lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/unordered/main.cpp
    lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/vbool/main.cpp

Modified: lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/iterator/main.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/iterator/main.cpp?rev=336403&r1=336402&r2=336403&view=diff
==============================================================================
--- lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/iterator/main.cpp (original)
+++ lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/iterator/main.cpp Thu Jul  5 17:16:21 2018
@@ -1,8 +1,4 @@
 #include <string>
-#ifdef _LIBCPP_INLINE_VISIBILITY
-#undef _LIBCPP_INLINE_VISIBILITY
-#endif
-#define _LIBCPP_INLINE_VISIBILITY
 #include <map>
 #include <vector>
 
@@ -39,4 +35,4 @@ int main()
 	svter svI = sv.begin();
 
 	return 0; // Set break point at this line.
-}
\ No newline at end of file
+}

Modified: lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/list/TestDataFormatterLibcxxList.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/list/TestDataFormatterLibcxxList.py?rev=336403&r1=336402&r2=336403&view=diff
==============================================================================
--- lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/list/TestDataFormatterLibcxxList.py (original)
+++ lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/list/TestDataFormatterLibcxxList.py Thu Jul  5 17:16:21 2018
@@ -88,8 +88,9 @@ class LibcxxListDataFormatterTestCase(Te
                     substrs=['list has 0 items',
                              '{}'])
 
-        self.runCmd("n")
-
+        self.runCmd("n") # This gets up past the printf
+        self.runCmd("n") # Now advance over the first push_back.
+        
         self.expect("frame variable numbers_list",
                     substrs=['list has 1 items',
                              '[0] = ',
@@ -187,6 +188,7 @@ class LibcxxListDataFormatterTestCase(Te
                              '\"is\"',
                              '\"smart\"'])
 
+        self.runCmd("n") # This gets us past the printf
         self.runCmd("n")
 
         # check access-by-index

Modified: lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/list/loop/main.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/list/loop/main.cpp?rev=336403&r1=336402&r2=336403&view=diff
==============================================================================
--- lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/list/loop/main.cpp (original)
+++ lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/list/loop/main.cpp Thu Jul  5 17:16:21 2018
@@ -3,12 +3,8 @@
 #define private public
 #define protected public
 
-#ifdef _LIBCPP_INLINE_VISIBILITY
-#undef _LIBCPP_INLINE_VISIBILITY
-#endif
-#define _LIBCPP_INLINE_VISIBILITY
 #include <list>
-
+#include <stdio.h>
 #include <assert.h>
 
 typedef std::list<int> int_list;
@@ -18,7 +14,8 @@ int main()
 #ifdef LLDB_USING_LIBCPP
     int_list *numbers_list = new int_list{1,2,3,4,5,6,7,8,9,10};
 
-    auto *third_elem = numbers_list->__end_.__next_->__next_->__next_; // Set break point at this line.
+    printf("// Set break point at this line.");
+    auto *third_elem = numbers_list->__end_.__next_->__next_->__next_;
     assert(third_elem->__value_ == 3);
     auto *fifth_elem = third_elem->__next_->__next_;
     assert(fifth_elem->__value_ == 5);

Modified: lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/list/main.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/list/main.cpp?rev=336403&r1=336402&r2=336403&view=diff
==============================================================================
--- lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/list/main.cpp (original)
+++ lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/list/main.cpp Thu Jul  5 17:16:21 2018
@@ -1,10 +1,6 @@
 #include <string>
-#ifdef _LIBCPP_INLINE_VISIBILITY
-#undef _LIBCPP_INLINE_VISIBILITY
-#endif
-#define _LIBCPP_INLINE_VISIBILITY
 #include <list>
-
+#include <stdio.h>
 
 typedef std::list<int> int_list;
 typedef std::list<std::string> string_list;
@@ -13,7 +9,8 @@ int main()
 {
     int_list numbers_list;
     
-    (numbers_list.push_back(0x12345678)); // Set break point at this line.
+    printf("// Set break point at this line.");
+    (numbers_list.push_back(0x12345678));
     (numbers_list.push_back(0x11223344));
     (numbers_list.push_back(0xBEEFFEED));
     (numbers_list.push_back(0x00ABBA00));
@@ -32,12 +29,15 @@ int main()
     (text_list.push_back(std::string("is")));
     (text_list.push_back(std::string("smart")));
     
-    (text_list.push_back(std::string("!!!"))); // Set second break point at this line.
+    printf("// Set second break point at this line.");
+    (text_list.push_back(std::string("!!!"))); 
     
     std::list<int> countingList = {3141, 3142, 3142,3142,3142, 3142, 3142, 3141};
     countingList.sort();
-    countingList.unique(); // Set third break point at this line.
-    countingList.size(); // Set fourth break point at this line.
+    printf("// Set third break point at this line.");
+    countingList.unique();
+    printf("// Set fourth break point at this line.");
+    countingList.size();
 
     return 0;
 }

Modified: lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/multiset/main.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/multiset/main.cpp?rev=336403&r1=336402&r2=336403&view=diff
==============================================================================
--- lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/multiset/main.cpp (original)
+++ lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/multiset/main.cpp Thu Jul  5 17:16:21 2018
@@ -1,8 +1,4 @@
 #include <string>
-#ifdef _LIBCPP_INLINE_VISIBILITY
-#undef _LIBCPP_INLINE_VISIBILITY
-#endif
-#define _LIBCPP_INLINE_VISIBILITY
 #include <set>
 
 typedef std::multiset<int> intset;

Modified: lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/set/main.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/set/main.cpp?rev=336403&r1=336402&r2=336403&view=diff
==============================================================================
--- lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/set/main.cpp (original)
+++ lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/set/main.cpp Thu Jul  5 17:16:21 2018
@@ -1,8 +1,4 @@
 #include <string>
-#ifdef _LIBCPP_INLINE_VISIBILITY
-#undef _LIBCPP_INLINE_VISIBILITY
-#endif
-#define _LIBCPP_INLINE_VISIBILITY
 #include <set>
 
 typedef std::set<int> intset;

Modified: lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/unordered/main.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/unordered/main.cpp?rev=336403&r1=336402&r2=336403&view=diff
==============================================================================
--- lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/unordered/main.cpp (original)
+++ lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/unordered/main.cpp Thu Jul  5 17:16:21 2018
@@ -1,8 +1,4 @@
 #include <string>
-#ifdef _LIBCPP_INLINE_VISIBILITY
-#undef _LIBCPP_INLINE_VISIBILITY
-#endif
-#define _LIBCPP_INLINE_VISIBILITY
 #include <unordered_map>
 #include <unordered_set>
 
@@ -81,4 +77,4 @@ int main()
 	thefoo_rw();  // Set break point at this line.
 	
     return 0;
-}
\ No newline at end of file
+}

Modified: lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/vbool/main.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/vbool/main.cpp?rev=336403&r1=336402&r2=336403&view=diff
==============================================================================
--- lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/vbool/main.cpp (original)
+++ lldb/trunk/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/vbool/main.cpp Thu Jul  5 17:16:21 2018
@@ -1,9 +1,4 @@
 #include <string>
-#ifdef _LIBCPP_INLINE_VISIBILITY
-#undef _LIBCPP_INLINE_VISIBILITY
-#endif
-#define _LIBCPP_INLINE_VISIBILITY
-
 #include <vector>
 
 int main()




More information about the lldb-commits mailing list