Re: [PATCH] Add counter histogram to fdo summary (issue6465057)

2012-09-05 Thread Teresa Johnson
On Wed, Sep 5, 2012 at 9:13 AM, H.J. Lu wrote: > On Wed, Sep 5, 2012 at 8:50 AM, Teresa Johnson wrote: >> On Wed, Sep 5, 2012 at 8:44 AM, H.J. Lu wrote: >>> On Wed, Sep 5, 2012 at 8:09 AM, Teresa Johnson wrote: Sorry about that. I am right now trying to reproduce the profiledbootstrap

Re: [PATCH] Add counter histogram to fdo summary (issue6465057)

2012-09-05 Thread H.J. Lu
On Wed, Sep 5, 2012 at 8:50 AM, Teresa Johnson wrote: > On Wed, Sep 5, 2012 at 8:44 AM, H.J. Lu wrote: >> On Wed, Sep 5, 2012 at 8:09 AM, Teresa Johnson wrote: >>> Sorry about that. I am right now trying to reproduce the >>> profiledbootstrap problem that H.J. reported, which is on >>> x86_64-un

Re: [PATCH] Add counter histogram to fdo summary (issue6465057)

2012-09-05 Thread Teresa Johnson
On Wed, Sep 5, 2012 at 8:44 AM, H.J. Lu wrote: > On Wed, Sep 5, 2012 at 8:09 AM, Teresa Johnson wrote: >> Sorry about that. I am right now trying to reproduce the >> profiledbootstrap problem that H.J. reported, which is on >> x86_64-unknown-linux-gnu where I had successfully done a >> profiledbo

Re: [PATCH] Add counter histogram to fdo summary (issue6465057)

2012-09-05 Thread H.J. Lu
On Wed, Sep 5, 2012 at 8:09 AM, Teresa Johnson wrote: > Sorry about that. I am right now trying to reproduce the > profiledbootstrap problem that H.J. reported, which is on > x86_64-unknown-linux-gnu where I had successfully done a > profiledbootstrap before my commit. > > Unfortunately after svn

Re: [PATCH] Add counter histogram to fdo summary (issue6465057)

2012-09-05 Thread Teresa Johnson
Sorry about that. I am right now trying to reproduce the profiledbootstrap problem that H.J. reported, which is on x86_64-unknown-linux-gnu where I had successfully done a profiledbootstrap before my commit. Unfortunately after svn updating my client I am hitting an unrelated build problem with my

Re: [PATCH] Add counter histogram to fdo summary (issue6465057)

2012-09-05 Thread Markus Trippelsdorf
On 2012.09.04 at 14:23 -0700, Teresa Johnson wrote: > I just committed the patch (included below). I implemented the > occupancy bit vector approach for recording non-zero histogram > entries, and a few issues uncovered with the merging in a profiled > bootstrap. > > Passes both bootstrap and prof

Re: [PATCH] Add counter histogram to fdo summary (issue6465057)

2012-09-04 Thread Teresa Johnson
I just committed the patch (included below). I implemented the occupancy bit vector approach for recording non-zero histogram entries, and a few issues uncovered with the merging in a profiled bootstrap. Passes both bootstrap and profiledbootstrap builds and regression tests. Thanks, Teresa Enha

Re: [PATCH] Add counter histogram to fdo summary (issue6465057)

2012-08-30 Thread Jan Hubicka
> On Wed, Aug 29, 2012 at 6:12 AM, Jan Hubicka wrote: > >> Index: libgcc/libgcov.c > >> === > >> --- libgcc/libgcov.c (revision 190736) > >> +++ libgcc/libgcov.c (working copy) > >> @@ -276,6 +276,78 @@ gcov_version (struct gcov_inf

Re: [PATCH] Add counter histogram to fdo summary (issue6465057)

2012-08-29 Thread Teresa Johnson
On Wed, Aug 29, 2012 at 6:12 AM, Jan Hubicka wrote: >> Index: libgcc/libgcov.c >> === >> --- libgcc/libgcov.c (revision 190736) >> +++ libgcc/libgcov.c (working copy) >> @@ -276,6 +276,78 @@ gcov_version (struct gcov_info *ptr, gcov

Re: [PATCH] Add counter histogram to fdo summary (issue6465057)

2012-08-29 Thread Jan Hubicka
> Index: libgcc/libgcov.c > === > --- libgcc/libgcov.c (revision 190736) > +++ libgcc/libgcov.c (working copy) > @@ -276,6 +276,78 @@ gcov_version (struct gcov_info *ptr, gcov_unsigned >return 1; > } > > +/* Insert counter VAL

[PATCH] Add counter histogram to fdo summary (issue6465057)

2012-08-28 Thread Teresa Johnson
Revision to earlier patch to augment the gcov program summary with working set information. We now emit the non-zero arc counter histogram entries into the summary, instead of computing the working set information and emitting that into the summary. The working set information is computed by the c