[PATCH] D42475: [ELF] Add warnings for various symbols that cannot be ordered

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 31 06:40:27 PST 2018


jhenderson updated this revision to Diff 132166.
jhenderson edited the summary of this revision.
jhenderson added a comment.

Rebased following r323779. Thanks to that change, buildSectionOrder() is now only called in one place, and only after processSectionCommands(). This means that we don't need the additional boolean to prevent warnings being emitted twice in some situations, and can also now warn for symbols in /DISCARD/ sections that would otherwise be ordered.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D42475

Files:
  ELF/Config.h
  ELF/Driver.cpp
  ELF/Options.td
  ELF/Writer.cpp
  test/ELF/Inputs/symbol-ordering-file-warnings1.s
  test/ELF/Inputs/symbol-ordering-file-warnings2.s
  test/ELF/symbol-ordering-file-warnings.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42475.132166.patch
Type: text/x-patch
Size: 12599 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180131/9bafe5ca/attachment-0001.bin>


More information about the llvm-commits mailing list