clayborg requested changes to this revision.
clayborg added a comment.
This revision now requires changes to proceed.

See inline comments.



================
Comment at: source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp:2050-2052
+        if (dwarf_cu->ExtractDIEsIfNeeded(DWARFCompileUnit::
+            ExtractDIEsIfNeededKind::AllDiesButCuDieOnlyForPartialUnits) > 1)
           clear_cu_dies[cu_idx] = true;
----------------
All of these changes can be avoided by just appropriately checking for partial 
units and skipping them in this index function. Do you plan on using this 
functionality in any other place? I would rather you just fix this function to 
filter out the partial units.


https://reviews.llvm.org/D40471



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
  • [Lldb-commits] [PATCH] D40... Jan Kratochvil via Phabricator via lldb-commits
    • [Lldb-commits] [PATCH... Greg Clayton via Phabricator via lldb-commits

Reply via email to