[PATCH] D37188: [clang-tools-extra] [cmake] Support running extra clang tool tests without static analyzer

Michał Górny via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Aug 27 03:18:02 PDT 2017


mgorny created this revision.

Support running the extra clang tool tests when the static analyzer
is disabled. Disable the relevant clang-tidy tests and one include-fixer
test that require it to work.

Previously, the tests were disabled entirely with
CLANG_ENABLE_STATIC_ANALYZER being false. Now, the tests are being
enabled and the relevant tests are excluded and marked unsupported
appropriately.

In order to disable clang-tidy tests, the whole test directory is added
to the exclude lists, to avoid having to explicitly add 'REQUIRES' line
to every single test. If the other solution is preferable, I can update
the patch.

The yamldb_plugin include-fixer test is also updated to be disabled
without static analyzer. It fails in that case because clang is not
outputting a replacement suggestion -- but I don't know the exact
reason why it does not do that.


Repository:
  rL LLVM

https://reviews.llvm.org/D37188

Files:
  CMakeLists.txt
  test/CMakeLists.txt
  test/include-fixer/yamldb_plugin.cpp
  test/lit.cfg
  test/lit.site.cfg.in
  unittests/CMakeLists.txt

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37188.112811.patch
Type: text/x-patch
Size: 5727 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170827/9fd16cad/attachment.bin>


More information about the cfe-commits mailing list