It's worth profiling but I would imagine performance to be the same or faster.  Conceptually the only difference is storing longs internally or bytes internally.  The latter allows fewer bitwise operations on average to extract or set a given bit, so i would imagine performance to be the same or faster.  Regardless, since this is a performance sensitive data structure, we should profile.<br><br>Design wise, i think this actually makes it more generic, not less.  If you have a bit vector, it's reasonable to expect you can access the underlying bytes, but you can't when the internal representation is a sequence of longs.<br><br><div class="gmail_quote"><div dir="ltr">On Wed, Aug 10, 2016 at 9:25 PM Madhur Amilkanthwar <<a href="mailto:madhur13490@gmail.com">madhur13490@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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"></div></div><div class="gmail_extra"><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></div></div><div class="gmail_extra"><div class="gmail_quote"><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>
</blockquote></div></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="m_-4111678237413071207HOEnZb"><div class="m_-4111678237413071207h5"><br>
<br>
<a href="https://reviews.llvm.org/D23384" rel="noreferrer" target="_blank">https://reviews.llvm.org/D23384</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">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/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div></div><div class="gmail_extra"><br><br clear="all"><div><br></div>-- <br><div class="m_-4111678237413071207gmail_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></blockquote></div>