[PATCH] D32612: Remove LinkerScript::flush

[Rafael Ávila de Espíndola via Phabricator via llvm-commits]

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>