r244867 - Add sanitizer blacklists to the rules generated with -M/-MM/-MD/-MMD.

Ivan Krasin via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 13 16:13:24 PDT 2015


Hi there,

I will remove both test cases, which rely on the existence of the default
blacklist. It's handled by the driver anyway, and the code I changed is in
the frontend; at this point, there's no such a thing, as a default
blacklist, all of them are explicit.
Just a sec.

krasin

On Thu, Aug 13, 2015 at 4:09 PM, NAKAMURA Takumi <geek4civic at gmail.com>
wrote:

> Tweaked a test in r244970.
> Could you split it out if you would like to run it for the default target?
> On Fri, Aug 14, 2015 at 5:07 AM Ivan Krasin via cfe-commits <
> cfe-commits at lists.llvm.org> wrote:
>
>> Thank you, Yaron. Understood.
>>
>> On Thu, Aug 13, 2015 at 11:04 AM, Yaron Keren <yaron.keren at gmail.com>
>> wrote:
>>
>>> There is great variety of OSs, compilers, configs in the bots. You can't
>>> get the smae with try bots unless everything is duplicated which is a
>>> waste. You test locally, commit, watch the bots here:
>>>
>>>  http://lab.llvm.org:8011/grid
>>>
>>> or wait for buildbot failure e-mails which arrive a bit slower.
>>> Breaks may happen even if you test locally, so it's critical be around
>>> after the commit to to resolve such issues.
>>>
>>>  http://llvm.org/docs/DeveloperPolicy.html#quality
>>>
>>>
>>>
>>> 2015-08-13 20:51 GMT+03:00 Ivan Krasin <krasin at google.com>:
>>>
>>>> Thank you, Yaron.
>>>>
>>>> Sorry for breaking the build, it obviously passed locally and I didn't
>>>> think about all the variety of the supported configs. :(
>>>>
>>>> Does LLVM have try bots, so that I can run the tests with my patch
>>>> before committing it?
>>>>
>>>> krasin
>>>>
>>>
>>> On Thu, Aug 13, 2015 at 3:40 AM, Renato Golin <renato.golin at linaro.org>
>>> wrote:
>>>
>>>> On 13 August 2015 at 07:15, Yaron Keren via cfe-commits
>>>> <cfe-commits at lists.llvm.org> wrote:
>>>> > CHECK-EIGHT is failing bots, see
>>>> >
>>>> >
>>>> http://lab.llvm.org:8011/builders/clang-x86_64-ubuntu-gdb-75/builds/24306/steps/check-all/logs/FAIL%3A%20Clang%3A%3Adependency-gen.c
>>>>
>>>>
>>>> That check only works if compiler-rt is built in, and supports the
>>>> target. I wouldn't put that in a Clang test.
>>>>
>>>
>>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150813/8b666608/attachment-0001.html>


More information about the cfe-commits mailing list