[PATCH] D43437: Fix link failures for Preprocessor::addCommentHandler
Heejin Ahn via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Feb 18 00:22:12 PST 2018
aheejin created this revision.
aheejin added a reviewer: ioeric.
Herald added subscribers: cfe-commits, jkorous-apple, ilya-biryukov, mgorny, klimek.
https://reviews.llvm.org/D42640 adds calls to `Preprocessor::addCommentHandler` in
`unittests/clangd/SymbolCollectorTests.cpp` and
`clangd/global-symbol-builder/GlobalSymbolBuilderMain.cpp` but does not
link `clangLex` library. This causes undefined reference errors when
built with `-DBUILD_SHARED_LIBS=ON`.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D43437
Files:
clangd/global-symbol-builder/CMakeLists.txt
unittests/clangd/CMakeLists.txt
Index: unittests/clangd/CMakeLists.txt
===================================================================
--- unittests/clangd/CMakeLists.txt
+++ unittests/clangd/CMakeLists.txt
@@ -38,6 +38,7 @@
clangFormat
clangFrontend
clangIndex
+ clangLex
clangSema
clangTooling
clangToolingCore
Index: clangd/global-symbol-builder/CMakeLists.txt
===================================================================
--- clangd/global-symbol-builder/CMakeLists.txt
+++ clangd/global-symbol-builder/CMakeLists.txt
@@ -15,5 +15,6 @@
clangDaemon
clangBasic
clangFrontend
+ clangLex
clangTooling
)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D43437.134825.patch
Type: text/x-patch
Size: 614 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180218/af728bc6/attachment.bin>
More information about the cfe-commits
mailing list