[PATCH] D38239: [ELF] - Define linkerscript symbols early.

[George Rimar via Phabricator via llvm-commits]

grimar updated this revision to Diff 128019.
grimar edited the summary of this revision.
grimar added a comment.

- Reimplemented.

I think correct solution is to add linker script symbols as undefined and
set flag for them, showing we know they be defined/adjusted later by script. 
That looks cleaner and allows to avod multiple definition error I mentioned earlier.

Patch do this.


https://reviews.llvm.org/D38239

Files:
  ELF/Driver.cpp
  ELF/LTO.cpp
  ELF/LinkerScript.cpp
  ELF/LinkerScript.h
  ELF/SymbolTable.cpp
  ELF/Symbols.h
  test/ELF/linkerscript/symbols-synthetic.s
  test/ELF/linkerscript/version-script.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D38239.128019.patch
Type: text/x-patch
Size: 11170 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171222/4e0d8413/attachment.bin>