[PATCH] D25429: [analyzer] Link libStaticAnalyzerCheckers to libASTMatchers.

[Artem Dergachev via cfe-commits]

NoQ created this revision.
NoQ added reviewers: zaks.anna, dcoughlin, xazax.hun, a.sidorin, alexfh.
NoQ added a subscriber: cfe-commits.
Herald added subscribers: mgorny, beanz.

ASTMatchers are sometimes handy in the Analyzer, and there are more patches to make use of them.

However, it turns out that not only the Analyzer, but also the whole clang binary was never linked to libASTMatchers. So there's a binary size concern.

On my machine (mac) binary size increases by:

- 0.5% (73 731 296 bytes => 74 118 172 bytes) in release mode,
- 2% (243 942 672 bytes => 249 748 052 bytes) in debug mode.

Is such increase bearable? Would anybody approve that?


https://reviews.llvm.org/D25429

Files:
  lib/StaticAnalyzer/Checkers/CMakeLists.txt


Index: lib/StaticAnalyzer/Checkers/CMakeLists.txt
===================================================================
--- lib/StaticAnalyzer/Checkers/CMakeLists.txt
+++ lib/StaticAnalyzer/Checkers/CMakeLists.txt
@@ -91,6 +91,7 @@
 
   LINK_LIBS
   clangAST
+  clangASTMatchers
   clangAnalysis
   clangBasic
   clangLex


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D25429.74109.patch
Type: text/x-patch
Size: 320 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161010/86f6642d/attachment.bin>