[PATCH] D145009: Split getCompileUnitFor{Data,Code}Address.
Mitch Phillips via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 28 14:55:58 PST 2023
hctim created this revision.
hctim added reviewers: arsenm, dblaikie.
Herald added a subscriber: hiraditya.
Herald added a project: All.
hctim requested review of this revision.
Herald added subscribers: llvm-commits, wdng.
Herald added a project: LLVM.
Getting compile units for data addresses is much slower, as it often
requires a slow fallback path to walk every DWARF entry, as
data addresses don't fall into the compilation unit ranges.
Most lookups are code addresses, and don't need this logic. Split the
functionality out so that we restore the fast-path behaviour for the
code lookups.
More context at:
https://discourse.llvm.org/t/llvm-symbolizer-has-gotten-extremely-slow/67262
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D145009
Files:
llvm/include/llvm/DebugInfo/DWARF/DWARFContext.h
llvm/lib/DebugInfo/DWARF/DWARFContext.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D145009.501300.patch
Type: text/x-patch
Size: 4302 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230228/0ca14e33/attachment.bin>
More information about the llvm-commits
mailing list