[PATCH] D59464: [analyzer] Add an example plugin for checker dependency handling
Kristóf Umann via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 16 15:33:27 PDT 2019
Szelethus marked 2 inline comments as done.
Szelethus added a comment.
If there's a cmake variable for fuchsia, Could you please try to add them here? (see inlines) I don't have access to that particular platform.
================
Comment at: cfe/trunk/test/Analysis/plugins/CheckerDependencyHandling/CMakeLists.txt:4
+
+if(LLVM_ENABLE_PLUGINS AND (WIN32 OR CYGWIN))
+ target_link_libraries(CheckerDependencyHandlingAnalyzerPlugin PRIVATE
----------------
```
if(LLVM_ENABLE_PLUGINS AND (WIN32 OR CYGWIN OR FUCHSIA))
```
================
Comment at: cfe/trunk/test/Analysis/plugins/SampleAnalyzer/CMakeLists.txt:4
+
+if(LLVM_ENABLE_PLUGINS AND (WIN32 OR CYGWIN))
+ target_link_libraries(SampleAnalyzerPlugin PRIVATE
----------------
Szelethus wrote:
> Is it possible that if we tweaked this line the error would go away?
```
if(LLVM_ENABLE_PLUGINS AND (WIN32 OR CYGWIN OR FUCHSIA))
```
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59464/new/
https://reviews.llvm.org/D59464
More information about the llvm-commits
mailing list