fhahn added a comment. Yeah, I am not really sure there is a strong reason to change it. I just went for SmallPtrSet, because it is much wider used than SmallSet. And using `SmallPtrSetImpl<> &` on a SmallSet looks suprising. https://reviews.llvm.org/D47836