[PATCH] D47068: Move #include manipulation code to new lib/Tooling/Inclusions.

[Ilya Biryukov via Phabricator via cfe-commits]

ilya-biryukov accepted this revision.
ilya-biryukov added a comment.
This revision is now accepted and ready to land.

LGTM with a minor naming suggestion.



================
Comment at: include/clang/Format/Format.h:20
 #include "clang/Tooling/Core/Replacement.h"
+#include "clang/Tooling/Inclusions/IncludeStyle.h"
 #include "llvm/ADT/ArrayRef.h"
----------------
Maybe `Tooling/Includes`?
I think inclusions is somewhat uncommon, while includes is more widely used. Don't have a strong opinion, though.


Repository:
  rC Clang

https://reviews.llvm.org/D47068