diff options
| author | Konrad Kleine <kkleine@redhat.com> | 2019-09-11 10:00:30 +0000 |
|---|---|---|
| committer | Konrad Kleine <kkleine@redhat.com> | 2019-09-11 10:00:30 +0000 |
| commit | 3a4781bbf4f39a25562b4c61c9a9ab2483a96b41 (patch) | |
| tree | 7c8cb736d18d34b1ef072e3ab7f8ed60b11c576a /llvm/lib/Transforms | |
| parent | 57256af307ab7bdf42c47da019a1a288b9f9451a (diff) | |
| download | bcm5719-llvm-3a4781bbf4f39a25562b4c61c9a9ab2483a96b41.tar.gz bcm5719-llvm-3a4781bbf4f39a25562b4c61c9a9ab2483a96b41.zip | |
[LLDB][ELF] Load both, .symtab and .dynsym sections
Summary:
This change ensures that the .dynsym section will be parsed even when there's already is a .symtab.
It is motivated because of minidebuginfo (https://sourceware.org/gdb/current/onlinedocs/gdb/MiniDebugInfo.html#MiniDebugInfo).
There it says:
Keep all the function symbols not already in the dynamic symbol table.
That means the .symtab embedded inside the .gnu_debugdata does NOT contain the symbols from .dynsym. But in order to put a breakpoint on all symbols we need to load both. I hope this makes sense.
My other patch D66791 implements support for minidebuginfo, that's why I need this change.
Reviewers: labath, espindola, alexshap
Subscribers: JDevlieghere, emaste, arichardson, MaskRay, lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D67390
llvm-svn: 371599
Diffstat (limited to 'llvm/lib/Transforms')
0 files changed, 0 insertions, 0 deletions
