[PATCH] [LNT] Aggregate most recent runs

Chris Matthews chris.matthews at apple.com
Mon Jun 2 13:38:36 PDT 2014


LGTM.

> On May 31, 2014, at 3:24 PM, Yi Kong <kongy.dev at gmail.com> wrote:
> 
> Fixed by removing comparison_window.
> 
> On 30 May 2014 00:34, Chris Matthews <chris.matthews at apple.com> wrote:
>> This patch breaks the daily report view.
>> 
>> This daily report uses a custom run window, so that it can do comparisons back in time. I’m not sure we can ignore the comparison_window in that case?
>> 
>> Yi, can you take a look? Thanks!
>> 
>> On May 19, 2014, at 11:49 AM, Chris Matthews <chris.matthews at apple.com> wrote:
>> 
>>> Patch LGTM.
>>> 
>>> On May 10, 2014, at 10:50 AM, Renato Golin <renato.golin at linaro.org> wrote:
>>> 
>>>> On 9 May 2014 21:57, Chris Matthews <chris.matthews at apple.com> wrote:
>>>>> Yes, I’d like to test this on a few of our data sets.  I just want to make sure we are not going to miss regressions.
>>>> 
>>>> This should be an option of the buildbot, so that only those that
>>>> really needed it (slow machine) enable it, with the default being
>>>> what's available today.
>>>> 
>>>> In the case of slow machines, we're not getting *anything*, so we're
>>>> already missing 100% of the regressions. Even if we could only get
>>>> less than 50% of the regressions, it would still be a major
>>>> improvement.
>>>> 
>>>> cheers,
>>>> --renato
>>> 
>>> 
>>> _______________________________________________
>>> llvm-commits mailing list
>>> llvm-commits at cs.uiuc.edu
>>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>> 
>> 
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> <0001-Fix-daily-report.patch>





More information about the llvm-commits mailing list