<div dir="ltr">I ran into this a couple of months ago, when doing something else, but was too caught up in that so didn't get around to raising it. I agree that converting the symbols from lazy after we're done with them makes sense. It would also allow us to simplify a number of places where we check for !Lazy && !Undefined or similar.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On 14 October 2017 at 07:03, Rafael Avila de Espindola via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Shoaib Meenai <<a href="mailto:smeenai@fb.com">smeenai@fb.com</a>> writes:<br>
<br>
> (switching list from llvm-commits to llvm-dev, which seems more appropriate here)<br>
</span>Thanks.<br>
<span class=""><br>
> I think I'm not too excited about adding more predicates or inheritance. Lazy symbols are useful only when we are adding files to the symbol table. After that, there's no use of them. So maybe we can just visit all symbols at some point after reading all files but before writeResult to nuke all Lazy symbols (by replacing local undefined symbol or something), so that we don't have to think about it after that point.<br>
<br>
</span>True. We should be able to either remove or convert to undef every lazy<br>
symbol after we read all the files.<br>
<br>
Cheers,<br>
Rafael<br>
______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org">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/<wbr>mailman/listinfo/llvm-dev</a><br>
</blockquote></div><br></div>