From f36e01de6187e25d857f5e4245fa956be89375d9 Mon Sep 17 00:00:00 2001 From: Chih-Hung Hsieh Date: Fri, 8 Oct 2021 13:14:41 -0700 Subject: Add new clang-tidy and clang warning patterns * New "clang-tidy used ... seconds" warnings are reported when clang-tidy runs for more than 1/2 of TIDY_TIMEOUT seconds. * Recognize other clang-tidy warnings in separate groups; they should not be used in normal Android builds. * Add two more variants of C++ warnings. Test: warn.py --url=http://cs/android --separator='?l=' build.log > warnings.html Test: warn.py --gencsv build.log > warnings.csv Change-Id: I4a2e1c3c817c586cfe3da125b920cca77fcc63b6 --- tools/warn/tidy_warn_patterns.py | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'tools/warn/tidy_warn_patterns.py') diff --git a/tools/warn/tidy_warn_patterns.py b/tools/warn/tidy_warn_patterns.py index 1297966b53..a5842ea125 100644 --- a/tools/warn/tidy_warn_patterns.py +++ b/tools/warn/tidy_warn_patterns.py @@ -81,6 +81,7 @@ def analyzer_group_check(check): warn_patterns = [ # pylint does not recognize g-inconsistent-quotes # pylint:disable=line-too-long,bad-option-value,g-inconsistent-quotes + group_tidy_warn_pattern('altera'), group_tidy_warn_pattern('android'), simple_tidy_warn_pattern('abseil-string-find-startswith'), simple_tidy_warn_pattern('bugprone-argument-comment'), @@ -127,8 +128,9 @@ warn_patterns = [ simple_tidy_warn_pattern('cert-oop54-cpp'), group_tidy_warn_pattern('cert'), group_tidy_warn_pattern('clang-diagnostic'), + group_tidy_warn_pattern('concurrency'), group_tidy_warn_pattern('cppcoreguidelines'), - group_tidy_warn_pattern('llvm'), + group_tidy_warn_pattern('fuchsia'), simple_tidy_warn_pattern('google-default-arguments'), simple_tidy_warn_pattern('google-runtime-int'), simple_tidy_warn_pattern('google-runtime-operator'), @@ -152,8 +154,10 @@ warn_patterns = [ simple_tidy_warn_pattern('hicpp-noexcept-move'), simple_tidy_warn_pattern('hicpp-use-override'), group_tidy_warn_pattern('hicpp'), - group_tidy_warn_pattern('modernize'), + group_tidy_warn_pattern('llvm'), + group_tidy_warn_pattern('llvmlibc'), group_tidy_warn_pattern('misc'), + group_tidy_warn_pattern('modernize'), simple_tidy_warn_pattern('performance-faster-string-find'), simple_tidy_warn_pattern('performance-for-range-copy'), simple_tidy_warn_pattern('performance-implicit-cast-in-loop'), @@ -173,6 +177,7 @@ warn_patterns = [ group_tidy_warn_pattern('portability'), tidy_warn('TIMEOUT', [r".*: warning: clang-tidy aborted "]), + tidy_warn('Long Runs', [r".*: warning: clang-tidy used "]), # warnings from clang-tidy's clang-analyzer checks analyzer_high('clang-analyzer-core, null pointer', -- cgit v1.2.3-59-g8ed1b