[lld][buildbot] Add more buildbots with sanitize/thread

Dmitri Gribenko gribozavr at gmail.com
Tue Oct 29 16:07:01 PDT 2013


On Tue, Oct 29, 2013 at 4:02 PM, Shankar Easwaran
<shankare at codeaurora.org> wrote:
> On 10/29/2013 4:53 PM, Dmitri Gribenko wrote:
>>
>> On Tue, Oct 29, 2013 at 11:07 AM, Shankar Easwaran
>> <shankare at codeaurora.org> wrote:
>>>
>>> Hi Galina, Dmitri,
>>>
>>> Attached is the patch which adds two new bots for compiling lld/clang
>>> with
>>> -fsanitize options
>>>
>>> * address
>>> * thread
>>>
>>> Let me know if you could review and commit it.
>>>
>>> Please let me know if llvm-reviews is the place to submit changes for
>>> buildbots too. I could move the diff over there if its convenient for
>>> you.
>>
>> Shankar,
>>
>> I am not entirely sure if the hardware is good enough to support this
>> additional load and keep responsiveness at the same level.  The goal
>> of clang_fast bots is to be fast and test as many commits as possible.
>>   Adding more configurations will slow down the build cycle for LLD
>> bots, and will test fewer commits.  But maybe this is a reasonable
>> tradeoff for you.
>
> After discussing with Michael on IRC, we feel that that testing each commit
> is more important.
>
> Possibly this could be visited again when there is more hardware resources
> available ?

Of course.  No promises, though.

> Do you think of any other options like running the bot with fsanitize
> options once a day or once a month, when the bot is more idle ? If there is
> a way to configure it.

I think it would be reasonable to configure a second builder on the
same machine given that:
- uses only one CPU,
- uses a less aggressive schedule,
- runs in parallel with the fast builder (does not block it).

Dmitri

-- 
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/



More information about the llvm-commits mailing list