diff options
author | 2020-06-16 17:51:46 -0700 | |
---|---|---|
committer | 2020-06-22 11:13:42 -0700 | |
commit | 1a6acd4f39b8ea2b030444deb4ecdb0af652e5aa (patch) | |
tree | 7f7671a15e3f9d72af1a8c67807f15f80bc8605f /cc/coverage.go | |
parent | 181c2a40d8465372a44ee34109c7ea1d7b9ce21d (diff) |
Make native_coverage clause work with ClangCoverage
Make uses NATIVE_COVERAGE to enable gcov coverage and CLANG_COVERAGE
to enable clang coverage. NATIVE_COVERAGE is translated to the Soong
Native_coverage product variable which triggers the native_coverage
clause in Android.bp files. The clause also needs to be triggered
for CLANG_COVERAGE.
Rename the existing Native_coverage product variable to GcovCoverage,
and regenerate Native_coverage when either GcovCoverage or
ClangCoverage are set.
Also remove NativeLineCoverage, it wasn't doing anything differently
than Native_coverage.
Bug: 159059537
Test: m checkbuild
Change-Id: I215124a9b35a2ad50ad562079d392e3d33da11f4
Diffstat (limited to 'cc/coverage.go')
-rw-r--r-- | cc/coverage.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/cc/coverage.go b/cc/coverage.go index 1a559a9a5..c8233247f 100644 --- a/cc/coverage.go +++ b/cc/coverage.go @@ -74,8 +74,8 @@ func (cov *coverage) deps(ctx DepsContext, deps Deps) Deps { } func (cov *coverage) flags(ctx ModuleContext, flags Flags, deps PathDeps) (Flags, PathDeps) { - gcovCoverage := ctx.DeviceConfig().NativeCoverageEnabled() clangCoverage := ctx.DeviceConfig().ClangCoverageEnabled() + gcovCoverage := ctx.DeviceConfig().GcovCoverageEnabled() if !gcovCoverage && !clangCoverage { return flags, deps @@ -161,7 +161,7 @@ func (cov *coverage) begin(ctx BaseModuleContext) { func SetCoverageProperties(ctx android.BaseModuleContext, properties CoverageProperties, moduleTypeHasCoverage bool, useSdk bool, sdkVersion string) CoverageProperties { // Coverage is disabled globally - if !ctx.DeviceConfig().NativeCoverageEnabled() && !ctx.DeviceConfig().ClangCoverageEnabled() { + if !ctx.DeviceConfig().NativeCoverageEnabled() { return properties } |