llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-lldb Author: Michael Buch (Michael137) <details> <summary>Changes</summary> The accepted DWARFv5 issue 161118.1: "DW_TAG for C++ static data members" specifies that static data member declaration be described by DW_TAG_variable. Make sure we recognize such members. --- Full diff: https://github.com/llvm/llvm-project/pull/72236.diff 1 Files Affected: - (modified) lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp (+6-2) ``````````diff diff --git a/lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp b/lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp index f5628b2753da2a7..79d3199855e1be7 100644 --- a/lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp +++ b/lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp @@ -144,7 +144,7 @@ static bool ShouldIgnoreArtificialField(llvm::StringRef FieldName) { std::optional<DWARFFormValue> DWARFASTParserClang::FindConstantOnVariableDefinition(DWARFDIE die) { - assert(die.Tag() == llvm::dwarf::DW_TAG_member); + assert(die.Tag() == DW_TAG_member || die.Tag() == DW_TAG_variable); auto *dwarf = die.GetDWARF(); if (!dwarf) @@ -2889,7 +2889,7 @@ void DWARFASTParserClang::CreateStaticMemberVariable( const DWARFDIE &die, const MemberAttributes &attrs, const lldb_private::CompilerType &class_clang_type) { Log *log = GetLog(DWARFLog::TypeCompletion | DWARFLog::Lookups); - assert(die.Tag() == DW_TAG_member); + assert(die.Tag() == DW_TAG_member || die.Tag() == DW_TAG_variable); Type *var_type = die.ResolveTypeUID(attrs.encoding_form.Reference()); @@ -3195,6 +3195,10 @@ bool DWARFASTParserClang::ParseChildMembers( } break; + case DW_TAG_variable: { + const MemberAttributes attrs(die, parent_die, module_sp); + CreateStaticMemberVariable(die, attrs, class_clang_type); + } break; case DW_TAG_member: ParseSingleMember(die, parent_die, class_clang_type, default_accessibility, layout_info, last_field_info); `````````` </details> https://github.com/llvm/llvm-project/pull/72236 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits