Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1689)

Issue 6430053: [google main] change the gcda tag for module info

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 6 months ago by xur
Modified:
12 years, 6 months ago
Reviewers:
davidxl
CC:
gcc-patches_gcc.gnu.org
Base URL:
svn+ssh://gcc.gnu.org/svn/gcc/branches/google/main/
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M gcc/gcov-io.h View 2 chunks +1 line, -1 line 0 comments Download

Messages

Total messages: 1
xur
12 years, 6 months ago (2012-07-19 22:31:43 UTC) #1
Hi,

This is for google branches only.

Change the module_info tag to an odd number to avoid the warning during dump.

-Rong

2012-07-19  Rong Xu  <xur@google.com>

	* gcc/gcov-io.h (GCOV_TAG_MODULE_INFO): tag needs to be
        an odd number in each active level.

Index: gcc/gcov-io.h
===================================================================
--- gcc/gcov-io.h	(revision 189690)
+++ gcc/gcov-io.h	(working copy)
@@ -441,7 +441,6 @@ typedef HOST_WIDEST_INT gcov_type;
 #define GCOV_TAG_PROGRAM_SUMMARY ((gcov_unsigned_t)0xa3000000)
 #define GCOV_TAG_SUMMARY_LENGTH  \
 	(1 + GCOV_COUNTERS_SUMMABLE * (3 + 3 * 2))
-#define GCOV_TAG_MODULE_INFO ((gcov_unsigned_t)0xa4000000)
 #define GCOV_TAG_PMU_LOAD_LATENCY_INFO ((gcov_unsigned_t)0xa5000000)
 #define GCOV_TAG_PMU_LOAD_LATENCY_LENGTH(filename)  \
   (gcov_string_length (filename) + 12 + 2)
@@ -449,6 +448,7 @@ typedef HOST_WIDEST_INT gcov_type;
 #define GCOV_TAG_PMU_BRANCH_MISPREDICT_LENGTH(filename)  \
   (gcov_string_length (filename) + 5 + 2)
 #define GCOV_TAG_PMU_TOOL_HEADER ((gcov_unsigned_t)0xa9000000)
+#define GCOV_TAG_MODULE_INFO ((gcov_unsigned_t)0xab000000)
 
 /* Counters that are collected.  */
 #define GCOV_COUNTER_ARCS 	0  /* Arc transitions.  */

--
This patch is available for review at http://codereview.appspot.com/6430053
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b