[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