summaryrefslogtreecommitdiffstats
path: root/libunwind/src
diff options
context:
space:
mode:
authorMartin Storsjo <martin@martin.st>2017-11-16 19:36:48 +0000
committerMartin Storsjo <martin@martin.st>2017-11-16 19:36:48 +0000
commite95d73958d64cbc514f3fee7dcc12a572eb82300 (patch)
tree0a29cfa807bf231bd73e0e80cf69a65f026403ea /libunwind/src
parent78381016785a3fc4b6304c8040198c9cdd0564d9 (diff)
downloadbcm5719-llvm-e95d73958d64cbc514f3fee7dcc12a572eb82300.tar.gz
bcm5719-llvm-e95d73958d64cbc514f3fee7dcc12a572eb82300.zip
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 llvm-svn: 318446
Diffstat (limited to 'libunwind/src')
-rw-r--r--libunwind/src/AddressSpace.hpp2
1 files changed, 0 insertions, 2 deletions
diff --git a/libunwind/src/AddressSpace.hpp b/libunwind/src/AddressSpace.hpp
index 32428400526..2be16afe365 100644
--- a/libunwind/src/AddressSpace.hpp
+++ b/libunwind/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;
OpenPOWER on IntegriCloud