[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
Fri Feb 9 09:29:39 PST 2018


jhenderson updated this revision to Diff 133633.
jhenderson marked 5 inline comments as done.
jhenderson added a comment.
Herald added a subscriber: arichardson.

Addressed Rui's review comments:

- Renamed some variables
- Removed some braces
- Reformatted messages to put symbols at end and source/object file at front (but see inline comments)


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.133633.patch
Type: text/x-patch
Size: 12949 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180209/eb4e0fad/attachment.bin>


More information about the llvm-commits mailing list