arphaman added a comment.

I think this patch should be split into a number of smaller patches to help the 
review process.

Things like `tools/IndexStore`, `DirectoryWatcher` and other components that 
are not directly needed right now should definitely be in their own patches.
It would be nice to find some way to split the implementation into multiple 
patches as well.


https://reviews.llvm.org/D39050



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

Reply via email to