[Lldb-commits] [PATCH] D74964: [lldb/DWARF] Don't index dwp file multiple times

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Feb 24 00:15:37 PST 2020


labath added inline comments.


================
Comment at: lldb/source/Plugins/SymbolFile/DWARF/ManualDWARFIndex.h:66
 
   /// Non-null value means we haven't built the index yet.
+  SymbolFileDWARF *m_dwarf;
----------------
JDevlieghere wrote:
> This seems inverted? 
Actually, it is correct, although somewhat confusing -- the member is set to nullptr once we finished indexing. I think I can rephrase that comment to be a bit clearer..


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74964/new/

https://reviews.llvm.org/D74964





More information about the lldb-commits mailing list