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