<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Apr 7, 2014 at 8:26 AM, Shankar Kalpathi Easwaran <span dir="ltr"><<a href="mailto:shankarke@gmail.com" target="_blank">shankarke@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  I am having some problem with my browser, with inline comments.  and I also think this be converted to 2 patches.<br>
<br>
  a) to remove the option (use-shlib-undefines)<br>
  b) fix --[no]allow-shlib-undefined.<br></blockquote><div><br></div><div>My bet is --use-shlib-undefines was introduced to deal with undefined symbols in a library file, under an incorrect assumption on how GNU LD works on them. So fixing it naturally linked to removing the option. Technically I can split this change into two parts; one to fix the issue and leave code for --use-shlib-undefined as dead one, and remove it in a followup patch. Not sure it worth doing, but if everybody wants it, I'll do that.</div>

<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  I am not sure the reason we added the option use-shlib-undefines, Bigcheese ?<br>
<br>
  Rui, can you wait on Bigcheese's review before removing the option use-shlib-undefines ?</blockquote><div><br></div><div>Sure, of course.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5">
<a href="http://reviews.llvm.org/D3304" target="_blank">http://reviews.llvm.org/D3304</a><br>
<br>
BRANCH<br>
  shlib<br>
<br>
ARCANIST PROJECT<br>
  lld<br>
<br>
<br>
</div></div></blockquote></div><br></div></div>