[PATCH] D108124: Fix type in DenseMap<BitVector, *> to match V.size()

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 16 06:44:41 PDT 2021


rengolin added a comment.

Or, being conservatively (with container sizes), we can use `size_type size()` on both cases, because that's what the typedef seems to imply in the first place.

It's also the type of `count()` for both versions, which is equal or larger than `size`, so `size_t` is overkill.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108124/new/

https://reviews.llvm.org/D108124



More information about the llvm-commits mailing list