[PATCH] D74299: [clang-tidy] extend tests of run-clang-tidy
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jun 5 08:18:55 PDT 2020
aaron.ballman added a comment.
Generally looks good to me, but I had a question about test coverage.
================
Comment at: clang-tools-extra/test/clang-tidy/infrastructure/run-clang-tidy_config-file.cpp:1
-// RUN: %run_clang_tidy --help
// RUN: rm -rf %t
----------------
Is there a reason to remove this test coverage? I believe this is testing that we can access the help without crashing.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D74299/new/
https://reviews.llvm.org/D74299
More information about the cfe-commits
mailing list