[Lldb-commits] [PATCH] D54751: [LLDB] - Fix setting the breakpoints when -gsplit-dwarf and DWARF 5 were used for building the executable.

George Rimar via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Nov 28 01:36:07 PST 2018


grimar marked an inline comment as done.
grimar added inline comments.


================
Comment at: source/Plugins/SymbolFile/DWARF/DWARFUnit.cpp:355-356
+  if (addr_base == LLDB_INVALID_ADDRESS)
+    addr_base = cu_die.GetAttributeValueAsUnsigned(m_dwarf, this,
+                                                   DW_AT_GNU_addr_base, 0);
   dwo_cu->SetAddrBase(addr_base);
----------------
clayborg wrote:
> Do we still want the addr_base to default to zero instead of LLDB_INVALID_ADDRESS here?
I believe so. Setting the addr_base to default zero looks fine to me. That way the existent code does not
need to check for LLDB_INVALID_ADDRESS and can just call the `GetAddrBase` and add the result
to the offset calculation.
That is how things already work and I do not see any benefits from changing this behavior probably?


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

https://reviews.llvm.org/D54751





More information about the lldb-commits mailing list