2012-12-13 Teresa Johnson <tejohnson@google.com> PR gcov-profile/55674 * lto-cgraph.c (merge_profile_summaries): Set min correctly the first time ...
12 years, 4 months ago
(2012-12-14 06:42:43 UTC)
#1
2012-12-13 Teresa Johnson <tejohnson@google.com>
PR gcov-profile/55674
* lto-cgraph.c (merge_profile_summaries): Set min correctly the
first time we merge into a histogram entry.
Index: lto-cgraph.c
===================================================================
--- lto-cgraph.c (revision 194496)
+++ lto-cgraph.c (working copy)
@@ -1368,7 +1368,9 @@ merge_profile_summaries (struct lto_file_decl_data
so we need to account for a non-zero histogram entry at new_ix. */
unsigned new_ix = gcov_histo_index (scaled_min);
lto_gcov_summary.histogram[new_ix].min_value
- = MIN (lto_gcov_summary.histogram[new_ix].min_value, scaled_min);
+ = (lto_gcov_summary.histogram[new_ix].num_counters
+ ? MIN (lto_gcov_summary.histogram[new_ix].min_value, scaled_min)
+ : scaled_min);
/* Some of the scaled counter values would ostensibly need to be placed
into different (larger) histogram buckets, but we keep things simple
here and place the scaled cumulative counter value in the bucket
--
This patch is available for review at http://codereview.appspot.com/6946049
> 2012-12-13 Teresa Johnson <tejohnson@google.com> > > PR gcov-profile/55674 > * lto-cgraph.c (merge_profile_summaries): Set min ...
12 years, 4 months ago
(2012-12-14 07:03:31 UTC)
#2
> 2012-12-13 Teresa Johnson <tejohnson@google.com>
>
> PR gcov-profile/55674
> * lto-cgraph.c (merge_profile_summaries): Set min correctly the
> first time we merge into a histogram entry.
OK,
thanks!
Honza
Issue 6946049: [PATCH] Fix LTO binary size increases (PR gcov-profile/55674)
Created 12 years, 4 months ago by tejohnson
Modified 10 years, 7 months ago
Reviewers: hubicka_ucw.cz
Base URL: svn+ssh://gcc.gnu.org/svn/gcc/trunk/gcc/
Comments: 0