This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy][CMake] Make clang-tidy usable as distribution component
ClosedPublic

Authored by phosek on May 3 2017, 10:28 AM.

Reviewers

Commits

Summary

Use add_clang_tool rather than add_clang_executable to support clang-tidy as a distribution component.

I'm not sure I understand all implications, but from a cursory glance this seems reasonable.

This revision is now accepted and ready to land.May 5 2017, 4:46 AM

Yep, this seems straight forward.

This revision was automatically updated to reflect the committed changes.

Path

Size

clang-tools-extra/

trunk/

clang-tidy/

tool/

2 lines

	set(LLVM_LINK_COMPONENTS			set(LLVM_LINK_COMPONENTS
	support			support
	)			)

	add_clang_executable(clang-tidy			add_clang_tool(clang-tidy
	ClangTidyMain.cpp			ClangTidyMain.cpp
	)			)
	add_dependencies(clang-tidy			add_dependencies(clang-tidy
	clang-headers			clang-headers
	)			)
	target_link_libraries(clang-tidy			target_link_libraries(clang-tidy
	clangAST			clangAST
	clangASTMatchers			clangASTMatchers
	Show All 22 Lines