[Lldb-commits] [PATCH] D65311: [dotest] Remove multiprocessing

2019-07-30 Thread Jonas Devlieghere via Phabricator via lldb-commits
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rL367331: [dotest] Remove multiprocessing (authored by JDevlieghere, committed by ). Herald added a project: LLVM. Herald ad

[Lldb-commits] [PATCH] D65311: [dotest] Remove multiprocessing

2019-07-29 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere added a comment. In D65311#1605147 , @jasonmolenda wrote: > I'm not sure it's really relevant when lit is the driver, but I would often > use --no-multiprocess with dotest to see the full trace output of a single > test while it was running

[Lldb-commits] [PATCH] D65311: [dotest] Remove multiprocessing

2019-07-29 Thread Jason Molenda via Phabricator via lldb-commits
jasonmolenda added a comment. I'm not sure it's really relevant when lit is the driver, but I would often use --no-multiprocess with dotest to see the full trace output of a single test while it was running. In multiprocess mode that wasn't displayed iirc. That's my only use of it. I do use

[Lldb-commits] [PATCH] D65311: [dotest] Remove multiprocessing

2019-07-26 Thread Jim Ingham via Phabricator via lldb-commits
jingham added a comment. This seems okay to me. I would also check with Jason. I don't know who coordinates running tests remotely - or even if they can run in parallel now. But that's the one bit I don't understand well enough to say yea or nay on. CHANGES SINCE LAST ACTION https://revie

[Lldb-commits] [PATCH] D65311: [dotest] Remove multiprocessing

2019-07-26 Thread Pavel Labath via Phabricator via lldb-commits
labath added a comment. Woohoo. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65311/new/ https://reviews.llvm.org/D65311 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [PATCH] D65311: [dotest] Remove multiprocessing

2019-07-25 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere updated this revision to Diff 211862. JDevlieghere added a comment. Herald added a subscriber: mgorny. Remove more options and external references. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65311/new/ https://reviews.llvm.org/D65311 Files: lldb/docs/resources/test.rs

[Lldb-commits] [PATCH] D65311: [dotest] Remove multiprocessing

2019-07-25 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere updated this revision to Diff 211861. JDevlieghere added a comment. Add context CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65311/new/ https://reviews.llvm.org/D65311 Files: lldb/packages/Python/lldbsuite/test/configuration.py lldb/packages/Python/lldbsuite/test/dose

[Lldb-commits] [PATCH] D65311: [dotest] Remove multiprocessing

2019-07-25 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere created this revision. JDevlieghere added a reviewer: LLDB. Herald added a project: LLDB. Herald added a subscriber: lldb-commits. JDevlieghere updated this revision to Diff 211861. JDevlieghere added a comment. Add context Now that the Xcode project is removed, I want to focus on `