[PATCH] D68172: Don't install example analyzer plugins

Aaron Puchert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 27 18:39:17 PDT 2019


aaronpuchert added a comment.

In D68172#1686772 <https://reviews.llvm.org/D68172#1686772>, @NoQ wrote:

> + at Szelethus because i'm a bit out-of-the-loop on plugins: i very vaguely remember that we decided to put them into tests(?)


The plugins were in `test`, but D62445 <https://reviews.llvm.org/D62445> changed that. Another solution would have been to move them into the `unittests` directory. But other example plugins are also part of the ordinary source tree, see for example `llvm/lib/Transforms/Hello`. In fact this change is based on LLVMHello, which also has `BUILDTREE_ONLY`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68172/new/

https://reviews.llvm.org/D68172





More information about the cfe-commits mailing list