[PATCH] D29892: ar: add llvm-dlltool support

[Martell Malone via llvm-commits]

We ran into this before with the casts.
It was previously a warning but someone made that warning an error on the
build bots.
Fixed in rL308360

On Tue, Jul 18, 2017 at 9:51 PM, Rui Ueyama <ruiu at google.com> wrote:

> Which error are you talking about? It doesn't seem like a gcc warning.
>
> E.g. http://lab.llvm.org:8011/builders/sanitizer-x86_64-
> linux-fast/builds/6544/steps/check-llvm%20asan/logs/stdio
>
> On Tue, Jul 18, 2017 at 1:36 PM, Martell Malone <martellmalone at gmail.com>
> wrote:
>
>> Thanks
>> I already replied to the thread from the build bot.
>> We have seen a similar cast gcc warning error before.
>> Easy fix in a follow up shortly.
>>
>> On Tue 18 Jul 2017 at 20:31, Aleksey Shlyapnikov via Phabricator <
>> reviews at reviews.llvm.org> wrote:
>>
>>> alekseyshl added a comment.
>>>
>>> http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-
>>> fast/builds/6544 is not happy about this patch.
>>>
>>>
>>> Repository:
>>>   rL LLVM
>>>
>>> https://reviews.llvm.org/D29892
>>>
>>>
>>>
>>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170718/29fb9284/attachment.html>