StrixOS/build_soong
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add clang-analyzer check if selected. am: 04f8d379e6

Browse Source 
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1554894

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: I5d02e3e380aa3c19053546a1b91a8878b5883bff
This commit is contained in:
Chih-Hung Hsieh
2021-01-20 07:47:05 +00:00
committed by Automerger Merge Worker
parent bc657cb78a 04f8d379e6
commit 6b6b7353f8
1 changed files with 9 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
cc/config/tidy.go
View File

@@ -29,15 +29,12 @@ func init() {
if override := ctx.Config().Getenv("DEFAULT_GLOBAL_TIDY_CHECKS"); override != "" { if override := ctx.Config().Getenv("DEFAULT_GLOBAL_TIDY_CHECKS"); override != "" {
return override return override
} }
return strings.Join([]string{ checks := strings.Join([]string{
"-*", "-*",
"abseil-*", "abseil-*",
"android-*", "android-*",
"bugprone-*", "bugprone-*",
"cert-*", "cert-*",
// clang-analyzer-* check is slow and enables clang-diagnostic-padded,
// which cannot be suppressed yet.
// "clang-analyzer-*",
"clang-diagnostic-unused-command-line-argument", "clang-diagnostic-unused-command-line-argument",
"google-*", "google-*",
"misc-*", "misc-*",
@@ -63,6 +60,14 @@ func init() {
// -readability-* // -readability-*
// -zircon-* // -zircon-*
}, ",") }, ",")
// clang-analyzer-* checks are too slow to be in the default for WITH_TIDY=1.
// nightly builds add CLANG_ANALYZER_CHECKS=1 to run those checks.
// Some test code have clang-diagnostic-padded warnings that cannot be
// suppressed, but only by disabling clang-analyzer-optin.performance.*.
if ctx.Config().IsEnvTrue("CLANG_ANALYZER_CHECKS") {
checks += ",clang-analyzer-*,-clang-analyzer-optin.performance.*"
}
return checks
}) })
// There are too many clang-tidy warnings in external and vendor projects. // There are too many clang-tidy warnings in external and vendor projects.