[PATCH] D74984: [ADT] Add CoalescingBitVector, implemented using IntervalMap [1/3]
Adrian Prantl via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 25 09:15:53 PST 2020
aprantl added inline comments.
================
Comment at: llvm/include/llvm/ADT/CoalescingBitVector.h:100
+ ///
+ /// This method does /not/ support setting a bit that has already been set,
+ /// for efficiency reasons. If possible, restructure your code to not set the
----------------
What happens if I set an already-set bit? Will the data structure get inconsistent, less efficient, crash? Should there be an assert(!test(Index)) here?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D74984/new/
https://reviews.llvm.org/D74984
More information about the llvm-commits
mailing list