[PATCH] D25127: [ARM][LLD] .ARM.exidx support for executables/shared libraries
Peter Smith via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 4 02:40:52 PDT 2016
peter.smith updated this revision to Diff 73437.
peter.smith added a comment.
Split out garbage collection changes from this patch on Rafael's request which didn't get copied into the review:
"Can you split this further?
It would be nice to have a patch taking care of only which sections
are gced. That is, it just makes sure that we keep only the sections
that refer to a kept section.
Cheers,
Rafael"
The only differences to the previous version are the removal of changes to MarkLive.cpp and the removal of the corresponding test that depended on it.
I'll work on Rui's suggestion for garbage collection in a separate review. Will hopefully have something later today.
Hope I've interpreted everyone correctly.
https://reviews.llvm.org/D25127
Files:
ELF/InputSection.cpp
ELF/InputSection.h
ELF/OutputSections.cpp
ELF/Writer.cpp
test/ELF/Inputs/arm-exidx-cantunwind.s
test/ELF/arm-exidx-canunwind.s
test/ELF/arm-exidx-order.s
test/ELF/arm-exidx-output.s
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D25127.73437.patch
Type: text/x-patch
Size: 18734 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161004/6edad716/attachment.bin>
More information about the llvm-commits
mailing list