[PATCH] D32612: Remove LinkerScript::flush
Rafael Ávila de Espíndola via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 27 12:14:08 PDT 2017
rafael created this revision.
This patch replaces flush with a last ditch attempt at synchronizing the section list with the linker script "AST".
The synchronization in a bit of a hack and should in time be avoided by creating the AST earlier so that modifications can be made directly to it instead of modifying the section list and synchronizing it back.
This is the main step for fixing https://bugs.llvm.org/show_bug.cgi?id=32816. With this in place I think the only missing thing would be to have processCommands assign section indexes as dummy offsets so that the sort in OutputSection::finalize works.
With this LinkerScript::assignAddresses becomes much simpler, which should help with the thunk work.
https://reviews.llvm.org/D32612
Files:
ELF/LinkerScript.cpp
ELF/LinkerScript.h
ELF/Writer.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D32612.96970.patch
Type: text/x-patch
Size: 6355 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170427/397cd99c/attachment.bin>
More information about the llvm-commits
mailing list