mclow.lists accepted this revision.
mclow.lists added inline comments.
This revision is now accepted and ready to land.


================
Comment at: include/__hash_table:2141
         __n = 2;
     else if (__n & (__n - 1))
         __n = __next_prime(__n);
----------------
danalbert wrote:
> With `rehash(0)` this is `0 & (0 - 1)`, which triggers 
> unsigned-integer-overflow.
Grumble, grumble. That's not UB, that's just UBSan whining.
On the other hand, this doesn't appear to change any behavior, and shuts UBSan 
up.


Repository:
  rCXX libc++

https://reviews.llvm.org/D40743



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to