arphaman requested changes to this revision.
arphaman added a comment.
This revision now requires changes to proceed.

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.



================
Comment at: include/clang/Index/IndexingAction.h:21
   class FrontendAction;
+  class ASTContext;
+  class Decl;
----------------
Please arrange the added declarations so that the whole list is in ordered 
alphabetically.


================
Comment at: lib/Index/IndexingAction.cpp:181
+void index::indexTopLevelDecls(ASTContext &Ctx, ArrayRef<const Decl *> Decls,
+                        std::shared_ptr<IndexDataConsumer> DataConsumer,
+                        IndexingOptions Opts) {
----------------
clang-format?


https://reviews.llvm.org/D35405



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to