<div dir="ltr">This seems to have strange behavior - returning the size/empty status of the BitVector for the BitSets operations is probably wrong, no? (the BitVector's length is the length of the field, not the number of set (1) bits in it)</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 1, 2016 at 4:58 PM, Matthias Braun 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">MatzeB retitled this revision from "BitVector: Add insert(), erase() and count() methods." to "Introduce BitSet: A BitVector based class behaving like std::set/DenseSet".<br>
MatzeB updated the summary for this revision.<br>
MatzeB updated this revision to Diff 52446.<br>
MatzeB added a comment.<br>
<br>
New revision introducing a new proxy class acting like a set but backed by a BitVector.<br>
<br>
Also comes with more extensive unittests for set classes.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D18578" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18578</a><br>
<br>
Files:<br>
  include/llvm/ADT/BitSet.h<br>
  unittests/ADT/CMakeLists.txt<br>
  unittests/ADT/DenseSetTest.cpp<br>
  unittests/ADT/SetTest.cpp<br>
<br>
<br>_______________________________________________<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/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>