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

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue May 19 09:16:56 PDT 2020


JDevlieghere marked 2 inline comments as done.
JDevlieghere added inline comments.


================
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
----------------
labath wrote:
> 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...
What behavior do you have in mind? Make these errors and exit early? It's just us reporting the warning, we can do anything we want with it. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80165





More information about the lldb-commits mailing list