[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