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



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to