gcov: fix TOPN streaming from shared libraries
authorSergei Trofimovich <siarheit@google.com>
Sun, 6 Sep 2020 11:13:54 +0000 (12:13 +0100)
committerSergei Trofimovich <siarheit@google.com>
Tue, 22 Sep 2020 10:41:04 +0000 (11:41 +0100)
Before the change gcc did not stream correctly TOPN counters
if counters belonged to a non-local shared object.

As a result zero-section optimization generated TOPN sections
in a form not recognizable by '__gcov_merge_topn'.

The problem happens because in a case of multiple shared objects
'__gcov_merge_topn' function is present in address space multiple
times (once per each object).

The fix is to never rely on function address and predicate on TOPN
counter types.

libgcc/ChangeLog:

PR gcov-profile/96913
* libgcov-driver.c (write_one_data): Avoid function pointer
comparison in TOP streaming decision.

libgcc/libgcov-driver.c

index 58914268d4ece0b3a3a7dcb9cb21c4fa197fd770..e53e4dc392a0cd6c4d0d90ff79385c1d5bb9086f 100644 (file)
@@ -424,7 +424,7 @@ write_one_data (const struct gcov_info *gi_ptr,
 
          n_counts = ci_ptr->num;
 
-         if (gi_ptr->merge[t_ix] == __gcov_merge_topn)
+         if (t_ix == GCOV_COUNTER_V_TOPN || t_ix == GCOV_COUNTER_V_INDIR)
            write_top_counters (ci_ptr, t_ix, n_counts);
          else
            {