<div dir="ltr"><a href="https://reviews.llvm.org/D25108">https://reviews.llvm.org/D25108</a><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 29, 2016 at 1:17 PM, Rui Ueyama <span dir="ltr"><<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.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"><div>Phab is not up yet, so I attached a patch to this mail.</div><div><br></div><div><br></div><div>When we create a PDB file using PDBFileBuilder, the information</div><div>in the superblock, such as the size of the resulting file, is not</div><div>available.</div><div><br></div><div>Previously, PDBFileBuilder::initialize took a superblock assuming</div><div>that all the members of the struct are correct. That is useful when</div><div>you want to restore the exact information from a YAML file, but</div><div>that's probably the only use case in which that is useful.</div><div>When we are creating a PDB file on the fly, we have to backfill the</div><div>members.</div><div><br></div><div>This patch redefines PDBFileBuilder::initialize to take only a</div><div>block size. Now all the other members are left as default values,</div><div>so that they'll be updated when commit() is called.</div></div>
</blockquote></div><br></div>