[PATCH] D43254: Use LLVM's DenseSet insteead of unordered_set.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 13 13:39:46 PST 2018


ruiu added inline comments.


================
Comment at: lld/wasm/SymbolTable.cpp:37
 void SymbolTable::reportRemainingUndefines() {
-  std::unordered_set<Symbol *> Undefs;
+  DenseSet<Symbol *> Undefs;
   for (Symbol *Sym : SymVector) {
----------------
efriedma wrote:
> Don't you need a SetVector here?
Forgive my ignorance, but what's the difference between DenseSet and SetVector?


https://reviews.llvm.org/D43254





More information about the llvm-commits mailing list