[PATCH] D98179: [lit] Sort test start times based on prior test timing data

David Zarzycki via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 16 10:37:08 PDT 2021


davezarzycki added a comment.

I've run out of time for today, but for the record, I think we should use slash as the canonical separator for the timing data file. The actual separator doesn't matter because these strings are never used as paths, just as keys into a dictionary. (Therefore any character would work.) This would let us harmonize the Unix and Windows timing data files.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98179



More information about the llvm-commits mailing list