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

2021-03-12 Thread Julian Lettner via Phabricator via lldb-commits
yln added a comment. Very happy with the patch! Thanks for all your hard work! :) I have a few small nits and one ask: can we model "test failed" explicitly instead of making their execution time really large? Comment at: llvm/utils/lit/lit/discovery.py:141 +test_times =

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

2021-03-17 Thread Julian Lettner via Phabricator via lldb-commits
yln added a comment. In D98179#2629578 , @davezarzycki wrote: > ... I think we should use slash as the canonical separator for the timing > data file. +1 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D9817