[PATCH] D58047: [LLD][ELF][ARM] Synthesise missing .ARM.exidx sections.

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 11 08:47:29 PST 2019


peter.smith updated this revision to Diff 186267.
peter.smith added a comment.

Uploaded new diff to address review comments, main changes:

- Added expanded comment explaining why we need these sections.
- Moved the addition of the synthetic sections to where we add the sentinel and added test that fails if we don't.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58047/new/

https://reviews.llvm.org/D58047

Files:
  ELF/InputSection.cpp
  ELF/SyntheticSections.cpp
  ELF/SyntheticSections.h
  ELF/Writer.cpp
  test/ELF/arm-exidx-add-missing.s
  test/ELF/arm-exidx-canunwind.s
  test/ELF/arm-exidx-dedup.s
  test/ELF/arm-exidx-gc.s
  test/ELF/arm-exidx-no-add-missing.s
  test/ELF/arm-exidx-shared.s
  test/ELF/linkerscript/arm-exidx-sentinel-and-assignment.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D58047.186267.patch
Type: text/x-patch
Size: 21279 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190211/d4d3e017/attachment.bin>


More information about the llvm-commits mailing list