[PATCH] D20804: [include-fixer] collect the number of times a symbols is found in an indexing run and use it for symbols popularity ranking.

Benjamin Kramer via cfe-commits cfe-commits at lists.llvm.org
Tue May 31 02:10:28 PDT 2016


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





More information about the cfe-commits mailing list