[Lldb-commits] [PATCH] D55991: DWARF: Fix a bug in array size computation

Jan Kratochvil via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Dec 25 08:41:31 PST 2018


jankratochvil added inline comments.


================
Comment at: source/Plugins/SymbolFile/DWARF/DWARFFormValue.cpp:620
+  case DW_FORM_ref_sig8:
+  case DW_FORM_GNU_ref_alt:
+    return die_offset;
----------------
clayborg wrote:
> Not sure that this is, but if it isn't an absolute .debug_info offset, then we shouldn't return it here
It is DWZ's reference from main debug file into shared DWZ common file - implemented by pending D40474.
I agree it needs more complicated handling than this implementation so it should not be here.



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

https://reviews.llvm.org/D55991





More information about the lldb-commits mailing list