blob: 8ca94efc89f6ce5d3bcb3c5df70e0c0d9e77b619 [file] [log] [blame]
Dan Willemsena03cf6d2016-09-26 15:45:04 -07001// Copyright 2016 Google Inc. All rights reserved.
2//
3// Licensed under the Apache License, Version 2.0 (the "License");
4// you may not use this file except in compliance with the License.
5// You may obtain a copy of the License at
6//
7// http://www.apache.org/licenses/LICENSE-2.0
8//
9// Unless required by applicable law or agreed to in writing, software
10// distributed under the License is distributed on an "AS IS" BASIS,
11// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12// See the License for the specific language governing permissions and
13// limitations under the License.
14
15package cc
16
17import (
18 "strings"
19
20 "github.com/google/blueprint/proptools"
21
22 "android/soong/cc/config"
23)
24
25type TidyProperties struct {
26 // whether to run clang-tidy over C-like sources.
27 Tidy *bool
28
29 // Extra flags to pass to clang-tidy
30 Tidy_flags []string
31
32 // Extra checks to enable or disable in clang-tidy
33 Tidy_checks []string
34}
35
36type tidyFeature struct {
37 Properties TidyProperties
38}
39
40func (tidy *tidyFeature) props() []interface{} {
41 return []interface{}{&tidy.Properties}
42}
43
44func (tidy *tidyFeature) begin(ctx BaseModuleContext) {
45}
46
Colin Cross37047f12016-12-13 17:06:13 -080047func (tidy *tidyFeature) deps(ctx DepsContext, deps Deps) Deps {
Dan Willemsena03cf6d2016-09-26 15:45:04 -070048 return deps
49}
50
51func (tidy *tidyFeature) flags(ctx ModuleContext, flags Flags) Flags {
Colin Cross379d2cb2016-12-05 17:11:06 -080052 CheckBadTidyFlags(ctx, "tidy_flags", tidy.Properties.Tidy_flags)
53 CheckBadTidyChecks(ctx, "tidy_checks", tidy.Properties.Tidy_checks)
54
Dan Willemsena03cf6d2016-09-26 15:45:04 -070055 // Check if tidy is explicitly disabled for this module
56 if tidy.Properties.Tidy != nil && !*tidy.Properties.Tidy {
57 return flags
58 }
59
60 // If not explicitly set, check the global tidy flag
Colin Cross6510f912017-11-29 00:27:14 -080061 if tidy.Properties.Tidy == nil && !ctx.Config().ClangTidy() {
Dan Willemsena03cf6d2016-09-26 15:45:04 -070062 return flags
63 }
64
65 // Clang-tidy requires clang
66 if !flags.Clang {
67 return flags
68 }
69
70 flags.Tidy = true
71
Dan Willemsena03cf6d2016-09-26 15:45:04 -070072 esc := proptools.NinjaAndShellEscape
73
74 flags.TidyFlags = append(flags.TidyFlags, esc(tidy.Properties.Tidy_flags)...)
75 if len(flags.TidyFlags) == 0 {
76 headerFilter := "-header-filter=\"(" + ctx.ModuleDir() + "|${config.TidyDefaultHeaderDirs})\""
77 flags.TidyFlags = append(flags.TidyFlags, headerFilter)
78 }
79
Chih-Hung Hsiehdc0c0302017-12-15 20:57:48 -080080 // If clang-tidy is not enabled globally, add the -quiet flag.
81 if !ctx.Config().ClangTidy() {
82 flags.TidyFlags = append(flags.TidyFlags, "-quiet")
Chih-Hung Hsieh669cb912018-01-04 01:41:16 -080083 flags.TidyFlags = append(flags.TidyFlags, "-extra-arg-before=-fno-caret-diagnostics")
Chih-Hung Hsiehdc0c0302017-12-15 20:57:48 -080084 }
85
George Burgess IV561a3fe2017-05-03 18:13:08 -070086 // We might be using the static analyzer through clang tidy.
87 // https://bugs.llvm.org/show_bug.cgi?id=32914
88 flags.TidyFlags = append(flags.TidyFlags, "-extra-arg-before=-D__clang_analyzer__")
89
Dan Willemsena03cf6d2016-09-26 15:45:04 -070090 tidyChecks := "-checks="
Colin Cross6510f912017-11-29 00:27:14 -080091 if checks := ctx.Config().TidyChecks(); len(checks) > 0 {
Dan Willemsena03cf6d2016-09-26 15:45:04 -070092 tidyChecks += checks
93 } else {
94 tidyChecks += config.TidyChecksForDir(ctx.ModuleDir())
95 }
96 if len(tidy.Properties.Tidy_checks) > 0 {
Dan Willemsena03cf6d2016-09-26 15:45:04 -070097 tidyChecks = tidyChecks + "," + strings.Join(esc(tidy.Properties.Tidy_checks), ",")
98 }
99 flags.TidyFlags = append(flags.TidyFlags, tidyChecks)
100
101 return flags
102}