bkramer added inline comments.

================
Comment at: include-fixer/find-all-symbols/tool/FindAllSymbolsMain.cpp:91
@@ -90,2 +90,3 @@
   std::set<SymbolInfo> UniqueSymbols;
+  std::map<SymbolInfo, int> NumOccurrences;
   std::mutex SymbolMutex;
----------------
Can we remove the set now? 2 copies of all SymbolInfos seems unnecessary.


http://reviews.llvm.org/D20804



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to