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

2018-03-17 Thread Jan Kratochvil via Phabricator via lldb-commits
jankratochvil updated this revision to Diff 138813. https://reviews.llvm.org/D40475 Files: packages/Python/lldbsuite/test/dotest.py packages/Python/lldbsuite/test/lldbinline.py packages/Python/lldbsuite/test/lldbtest.py packages/Python/lldbsuite/test/make/Makefile.rules packages/Python/

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

2018-03-11 Thread Jan Kratochvil via Phabricator via lldb-commits
jankratochvil updated this revision to Diff 137963. jankratochvil added a comment. > I have implemented dwz category like gmodules but even when checkDWZSupport > makes configuration.skipCategories.append("dwz") the DWZ-variants of tests > are still being run. I will investigate it more. This i

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

2018-03-08 Thread Jan Kratochvil via Phabricator via lldb-commits
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 und

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

2018-02-25 Thread Davide Italiano via Phabricator via lldb-commits
davide added a comment. 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`). Up to you though. https://reviews.llvm.org/D40475 _

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

2018-02-25 Thread Jan Kratochvil via Phabricator via lldb-commits
jankratochvil updated this revision to Diff 135847. jankratochvil added a comment. > The way we do these sorts of things is that we explicitly add the category to > the list of skipped categories (take a look at `checkLibcxxSupport` in > dotest.py). I have implemented `dwz` category like `gmodu

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

2018-02-20 Thread Jan Kratochvil via Phabricator via lldb-commits
jankratochvil updated this revision to Diff 135063. jankratochvil retitled this revision from "DWZ 12/12: DWZ test mode" to "DWZ 10/11: DWZ test mode". jankratochvil added a comment. Herald added subscribers: JDevlieghere, mgorny. It now includes also unittests/SymbolFile/DWZ/ which should PASS e