This is an archive of the discontinued LLVM Phabricator instance.

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

Authored by ioeric on May 31 2016, 1:55 AM.

Diff Detail

Event Timeline

ioeric updated this revision to Diff 59025.May 31 2016, 1:55 AM
ioeric retitled this revision from to [include-fixer] collect the number of times a symbols is found in an indexing run and use it for symbols popularity ranking..
ioeric updated this object.
ioeric added a reviewer: bkramer.
ioeric added subscribers: djasper, hokein, cfe-commits.
bkramer added inline comments.May 31 2016, 2:10 AM
include-fixer/find-all-symbols/tool/FindAllSymbolsMain.cpp
91

Can we remove the set now? 2 copies of all SymbolInfos seems unnecessary.

ioeric updated this revision to Diff 59027.May 31 2016, 2:31 AM
  • Removed a redundant set of symbols during merging.
bkramer accepted this revision.May 31 2016, 2:36 AM
bkramer edited edge metadata.

lg

This revision is now accepted and ready to land.May 31 2016, 2:36 AM
This revision was automatically updated to reflect the committed changes.