[PATCH] Initial attempt to restructure libTooling and move parts out that don't need to depend on lib/AST.

Daniel Jasper djasper at google.com
Fri Oct 24 07:21:15 PDT 2014


Hi klimek,

This is an approach to split libTooling into two parts. This should not require any change to libTooling users.

Alternatively, we could move some of these to lib/Rewrite. However, that would mean, we should also change the namespace of clang::tooling::Replacement and that is something that is used frequently by libTooling users.

This restructuring cuts the remaining dependency on clang-format on lib/AST effectively cutting its binary size in half.

Once we settle on a structure, I'll also update the Makefiles.

http://reviews.llvm.org/D5973

Files:
  include/clang/Tooling/Core/Replacement.h
  include/clang/Tooling/Refactoring.h
  lib/Format/CMakeLists.txt
  lib/Tooling/CMakeLists.txt
  lib/Tooling/Core/CMakeLists.txt
  lib/Tooling/Core/Replacement.cpp
  lib/Tooling/Refactoring.cpp
  tools/clang-format/CMakeLists.txt
  unittests/Format/CMakeLists.txt
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5973.15412.patch
Type: text/x-patch
Size: 41267 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141024/d12ee7c6/attachment.bin>


More information about the cfe-commits mailing list