<div dir="ltr">Please send this kind of patches pre-commit review to reduce post-commit confusion in future.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 23, 2015 at 1:40 PM, Shankar Easwaran <span dir="ltr"><<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Since its the same opinion raised by you/Rui, I can change the code to use a map. Will send it for review, soon.<span class="HOEnZb"><font color="#888888"><br>
<br>
Shankar Easwaran</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On 2/23/2015 3:22 PM, Simon Atanasyan wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
As far as I can see we need to keep links to symbols just while we<br>
create references. IMHO it is a bit redundant to keep these links<br>
infinitely. So maybe it is better to maintain a map reference=>symbol<br>
and use this map in the ELFFile::updateReferences() method.<br>
<br>
On Mon, Feb 23, 2015 at 11:27 PM, Shankar Easwaran<br>
<<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.org</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
You can see the function, ELFFile::updateReferences.<br>
<br>
On 2/23/2015 2:24 PM, Simon Atanasyan wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Shankar,<br>
<br>
Where can I see how we the use new `ELFReference::symbol()` method?<br>
<br>
On Mon, Feb 23, 2015 at 2:46 AM, Shankar Easwaran<br>
<<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.org</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: shankare<br>
Date: Sun Feb 22 17:46:21 2015<br>
New Revision: 230191<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=230191&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project?rev=230191&view=rev</a><br>
Log:<br>
[ELF] Add symbol to ELFReference.<br>
<br>
Relocation handling need more information about the Symbol that we are<br>
creating<br>
references for.<br>
<br>
No change in functionality.<br>
<br>
Modified:<br>
      lld/trunk/lib/ReaderWriter/<u></u>ELF/Atoms.h<br>
      lld/trunk/lib/ReaderWriter/<u></u>ELF/ELFFile.h<br>
      lld/trunk/lib/ReaderWriter/<u></u>ELF/Mips/MipsELFFile.h<br>
</blockquote></blockquote></blockquote></blockquote>
<br>
<br>
-- <br></div></div><span class="im HOEnZb">
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation<br>
<br></span><div class="HOEnZb"><div class="h5">
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>