[PATCH] D21794: [ELF] - Added support for --unresolved-symbols option.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 28 07:40:40 PDT 2016


grimar created this revision.
grimar added reviewers: ruiu, rafael.
grimar added subscribers: llvm-commits, grimar.

Option has next description (http://linux.die.net/man/1/ld):

--unresolved-symbols=method
Determine how to handle unresolved symbols. There are four possible values for method:

ignore-all: Do not report any unresolved symbols.
report-all: Report all unresolved symbols. This is the default.
ignore-in-object-files: Report unresolved symbols that are contained in shared libraries, but ignore them if they come from regular object files.
ignore-in-shared-libs: Report unresolved symbols that come from regular object files, but ignore them if they come from shared libraries. 

Patch implements it. This is PR24524.
Since report-all is default and we do not report about undefined symbols in lld,
report-all does not report about undefines from DSO. Only option that do that
is ignore-in-object-files.

Important note: when I started working on this few hours ago and reassigned it on myself, 
this PR was without movements for about 10 month and was reported for old ELF linker. 
I was not aware that Shridhar Joshi also working on it for new linker.
Since I was already somewhere in the middle, I decided to finish it. That probably means
that we will have 2 variants of this patch soon and probably reasonable to see both and choose 
the most appropriate.

http://reviews.llvm.org/D21794

Files:
  ELF/Config.h
  ELF/Driver.cpp
  ELF/Options.td
  ELF/Writer.cpp
  test/ELF/Inputs/unresolved-symbols.s
  test/ELF/unresolved-symbols.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21794.62091.patch
Type: text/x-patch
Size: 8564 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160628/6d634810/attachment.bin>


More information about the llvm-commits mailing list