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

Reid Kleckner via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Nov 20 16:04:59 PST 2018


rnk added a comment.

I think the code to implement this is fine, but before we add this complexity to lit, I just wanted to know if other folks who work on the LLDB test suite are on board and want to use this approach to abstract over building apps for different targets. I see @stella.stamenova is, but I wanted to hear from other people involved in the LLDB lit test suite stuff. The `%compile %debug %opt %s`  lit substitution approach is limiting, but do people feel strongly that this is much better?



================
Comment at: llvm/utils/lit/tests/Inputs/shtest-keyword-command/keyword_helper.py:2
+
+def customCommand(line):
+  return ('STDOUT: ' + line, 'STDERR: ' + line)
----------------
Oh, the joys of multiprocessing and pickling...


https://reviews.llvm.org/D54731





More information about the lldb-commits mailing list