This is the mail archive of the
mailing list for the GCC project.
[PATCH] Adapt the numbering scheme (PR gcov-profile/64874)
- From: Martin Liška <mliska at suse dot cz>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Cc: nathan at codesourcery dot com, Jan Hubicka <hubicka at ucw dot cz>
- Date: Fri, 22 Jul 2016 13:46:44 +0200
- Subject: [PATCH] Adapt the numbering scheme (PR gcov-profile/64874)
- Authentication-results: sourceware.org; auth=none
As described in the PR, current numbering scheme in gcov-io.h would overflow in couple of years.
Thus, I'm suggesting to switch from:
Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.
Ready to be installed?
>From 3e35f8f08558d95f5d6bd674f659c976ea86f311 Mon Sep 17 00:00:00 2001
From: marxin <email@example.com>
Date: Fri, 22 Jul 2016 11:54:20 +0200
Subject: [PATCH] Adapt the numbering scheme (PR gcov-profile/64874)
2016-07-22 Martin Liska <firstname.lastname@example.org>
* gcov-io.h: Update command about file format.
* gcov-iov.c (main): Adapt the numbering scheme.
gcc/gcov-io.h | 17 ++++++++---------
gcc/gcov-iov.c | 6 +++---
2 files changed, 11 insertions(+), 12 deletions(-)
diff --git a/gcc/gcov-io.h b/gcc/gcov-io.h
index 3446407..4475d1f 100644
@@ -63,19 +63,18 @@ see the files COPYING3 and COPYING.RUNTIME respectively. If not, see
Although the ident and version are formally 32 bit numbers, they
are derived from 4 character ASCII strings. The version number
- consists of the single character major version number, a two
- character minor version number (leading zero for versions less than
- 10), and a single character indicating the status of the release.
+ consists of a two character major version number (leading zero for versions
+ less than 10), the single character minor version number,
+ and a single character indicating the status of the release.
That will be 'e' experimental, 'p' prerelease and 'r' for release.
Because, by good fortune, these are in alphabetical order, string
collating can be used to compare version strings. Be aware that
the 'e' designation will (naturally) be unstable and might be
- incompatible with itself. For gcc 3.4 experimental, it would be
- '304e' (0x33303465). When the major version reaches 10, the
- letters A-Z will be used. Assuming minor increments releases every
- 6 months, we have to make a major increment every 50 years.
- Assuming major increments releases every 5 years, we're ok for the
- next 155 years -- good enough for me.
+ incompatible with itself. For gcc 7.0 experimental, it would be
+ '070e' (0x30373065). As we currently do not release more than 5 minor
+ releases, the single character should be always fine. Major number
+ is currently changed roughly every year, which gives us space
+ for next 90 years.
A record has a tag, length and variable amount of data.
diff --git a/gcc/gcov-iov.c b/gcc/gcov-iov.c
index 202f32a..6685c49 100644
@@ -58,9 +58,9 @@ main (int argc, char **argv)
|| strcmp (argv, "prerelease") == 0)
phase = '*';
- v = (major < 10 ? '0' : 'A' - 10) + major;
- v = (minor / 10) + '0';
- v = (minor % 10) + '0';
+ v = (major / 10) + '0';
+ v = (major % 10) + '0';
+ v = minor + '0';
v = phase;
for (ix = 0; ix != 4; ix++)