[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