[PATCH] D97046: [lit] Add "late_tests" test suite config option

[James Henderson via Phabricator via llvm-commits]

jhenderson added inline comments.


================
Comment at: llvm/utils/lit/lit/TestingConfig.py:130-133
         # List of tests to run early.
         self.early_tests = {}
+        # Regular expression for tests to run late.
+        self.late_tests = "^$"
----------------
davezarzycki wrote:
> jhenderson wrote:
> > Why are late tests a regular expression, but early tests a list? The inconsistency seems confusing to me.
> "late_tests" does not need to be regular expression. It was just expedient given my goals. The "late_tests" variable could be a list if directories if we treat tests therein as being "late".
I think users will expect the two to be the same type. I don't mind which way around it is personally, and I think applying the rule (whether early or late) to all tests within a directory, when a directory is specified, would be a good idea.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97046