[PATCH] D29150: Add --warn-unresolved-symbols and --error-unresolved-symbols options

[Ed Maste via Phabricator via llvm-commits]

emaste added inline comments.


================
Comment at: test/ELF/warn-unresolved-symbols.s:23-32
+## Do not report undefines if linking relocatable or shared.
+# RUN: ld.lld -r %t1.o %t2.o -o %t7 --error-unresolved-symbols 2>&1 | \
+# RUN:  FileCheck -allow-empty -check-prefix=NOERR %s
+# RUN: ld.lld -shared %t1.o %t2.o -o %t8.so --error-unresolved-symbols 2>&1 | \
+# RUN:  FileCheck -allow-empty -check-prefix=NOERR %s
+# RUN: ld.lld -r %t1.o %t2.o -o %t9 --warn-unresolved-symbols 2>&1 | \
+# RUN:  FileCheck -allow-empty -check-prefix=NOWARN %s
----------------
davide wrote:
> gold and bfd accept both the single dash and the double dash variant of this option.
> You can change some of the invocations here to `-` in order to make sure we support it correctly.
In general gold and bfd accept `-` and `--` for everything except options starting with 'o'.


https://reviews.llvm.org/D29150