[PATCH] D77985: Fix bug in SmallBitVector::find_next_unset

Denis Antrushin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 21 11:52:47 PDT 2020


dantrushin added inline comments.


================
Comment at: llvm/include/llvm/ADT/SmallBitVector.h:296
 
       if (Bits == ~uintptr_t(0) || Prev + 1 >= getSmallSize())
         return -1;
----------------
Wouldn't it be simpler to leave `++Prev` from line 290 in place and then just use `Prev` ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77985





More information about the llvm-commits mailing list