[PATCH] D17923: Add getSetRef() to SetVector

Amjad Aboud via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 7 01:56:52 PST 2016


aaboud added a comment.

I do not think we need this new method, no need to access the underlying set directly.

1. If you use "insert()" you assure that the element will be added only once to the SetVector.
2. If you want to check if the element is already in the SetVector, you may use the function "count()".


http://reviews.llvm.org/D17923





More information about the llvm-commits mailing list