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

Yury Gribov y.gribov@samsung.com
Tue Dec 29 17:38:00 GMT 2015


On 12/29/2015 08:05 PM, Sandra Loosemore wrote:
> 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?

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)

-Y

-------------- next part --------------
A non-text attachment was scrubbed...
Name: frandom-seed-fix-2.patch
Type: text/x-patch
Size: 2686 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20151229/b3dd2aba/attachment.bin>


More information about the Gcc-patches mailing list