[include-cleaner] Fix link errors when -DBUILD_SHARED_LIBS=ON

Kai Luo · Kai Luo · commit 2e73129483c4 · 2022-10-19T07:26:08.000Z
Fixed ppc buildbot https://lab.llvm.org/buildbot/#/builders/121/builds/24273 which is using `-DBUILD_SHARED_LIBS=ON`. Reviewed By: sammccall Differential Revision: https://reviews.llvm.org/D136229
diff --git a/clang-tools-extra/include-cleaner/lib/CMakeLists.txt b/clang-tools-extra/include-cleaner/lib/CMakeLists.txt
@@ -6,7 +6,8 @@ add_clang_library(clangIncludeCleaner
   WalkAST.cpp
 
   LINK_LIBS
-  clangBasic
   clangAST
+  clangBasic
+  clangLex
   )
 
diff --git a/clang-tools-extra/include-cleaner/tool/CMakeLists.txt b/clang-tools-extra/include-cleaner/tool/CMakeLists.txt
@@ -4,6 +4,8 @@ include_directories("../lib") # FIXME: use public APIs instead.
 add_clang_tool(clang-include-cleaner IncludeCleaner.cpp)
 clang_target_link_libraries(clang-include-cleaner PRIVATE
   clangBasic
+  clangFrontend
+  clangSerialization
   clangTooling
   )
 target_link_libraries(clang-include-cleaner PRIVATE

Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,8 @@ add_clang_library(clangIncludeCleaner`
`6`	`6`	`WalkAST.cpp`
`7`	`7`
`8`	`8`	`LINK_LIBS`
`9`		`- clangBasic`
`10`	`9`	`clangAST`
	`10`	`+ clangBasic`
	`11`	`+ clangLex`
`11`	`12`	`)`
`12`	`13`
Original file line number	Diff line number	Diff line change
`@@ -4,6 +4,8 @@ include_directories("../lib") # FIXME: use public APIs instead.`
`4`	`4`	`add_clang_tool(clang-include-cleaner IncludeCleaner.cpp)`
`5`	`5`	`clang_target_link_libraries(clang-include-cleaner PRIVATE`
`6`	`6`	`clangBasic`
	`7`	`+ clangFrontend`
	`8`	`+ clangSerialization`
`7`	`9`	`clangTooling`
`8`	`10`	`)`
`9`	`11`	`target_link_libraries(clang-include-cleaner PRIVATE`