Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e59f0f66 authored by Pirama Arumuga Nainar's avatar Pirama Arumuga Nainar
Browse files

Enable continuous coverage: use '%c' specifier in LLVM_PROFILE_FILE 

Bug: http://b/194128476
Bug: http://b/210012154

- Do not use %c if coverage is enabled for bionic/libc.

Test: Run tests with this topic and verify coverage still works and also
test memory-mapped coverage (death tests, JNI code in CTS)

Change-Id: Id1ade9c6f45d69a1da912e3e57acd1d0197c11b5
parent e797a5c7

rootdir/Android.mk

+7 −2
Original line number Diff line number Diff line
@@ -75,10 +75,15 @@ ifeq ($(NATIVE_COVERAGE),true) 
  EXPORT_GLOBAL_GCOV_OPTIONS := export GCOV_PREFIX /data/misc/trace

endif



EXPORT_GLOBAL_CLANG_COVERAGE_OPTIONS :=

ifeq ($(CLANG_COVERAGE),true)

  ifeq ($(BIONIC_COVERAGE),false)

    # http://b/210012154 Disable continuous coverage if instrumentation is on

    # for bionic/libc

    EXPORT_GLOBAL_CLANG_COVERAGE_OPTIONS := export LLVM_PROFILE_FILE /data/misc/trace/clang%c-%20m.profraw

  else

    EXPORT_GLOBAL_CLANG_COVERAGE_OPTIONS := export LLVM_PROFILE_FILE /data/misc/trace/clang-%20m.profraw

  endif

endif



# Put it here instead of in init.rc module definition,

# because init.rc is conditionally included.