[PATCH] D23690: [ELF] Improve error reporting for relocations

[Petr Hosek via llvm-commits]

phosek created this revision.
phosek added reviewers: ruiu, rafael.
phosek added a subscriber: llvm-commits.
phosek added a project: lld.

We should always include symbol name when reporting relocations error to simplify debugging of these issues. Without symbol names users have to manually investigate which of the libraries contain invalid relocations which can be cumbersome when linking multiple libraries.

https://reviews.llvm.org/D23690

Files:
  ELF/Relocations.cpp
  test/ELF/aarch64-fpic-add_abs_lo12_nc.s
  test/ELF/aarch64-fpic-adr_prel_lo21.s
  test/ELF/aarch64-fpic-adr_prel_pg_hi21.s
  test/ELF/aarch64-fpic-ldst32_abs_lo12_nc.s
  test/ELF/aarch64-fpic-ldst64_abs_lo12_nc.s
  test/ELF/aarch64-fpic-ldst8_abs_lo12_nc.s
  test/ELF/copy-errors.s
  test/ELF/copy-in-shared.s
  test/ELF/copy-rel-corrupted.s
  test/ELF/copy-rel-pie-error.s
  test/ELF/eh-frame-dyn-rel.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23690.68591.patch
Type: text/x-patch
Size: 7277 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160818/939df908/attachment.bin>