[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