clayborg added inline comments.
================
Comment at:
lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp:2265-2267
+ auto base_address = func_range.GetBaseAddress();
+ if (base_address.IsSectionOffset() &&
+ base_address.GetSection()->GetType() == eSectionTypeCode) {
----------------
This is a bit more expensive than comparing against the m_first_code_address
like we do in D87172. We could just modify the "if" statement on line 2256:
```
if (lowest_func_addr >= m_first_code_address &&
lowest_func_addr != LLDB_INVALID_ADDRESS &&
lowest_func_addr <= highest_func_addr) {
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87173/new/
https://reviews.llvm.org/D87173
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
