[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