[PATCH] D53260: [ADT] Fix a bug in DenseSet's initializer_list constructor.

[Lang Hames via Phabricator via llvm-commits]

lhames closed this revision.
lhames added a comment.

Committed as r344542.

In https://reviews.llvm.org/D53260#1265686, @dblaikie wrote:

> Looks good to me (I guess the capacity thing there could be tested too - ensuring that it didn't roll up to a larger capacity than intended?)


Unfortunately DenseSet / DenseMap do not expose a capacity function at the moment. I believe it's just getNumBuckets, so unless there is some reason not to I would be in favor of adding a public 'capacity' function that returns that. Then we could add that check to the unit test.


Repository:
  rL LLVM

https://reviews.llvm.org/D53260