<div dir="ltr">Hi Keno,<br><br>r231724 and r231726 should get us most of the way towards re-introducing your patch. I see a couple more steps before we get all the way there though:<div><br></div><div>1) Teach RuntimeDyld to let strong definitions squash weak ones when building the symbol table. This will ensure that relocations resolve correctly within a single RuntimeDyld instance.</div><div><br></div><div>2) Teach RTDyldMemoryManager and MCJIT to pass around RuntimeDyld::SymbolInfo data structures. This will allow us to perform lookup correctly in the casen where people chain MCJIT instances.</div><div><br></div><div>The second change will require some community consultation. It's not technically a change to the C-API, but it will hit all C-API users with custom memory managers. They'll want a heads up so they can prepare.</div><div><br></div><div>- Lang.</div><div><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 24, 2015 at 3:36 PM, Keno Fischer <span dir="ltr"><<a href="mailto:kfischer@college.harvard.edu" target="_blank">kfischer@college.harvard.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Lang,<div><br></div><div>it's not actively causing problems for me right now, but I expect it will in the near future. Let me know if you have a patch I can take a look at.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Keno</div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 20, 2015 at 8:17 PM, Lang Hames <span dir="ltr"><<a href="mailto:lhames@gmail.com" target="_blank">lhames@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Keno,<div><br></div><div>I've had a chance to look at this at long last. It turns out AbortOnFailure is innocent (if ugly), the problem is actually in getSymbolAddress. With this patch it doesn't return an address for weak symbols, so RuntimeDyld clients can't see weak symbols. We probably shouldn't just expose them either though: There would be no way for a client to tell the difference between a weak symbol and a strong one.</div><div><br></div><div>I think we actually need to add a weak/strong flag to the symbol table, along the same lines as the visibility flag. I've filed <a href="http://llvm.org/PR22653" target="_blank">http://llvm.org/PR22653</a> to track this.</div><div><br></div><div>Is this issue actively causing problems for you? I expect I'll want to deal with it soonish either way, but if it's an active headache for you I'll try to make it sooner rather than later.</div><div><br>Cheers,</div><div>Lang.</div></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 4, 2015 at 2:14 PM, Lang Hames <span dir="ltr"><<a href="mailto:lhames@gmail.com" target="_blank">lhames@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Keno,<div><br></div><div>Sorry I haven't gotten to this yet. I haven't forgotten about it - I'm planning to get it fixed up tomorrow or Friday.</div><div><br></div><div>Cheers,</div><div>Lang.</div><div><br></div></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 27, 2015 at 5:48 PM, Keno Fischer <span dir="ltr"><<a href="mailto:kfischer@college.harvard.edu" target="_blank">kfischer@college.harvard.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">No problem.</div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 27, 2015 at 8:33 PM, Lang Hames <span dir="ltr"><<a href="mailto:lhames@gmail.com" target="_blank">lhames@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Keno,<div><br></div><div>Sorry - I had to revert in r227287. Hopefully I'll have a solution soon.</div><span><font color="#888888"><div><br></div><div>- Lang.</div><div><br></div></font></span></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>