This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rLLDB357678: Add dropped ManualDWARFIndex assert() (authored by jankratochvil, committed by ).
Changed prior to commit: https://reviews.llvm.org/D60254?vs=193669&id=193677#toc Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60254/new/ https://reviews.llvm.org/D60254 Files: source/Plugins/SymbolFile/DWARF/ManualDWARFIndex.cpp Index: source/Plugins/SymbolFile/DWARF/ManualDWARFIndex.cpp =================================================================== --- source/Plugins/SymbolFile/DWARF/ManualDWARFIndex.cpp +++ source/Plugins/SymbolFile/DWARF/ManualDWARFIndex.cpp @@ -92,6 +92,9 @@ } void ManualDWARFIndex::IndexUnit(DWARFUnit &unit, IndexSet &set) { + assert(!unit.GetSymbolFileDWARF()->GetBaseCompileUnit() && + "DWARFUnit associated with .dwo or .dwp should not be indexed directly"); + Log *log = LogChannelDWARF::GetLogIfAll(DWARF_LOG_LOOKUPS); if (log) {
Index: source/Plugins/SymbolFile/DWARF/ManualDWARFIndex.cpp =================================================================== --- source/Plugins/SymbolFile/DWARF/ManualDWARFIndex.cpp +++ source/Plugins/SymbolFile/DWARF/ManualDWARFIndex.cpp @@ -92,6 +92,9 @@ } void ManualDWARFIndex::IndexUnit(DWARFUnit &unit, IndexSet &set) { + assert(!unit.GetSymbolFileDWARF()->GetBaseCompileUnit() && + "DWARFUnit associated with .dwo or .dwp should not be indexed directly"); + Log *log = LogChannelDWARF::GetLogIfAll(DWARF_LOG_LOOKUPS); if (log) {
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits