diff options
| author | Pirama Arumuga Nainar <pirama@google.com> | 2018-04-23 22:44:39 +0000 |
|---|---|---|
| committer | Pirama Arumuga Nainar <pirama@google.com> | 2018-04-23 22:44:39 +0000 |
| commit | 0b882f0394f0deaf742e09b72123ac83d455c6c1 (patch) | |
| tree | 326193cf51b980662fb9ed4cfabc6bd65d588f50 /cc/coverage.go | |
| parent | 358056c058e43bfb2bbae8978fbfe4a8e4c84fb5 (diff) | |
| download | android_build_soong-0b882f0394f0deaf742e09b72123ac83d455c6c1.tar.gz android_build_soong-0b882f0394f0deaf742e09b72123ac83d455c6c1.tar.bz2 android_build_soong-0b882f0394f0deaf742e09b72123ac83d455c6c1.zip | |
Revert "Support coverage instrumentation for Linux host"
This reverts commit 358056c058e43bfb2bbae8978fbfe4a8e4c84fb5.
Reason for revert: Breaks build_test target in some branches.
Change-Id: I604561033038d4ff15b74caf7b81ff5c8dd9632f
Diffstat (limited to 'cc/coverage.go')
| -rw-r--r-- | cc/coverage.go | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/cc/coverage.go b/cc/coverage.go index 671353c9..391b1181 100644 --- a/cc/coverage.go +++ b/cc/coverage.go @@ -16,7 +16,6 @@ package cc import ( "android/soong/android" - "android/soong/cc/config" ) type CoverageProperties struct { @@ -39,10 +38,6 @@ func (cov *coverage) props() []interface{} { func (cov *coverage) begin(ctx BaseModuleContext) {} func (cov *coverage) deps(ctx BaseModuleContext, deps Deps) Deps { - if cov.Properties.CoverageEnabled { - runtimeLibrary := config.ProfileRuntimeLibrary(ctx.toolchain()) - deps.LateStaticLibs = append(deps.LateStaticLibs, runtimeLibrary) - } return deps } @@ -104,8 +99,9 @@ func coverageLinkingMutator(mctx android.BottomUpMutatorContext) { if !mctx.DeviceConfig().NativeCoverageEnabled() { // Coverage is disabled globally - } else if mctx.Darwin() || mctx.Windows() { - // Coverage not supported for Darwin and Windows + } else if mctx.Host() { + // TODO(dwillemsen): because of -nodefaultlibs, we must depend on libclang_rt.profile-*.a + // Just turn off for now. } else if c.coverage.Properties.Native_coverage != nil { enabled = *c.coverage.Properties.Native_coverage } else { |