r201256 - [examples] Use loadable modules instead of shared libraries for Clang plugins.

NAKAMURA Takumi geek4civic at gmail.com
Thu Feb 13 07:57:31 PST 2014


Jordan, thanks for working!
And I am sorry I mis-introduced dependent libraries despite of plugin module.

FYI, I woked llvm_add_library(MODULE) would be semantically equivalent
to add_llvm_loadable_module.


> ==============================================================================
> --- cfe/trunk/examples/analyzer-plugin/CMakeLists.txt (original)
> +++ cfe/trunk/examples/analyzer-plugin/CMakeLists.txt Wed Feb 12 15:04:23 2014
> @@ -1,8 +1,4 @@
> -set(LLVM_LINK_COMPONENTS
> -  Support
> -  )
> -
> -add_clang_library(SampleAnalyzerPlugin MODULE MainCallChecker.cpp)
> +add_llvm_loadable_module(SampleAnalyzerPlugin MainCallChecker.cpp)
>
>  add_dependencies(SampleAnalyzerPlugin
>    ClangAttrClasses
> @@ -11,15 +7,7 @@ add_dependencies(SampleAnalyzerPlugin
>    ClangDeclNodes
>    ClangDiagnosticCommon
>    ClangStmtNodes
> -  )
> -
> -target_link_libraries(SampleAnalyzerPlugin
>    clangAST
>    clangAnalysis
>    clangStaticAnalyzerCore
>    )

One-point nitpick; Did you really need to add dependencies to these libraries?
I guess you would like ClangCommentCommandList here.

I will prune whole them soon.

> -
> -set_target_properties(SampleAnalyzerPlugin
> -  PROPERTIES
> -  LINKER_LANGUAGE CXX
> -  PREFIX "")



More information about the cfe-commits mailing list