[LNT] r249026 - Add support for storing hash of test binaries.

Tobias Grosser via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 12 00:51:27 PDT 2015


On 10/12/2015 09:37 AM, Kristof Beyls wrote:
> Hi Tobias,
>
> The instance of LNT running at llvm.org/perf needs to be upgraded
> to at least this commit, to be able to understand these new hash fields.
> Chris is planning to do so.
>
> Given that nobody noticed this failure for a couple of days, we
> thought it means that currently, nobody is critically dependent on
> llvm.org/perf. Therefore, we decided to move forward and get llvm.org/perf
> upgraded, rather than trying to downgrade the clients again, which may be
> non-trivial.

Yes, please update quickly.

I did not notice this change earlier as some new library dependences 
added by Chris broke my LNT builds (now fixed). Also for some reason the 
LNT builder does not turn red if the submission fails. So it took a 
while to notice this.

In general my current use case is not to look every day but to look at 
the full per-commit performance history for reference whenever I do 
something performance critical or when I track down performance bugs.
Unfortunately, now a couple of days performance data is lost. :( It 
would be nice if this could be fixed quickly (or the offending LNT 
changes be disabled until the LNT builder is upgraded).

> The upgrade will also pick up a number of stability fixes, so hopefully
> llvm.org/perf will be a bit more stable after the upgrade too.

Cool.

Best,
Tobias

P.S: I know we did not have LNT access for now, but maybe in the future 
first updating the LNT server and then committing problematic client 
changes would be nice.


More information about the llvm-commits mailing list