From: Eric Liu Date: Wed, 30 May 2018 11:51:48 +0000 (+0000) Subject: Reland "Move #include manipulation code to new lib/Tooling/Inclusions." X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=f311608c6cbf4e90fa2fb2e2ddbf440e25947edb;p=clang Reland "Move #include manipulation code to new lib/Tooling/Inclusions." This reverts commit r332751 (i.e. reland r332720) after fixing module build. Differential Revision: https://reviews.llvm.org/D47068 git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@333532 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/clang/Format/Format.h b/include/clang/Format/Format.h index b4d99f0754..7565487541 100644 --- a/include/clang/Format/Format.h +++ b/include/clang/Format/Format.h @@ -16,8 +16,8 @@ #define LLVM_CLANG_FORMAT_FORMAT_H #include "clang/Basic/LangOptions.h" -#include "clang/Tooling/Core/IncludeStyle.h" #include "clang/Tooling/Core/Replacement.h" +#include "clang/Tooling/Inclusions/IncludeStyle.h" #include "llvm/ADT/ArrayRef.h" #include "llvm/Support/Regex.h" #include diff --git a/include/clang/Tooling/Core/HeaderIncludes.h b/include/clang/Tooling/Inclusions/HeaderIncludes.h similarity index 96% rename from include/clang/Tooling/Core/HeaderIncludes.h rename to include/clang/Tooling/Inclusions/HeaderIncludes.h index 1aaa3de441..d99a328316 100644 --- a/include/clang/Tooling/Core/HeaderIncludes.h +++ b/include/clang/Tooling/Inclusions/HeaderIncludes.h @@ -7,12 +7,12 @@ // //===----------------------------------------------------------------------===// -#ifndef LLVM_CLANG_TOOLING_CORE_HEADERINCLUDES_H -#define LLVM_CLANG_TOOLING_CORE_HEADERINCLUDES_H +#ifndef LLVM_CLANG_TOOLING_INCLUSIONS_HEADERINCLUDES_H +#define LLVM_CLANG_TOOLING_INCLUSIONS_HEADERINCLUDES_H #include "clang/Basic/SourceManager.h" #include "clang/Tooling/Core/Replacement.h" -#include "clang/Tooling/Core/IncludeStyle.h" +#include "clang/Tooling/Inclusions/IncludeStyle.h" #include "llvm/Support/Path.h" #include "llvm/Support/Regex.h" #include @@ -134,4 +134,4 @@ private: } // namespace tooling } // namespace clang -#endif // LLVM_CLANG_TOOLING_CORE_HEADERINCLUDES_H +#endif // LLVM_CLANG_TOOLING_INCLUSIONS_HEADERINCLUDES_H diff --git a/include/clang/Tooling/Core/IncludeStyle.h b/include/clang/Tooling/Inclusions/IncludeStyle.h similarity index 96% rename from include/clang/Tooling/Core/IncludeStyle.h rename to include/clang/Tooling/Inclusions/IncludeStyle.h index ed7bd08ce0..42d6e1a5cf 100644 --- a/include/clang/Tooling/Core/IncludeStyle.h +++ b/include/clang/Tooling/Inclusions/IncludeStyle.h @@ -7,8 +7,8 @@ // //===----------------------------------------------------------------------===// -#ifndef LLVM_CLANG_TOOLING_CORE_INCLUDESTYLE_H -#define LLVM_CLANG_TOOLING_CORE_INCLUDESTYLE_H +#ifndef LLVM_CLANG_TOOLING_INCLUSIONS_INCLUDESTYLE_H +#define LLVM_CLANG_TOOLING_INCLUSIONS_INCLUDESTYLE_H #include "llvm/Support/YAMLTraits.h" #include @@ -130,4 +130,4 @@ struct ScalarEnumerationTraits< } // namespace yaml } // namespace llvm -#endif // LLVM_CLANG_TOOLING_CORE_INCLUDESTYLE_H +#endif // LLVM_CLANG_TOOLING_INCLUSIONS_INCLUDESTYLE_H diff --git a/include/clang/module.modulemap b/include/clang/module.modulemap index 4097ad2dc7..28df963120 100644 --- a/include/clang/module.modulemap +++ b/include/clang/module.modulemap @@ -153,3 +153,8 @@ module Clang_ToolingCore { requires cplusplus umbrella "Tooling/Core" module * { export * } } + +module Clang_ToolingInclusions { + requires cplusplus + umbrella "Tooling/Inclusions" module * { export * } +} diff --git a/lib/Format/CMakeLists.txt b/lib/Format/CMakeLists.txt index 42e6d53d9f..0019d045cd 100644 --- a/lib/Format/CMakeLists.txt +++ b/lib/Format/CMakeLists.txt @@ -20,4 +20,5 @@ add_clang_library(clangFormat clangBasic clangLex clangToolingCore + clangToolingInclusions ) diff --git a/lib/Format/Format.cpp b/lib/Format/Format.cpp index c6de92b3ca..d1a5dc8e6c 100644 --- a/lib/Format/Format.cpp +++ b/lib/Format/Format.cpp @@ -31,7 +31,7 @@ #include "clang/Basic/SourceManager.h" #include "clang/Basic/VirtualFileSystem.h" #include "clang/Lex/Lexer.h" -#include "clang/Tooling/Core/HeaderIncludes.h" +#include "clang/Tooling/Inclusions/HeaderIncludes.h" #include "llvm/ADT/STLExtras.h" #include "llvm/ADT/StringRef.h" #include "llvm/Support/Allocator.h" diff --git a/lib/Tooling/CMakeLists.txt b/lib/Tooling/CMakeLists.txt index eaae49b431..031d8b51de 100644 --- a/lib/Tooling/CMakeLists.txt +++ b/lib/Tooling/CMakeLists.txt @@ -4,6 +4,7 @@ set(LLVM_LINK_COMPONENTS ) add_subdirectory(Core) +add_subdirectory(Inclusions) add_subdirectory(Refactoring) add_subdirectory(ASTDiff) diff --git a/lib/Tooling/Core/CMakeLists.txt b/lib/Tooling/Core/CMakeLists.txt index 84d17984b4..f7f423c522 100644 --- a/lib/Tooling/Core/CMakeLists.txt +++ b/lib/Tooling/Core/CMakeLists.txt @@ -2,8 +2,6 @@ set(LLVM_LINK_COMPONENTS support) add_clang_library(clangToolingCore Diagnostic.cpp - HeaderIncludes.cpp - IncludeStyle.cpp Lookup.cpp Replacement.cpp diff --git a/lib/Tooling/Inclusions/CMakeLists.txt b/lib/Tooling/Inclusions/CMakeLists.txt new file mode 100644 index 0000000000..00afb50f3a --- /dev/null +++ b/lib/Tooling/Inclusions/CMakeLists.txt @@ -0,0 +1,12 @@ +set(LLVM_LINK_COMPONENTS support) + +add_clang_library(clangToolingInclusions + HeaderIncludes.cpp + IncludeStyle.cpp + + LINK_LIBS + clangBasic + clangLex + clangRewrite + clangToolingCore + ) diff --git a/lib/Tooling/Core/HeaderIncludes.cpp b/lib/Tooling/Inclusions/HeaderIncludes.cpp similarity index 99% rename from lib/Tooling/Core/HeaderIncludes.cpp rename to lib/Tooling/Inclusions/HeaderIncludes.cpp index d974d16f01..99c0866a68 100644 --- a/lib/Tooling/Core/HeaderIncludes.cpp +++ b/lib/Tooling/Inclusions/HeaderIncludes.cpp @@ -7,7 +7,7 @@ // //===----------------------------------------------------------------------===// -#include "clang/Tooling/Core/HeaderIncludes.h" +#include "clang/Tooling/Inclusions/HeaderIncludes.h" #include "clang/Basic/SourceManager.h" #include "clang/Lex/Lexer.h" diff --git a/lib/Tooling/Core/IncludeStyle.cpp b/lib/Tooling/Inclusions/IncludeStyle.cpp similarity index 95% rename from lib/Tooling/Core/IncludeStyle.cpp rename to lib/Tooling/Inclusions/IncludeStyle.cpp index 74eec481c2..3597710f1f 100644 --- a/lib/Tooling/Core/IncludeStyle.cpp +++ b/lib/Tooling/Inclusions/IncludeStyle.cpp @@ -7,8 +7,7 @@ // //===----------------------------------------------------------------------===// -#include "clang/Tooling/Core/IncludeStyle.h" - +#include "clang/Tooling/Inclusions/IncludeStyle.h" using clang::tooling::IncludeStyle; diff --git a/unittests/Tooling/HeaderIncludesTest.cpp b/unittests/Tooling/HeaderIncludesTest.cpp index 4407221f8a..ff68f75a6e 100644 --- a/unittests/Tooling/HeaderIncludesTest.cpp +++ b/unittests/Tooling/HeaderIncludesTest.cpp @@ -7,7 +7,7 @@ // //===----------------------------------------------------------------------===// -#include "clang/Tooling/Core/HeaderIncludes.h" +#include "clang/Tooling/Inclusions/HeaderIncludes.h" #include "../Tooling/ReplacementTest.h" #include "../Tooling/RewriterTestContext.h" #include "clang/Format/Format.h"