<div dir="ltr">I agree with David. You mentioned the reasons about design choices and things which would be allowed with this patch but what about performance?</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 11, 2016 at 8:36 AM, David Majnemer via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">majnemer added a subscriber: majnemer.<br>
majnemer added a comment.<br>
<br>
Have you actually measured this to be a major improvement? Are there so many bits that the old way is a major bottleneck?<br>
The BitVector is used for the CodeGen and optimizer, I'm not entirely convinced it makes sense to modify this generic datastructure so drastically for PDBs...<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="https://reviews.llvm.org/D23384" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D23384</a><br>
<br>
<br>
<br>
______________________________<wbr>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><i style="font-size:12.8px">Disclaimer: Views, concerns, thoughts, questions, ideas expressed in this mail are of my own and my employer has no take in it. </i><br></div><div>Thank You.<br>Madhur D. Amilkanthwar<br><br></div></div></div>
</div>