[Lldb-commits] [PATCH] D80165: [lldb/Driver] Fix handling on positional arguments

Tue May 19 06:28:30 PDT 2020

labath added a comment.

Thanks for fixing this. I remember running into this a couple of times, but I never got around to do something about it...

================
Comment at: lldb/test/Shell/Driver/TestPositionalArgs.test:27-28
+
+UNKNOWN: warning: ignoring unknown option: -baz
+UNKNOWN: warning: ignoring unknown option: --quux
+UNKNOWN: Current executable set to {{.*}}foo
----------------
Unrelated to this patch, but.. do we actually want this behavior? I don't think these were just "warnings" before libOption. Given that clang treats (at least some) unknown options as errors, I'm guessing there must be some libOption knob for controlling the behavior of this...

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80165