[PATCH] D22046: [clang-tidy] Add dependency on clang-headers

Nico Weber via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 6 07:32:15 PDT 2016


thakis created this revision.
thakis added a reviewer: alexfh.
thakis added a subscriber: cfe-commits.

Currently, to be able to process a source file including e.g. stddef.h with clang-tidy, one has to build both clang-tidy and the clang-headers target.  Since stddef.h is needed for virtually any source file, let clang-tidy depend on clang-headers, so that it Just Works after it has been built.

http://reviews.llvm.org/D22046

Files:
  clang-tidy/tool/CMakeLists.txt

Index: clang-tidy/tool/CMakeLists.txt
===================================================================
--- clang-tidy/tool/CMakeLists.txt
+++ clang-tidy/tool/CMakeLists.txt
@@ -5,6 +5,9 @@
 add_clang_executable(clang-tidy
   ClangTidyMain.cpp
   )
+add_dependencies(clang-tidy
+  clang-headers
+  )
 target_link_libraries(clang-tidy
   clangAST
   clangASTMatchers


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22046.62862.patch
Type: text/x-patch
Size: 368 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160706/9e4cc395/attachment.bin>


More information about the cfe-commits mailing list