[Lldb-commits] [PATCH] D53175: [dotest] Make a missing FileCheck binary a warning, not an error
Stella Stamenova via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Oct 11 15:30:14 PDT 2018
stella.stamenova added a comment.
I spent a bit more time on this and I think I have a better idea of what's happening.
Vedant added functionality so that we can use FileCheck-style checks inside non-lit LLDB tests. Part of that change was to require a parameter --filecheck to be passed when calling the scripts (such as dotests.py). Also as part of that change, the CMakeFiles now propagate a default location for FileCheck as an input to the scripts, so that if you just run the tests by default using any of the generated outputs (for ninja, or make, or VS), they work correctly
The bots (at least some bots) don't just use the generated solutions to run the tests. There's some complicated logic in the zorg repo to create the list of parameters that are passed to the bots and they do not include the new --filecheck argument. So it's not individual bots that need to be updated, but the test harness. I haven't really looked at it before, so I'm still trying to understand it - especially how we would get FileCheck to any of the bots that run the tests remotely using lldbserver today.
More information about the lldb-commits