PATCH: suppress warnings in llvm-link

[Eli Bendersky]

On Thu, Feb 6, 2014 at 9:55 AM, Bill Wendling <isanbard at gmail.com> wrote:

> LGTM too.
>
> -bw
>
>
> On Thu, Feb 6, 2014 at 9:37 AM, Nadav Rotem <nrotem at apple.com> wrote:
>
>> LGTM.
>>
>>

Thanks for the quick review, guys. Committed in r200927.

Eli















>  On Feb 6, 2014, at 9:32 AM, Eli Bendersky <eliben at google.com> wrote:
>>
>> > Hello,
>> >
>> > Attaching a patch that adds a -suppress-warnings flag to the LLVM
>> bitcode linker, with the expected effect.
>> >
>> > The option is currently added directly into LinkModules.cpp; an
>> alternative would be to add it to llvm-link.cpp, but then it would have to
>> be percolated down to ModuleLinker through Linker etc. I can modify the
>> patch to take that path, if that's deemed better. If the warning goes into
>> llvm-link.cpp, it may also make sense to turn it to "-w" to be just like in
>> Clang.
>> >
>> > PTAL,
>> >
>> > Eli
>> >
>> >
>> <llvm-link-suppress-warnings.1.patch>_______________________________________________
>> > llvm-commits mailing list
>> > llvm-commits at cs.uiuc.edu
>> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140206/e24f5b1f/attachment.html>