<div dir="ltr">I think no such plumbing is needed. That can be done in a few lines like this <a href="http://reviews.llvm.org/D7445">http://reviews.llvm.org/D7445</a><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Feb 5, 2015 at 11:21 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">I tried making changes and this was the flow.<br>
<br>
instantiateMember -> calls Registry::loadFile(memory buffer) -> calls Reader::loadFile(memory buffer) -> calls ELFObjectReader::loadFile -> Creates lld::File.<br>
<br>
So memberName and fileName needs to be added to all these calls, and the intermediate API's are called from YAML/MachO and there are places it does not make sense.<br>
<br>
MemoryBuffer need to owned by lld::File which is not being done currently. But with all this change, MemoryBuffer would<br>
<br>
So I feel this is much cleaner. If you see a better way to do this, let me know.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D7428" target="_blank">http://reviews.llvm.org/D7428</a><br>
<br>
EMAIL PREFERENCES<br>
<a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br></div>