This is an archive of the discontinued LLVM Phabricator instance.

[index] Added a method indexTopLevelDecls to run indexing on a list of Decls.
ClosedPublic

Authored by ilya-biryukov on Jul 14 2017, 2:36 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

ilya-biryukov created this revision.Jul 14 2017, 2:36 AM
arphaman requested changes to this revision.Jul 14 2017, 3:20 AM

Nice, I will need something like this for the refactoring stress test tool in the future that verifies that the indexer and renaming engine have a similar view of the code.

include/clang/Index/IndexingAction.h
21 ↗(On Diff #106595)

Please arrange the added declarations so that the whole list is in ordered alphabetically.

lib/Index/IndexingAction.cpp
181 ↗(On Diff #106595)

clang-format?

This revision now requires changes to proceed.Jul 14 2017, 3:20 AM
ilya-biryukov edited edge metadata.
  • Fixed formatting.
ilya-biryukov marked 2 inline comments as done.Jul 14 2017, 3:24 AM
ilya-biryukov added inline comments.
lib/Index/IndexingAction.cpp
181 ↗(On Diff #106595)

Sorry, didn't rerun it after a rename of the method. Fixed now.

arphaman accepted this revision.Jul 14 2017, 3:25 AM

LGTM, thanks.

This revision is now accepted and ready to land.Jul 14 2017, 3:25 AM
This revision was automatically updated to reflect the committed changes.