[all-commits] [llvm/llvm-project] c7eb1b: [lldb] Consult summary provider before printing ch...
Dave Lee via All-commits
all-commits at lists.llvm.org
Tue May 30 15:35:18 PDT 2023
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: c7eb1b07470b9babfcd258f014df3661e5f84b30
https://github.com/llvm/llvm-project/commit/c7eb1b07470b9babfcd258f014df3661e5f84b30
Author: Dave Lee <davelee.com at gmail.com>
Date: 2023-05-30 (Tue, 30 May 2023)
Changed paths:
M lldb/source/DataFormatters/ValueObjectPrinter.cpp
A lldb/test/API/functionalities/data-formatter/root-reference-children/Makefile
A lldb/test/API/functionalities/data-formatter/root-reference-children/TestRootReferenceChildren.py
A lldb/test/API/functionalities/data-formatter/root-reference-children/main.cpp
Log Message:
-----------
[lldb] Consult summary provider before printing children of root references
When printing the root of a value, if it's a reference its children are unconditionally
printed - in contrast to pointers whose children are only printed if a sufficient
pointer depth is given.
However, the children are printed even when there's a summary provider that says not to.
If a summary provider exists, this change consults it to determine if children should be
printed.
For example, given a variable of type `std::string &`, this change has the following
effect:
Before:
```
(lldb) p string_ref
(std::string &) string_ref = "one two three four five six seven eight nine ten": {
__r_ = {
std::__1::__compressed_pair_elem<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >::__rep, 0, false> = {
__value_ = {
= {
__l = (__data_ = "one two three four five six seven eight nine ten", __size_ = 48, __cap_ = 64, __is_long_ = 1)
__s = (__data_ = "@\0p\U00000001\0`\0\00\0\0\0\0\0\0\0@", __padding_ = "\x80t<", __size_ = '\0', __is_long_ = '\x01')
__r = {
__words ={...}
}
}
}
}
}
}
```
After:
```
(lldb) p string_ref
(std::string &) string_ref = "one two three four five six seven eight nine ten"
```
rdar://73248786
Differential Revision: https://reviews.llvm.org/D151748
More information about the All-commits
mailing list