This is an archive of the discontinued LLVM Phabricator instance.

llvm-readobj: Try the DWARF CFI dumper on all machines.
ClosedPublic

Authored by pcc on Feb 27 2019, 9:11 PM.

Details

Summary

There's no reason to limit the DWARF CFI dumper to EM_386 and EM_X86_64;
ELF files could contain DWARF CFI on almost any platform (even 32-bit ARM;
NetBSD uses DWARF CFI on that platform). So start using the DWARF CFI dumper
unconditionally so that we can dump .eh_frame sections on the remaining ELF
platforms as well as in NetBSD binaries.

Diff Detail

Repository
rL LLVM

Event Timeline

pcc created this revision.Feb 27 2019, 9:11 PM
efriedma accepted this revision.Feb 28 2019, 1:01 PM
efriedma added a subscriber: efriedma.

LGTM

I had a very similar change in my tree recently to debug an issue with unwinding on AArch64.

This revision is now accepted and ready to land.Feb 28 2019, 1:01 PM
This revision was automatically updated to reflect the committed changes.