From 6703812688b84100a159ec793f677952ea8e3eba Mon Sep 17 00:00:00 2001 From: Michael Buch Date: Sun, 17 Jul 2022 12:16:39 +0100 Subject: [PATCH] [LLDB][DataFormatter] Add support for std::__map_const_iterator This patch adds support for formatting `std::map::const_iterator`. It's just a matter of adding `const_` to the existing regex. **Testing** * Added test case to existing API tests Differential Revision: https://reviews.llvm.org/D129962 --- lldb/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp | 2 +- .../data-formatter-stl/libcxx/map/TestDataFormatterLibccMap.py | 10 ++++++++++ .../data-formatter/data-formatter-stl/libcxx/map/main.cpp | 7 ++++++- 3 files changed, 17 insertions(+), 2 deletions(-) diff --git a/lldb/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp b/lldb/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp index db87aae..23ce165 100644 --- a/lldb/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp +++ b/lldb/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp @@ -909,7 +909,7 @@ static void LoadLibCxxFormatters(lldb::TypeCategoryImplSP cpp_category_sp) { cpp_category_sp, lldb_private::formatters::LibCxxMapIteratorSyntheticFrontEndCreator, "std::map iterator synthetic children", - ConstString("^std::__[[:alnum:]]+::__map_iterator<.+>$"), stl_synth_flags, + ConstString("^std::__[[:alnum:]]+::__map_(const_)?iterator<.+>$"), stl_synth_flags, true); AddCXXSynthetic( diff --git a/lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/map/TestDataFormatterLibccMap.py b/lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/map/TestDataFormatterLibccMap.py index e0111ea..097a1e6 100644 --- a/lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/map/TestDataFormatterLibccMap.py +++ b/lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/map/TestDataFormatterLibccMap.py @@ -116,6 +116,16 @@ class LibcxxMapDataFormatterTestCase(TestBase): substrs=['first =', 'second =']) + # (Non-)const key/val iterators + self.expect_expr("it", result_children=[ + ValueCheck(name="first", value="0"), + ValueCheck(name="second", value="0") + ]) + self.expect_expr("const_it", result_children=[ + ValueCheck(name="first", value="0"), + ValueCheck(name="second", value="0") + ]) + # check that MightHaveChildren() gets it right self.assertTrue( self.frame().FindVariable("ii").MightHaveChildren(), diff --git a/lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/map/main.cpp b/lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/map/main.cpp index da6eca9..43ce6ae 100644 --- a/lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/map/main.cpp +++ b/lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/map/main.cpp @@ -24,7 +24,12 @@ int main() ii[0] = 0; // Set break point at this line. ii[1] = 1; - thefoo_rw(1); // Set break point at this line. + + intint_map::iterator it = ii.begin(); + intint_map::const_iterator const_it = ii.cbegin(); + std::printf("%d %d\n", it->second, const_it->second); + + thefoo_rw(1); // Set break point at this line. ii[2] = 0; ii[3] = 1; thefoo_rw(1); // Set break point at this line. -- 2.7.4