r283827 - [Driver][Diagnostics] Make 'show option names' default for driver warnings

Bruno Cardoso Lopes via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 11 11:30:40 PDT 2016


Let's see how it goes: Committed r283913

On Tue, Oct 11, 2016 at 11:13 AM, Bruno Cardoso Lopes
<bruno.cardoso at gmail.com> wrote:
> On Tue, Oct 11, 2016 at 10:09 AM, Renato Golin <renato.golin at linaro.org> wrote:
>> On 11 October 2016 at 16:34, Bruno Cardoso Lopes
>> <bruno.cardoso at gmail.com> wrote:
>>> Thanks Renato!
>>
>> So, Daniel Jasper did a trick on r283853 (clang || true) to make it
>> not fail when it returns on error. However, I wasn't able to make it
>> return anything but 0, so I'm at odds why this fails on the bot.
>>
>> I was expecting it to actually return non-zero, since it emits an
>> error, which is even weirder. So, using "not clang" doesn't work in
>> this case. Do you have any ideas why this behaviour is like that?
>
> I have no idea. I'm gonna try to re-introduce the commit without a
> target specifc triple, that hopefully should be enough to get it
> working (there was no specific reason this test should be using a
> specific target anyway)
>
> Thanks again Renato
>
>
>
> --
> Bruno Cardoso Lopes
> http://www.brunocardoso.cc



-- 
Bruno Cardoso Lopes
http://www.brunocardoso.cc


More information about the cfe-commits mailing list