This is an archive of the discontinued LLVM Phabricator instance.

Differential D43371

[clang-include-fixer] Use add_clang_tool instead of add_clang_executable
ClosedPublic

Authored by phosek on Feb 15 2018, 11:25 PM.

Details

Reviewers
klimek
sammccall
bkramer
hokein
beanz
Commits
rG330a00c41e3f: [clang-include-fixer] Use add_clang_tool instead add_clang_executable
rL325381: [clang-include-fixer] Use add_clang_tool instead add_clang_executable
rCTE325381: [clang-include-fixer] Use add_clang_tool instead add_clang_executable
Summary

This makes it possible to include clang-include-fixer as distribution
component when building Clang based toolchain using CMake.

Diff Detail

Event Timeline

phosek created this revision.Feb 15 2018, 11:25 PM
Herald added subscribers: cfe-commits, mgorny. · View Herald TranscriptFeb 15 2018, 11:25 PM
phosek added a reviewer: beanz.Feb 15 2018, 11:26 PM
phosek retitled this revision from [clang-include-fixer] Use add_clang_tool instead add_clang_executable to [clang-include-fixer] Use add_clang_tool instead of add_clang_executable.Feb 15 2018, 11:47 PM
bkramer accepted this revision.Feb 16 2018, 6:52 AM

lg

This revision is now accepted and ready to land.Feb 16 2018, 6:52 AM
Closed by commit rCTE325381: [clang-include-fixer] Use add_clang_tool instead add_clang_executable (authored by phosek). · Explain WhyFeb 16 2018, 12:28 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
include-fixer/
tool/
2 lines

Diff 134691

include-fixer/tool/CMakeLists.txt

include_directories(${CMAKE_CURRENT_SOURCE_DIR}/..) include_directories(${CMAKE_CURRENT_SOURCE_DIR}/..)
add_clang_executable(clang-include-fixer add_clang_tool(clang-include-fixer
ClangIncludeFixer.cpp ClangIncludeFixer.cpp
) )
target_link_libraries(clang-include-fixer target_link_libraries(clang-include-fixer
PRIVATE PRIVATE
clangBasic clangBasic
clangFormat clangFormat
clangFrontend clangFrontend
Show All 16 Lines