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 */ Is this field necessary? http://codereview.appspot.com/6427063/diff/11002/gcc/gcov-io.h#newcode689 gcc/gcov-io.h:689: char* filename; /* The filename that belongs at this index */ Can this field name be generalized? http://codereview.appspot.com/6427063/diff/11002/libgcc/pmu-profile.c File libgcc/pmu-profile.c (right): http://codereview.appspot.com/6427063/diff/11002/libgcc/pmu-profile.c#newcode83 libgcc/pmu-profile.c:83: Who is the caller to this interface? It should be declared in gcov-io.h http://codereview.appspot.com/6427063/