[PATCH] D55446: Show "Unknown -z option" error message early.

Mon Dec 10 00:36:20 PST 2018

grimar added a comment.

LGTM with the description clarification.

================
Comment at: lld/test/ELF/driver.test:62

 # RUN: not ld.lld %t -z foo 2>&1 | FileCheck -check-prefix=ERR10 %s
+# RUN: not ld.lld %t -z foo --version 2>&1 | FileCheck -check-prefix=ERR10 %s
----------------
thakis wrote:
> This test suggests that the patch description isn't quite right -- lld already prints an error for unknown -z flags. The only effect this patch has is that it now also does this if --version or --help is passed, right?
+1

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55446/new/

https://reviews.llvm.org/D55446