Skip to content

Commit 59e9160

Browse files
authored
[lldb] UpdateFormatsIfNeeded should respect the dynamic value type (#93262)
UpdateFormatsIfNeeded has hardcoded the call to GetFormat with no dynamic values. GetFormat will try to find the synthetic children of the ValueObject, and passing the wrong one can fail, which can be bad for performance but should not be user visible. Fix the performace bug by passing the dynamic value type of the ValueObject. rdar://122506593
1 parent 70550cd commit 59e9160

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lldb/source/Core/ValueObject.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,7 @@ bool ValueObject::UpdateFormatsIfNeeded() {
216216
m_last_format_mgr_revision = DataVisualization::GetCurrentRevision();
217217
any_change = true;
218218

219-
SetValueFormat(DataVisualization::GetFormat(*this, eNoDynamicValues));
219+
SetValueFormat(DataVisualization::GetFormat(*this, GetDynamicValueType()));
220220
SetSummaryFormat(
221221
DataVisualization::GetSummaryFormat(*this, GetDynamicValueType()));
222222
SetSyntheticChildren(

0 commit comments

Comments
 (0)