[Lldb-commits] [PATCH] D54731: [lit] Enable the use of custom user-defined lit commands

[Pavel Labath via Phabricator via lldb-commits]

labath added a comment.

I'd go with the "conservative" approach first. The idea of having lldb loaded inside a lit process does not excite me. One of the problems we have with dotest is that when lldb crashes during the test, it takes a part of the test driver with it which causes some tests to be skipped and the complicates the reporting of the result of the crashed test. It's not as bad right now, as there is still the main process left to report some kind of an error (back in the days when tests were run sequentially in a single process, the entire test suite would just stop), but I still think it would be nice to avoid these issues in the new framework.


https://reviews.llvm.org/D54731