Re: [google] Modification of gcov pmu format to reduce gcda size bloat (issue 6427063)

Hi David, Could you take a look at the patch I mailed to gcc-patches when you get a chance? Thanks, Chris http://codereview.appspot.com/6427063/

Re: [google] Modification of gcov pmu format to reduce gcda size bloat (issue 6427063)

Hi Rong, Could you take a look at the patch I mailed to gcc-patches when you get a chance? It reduces the gcda size bloat issue by replacing gcov_pmu data with a filetag field that holds the position of the correct filename inside of the newly added string table. Thanks, Chris http://coderevie

Re: [google] Modification of gcov pmu format to reduce gcda size bloat (issue 6427063)

Ok, I fixed most of the problem you found. I'm not sure what happened with gcov.c, but I'll try uploading it again. http://codereview.appspot.com/6427063/diff/5001/gcc/gcov-io.c File gcc/gcov-io.c (right): http://codereview.appspot.com/6427063/diff/5001/gcc/gcov-io.c#newcode280 gcc/gcov-io.c:28

Re: [google] Modification of gcov pmu format to reduce gcda size bloat (issue 6427063)

http://codereview.appspot.com/6427063/diff/11002/gcc/gcov-io.h File gcc/gcov-io.h (right): http://codereview.appspot.com/6427063/diff/11002/gcc/gcov-io.h#newcode688 gcc/gcov-io.h:688: gcov_unsigned_t index; /* The corresponding string table index */ On 2012/08/24 22:30:30, davidxl wrote: Is t

Re: [google] Modification of gcov pmu format to reduce gcda size bloat (issue 6427063)

On 2012/08/24 23:20:21, davidxl wrote: On Fri, Aug 24, 2012 at 3:56 PM, wrote: > > http://codereview.appspot.com/6427063/diff/11002/gcc/gcov-io.h > File gcc/gcov-io.h (right): > > http://codereview.appspot.com/6427063/diff/11002/gcc/gcov-io.h#newcode688 > gcc/gcov-io.