[PATCH] D97046: [lit] Generalize `early_tests`

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 2 11:42:56 PST 2021


mehdi_amini added a comment.

In D97046#2587436 <https://reviews.llvm.org/D97046#2587436>, @lebedev.ri wrote:

> I feel like this should be handled by lit transparently, like recording the time it took to run each test last time, then somehow deciding on the new test run order based on that,

+1.

This seems like the right thing to do to me. The "early" or phase looks very ad-hoc to me. 
It also isn't clear to me how it'd would play with a "recording time and replay" solution that would be the desired default for me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97046



More information about the llvm-commits mailing list