Update {Small}BitVector size_type definition
authorRenato Golin <rengolin@systemcall.eu>
Wed, 18 Aug 2021 10:50:15 +0000 (11:50 +0100)
committerRenato Golin <rengolin@microsoft.com>
Thu, 19 Aug 2021 10:13:38 +0000 (11:13 +0100)
commit894ad26bd55f91cbcfd12f5f54426efb859dc14d
tree33e60975704df32c5e09c57b6af059851d5b33b4
parentdcc6b7b1d5e5a0f9537ce1bf919ac2338bd7ad7b
Update {Small}BitVector size_type definition

SmallBitVector implements a level of indirection over BitVector by
storing a smaller bit-vector in a pointer-sized element, or in case the
number of elements exceeds the bucket size, it creates a new pointer to
a BitVector and uses that as its storage.

However, the functions returning the vector size were using `unsigned`,
which is ok for BitVector, but not for SmallBitVector, which is actually
`uintptr_t`.

This commit reuses the `size_type` definition to more than just `count`
and propagates them into range iteration, size calculation, etc.

This is a continuation of D108124.

I haven't changed all occurrences of `unsigned` or `uintptr_t` to
`size_type`, just those that were directly related.

Following directions from clang-tidy on case of variables.

Differential Revision: https://reviews.llvm.org/D108290
llvm/include/llvm/ADT/BitVector.h
llvm/include/llvm/ADT/SmallBitVector.h