[PATCH] D47590: DWARFAcceleratorTable: Add an iterator-based api for accessing names in the index

Pavel Labath via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 31 07:41:37 PDT 2018


labath created this revision.
labath added reviewers: JDevlieghere, aprantl, dblaikie.

Back when we were introducing the DWARF v5 name index, there was a
short discussion whether we shouldn't have a nicer api for iterating
over the index. At that time, I did not find it necessary since the
iteration over names was done only from within the index itself (and I
figured the internal implementation can deal with a slightly rough
interface).

However, now I ran into a use for this kind of API in LLDB (for finding
all names matching a regular expression), so it looked like a nice
opportunity to introduce one. To make the API more useful, I've made the
NameTableEntry class a bit smarter: it now stores the string section
reference (so it can return its name) and its position in the name index
(mainly useful for dumping/logging).

I also convert the internal users to use the new API, which should also
give test coverage for the added code.


Repository:
  rL LLVM

https://reviews.llvm.org/D47590

Files:
  include/llvm/DebugInfo/DWARF/DWARFAcceleratorTable.h
  include/llvm/DebugInfo/DWARF/DWARFVerifier.h
  lib/DebugInfo/DWARF/DWARFAcceleratorTable.cpp
  lib/DebugInfo/DWARF/DWARFVerifier.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D47590.149293.patch
Type: text/x-patch
Size: 11491 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180531/a2b51877/attachment.bin>


More information about the llvm-commits mailing list