[PATCH] D151664: [lit] Add a method to lit.TestFormat to get the list of tests associated to a path

Louis Dionne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 29 14:05:47 PDT 2023


ldionne added reviewers: jdenny, yln.
ldionne added a comment.

For context, the problem this solves is that libc++ now has a special kind of test called `.gen` tests. Those tests are basically a script that generates other Lit tests on the fly. This means that if I run for example `lit libcxx/test/libcxx/assertions/headers_declare_verbose_abort.gen.py`, I need Lit to actually understand that this corresponds to more than one test, i.e. all the tests generated in that `.gen.py` file.

I think this abstraction is actually what we want most of the time instead of `getTestsInDirectory` -- indeed, I looked at several custom test formats and most seemed to define `getTestsInDirectory` in a way that defining `getTestsForFilename` would have worked better, and then `getTestsInDirectory` would have been implemented generically on top of that.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151664/new/

https://reviews.llvm.org/D151664



More information about the llvm-commits mailing list