<div dir="ltr"><div class="gmail_extra"><div>On Tue, May 7, 2013 at 10:08 PM, Nick Kledzik <span dir="ltr"><<a href="mailto:kledzik@apple.com" target="_blank">kledzik@apple.com</a>></span> wrote:<br></div><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Shankar,<br>
<br>
Do you mean add a method like:<br>
<br>
     virtual unsigned contentHash() const = 0;<br>
<br>
or maybe:<br>
<br>
     virtual llvm::hash_code contentHash() const = 0<br>
<br>
to lld::DefinedAtom?  That seems good to me.   We just need to figure out what should happen with atoms not intended to be merged.  Should the method assert?  In the case where we want there to be a hash available, is it computed lazily?<br>

<br>
Regarding the NativeReader/NativeWriter if the resolver is using the hash, then it would make sense to add the hash to the file format so reading native format is faster.<br>
<span class=""><font color="#888888"><br>
-Nick<br></font></span></blockquote><div><br></div><div>I'd rather we use a crypto hash so we don't have to compare content at all.</div><div><br></div><div><div>- Michael Spencer</div></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<span class=""><font color="#888888">
</font></span><div class=""><div class="h5"><br>
On May 7, 2013, at 4:43 PM, Shankar Easwaran wrote:<br>
> Can we add a atomContentHash for DefinedAtoms when the atoms are being created. This can essentially speed up comparisons of atoms especially for<br>
><br>
> * ICF (Identical code folding)<br>
> * Section groups (to identify similiar sections)<br>
><br>
> Not sure where else this would help. This would essentially be used only by the Reader and the Resolver.<br>
><br>
> There would be no change to the NativeReader/NativeWriter.<br>
><br>
> Thanks<br>
><br>
> Shankar Easwaran<br>
><br>
> --<br>
> Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation<br>
><br>
<br>
</div></div></blockquote></div><br></div></div>