[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