<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 11, 2016 at 1:50 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Thu, Aug 11, 2016 at 1:45 PM, Daniel Berlin <span dir="ltr"><<a href="mailto:dberlin@dberlin.org" target="_blank">dberlin@dberlin.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span>On Thu, Aug 11, 2016 at 1:39 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">This patch is to store bits in the little-endian order regardless of host endianness (or, in other words, use a byte array as the internal storage instead of a word array), so that you can easily set/get internal buffers. We could define a new class for doing this, but is this bad for the existing BitVector?</div></blockquote><div><br></div></span><div>Let's see!</div>Have you benchmarked it?</div></div></div></blockquote><div><br></div></span><div>Nope, will do. Do we have a benchmark suite or something?</div></div></div></div></blockquote><div><br></div><div>Yeah, folks measure compile time on the large stuff in tests and other random code in as less noisy a way as they can.</div><div>  <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> How do you usually measure the compiler performance?</div></div></div></div></blockquote><div><br></div><div>For something like this, you could probably just microbenchmark the set operations/etc before and after and that would IMHO, be okay enough.</div><div><br></div><div>Pretty sure you should be able to get <a href="https://github.com/google/benchmark">https://github.com/google/benchmark</a> working with these functions without too much trouble.</div><div><br></div><div>In fact, BitVector.h only depends on countTrailingZeros and alignTo (if you comment out the llvm/support/MathExtras.h, that is what fails)</div><div><br></div><div>You can just copy those definitions pretty easily.</div><div><br></div><div><br></div><div><br></div></div></div></div>