I can probably also just use the pdb::HashTable class where the key is the actual hash value.  I’ll go with the vector first since it’s simple and obviously correct then try to change it to something more efficient in a followup <br><div class="gmail_quote"><div dir="ltr">On Wed, Sep 19, 2018 at 4:23 PM Reid Kleckner via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">rnk added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D52283#1239983" rel="noreferrer" target="_blank">https://reviews.llvm.org/D52283#1239983</a>, @zturner wrote:<br>
<br>
> It’s true that nested containers don’t usually perform well, but in this<br>
>  case the outer vector *never* resizes. So it seems like it should be ok?<br>
<br>
<br>
I'd be fine with the simple code, sure. We want the PDB to load fast, though, being cache friendly is the best way to do that, IMO. :)<br>
<br>
<br>
<a href="https://reviews.llvm.org/D52283" rel="noreferrer" target="_blank">https://reviews.llvm.org/D52283</a><br>
<br>
<br>
<br>
</blockquote></div>