diff options
author | Ed Schouten <ed@nuxi.nl> | 2017-02-23 08:05:58 +0000 |
---|---|---|
committer | Ed Schouten <ed@nuxi.nl> | 2017-02-23 08:05:58 +0000 |
commit | fd939c0f6c9e036848f22f94ab00fbc4fadb1e2d (patch) | |
tree | cde61cc35162417240b94f8245895261022d58f9 /lldb/packages/Python/lldbsuite/test/python_api/breakpoint/TestBreakpointAPI.py | |
parent | 21467876096fa366f95f2058b4cbbdaa339769de (diff) | |
download | bcm5719-llvm-fd939c0f6c9e036848f22f94ab00fbc4fadb1e2d.tar.gz bcm5719-llvm-fd939c0f6c9e036848f22f94ab00fbc4fadb1e2d.zip |
Drop the dependency on dl_unwind_find_exidx().
While porting libunwind over to CloudABI for ARMv6, I observed that this
source file doesn't build, as it depends on dl_unwind_find_exidx(),
which CloudABI's C library was lacking. After I added that function,
I still needed to patch up libunwind to define _Unwind_Ptr.
Taking a step back, I wonder why we need to make use of this function
anyway. The unwinder already has some nice code to use dl_iterate_phdr()
to scan for a PT_GNU_EH_FRAME header. The dl_unwind_find_exidx() does
the same thing, except matching PT_ARM_EXIDX instead. We could also do
that ourselves.
This change gets rid of the dl_unwind_find_exidx() call and extends the
dl_iterate_phdr() loop. In addition to making the code a bit shorter, it
has the advantage of getting rid of some of those OS-specific #ifdefs.
This now means that if an operating system only provides
dl_iterate_phdr(), it gets support for unwinding on all architectures.
There is no need to add more stuff, just to get ARMv6 support.
Differential Revision: https://reviews.llvm.org/D28082
llvm-svn: 295944
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/python_api/breakpoint/TestBreakpointAPI.py')
0 files changed, 0 insertions, 0 deletions