This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Link clangTidy into clangd tests
ClosedPublic

Authored by nridge on Jan 22 2019, 6:50 PM.

Reviewers

Commits

After pulling the latest code, I found this change was necessary to get past the following linker error:

clang-tools-extra/unittests/clangd/TestTU.cpp:9443: error: undefined reference to 'clang::tidy::ClangTidyOptions::getDefaults()'

I assume this is a regression from https://reviews.llvm.org/D55256.

LGtM, thanks!

This revision is now accepted and ready to land.Jan 23 2019, 12:00 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

clang-tools-extra/

trunk/

unittests/

clangd/

1 line

Show First 20 Lines • Show All 53 Lines • ▼ Show 20 Lines	target_link_libraries(ClangdTests
clangBasic		clangBasic
clangDaemon		clangDaemon
clangFormat		clangFormat
clangFrontend		clangFrontend
clangIndex		clangIndex
clangLex		clangLex
clangSema		clangSema
clangSerialization		clangSerialization
		clangTidy
clangTooling		clangTooling
clangToolingCore		clangToolingCore
clangToolingInclusions		clangToolingInclusions
LLVMSupport		LLVMSupport
LLVMTestingSupport		LLVMTestingSupport
)		)

if (CLANGD_BUILD_XPC)		if (CLANGD_BUILD_XPC)
add_subdirectory(xpc)		add_subdirectory(xpc)
endif ()		endif ()