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

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