<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Apr 3, 2018 at 6:20 PM Rafael Avila de Espindola 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">espindola added a comment.<br>
<br>
The option and general algorithm makes sense to me.<br>
<br>
I am just not sure about the name since cyclic dependencies are still supported with --start-group/--end-group for example.<br>
<br>
BTW, I think this is a conservative heuristic since it would warn on "foo.a --start-group bar.a foo.a --end-group" if bar.a fetches a member of foo.a. That is probably fine.<br></blockquote><div><br>Yeah, the fact that this would produce a warning for code that a traditional unix linker would succeed at (when repeating libs on a link line to resolve those cycles) is a bit unfortunate, but at least not something I care about for my own uses, I don't think (I mean, I haven't looked at LLVM's build, but I doubt it uses this specify-a-library-more-than-once trick)</div><div><br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Taking a look at the implementation now.<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></div>