Re: [PATCH] gcov: add new option (--hash-names) (PR gcov-profile/36412).

2016-08-16 Thread Martin Liška
On 08/16/2016 04:58 PM, Nathan Sidwell wrote: > I see the 6 branch is now frozen, except for blocking regressions & docs. > > nathan Sure, I'm planning to commit after 6.2 will be released, thus the patch can land 6.3? Martin

Re: [PATCH] gcov: add new option (--hash-names) (PR gcov-profile/36412).

2016-08-16 Thread Nathan Sidwell
On 08/16/16 10:18, Martin Liška wrote: On 08/16/2016 03:28 PM, Nathan Sidwell wrote: On 08/16/16 08:41, Martin Liška wrote: Installed as r239503. Well, the creator of the PR reported that he's been using GCC 6.x, may I install the same patch to gcc-6 branch? I see the 6 branch is now froze

Re: [PATCH] gcov: add new option (--hash-names) (PR gcov-profile/36412).

2016-08-16 Thread Martin Liška
On 08/16/2016 03:28 PM, Nathan Sidwell wrote: > On 08/16/16 08:41, Martin Liška wrote: > >> Thank you for the improvement, all suggested changes are part of new version >> of the patch. >> >> Ready for trunk? > > ok, thanks > > nathan > Installed as r239503. Well, the creator of the PR reporte

Re: [PATCH] gcov: add new option (--hash-names) (PR gcov-profile/36412).

2016-08-16 Thread Nathan Sidwell
On 08/16/16 08:41, Martin Liška wrote: Thank you for the improvement, all suggested changes are part of new version of the patch. Ready for trunk? ok, thanks nathan

Re: [PATCH] gcov: add new option (--hash-names) (PR gcov-profile/36412).

2016-08-16 Thread Martin Liška
On 08/15/2016 02:18 PM, Nathan Sidwell wrote: > On 08/15/16 07:43, Martin Liška wrote: > >> All nits are applied in the second version of patch. >> >>> >>> don't seem to match? Why 'e'? >> >> I've renamed it to -x, well, a lot of letters are already occupied. > > I guess 'x' may be better. If t

Re: [PATCH] gcov: add new option (--hash-names) (PR gcov-profile/36412).

2016-08-15 Thread Nathan Sidwell
On 08/15/16 07:43, Martin Liška wrote: All nits are applied in the second version of patch. don't seem to match? Why 'e'? I've renamed it to -x, well, a lot of letters are already occupied. I guess 'x' may be better. If there is no good choice, do we really need a short name? (There di

Re: [PATCH] gcov: add new option (--hash-names) (PR gcov-profile/36412).

2016-08-15 Thread Martin Liška
On 08/15/2016 12:47 PM, Nathan Sidwell wrote: > On 08/09/16 10:32, Martin Liška wrote: >> Hello. >> >> Following enhancement for gcov solves issues when we cannot create a file >> due to a filesystem >> path length limit. Selected approach utilizes existing md5sum functions. >> >> Patch survives m

Re: [PATCH] gcov: add new option (--hash-names) (PR gcov-profile/36412).

2016-08-15 Thread Nathan Sidwell
On 08/09/16 10:32, Martin Liška wrote: Hello. Following enhancement for gcov solves issues when we cannot create a file due to a filesystem path length limit. Selected approach utilizes existing md5sum functions. Patch survives make check -k RUNTESTFLAGS="gcov.exp" on x86_64-linux-gnu. Ready

[PATCH] gcov: add new option (--hash-names) (PR gcov-profile/36412).

2016-08-09 Thread Martin Liška
tin >From 42393a73c0433d7de28dc560f47ff61e615718bf Mon Sep 17 00:00:00 2001 From: marxin Date: Tue, 9 Aug 2016 16:27:10 +0200 Subject: [PATCH] gcov: add new option (--hash-names) (PR gcov-profile/36412). gcc/ChangeLog: 2016-08-09 Martin Liska PR gcov-profile/36412 * doc/gcov.texi: Document --h