<div dir="ltr">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.</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Apr 3, 2018 at 10:21 AM Rui Ueyama via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ruiu added a comment.<br>
<br>
I'd like to not reply on each comment as I want reviewers focus on design and feature themselves first.<br>
<br>
Guys, do you think the new command line option name make sense? Do you think it's actually useful?<br>
<br>
<br>
<a href="https://reviews.llvm.org/D45195" rel="noreferrer" target="_blank">https://reviews.llvm.org/D45195</a><br>
<br>
<br>
<br>
</blockquote></div>