[PATCH] D32060: BitVector: add iterators for bits that are set

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 21 11:45:14 PDT 2017


MatzeB added a comment.

This is looking pretty good, but I'd expect the range based for loops to use the same type as operator() of `operator*()`, i.e. `for (unsigned Bit : Empty.set_bits()) {}.


https://reviews.llvm.org/D32060





More information about the llvm-commits mailing list