[Lldb-commits] [PATCH] D45332: [LIT] Add new LLDB test format

2018-04-09 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere abandoned this revision. JDevlieghere added a comment. In https://reviews.llvm.org/D45332#1059429, @labath wrote: > In https://reviews.llvm.org/D45332#1058976, @JDevlieghere wrote: > > > In https://reviews.llvm.org/D45332#1058970, @zturner wrote: > > > > > I don't think `sys.path` is

[Lldb-commits] [PATCH] D45332: [LIT] Add new LLDB test format

2018-04-06 Thread Pavel Labath via Phabricator via lldb-commits
labath added a comment. In https://reviews.llvm.org/D45332#1058976, @JDevlieghere wrote: > In https://reviews.llvm.org/D45332#1058970, @zturner wrote: > > > I don't think `sys.path` is set up correctly to be able to find the > > lldbtest package from the `lldb/lit` folder. > > > > These things k

[Lldb-commits] [PATCH] D45332: [LIT] Add new LLDB test format

2018-04-05 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere added a comment. In https://reviews.llvm.org/D45332#1058970, @zturner wrote: > I don't think `sys.path` is set up correctly to be able to find the lldbtest > package from the `lldb/lit` folder. > > These things kind of evolved separately, and the `lldb/lit` folder was > created as

[Lldb-commits] [PATCH] D45332: [LIT] Add new LLDB test format

2018-04-05 Thread Zachary Turner via Phabricator via lldb-commits
zturner added a comment. I don't think `sys.path` is set up correctly to be able to find the lldbtest package from the `lldb/lit` folder. These things kind of evolved separately, and the `lldb/lit` folder was created as a place to start iterating on LLVM-style lit / FileCheck tests. These kind

[Lldb-commits] [PATCH] D45332: [LIT] Add new LLDB test format

2018-04-05 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere added a comment. In https://reviews.llvm.org/D45332#1058914, @zturner wrote: > It should be possible to define it outside the LLVM repo. Just in > `llvm/lldb/lit/lit.cfg` replace this line: > > config.test_format = lit.formats.ShTest(execute_external) > > > with something like th

[Lldb-commits] [PATCH] D45332: [LIT] Add new LLDB test format

2018-04-05 Thread Zachary Turner via Phabricator via lldb-commits
zturner added a comment. It should be possible to define it outside the LLVM repo. Just in `llvm/lldb/lit/lit.cfg` replace this line: config.test_format = lit.formats.ShTest(execute_external) with something like this: import lldb_format config.test_format = lldb_format.LLDBTestFormat()

[Lldb-commits] [PATCH] D45332: [LIT] Add new LLDB test format

2018-04-05 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere added a comment. As discussed in https://reviews.llvm.org/D45215 Repository: rL LLVM https://reviews.llvm.org/D45332 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commit

[Lldb-commits] [PATCH] D45332: [LIT] Add new LLDB test format

2018-04-05 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere created this revision. JDevlieghere added reviewers: zturner, labath, davide, aprantl. Herald added a subscriber: delcypher. JDevlieghere added a comment. As discussed in https://reviews.llvm.org/D45215 This adds a new test format to lit which will be used for LLDB. (If it's possib