This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCHv2][PR 67425] Fix docs for -frandom-seed
- From: Sandra Loosemore <sandra at codesourcery dot com>
- To: Yury Gribov <y dot gribov at samsung dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Cc: Martin Sebor <msebor at redhat dot com>, Jonathan Wakely <redi at gcc dot gnu dot org>, Jakub Jelinek <jakub at redhat dot com>
- Date: Tue, 29 Dec 2015 18:47:51 -0700
- Subject: Re: [PATCHv2][PR 67425] Fix docs for -frandom-seed
- Authentication-results: sourceware.org; auth=none
- References: <567D2A37 dot 7020501 at samsung dot com> <56823756 dot 8040908 at samsung dot com> <5682BD6B dot 4040209 at codesourcery dot com> <5682C531 dot 60903 at samsung dot com>
On 12/29/2015 10:38 AM, Yury Gribov wrote:
On 12/29/2015 08:05 PM, Sandra Loosemore wrote:
[snip]
As a user reading this documentation, I wouldn't understand what kind of
string to use as an argument here, or how GCC actually uses it. (Usually
a "random seed" is something that is used to generate a reproductible
sequence of "random" numbers, not something used in place of "random"
numbers.) Can you do something to clarify the usage, instead of just
renaming the parameter?
Right, thanks. I've only added description of input argument in this
new patch. I'll be able to submit a detailed description of
functionality when I'm back from holidays in 2016)
Actually, I think your revised patch is fine for user documention. It
just needs one little grammar tweak:
@@ -7473,7 +7473,11 @@ place unique stamps in coverage data files and the object files that
produce them. You can use the @option{-frandom-seed} option to produce
reproducibly identical object files.
-The @var{number} should be different for every file you compile.
+The @var{string} can either be a number (decimal, octal or hex) or an
+arbitrary string (in which case it's converted to number by
s/number/a number/
+computing CRC32).
+
+The @var{string} should be different for every file you compile.
@item -fsched-verbose=@var{n}
@opindex fsched-verbose
OK with that change.
-Sandra