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:13:34 PDT 2016


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


More information about the cfe-commits mailing list