[PATCH] D154987: [lit] Implement PYTHON directive and config.prologue
Vlad Serebrennikov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 25 04:12:47 PDT 2023
Endill added a comment.
In D154987#4529228 <https://reviews.llvm.org/D154987#4529228>, @sammccall wrote:
> Could the tests you're targeting be written as actual python files, with an appropriate library of operations? (Some of this library could of course be shared with the shell test implementation).
I think it would accurate to say that the idea this patch implements was born out of use case I have for C++ defect report test suite. As I see it, restructuring that test suite as actual Python files will introduce more noise to the tests than other options. We would also lose C++ syntax highlighting, which would make tests even less readable in majority of contexts.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154987/new/
https://reviews.llvm.org/D154987
More information about the llvm-commits
mailing list