[PATCH] D89349: Added utility to launch tests on a target remotely.

Louis Dionne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 22 22:24:41 PDT 2020


ldionne added a comment.

In D89349#2349038 <https://reviews.llvm.org/D89349#2349038>, @ldionne wrote:

> Do we still need this utility if we add `--test-executable` (or as you did `--exec-pattern`) to the original libcxx `ssh.py`?

Note, what happened is that I spoke with @broadwaylamb offline and he made a point that supporting `--test-executable` was worthwhile, and it would remove existing hacks in `ssh.py`. In the meantime, I also came across other use cases that would be helped by having `--test-executable`, which is why I came back and changed my mind on D83429 <https://reviews.llvm.org/D83429>.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89349



More information about the llvm-commits mailing list