Fwd: [PATCH] Link Clang/ARM/Arch Linux builder into Zorg

[Mikael Lyngvig]

I did the unthinkable and edited the previous patch when I enabled the test
so now that change got lost when I sent you the last patch.  Do you mind
editing the "test=False" to become "test=True" in the patch I just sent you?

---------- Forwarded message ----------
From: Mikael Lyngvig <mikael at lyngvig.org>
Date: 2013/11/25
Subject: Re: [PATCH] Link Clang/ARM/Arch Linux builder into Zorg
To: Renato Golin <renato.golin at linaro.org>
Cc: Mikael Lyngvig <mikael at lyngvig.org>, LLVM-commits <
llvm-commits at cs.uiuc.edu>


I accept your test.  Must admit I forgot to time the build with various
values...  I don't know what's inside a Chromebook, but the XU isn't that
powerful again.  I'll reduce it to four.

Which reminds me that I forgot the ccache environment variables.

Here's an updated patch.

These peer reviews are truly wonderful once you get the hang of it :-)



2013/11/25 Renato Golin <renato.golin at linaro.org>

> Looks good, though I have to ask, isn't 6 jobs too much?
>
> I once did a test on a panda and the chromebook and found that the general
> 50%+ rule didn't apply to ARM targets, at least not those. Maybe the XU is
> closer to a desktop than the other two...
>
> cheers,
> --renato
>
>
> On 25 November 2013 11:26, Mikael Lyngvig <mikael at lyngvig.org> wrote:
>
>> Yup :-)  Please see the attachment.
>>
>>
>>
>>
>> 2013/11/25 Renato Golin <renato.golin at linaro.org>
>>
>>> Hi Michael,
>>>
>>> Running the tests take about 10 minutes, can you turn them on?
>>>
>>> --renato
>>>
>>>
>>> On 25 November 2013 09:29, Mikael Lyngvig <mikael at lyngvig.org> wrote:
>>>
>>>> With this patch, a new ARM Cortex A15 goes online - if the patch works,
>>>> that is.
>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> llvm-commits mailing list
>>>> llvm-commits at cs.uiuc.edu
>>>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>>>
>>>>
>>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131125/f064a017/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: zorg.patch
Type: application/octet-stream
Size: 1997 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131125/f064a017/attachment.obj>