[Lldb-commits] [lldb] [lldb][DWARF] Fix adding children to clang type that hasn't started definition. (PR #93839)
Michael Buch via lldb-commits
lldb-commits at lists.llvm.org
Thu May 30 09:19:26 PDT 2024
================
@@ -2232,6 +2232,11 @@ bool DWARFASTParserClang::CompleteRecordType(const DWARFDIE &die,
// For objective C we don't start the definition when the class is
// created.
TypeSystemClang::StartTagDeclarationDefinition(clang_type);
+ } else if (!clang_type.IsBeingDefined()) {
+ // In case of some weired DWARF causing we don't start definition on this
+ // definition DIE because we failed to find existing clang_type from
+ // UniqueDWARFASTTypeMap due to overstrict checking.
+ TypeSystemClang::StartTagDeclarationDefinition(clang_type);
----------------
Michael137 wrote:
Ah i see, I'd prefer not to add this then if it's going to be an untested codepath. Perhaps it's worth making this an `lldbassert`?
https://github.com/llvm/llvm-project/pull/93839
More information about the lldb-commits
mailing list