[PATCH] D18578: Introduce BitSet: A BitVector based class behaving like std::set/DenseSet

[David Blaikie via llvm-commits]

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)

On Fri, Apr 1, 2016 at 4:58 PM, Matthias Braun via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> MatzeB retitled this revision from "BitVector: Add insert(), erase() and
> count() methods." to "Introduce BitSet: A BitVector based class behaving
> like std::set/DenseSet".
> MatzeB updated the summary for this revision.
> MatzeB updated this revision to Diff 52446.
> MatzeB added a comment.
>
> New revision introducing a new proxy class acting like a set but backed by
> a BitVector.
>
> Also comes with more extensive unittests for set classes.
>
>
> Repository:
>   rL LLVM
>
> http://reviews.llvm.org/D18578
>
> Files:
>   include/llvm/ADT/BitSet.h
>   unittests/ADT/CMakeLists.txt
>   unittests/ADT/DenseSetTest.cpp
>   unittests/ADT/SetTest.cpp
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160413/d38dc417/attachment.html>