<div dir="ltr"><div><span style="font-size:12.8000001907349px">>It's better to define enums for these strings and use them in Resolver.cpp instead of strings.</span><br></div>Yes, enum will be good<div><br></div><div><span style="font-size:12.8000001907349px">>I'm wondering if this is correct. Does --unresolved-symbols really negate --allow-shlib-undefines?</span><br></div><div><span style="font-size:12.8000001907349px">Not all values of </span><span style="font-size:12.8000001907349px">--unresolved-symbols option negate </span><span style="font-size:12.8000001907349px">--allow-shlib-undefines. </span></div><div><span style="font-size:12.8000001907349px">But --allow-shlib-undefines option is only for shared libraries whereas --unresolved-symbols is for relocatable object files, shared libraries as well as executable.</span></div><div><span style="font-size:12.8000001907349px">I have tested behavior of all values of  </span><span style="font-size:12.8000001907349px">--unresolved-symbols in combination with --allow-shlib-undefines and --no-allow-shlib-undefines with gnu ld and gold.</span></div><div><br></div><div><span style="font-size:12.8000001907349px">Implementation of </span><span style="font-size:12.8000001907349px">--unresolved-symbols</span><span style="font-size:12.8000001907349px"> option submitted in lld is in compliance with gnu ld and gold.</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">@</span><span style="font-size:12.8000001907349px;font-weight:bold;white-space:nowrap">Simon Atanasyan,</span></div><div><span style="font-size:12.8000001907349px;font-weight:bold;white-space:nowrap"><br></span></div><div><span style="font-size:12.8000001907349px">>Is it possible to escape using Clang in the test? Usually build-bots do not build Clang with LLD</span><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">Sure, I will update test case by avoiding clang usage.</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><br></div><div class="gmail_extra"><div class="gmail_quote">On Wed, Sep 2, 2015 at 12:44 PM, Rui Ueyama <span dir="ltr"><<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">ruiu added a subscriber: ruiu.<br>
<br>
================<br>
Comment at: lib/Driver/GnuLdDriver.cpp:439-442<br>
@@ +438,6 @@<br>
+    }<br>
+    else if (val.compare("ignore-all") &&<br>
+             val.compare("report-all") &&<br>
+             val.compare("ignore-in-object-files") &&<br>
+             val.compare("ignore-in-shared-libs")) {<br>
+<br>
----------------<br>
It's better to define enums for these strings and use them in Resolver.cpp instead of strings.<br>
<br>
================<br>
Comment at: lib/Driver/GnuLdDriver.cpp:458<br>
@@ +457,3 @@<br>
+    ctx->setPrintRemainingUndefines(true);<br>
+    ctx->setAllowRemainingUndefines(false);<br>
+  }<br>
----------------<br>
I'm wondering if this is correct. Does --unresolved-symbols really negate --allow-shlib-undefines?<br>
<br>
<br>
<a href="http://reviews.llvm.org/D12226" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12226</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div>