From 8af8bb9ccfe940efd7aaf52e6b14dbfc4d955a70 Mon Sep 17 00:00:00 2001 From: marxin Date: Fri, 28 Apr 2017 12:49:26 +0000 Subject: [PATCH] gcno file: do not stream block flags (PR gcov-profile/80031). 2017-04-28 Martin Liska PR gcov-profile/80031 * gcov-dump.c (tag_blocks): Just print number of basic blocks. * gcov-io.h (GCOV_TAG_BLOCKS_NUM): Remove unused macro. * gcov.c (read_graph_file): Read just number of blocks. * profile.c (branch_prob): Do not stream 0 flags per a basic block. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@247370 138bc75d-0d04-0410-961f-82ee72b054a4 --- gcc/ChangeLog | 9 +++++++++ gcc/gcov-dump.c | 22 ++-------------------- gcc/gcov-io.h | 1 - gcc/gcov.c | 7 +------ gcc/profile.c | 3 +-- 5 files changed, 13 insertions(+), 29 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 4b95fc7d1aa..8e21c35c979 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,5 +1,14 @@ 2017-04-28 Martin Liska + PR gcov-profile/80031 + * gcov-dump.c (tag_blocks): Just print number of basic blocks. + * gcov-io.h (GCOV_TAG_BLOCKS_NUM): Remove unused macro. + * gcov.c (read_graph_file): Read just number of blocks. + * profile.c (branch_prob): Do not stream 0 flags per a basic + block. + +2017-04-28 Martin Liska + * gcov-dump.c (tag_*): Add new argument to declarations. (dump_gcov_file): Likewise. (tag_blocks): Add and use new argument depth. diff --git a/gcc/gcov-dump.c b/gcc/gcov-dump.c index 47db1795313..f2522577e9d 100644 --- a/gcc/gcov-dump.c +++ b/gcc/gcov-dump.c @@ -318,27 +318,9 @@ tag_function (const char *filename ATTRIBUTE_UNUSED, static void tag_blocks (const char *filename ATTRIBUTE_UNUSED, unsigned tag ATTRIBUTE_UNUSED, unsigned length ATTRIBUTE_UNUSED, - unsigned depth) + unsigned depth ATTRIBUTE_UNUSED) { - unsigned n_blocks = GCOV_TAG_BLOCKS_NUM (length); - - printf (" %u blocks", n_blocks); - - if (flag_dump_contents) - { - unsigned ix; - - for (ix = 0; ix != n_blocks; ix++) - { - if (!(ix & 7)) - { - printf ("\n"); - print_prefix (filename, depth, gcov_position ()); - printf (VALUE_PADDING_PREFIX VALUE_PREFIX, ix); - } - printf ("%04x ", gcov_read_unsigned ()); - } - } + printf (" %u blocks", gcov_read_unsigned ()); } static void diff --git a/gcc/gcov-io.h b/gcc/gcov-io.h index 1fb58dd918e..1c8ee8f9a2a 100644 --- a/gcc/gcov-io.h +++ b/gcc/gcov-io.h @@ -230,7 +230,6 @@ typedef uint64_t gcov_type_unsigned; #define GCOV_TAG_FUNCTION_LENGTH (3) #define GCOV_TAG_BLOCKS ((gcov_unsigned_t)0x01410000) #define GCOV_TAG_BLOCKS_LENGTH(NUM) (NUM) -#define GCOV_TAG_BLOCKS_NUM(LENGTH) (LENGTH) #define GCOV_TAG_ARCS ((gcov_unsigned_t)0x01430000) #define GCOV_TAG_ARCS_LENGTH(NUM) (1 + (NUM) * 2) #define GCOV_TAG_ARCS_NUM(LENGTH) (((LENGTH) - 1) / 2) diff --git a/gcc/gcov.c b/gcc/gcov.c index bb26a1a9787..63f6a75f1af 100644 --- a/gcc/gcov.c +++ b/gcc/gcov.c @@ -129,7 +129,6 @@ typedef struct block_info /* Block execution count. */ gcov_type count; - unsigned flags : 12; unsigned count_valid : 1; unsigned valid_chain : 1; unsigned invalid_chain : 1; @@ -1374,12 +1373,8 @@ read_graph_file (void) bbg_file_name, fn->name); else { - unsigned ix, num_blocks = GCOV_TAG_BLOCKS_NUM (length); - fn->num_blocks = num_blocks; - + fn->num_blocks = gcov_read_unsigned (); fn->blocks = XCNEWVEC (block_t, fn->num_blocks); - for (ix = 0; ix != num_blocks; ix++) - fn->blocks[ix].flags = gcov_read_unsigned (); } } else if (fn && tag == GCOV_TAG_ARCS) diff --git a/gcc/profile.c b/gcc/profile.c index c6f462d2f7a..c7eed0e3dfd 100644 --- a/gcc/profile.c +++ b/gcc/profile.c @@ -1195,8 +1195,7 @@ branch_prob (void) /* Basic block flags */ offset = gcov_write_tag (GCOV_TAG_BLOCKS); - for (i = 0; i != (unsigned) (n_basic_blocks_for_fn (cfun)); i++) - gcov_write_unsigned (0); + gcov_write_unsigned (n_basic_blocks_for_fn (cfun)); gcov_write_length (offset); /* Arcs */ -- 2.11.4.GIT