[PATCH] D45195: Add --check-library-dependency to maintain compatibility with other linkers

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 3 10:39:24 PDT 2018


Makes sense to me - I haven't applied/tested the patch against some recent
commits to LLVM I've made that broke gold/binutils ld based builders but
didn't break for me locally (using lld), but assuming it catches those -
yeah, it'd be useful to me & I'd totally turn it on locally & suggest
anyone else using lld with LLVM to do so as well.

On Tue, Apr 3, 2018 at 10:21 AM Rui Ueyama via Phabricator <
reviews at reviews.llvm.org> wrote:

> ruiu added a comment.
>
> I'd like to not reply on each comment as I want reviewers focus on design
> and feature themselves first.
>
> Guys, do you think the new command line option name make sense? Do you
> think it's actually useful?
>
>
> https://reviews.llvm.org/D45195
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180403/54a4a80e/attachment.html>


More information about the llvm-commits mailing list