[PATCH] D45454: Add llvm_gcov_flush to be called outside a shared library

Chih-Hung Hsieh via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 29 11:37:31 PDT 2018


chh added a comment.

Marco, latest patch does not change __gcov_flush, which is also hidden in libgcov.
Android coverage test programs have depended on an earlier compiler-rt that did not hide __gcov_flush.
If that's the only use case broken by recent change of compiler-rt, which hide __gcov_flush,
I think it is okay to keep compiler-rt the same as it is now.
Adding a visible llvm_gcov_flush will allow Android coverage test program to use it and dump system .so profiling data,
from the main test program.
Keeping an hidden __gcov_flush has the purpose that David mentioned earlier, to let each .so file dump its own profiling data.


https://reviews.llvm.org/D45454





More information about the cfe-commits mailing list