[Lldb-commits] [PATCH] D52375: [WIP] Support multiple compile units per OSO entry in SymbolFileDWARFDebugMap
Stefan Gränitz via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Sep 24 09:42:58 PDT 2018
sgraenitz added inline comments.
================
Comment at: source/Plugins/SymbolFile/DWARF/DWARFUnit.cpp:431
const size_t num_ranges =
- die->GetAttributeAddressRanges(dwarf, this, ranges, false);
+ die->GetAttributeAddressRanges(dwarf, this, ranges, check_hi_lo_pc);
if (num_ranges > 0) {
----------------
@JDevlieghere Thanks for your feedback!
@everyone:
This parameter is, actually, one of the key questions in this review: If there is no DW_AT_range attribute for the compile unit, can we safely fall back to DW_AT_low/high_pc?
https://reviews.llvm.org/D52375
More information about the lldb-commits
mailing list