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

Paul Robinson via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Aug 10 08:54:31 PDT 2018


probinson added inline comments.


================
Comment at: lit/tools/lldb-mi/interpreter/cli-support/target-list.test:5
+# We should hardcode executable name since at the moment of running of
+# lldb-mi the name must be known.
+# RUN: %cxx -o a.exe %p/inputs/main.cpp -g
----------------
aprantl wrote:
> That's totally fine, we just need to choose a name that works on all platforms, and `a.exe` does.
Not `%t/a.exe` ?  I thought CWD is sometimes not writeable (or might be the working copy, and we don't want to leave trash there).


Repository:
  rLLDB LLDB

https://reviews.llvm.org/D50525





More information about the lldb-commits mailing list