This is an archive of the discontinued LLVM Phabricator instance.

Differential D22046

[clang-tidy] Add dependency on clang-headers
ClosedPublic

Authored by thakis on Jul 6 2016, 7:32 AM.

Details

Reviewers
alexfh
Summary

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.

Diff Detail

Event Timeline

thakis updated this revision to Diff 62862.Jul 6 2016, 7:32 AM
thakis retitled this revision from to [clang-tidy] Add dependency on clang-headers.
thakis updated this object.
thakis added a reviewer: alexfh.
thakis added a subscriber: cfe-commits.
alexfh accepted this revision.Jul 6 2016, 2:27 PM
alexfh edited edge metadata.

LG

This revision is now accepted and ready to land.Jul 6 2016, 2:27 PM
thakis added a comment.Jul 7 2016, 6:27 AM

r274751, thanks!

Eugene.Zelenko closed this revision.Jul 7 2016, 8:00 AM
Eugene.Zelenko added a subscriber: Eugene.Zelenko.

Committed in r274751.

Revision Contents

PathSize
clang-tidy/
tool/
3 lines

Diff 62862

clang-tidy/tool/CMakeLists.txt

set(LLVM_LINK_COMPONENTS set(LLVM_LINK_COMPONENTS
support support
) )
add_clang_executable(clang-tidy add_clang_executable(clang-tidy
ClangTidyMain.cpp ClangTidyMain.cpp
) )
add_dependencies(clang-tidy
clang-headers
)
target_link_libraries(clang-tidy target_link_libraries(clang-tidy
clangAST clangAST
clangASTMatchers clangASTMatchers
clangBasic clangBasic
clangTidy clangTidy
clangTidyBoostModule clangTidyBoostModule
clangTidyCERTModule clangTidyCERTModule
clangTidyCppCoreGuidelinesModule clangTidyCppCoreGuidelinesModule
Show All 14 Lines