JDevlieghere updated this revision to Diff 138370.
JDevlieghere added a comment.
Yeah, that's more sensible indeed. I must have had `llvm-lit` in mind and never
gave it a second thought.
Repository:
rL LLVM
https://reviews.llvm.org/D44473
Files:
test/CMakeLists.txt
test/lldb-dotest.in
davide added a comment.
The name should be changed (also the utility name), but that should be done
separately.
Repository:
rL LLVM
https://reviews.llvm.org/D44473
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/c
davide added a comment.
Also, I second the feeling of having `lldb` somewhere in the name for the
utility (rather than `llvm` :)
Repository:
rL LLVM
https://reviews.llvm.org/D44473
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http:/
zturner added a subscriber: JDevlieghere.
zturner added a comment.
Shouldn’t it be lldb-dotest? I’m confused about what this target does
Repository:
rL LLVM
https://reviews.llvm.org/D44473
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
Shouldn’t it be lldb-dotest? I’m confused about what this target does
On Wed, Mar 14, 2018 at 8:21 AM Davide Italiano via Phabricator <
revi...@reviews.llvm.org> wrote:
> davide accepted this revision.
> davide added a comment.
> This revision is now accepted and ready to land.
>
> LGTM
>
>
> Repo
davide accepted this revision.
davide added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rL LLVM
https://reviews.llvm.org/D44473
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-b
JDevlieghere created this revision.
JDevlieghere added reviewers: davide, labath, zturner.
Herald added subscribers: llvm-commits, mgorny.
This makes llvm-dotest a custom target so you can run `ninja
llvm-dotest` to rebuild whatever is necessary before rerunning the
tests.
Repository:
rL LLVM
