[PATCH] D27415: [ELF] - Replace MergeOutputSection with synthetic input section MergeSection.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 27 08:29:28 PST 2017


grimar updated this revision to Diff 86053.
grimar added a comment.

- Addressed review comments.
- Rebased and performed cleanup.

This continues the review flow.

Patch that do different way (based on diff 2) is not ready yet, but available as demo here https://reviews.llvm.org/D29223.
It has very close source base, the major difference is when combineMergableSections is called.

I suggest the next thing for now: land this first as this it should help to fix the issue with script found in linux kernel.
Then we can discuss if we should fix the rest problem I mentioned earlier, which does not affect kernel for now,
but still probably worth to fix. I'll be able to update https://reviews.llvm.org/D29223 and minimize the amount of changes as well then.


https://reviews.llvm.org/D27415

Files:
  ELF/InputSection.cpp
  ELF/InputSection.h
  ELF/OutputSections.cpp
  ELF/OutputSections.h
  ELF/Symbols.cpp
  ELF/SyntheticSections.cpp
  ELF/SyntheticSections.h
  ELF/Writer.cpp
  test/ELF/comment-gc.s
  test/ELF/debug-gc.s
  test/ELF/gc-merge-local-sym.s
  test/ELF/i386-merge.s
  test/ELF/linkerscript/alternate-sections.s
  test/ELF/linkerscript/merge-sections.s
  test/ELF/map-file.s
  test/ELF/merge-shared-str.s
  test/ELF/merge-shared.s
  test/ELF/merge-string-align.s
  test/ELF/merge-string.s
  test/ELF/merge-sym.s
  test/ELF/no-merge.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27415.86053.patch
Type: text/x-patch
Size: 34898 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170127/159ad372/attachment.bin>


More information about the llvm-commits mailing list