[Lldb-commits] [PATCH] D50525: [WIP] Move lldb-mi interpreter tests to LIT

[Alexander Polyakov via Phabricator via lldb-commits]

apolyakov added inline comments.


================
Comment at: lit/tools/lldb-mi/interpreter/cli-support/breakpoint-set.test:4
+#
+# RUN: %cxx -o %t %p/inputs/main.cpp -g
+# RUN: %lldbmi %t < %s | FileCheck %s
----------------
stella.stamenova wrote:
> apolyakov wrote:
> > stella.stamenova wrote:
> > > One thing to consider here is that any extra parameters passed with -E to the test suite will not propagate to lit at the moment.
> > > 
> > > I ran into this with some internal testing where we need to pass parameters to the compiler - all of the lldb suite tests (c++ and c) build correctly, but any lit tests that directly invoke the compiler do not because the parameters do not get propagated all the way.
> > Could you give an example of extra parameters? I didn't see them before so I don't completely understand you.
> -E "--sysroot=path/to/sys/root -lc++abi -lunwind"
Ok, I think we won't use something like it here. Thank you.


Repository:
  rLLDB LLDB

https://reviews.llvm.org/D50525