<div dir="ltr">I believe this will report an error on more command-lines than a traditional linker would fail to link. E.g.:<div>  ld.lld foo.o A.a B.a A.a B.a</div><div>That "should" work fine if the libs are circularly-dependent, as long as there's only a 1-level cycle. I've seen that sort of construct used a fair amount, and while the start-group/end-group flags are pretty clearly a superior option, but are not universally used.</div><div><br></div><div>It will also fail to detect some problematic situations in the face of symbols defined in multiple objects within a group. For example, if you have:</div><div>ld foo.o --start-group A.a B.a --end-group</div><div>and:</div><div> foo.o needs a definition of "f"</div><div> A.a:a.o provides "g".</div><div> B.a:b.o provides "f" and requires "g"</div><div> B.a:b2.o provides "g".</div><div><br></div><div>IIUC, lld will pull in b1.o to get "f", and then a.o to get "g". This flag won't report a failure, as they're in a group. But GNU ld will pull in b1.o to get "f" and then continue searching to the right before circling back around, and thus find b2.o to provide "g". So, despite no warning message, you'll end up with a different link. I mean -- obviously don't do crazy things like that, but still, no warning.</div><div><br></div><div><span style="color:rgb(34,34,34);font-family:sans-serif;font-size:13px;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">However, both those issues seem perfectly acceptable for an opt-in warning flag, and should just be documented so that people know its limitations.</span><br></div><div><span style="color:rgb(34,34,34);font-family:sans-serif;font-size:13px;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline"><br></span></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Apr 4, 2018 at 6:47 PM Rui Ueyama via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi guys,</div><div><br></div><div>I'd like to propose a new lld command line option, --check-library-dependency. We already have a patch (<a href="https://reviews.llvm.org/D45195" target="_blank">https://reviews.llvm.org/D45195</a>), and I'm satisfied with it, but I'd like to get feedback from those who are using lld before submitting.</div><div><br></div><div>--check-library-dependency is an option to prevent an undefined reference from fetching an archive member written earlier in the command line. It can be used to keep your program compatible with GNU linkers after you switch to lld. I'll explain the feature and why you may find it useful below.<br></div><div><br></div><div>lld's symbol resolution semantics is more relaxed than traditional Unix linkers. For example,</div><div><br></div><div>  ld.lld foo.a bar.o</div><div><br></div><div>succeeds even if bar.o contains an undefined symbol that have to be resolved by some object file in foo.a. Traditional Unix linkers don't allow this kind of backward reference, as they visit each file only once from left to right in the command line while resolving all undefined symbols at the moment of visiting.</div><div><br></div><div>In the above case, since there's no undefined symbol when a linker visits foo.a, no files are pulled out from foo.a, and because the linker forgets about foo.a after visiting, it can't resolve undefined symbols in bar.o that could have been resolved otherwise.</div><div><br></div><div>That lld accepts more relaxed form means that (besides it'd make more sense) you can accidentally write a command line or a build file that works only with lld, even if you have a plan to distribute it to wider users who may be using GNU linkers. With --check-library-dependency, you can detect a library order that doesn't work with other Unix linkers.</div><div><br></div><div>The option is also useful to detect cyclic dependencies between static archives. Again, lld accepts</div><div><br></div><div>  ld.lld foo.a bar.a</div><div><br></div><div>even if foo.a and bar.a depend on each other. With --check-library-dependency, it is handled as an error.</div><div><br></div><div>Here is how the option works. We assign a group ID to each file. A file with a smaller group ID can pull out object files from an archive file with an equal or greater group ID. Otherwise, it is a reverse dependency and an error.</div><div><br></div><div>A file outside --{start,end}-group gets a fresh ID when instantiated. All files within the same --{start,end}-group get the same group ID. E.g.</div><div><br></div><div>  ld.lld A B --start-group C D --end-group E</div><div><br></div><div>A and B form group 0. C, D and their member object files form group 1. E forms group 2. I think that you can see how this group assignment rule simulates the traditional linker's semantics.</div><div><br></div><div>I think this is a reasonable option to add. What do you guys think?</div><div><br></div><div>Rui</div></div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>