On 12/29/2015 12:33 AM, Yury Gribov wrote:

Hi all,

this patch reverts invalid documentation change -frandom-seed which was
introduced by myself in r216773 a year ago.

I've checked the generated man and the only test for -frandom-seed
(gcc.dg/pr61868.c).

Ok for trunk?  I also want to backport to GCC5 branch.

[snip]

@@ -7464,7 +7464,7 @@ the first option takes effect and the subsequent options 
are
 ignored. Thus only @file{vec.miss} is produced which contains
 dumps from the vectorizer about missed opportunities.

-@item -frandom-seed=@var{number}
+@item -frandom-seed=@var{string}
 @opindex frandom-seed
 This option provides a seed that GCC uses in place of
 random numbers in generating certain symbol names

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?

-Sandra

Reply via email to