<div dir="ltr">Ahh, I think I get it.  I'm a little concerned about the performance impact of this though.  The whole reason the hash is stored in the file is so that it doesn't have to recompute the hash up front.  If we're re-computing it every time we load the type streams, it seems like it will be a big performance problem.  I think we should only compute the hash when we modify a record and/or write the file.</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Jun 15, 2016 at 12:02 PM Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ruiu added a comment.<br>
<br>
I think that's exactly what we are doing -- in this code, we read type records, compute hash values using our hash function, and compare them with hash values on the PDB file. If they are different, pdbdump reports that the file is corrupted.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D21393" rel="noreferrer" target="_blank">http://reviews.llvm.org/D21393</a><br>
<br>
<br>
<br>
</blockquote></div>