[PATCH] D33088: [LiveVariables] Switch Kill/Defs sets to be `DenseSet`

[Davide Italiano via Phabricator via llvm-commits]

davide added a comment.

In https://reviews.llvm.org/D33088#752754, @sanjoy wrote:

> How big does the set get?  Did you try using `SmallDenseSet` instead?


Hi Sanjoy, I tried to switch to `SmallDenseSet` and that slows down things a little 
`2min38sec` -> 2min55sec`

My set doesn't grow too much, it's < 100 elements, roughly.

Feel free to follow-up if you have any questions, and thanks for your comment!

-

Davide


Repository:
  rL LLVM

https://reviews.llvm.org/D33088