This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCHv2][PR 67425] Fix docs for -frandom-seed


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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]