diff options
author | Martin Storsjo <martin@martin.st> | 2017-11-16 19:36:48 +0000 |
---|---|---|
committer | Martin Storsjo <martin@martin.st> | 2017-11-16 19:36:48 +0000 |
commit | a08a7e455f51711f376087ca764149d125a87af5 (patch) | |
tree | 117dd496b41a6b36ced346a7c96e238034205fe5 | |
parent | 2ddcf2461daa5d61c543474aed06b12a8b9ad816 (diff) | |
download | libunwind_llvm-a08a7e455f51711f376087ca764149d125a87af5.tar.gz |
Remove a FIXME about truncated section names
If the linker chose to store the full section name instead of truncating
it, this field doesn't contain a truncated name, but an offset into
the string table of the binary. The string table isn't loaded/mapped
into memory during runtime though, so it's not possible to read the
full section name, unless we try to locate the DLL/EXE on disk that
the HMODULE corresponds to and load that manually.
Due to this, lld now always prefers writing a truncated section name
for sections that will be mapped at runtime, even when debug info is
enabled.
Differential Revision: https://reviews.llvm.org/D39918
git-svn-id: https://llvm.org/svn/llvm-project/libunwind/trunk@318446 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | src/AddressSpace.hpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/AddressSpace.hpp b/src/AddressSpace.hpp index 3242840..2be16af 100644 --- a/src/AddressSpace.hpp +++ b/src/AddressSpace.hpp @@ -382,8 +382,6 @@ inline bool LocalAddressSpace::findUnwindSections(pint_t targetAddr, found_obj = true; } else if (!strncmp((const char *)pish->Name, ".eh_frame", IMAGE_SIZEOF_SHORT_NAME)) { - // FIXME: This section name actually is truncated, ideally we - // should locate and check the full long name instead. info.dwarf_section = begin; info.dwarf_section_length = pish->Misc.VirtualSize; found_hdr = true; |