[PATCH] D20814: [include-fixer] Rank symbols based on the number of occurrences we found while merging.

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


bkramer created this revision.
bkramer added reviewers: ioeric, djasper.
bkramer added a subscriber: cfe-commits.

This sorts based on the popularity of the header, not the symbol. If
there are mutliple matching symbols in one header we take the maximum
popularity for that header and deduplicate. If we know nothing we sort
lexicographically based on the header path.

http://reviews.llvm.org/D20814

Files:
  include-fixer/SymbolIndexManager.cpp
  include-fixer/tool/ClangIncludeFixer.cpp
  test/include-fixer/Inputs/fake_yaml_db.yaml
  test/include-fixer/ranking.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D20814.59062.patch
Type: text/x-patch
Size: 4129 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160531/46d70e66/attachment-0001.bin>


More information about the cfe-commits mailing list