Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 1 | // Copyright 2017 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 | |
| 15 | package cc |
| 16 | |
| 17 | import ( |
Pirama Arumuga Nainar | 1acd447 | 2018-12-10 15:12:40 -0800 | [diff] [blame] | 18 | "strconv" |
| 19 | |
Pirama Arumuga Nainar | 82fe59b | 2019-07-02 14:55:35 -0700 | [diff] [blame] | 20 | "github.com/google/blueprint" |
| 21 | |
Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 22 | "android/soong/android" |
Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 23 | ) |
| 24 | |
Pirama Arumuga Nainar | f45e152 | 2020-08-05 10:08:30 -0700 | [diff] [blame] | 25 | const profileInstrFlag = "-fprofile-instr-generate=/data/misc/trace/clang-%p-%m.profraw" |
| 26 | |
Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 27 | type CoverageProperties struct { |
| 28 | Native_coverage *bool |
| 29 | |
Pirama Arumuga Nainar | ee30d5e | 2019-03-20 11:31:56 -0700 | [diff] [blame] | 30 | NeedCoverageVariant bool `blueprint:"mutated"` |
| 31 | NeedCoverageBuild bool `blueprint:"mutated"` |
| 32 | |
Pirama Arumuga Nainar | 1acd447 | 2018-12-10 15:12:40 -0800 | [diff] [blame] | 33 | CoverageEnabled bool `blueprint:"mutated"` |
| 34 | IsCoverageVariant bool `blueprint:"mutated"` |
Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 35 | } |
| 36 | |
| 37 | type coverage struct { |
| 38 | Properties CoverageProperties |
| 39 | |
| 40 | // Whether binaries containing this module need --coverage added to their ldflags |
| 41 | linkCoverage bool |
| 42 | } |
| 43 | |
| 44 | func (cov *coverage) props() []interface{} { |
| 45 | return []interface{}{&cov.Properties} |
| 46 | } |
| 47 | |
Oliver Nguyen | 1382ab6 | 2019-12-06 15:22:41 -0800 | [diff] [blame] | 48 | func getGcovProfileLibraryName(ctx ModuleContextIntf) string { |
Pirama Arumuga Nainar | 82fe59b | 2019-07-02 14:55:35 -0700 | [diff] [blame] | 49 | // This function should only ever be called for a cc.Module, so the |
| 50 | // following statement should always succeed. |
| 51 | if ctx.useSdk() { |
| 52 | return "libprofile-extras_ndk" |
| 53 | } else { |
| 54 | return "libprofile-extras" |
| 55 | } |
| 56 | } |
| 57 | |
Oliver Nguyen | 1382ab6 | 2019-12-06 15:22:41 -0800 | [diff] [blame] | 58 | func getClangProfileLibraryName(ctx ModuleContextIntf) string { |
| 59 | if ctx.useSdk() { |
| 60 | return "libprofile-clang-extras_ndk" |
Cindy Zhou | 5d5cfc1 | 2021-01-09 08:25:22 -0800 | [diff] [blame] | 61 | } else if ctx.isCfiAssemblySupportEnabled() { |
| 62 | return "libprofile-clang-extras_cfi_support" |
Oliver Nguyen | 1382ab6 | 2019-12-06 15:22:41 -0800 | [diff] [blame] | 63 | } else { |
| 64 | return "libprofile-clang-extras" |
| 65 | } |
| 66 | } |
| 67 | |
Pirama Arumuga Nainar | 82fe59b | 2019-07-02 14:55:35 -0700 | [diff] [blame] | 68 | func (cov *coverage) deps(ctx DepsContext, deps Deps) Deps { |
| 69 | if cov.Properties.NeedCoverageVariant { |
| 70 | ctx.AddVariationDependencies([]blueprint.Variation{ |
| 71 | {Mutator: "link", Variation: "static"}, |
Ivan Lozano | a0cd8f9 | 2020-04-09 09:56:02 -0400 | [diff] [blame] | 72 | }, CoverageDepTag, getGcovProfileLibraryName(ctx)) |
Oliver Nguyen | 1382ab6 | 2019-12-06 15:22:41 -0800 | [diff] [blame] | 73 | ctx.AddVariationDependencies([]blueprint.Variation{ |
| 74 | {Mutator: "link", Variation: "static"}, |
Ivan Lozano | a0cd8f9 | 2020-04-09 09:56:02 -0400 | [diff] [blame] | 75 | }, CoverageDepTag, getClangProfileLibraryName(ctx)) |
Pirama Arumuga Nainar | 65c95ff | 2019-03-25 10:21:31 -0700 | [diff] [blame] | 76 | } |
Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 77 | return deps |
| 78 | } |
| 79 | |
Pirama Arumuga Nainar | 82fe59b | 2019-07-02 14:55:35 -0700 | [diff] [blame] | 80 | func (cov *coverage) flags(ctx ModuleContext, flags Flags, deps PathDeps) (Flags, PathDeps) { |
Oliver Nguyen | 1382ab6 | 2019-12-06 15:22:41 -0800 | [diff] [blame] | 81 | clangCoverage := ctx.DeviceConfig().ClangCoverageEnabled() |
Colin Cross | 1a6acd4 | 2020-06-16 17:51:46 -0700 | [diff] [blame] | 82 | gcovCoverage := ctx.DeviceConfig().GcovCoverageEnabled() |
Oliver Nguyen | 1382ab6 | 2019-12-06 15:22:41 -0800 | [diff] [blame] | 83 | |
| 84 | if !gcovCoverage && !clangCoverage { |
Pirama Arumuga Nainar | 82fe59b | 2019-07-02 14:55:35 -0700 | [diff] [blame] | 85 | return flags, deps |
Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 86 | } |
| 87 | |
| 88 | if cov.Properties.CoverageEnabled { |
Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 89 | cov.linkCoverage = true |
Pirama Arumuga Nainar | c7679de | 2019-02-18 22:23:42 -0800 | [diff] [blame] | 90 | |
Oliver Nguyen | 1382ab6 | 2019-12-06 15:22:41 -0800 | [diff] [blame] | 91 | if gcovCoverage { |
Oliver Nguyen | 0452678 | 2020-04-21 12:40:27 -0700 | [diff] [blame] | 92 | flags.GcovCoverage = true |
Oliver Nguyen | 1382ab6 | 2019-12-06 15:22:41 -0800 | [diff] [blame] | 93 | flags.Local.CommonFlags = append(flags.Local.CommonFlags, "--coverage", "-O0") |
| 94 | |
| 95 | // Override -Wframe-larger-than and non-default optimization |
| 96 | // flags that the module may use. |
| 97 | flags.Local.CFlags = append(flags.Local.CFlags, "-Wno-frame-larger-than=", "-O0") |
| 98 | } else if clangCoverage { |
Pirama Arumuga Nainar | f45e152 | 2020-08-05 10:08:30 -0700 | [diff] [blame] | 99 | flags.Local.CommonFlags = append(flags.Local.CommonFlags, profileInstrFlag, "-fcoverage-mapping", "-Wno-pass-failed") |
Oliver Nguyen | 1382ab6 | 2019-12-06 15:22:41 -0800 | [diff] [blame] | 100 | } |
Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 101 | } |
| 102 | |
| 103 | // Even if we don't have coverage enabled, if any of our object files were compiled |
| 104 | // with coverage, then we need to add --coverage to our ldflags. |
| 105 | if !cov.linkCoverage { |
| 106 | if ctx.static() && !ctx.staticBinary() { |
| 107 | // For static libraries, the only thing that changes our object files |
| 108 | // are included whole static libraries, so check to see if any of |
| 109 | // those have coverage enabled. |
Colin Cross | 6e511a9 | 2020-07-27 21:26:48 -0700 | [diff] [blame] | 110 | ctx.VisitDirectDeps(func(m android.Module) { |
| 111 | if depTag, ok := ctx.OtherModuleDependencyTag(m).(libraryDependencyTag); ok { |
| 112 | if depTag.static() && depTag.wholeStatic { |
| 113 | if cc, ok := m.(*Module); ok && cc.coverage != nil { |
| 114 | if cc.coverage.linkCoverage { |
| 115 | cov.linkCoverage = true |
| 116 | } |
| 117 | } |
Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 118 | } |
| 119 | } |
| 120 | }) |
| 121 | } else { |
| 122 | // For executables and shared libraries, we need to check all of |
| 123 | // our static dependencies. |
Colin Cross | d11fcda | 2017-10-23 17:59:01 -0700 | [diff] [blame] | 124 | ctx.VisitDirectDeps(func(m android.Module) { |
Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 125 | cc, ok := m.(*Module) |
| 126 | if !ok || cc.coverage == nil { |
| 127 | return |
| 128 | } |
| 129 | |
| 130 | if static, ok := cc.linker.(libraryInterface); !ok || !static.static() { |
| 131 | return |
| 132 | } |
| 133 | |
| 134 | if cc.coverage.linkCoverage { |
| 135 | cov.linkCoverage = true |
| 136 | } |
| 137 | }) |
| 138 | } |
| 139 | } |
| 140 | |
| 141 | if cov.linkCoverage { |
Oliver Nguyen | 1382ab6 | 2019-12-06 15:22:41 -0800 | [diff] [blame] | 142 | if gcovCoverage { |
| 143 | flags.Local.LdFlags = append(flags.Local.LdFlags, "--coverage") |
Pirama Arumuga Nainar | 65c95ff | 2019-03-25 10:21:31 -0700 | [diff] [blame] | 144 | |
Ivan Lozano | a0cd8f9 | 2020-04-09 09:56:02 -0400 | [diff] [blame] | 145 | coverage := ctx.GetDirectDepWithTag(getGcovProfileLibraryName(ctx), CoverageDepTag).(*Module) |
Oliver Nguyen | 1382ab6 | 2019-12-06 15:22:41 -0800 | [diff] [blame] | 146 | deps.WholeStaticLibs = append(deps.WholeStaticLibs, coverage.OutputFile().Path()) |
Pirama Arumuga Nainar | 100bbdc | 2019-07-02 23:47:35 -0700 | [diff] [blame] | 147 | |
Oliver Nguyen | 1382ab6 | 2019-12-06 15:22:41 -0800 | [diff] [blame] | 148 | flags.Local.LdFlags = append(flags.Local.LdFlags, "-Wl,--wrap,getenv") |
| 149 | } else if clangCoverage { |
Pirama Arumuga Nainar | f45e152 | 2020-08-05 10:08:30 -0700 | [diff] [blame] | 150 | flags.Local.LdFlags = append(flags.Local.LdFlags, profileInstrFlag) |
Oliver Nguyen | 1382ab6 | 2019-12-06 15:22:41 -0800 | [diff] [blame] | 151 | |
Ivan Lozano | a0cd8f9 | 2020-04-09 09:56:02 -0400 | [diff] [blame] | 152 | coverage := ctx.GetDirectDepWithTag(getClangProfileLibraryName(ctx), CoverageDepTag).(*Module) |
Oliver Nguyen | 1382ab6 | 2019-12-06 15:22:41 -0800 | [diff] [blame] | 153 | deps.WholeStaticLibs = append(deps.WholeStaticLibs, coverage.OutputFile().Path()) |
Pirama Arumuga Nainar | 9464b6c | 2020-12-10 10:45:33 -0800 | [diff] [blame] | 154 | flags.Local.LdFlags = append(flags.Local.LdFlags, "-Wl,--wrap,open") |
Oliver Nguyen | 1382ab6 | 2019-12-06 15:22:41 -0800 | [diff] [blame] | 155 | } |
Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 156 | } |
| 157 | |
Pirama Arumuga Nainar | 82fe59b | 2019-07-02 14:55:35 -0700 | [diff] [blame] | 158 | return flags, deps |
Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 159 | } |
| 160 | |
Pirama Arumuga Nainar | ee30d5e | 2019-03-20 11:31:56 -0700 | [diff] [blame] | 161 | func (cov *coverage) begin(ctx BaseModuleContext) { |
Ivan Lozano | a0cd8f9 | 2020-04-09 09:56:02 -0400 | [diff] [blame] | 162 | if ctx.Host() { |
| 163 | // TODO(dwillemsen): because of -nodefaultlibs, we must depend on libclang_rt.profile-*.a |
| 164 | // Just turn off for now. |
| 165 | } else { |
| 166 | cov.Properties = SetCoverageProperties(ctx, cov.Properties, ctx.nativeCoverage(), ctx.useSdk(), ctx.sdkVersion()) |
| 167 | } |
| 168 | } |
| 169 | |
| 170 | func SetCoverageProperties(ctx android.BaseModuleContext, properties CoverageProperties, moduleTypeHasCoverage bool, |
| 171 | useSdk bool, sdkVersion string) CoverageProperties { |
Pirama Arumuga Nainar | 1acd447 | 2018-12-10 15:12:40 -0800 | [diff] [blame] | 172 | // Coverage is disabled globally |
Colin Cross | 1a6acd4 | 2020-06-16 17:51:46 -0700 | [diff] [blame] | 173 | if !ctx.DeviceConfig().NativeCoverageEnabled() { |
Ivan Lozano | a0cd8f9 | 2020-04-09 09:56:02 -0400 | [diff] [blame] | 174 | return properties |
Pirama Arumuga Nainar | 1acd447 | 2018-12-10 15:12:40 -0800 | [diff] [blame] | 175 | } |
Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 176 | |
Pirama Arumuga Nainar | ee30d5e | 2019-03-20 11:31:56 -0700 | [diff] [blame] | 177 | var needCoverageVariant bool |
| 178 | var needCoverageBuild bool |
Pirama Arumuga Nainar | 1acd447 | 2018-12-10 15:12:40 -0800 | [diff] [blame] | 179 | |
Ivan Lozano | a0cd8f9 | 2020-04-09 09:56:02 -0400 | [diff] [blame] | 180 | if moduleTypeHasCoverage { |
Pirama Arumuga Nainar | ee30d5e | 2019-03-20 11:31:56 -0700 | [diff] [blame] | 181 | // Check if Native_coverage is set to false. This property defaults to true. |
Ivan Lozano | a0cd8f9 | 2020-04-09 09:56:02 -0400 | [diff] [blame] | 182 | needCoverageVariant = BoolDefault(properties.Native_coverage, true) |
| 183 | if useSdk && sdkVersion != "current" { |
Pirama Arumuga Nainar | ee30d5e | 2019-03-20 11:31:56 -0700 | [diff] [blame] | 184 | // Native coverage is not supported for SDK versions < 23 |
Ivan Lozano | a0cd8f9 | 2020-04-09 09:56:02 -0400 | [diff] [blame] | 185 | if fromApi, err := strconv.Atoi(sdkVersion); err == nil && fromApi < 23 { |
Pirama Arumuga Nainar | ee30d5e | 2019-03-20 11:31:56 -0700 | [diff] [blame] | 186 | needCoverageVariant = false |
Pirama Arumuga Nainar | 1acd447 | 2018-12-10 15:12:40 -0800 | [diff] [blame] | 187 | } |
Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 188 | } |
| 189 | |
Pirama Arumuga Nainar | 1acd447 | 2018-12-10 15:12:40 -0800 | [diff] [blame] | 190 | if needCoverageVariant { |
Pirama Arumuga Nainar | ee30d5e | 2019-03-20 11:31:56 -0700 | [diff] [blame] | 191 | // Coverage variant is actually built with coverage if enabled for its module path |
Roland Levillain | 4f5297b | 2020-06-09 12:44:06 +0100 | [diff] [blame] | 192 | needCoverageBuild = ctx.DeviceConfig().NativeCoverageEnabledForPath(ctx.ModuleDir()) |
Pirama Arumuga Nainar | ee30d5e | 2019-03-20 11:31:56 -0700 | [diff] [blame] | 193 | } |
| 194 | } |
| 195 | |
Ivan Lozano | a0cd8f9 | 2020-04-09 09:56:02 -0400 | [diff] [blame] | 196 | properties.NeedCoverageBuild = needCoverageBuild |
| 197 | properties.NeedCoverageVariant = needCoverageVariant |
| 198 | |
| 199 | return properties |
Pirama Arumuga Nainar | ee30d5e | 2019-03-20 11:31:56 -0700 | [diff] [blame] | 200 | } |
| 201 | |
Jiyong Park | ee9a98d | 2019-08-09 14:44:36 +0900 | [diff] [blame] | 202 | // Coverage is an interface for non-CC modules to implement to be mutated for coverage |
| 203 | type Coverage interface { |
| 204 | android.Module |
| 205 | IsNativeCoverageNeeded(ctx android.BaseModuleContext) bool |
| 206 | PreventInstall() |
| 207 | HideFromMake() |
Jiyong Park | 83dc74b | 2020-01-14 18:38:44 +0900 | [diff] [blame] | 208 | MarkAsCoverageVariant(bool) |
Ivan Lozano | a0cd8f9 | 2020-04-09 09:56:02 -0400 | [diff] [blame] | 209 | EnableCoverageIfNeeded() |
Jiyong Park | ee9a98d | 2019-08-09 14:44:36 +0900 | [diff] [blame] | 210 | } |
| 211 | |
Pirama Arumuga Nainar | ee30d5e | 2019-03-20 11:31:56 -0700 | [diff] [blame] | 212 | func coverageMutator(mctx android.BottomUpMutatorContext) { |
| 213 | if c, ok := mctx.Module().(*Module); ok && c.coverage != nil { |
| 214 | needCoverageVariant := c.coverage.Properties.NeedCoverageVariant |
| 215 | needCoverageBuild := c.coverage.Properties.NeedCoverageBuild |
| 216 | if needCoverageVariant { |
Pirama Arumuga Nainar | 1acd447 | 2018-12-10 15:12:40 -0800 | [diff] [blame] | 217 | m := mctx.CreateVariations("", "cov") |
| 218 | |
| 219 | // Setup the non-coverage version and set HideFromMake and |
| 220 | // PreventInstall to true. |
| 221 | m[0].(*Module).coverage.Properties.CoverageEnabled = false |
| 222 | m[0].(*Module).coverage.Properties.IsCoverageVariant = false |
| 223 | m[0].(*Module).Properties.HideFromMake = true |
| 224 | m[0].(*Module).Properties.PreventInstall = true |
| 225 | |
| 226 | // The coverage-enabled version inherits HideFromMake, |
| 227 | // PreventInstall from the original module. |
| 228 | m[1].(*Module).coverage.Properties.CoverageEnabled = needCoverageBuild |
| 229 | m[1].(*Module).coverage.Properties.IsCoverageVariant = true |
Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 230 | } |
Jiyong Park | ee9a98d | 2019-08-09 14:44:36 +0900 | [diff] [blame] | 231 | } else if cov, ok := mctx.Module().(Coverage); ok && cov.IsNativeCoverageNeeded(mctx) { |
Ivan Lozano | a0cd8f9 | 2020-04-09 09:56:02 -0400 | [diff] [blame] | 232 | // APEX and Rust modules fall here |
Jiyong Park | ee9a98d | 2019-08-09 14:44:36 +0900 | [diff] [blame] | 233 | |
| 234 | // Note: variant "" is also created because an APEX can be depended on by another |
| 235 | // module which are split into "" and "cov" variants. e.g. when cc_test refers |
| 236 | // to an APEX via 'data' property. |
| 237 | m := mctx.CreateVariations("", "cov") |
Ivan Lozano | a0cd8f9 | 2020-04-09 09:56:02 -0400 | [diff] [blame] | 238 | m[0].(Coverage).MarkAsCoverageVariant(false) |
Jiyong Park | ee9a98d | 2019-08-09 14:44:36 +0900 | [diff] [blame] | 239 | m[0].(Coverage).PreventInstall() |
| 240 | m[0].(Coverage).HideFromMake() |
Ivan Lozano | a0cd8f9 | 2020-04-09 09:56:02 -0400 | [diff] [blame] | 241 | |
| 242 | m[1].(Coverage).MarkAsCoverageVariant(true) |
| 243 | m[1].(Coverage).EnableCoverageIfNeeded() |
Dan Willemsen | 581341d | 2017-02-09 16:16:31 -0800 | [diff] [blame] | 244 | } |
| 245 | } |