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:21:42 PDT 2015
I have created http://reviews.llvm.org/D12021, please take a look.
On Thu, Aug 13, 2015 at 4:13 PM, Ivan Krasin <krasin at google.com> wrote:
> 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/5736cbfc/attachment-0001.html>
More information about the cfe-commits
mailing list