[Lldb-commits] [PATCH] D40475: DWZ 10/11: DWZ test mode

Jan Kratochvil via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Mar 8 11:56:11 PST 2018


jankratochvil added a comment.

In https://reviews.llvm.org/D40475#1018951, @davide wrote:

> fwiw, you don't need unit tests or python tests to implement this.
>  If I understand the feature correctly you can probably extend `lldb-test` (which is run as part of `check-lldb-lit`).


I do not understand much yet all the testing frameworks LLDB implements.  Could you suggest what you find redundant?  So far I do want to run the whole testsuite also in `dwz` mode.
I understand `unittests/SymbolFile/DWZ/SymbolFileDWZTests.cpp` could be written also as an extension of `tools/lldb-test`. But then I do not see why. The unit test of mine looks fine to me.


https://reviews.llvm.org/D40475





More information about the lldb-commits mailing list