[PATCH] D17884: Disallow duplication of imported entities

don hinton via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 6 08:34:23 PST 2016


hintonda added a comment.

Could you use a SetVector or SmallSetVector instead of a DenseSet for DIImportedEntities?  It's equivalent to what you've proposed, but since it knows when it inserts a new one, maintaining the vector is much cheaper.


http://reviews.llvm.org/D17884





More information about the llvm-commits mailing list