Re: [PATCH 2/3] Add lto-dump tool.

[Jeff Law <jeffreyalaw@gmail.com>]

On 10/28/22 04:14, Thomas Schwinge wrote:
> Hi!
>
> This minor clean-up had fallen out of me working on something else in
> GCC's options machinery, several months ago:
>
> On 2019-03-12T18:14:04+0100, marxin <mliska@suse.cz> wrote:
>> gcc/lto/ChangeLog:
>>        * lang.opt: Add new language LTODump and options related
>>        to LTO dump tool.
> As this new "Language" 'LTODump' does not share any options with 'LTO'
> proper, it makes sense, in my opinion, to also make that obvious in
> 'gcc/lto/lang.opt', which your Subversion r270897 (Git
> commit 66d62d9f2e6b059be6a018397fba555147133a9a) "Add lto-dump tool"
> almost ;-) did:
>
>> --- a/gcc/lto/lang.opt
>> +++ b/gcc/lto/lang.opt
>> @@ -24,6 +24,9 @@
>>   Language
>>   LTO
>>
>> +Language
>> +LTODump
>> +
>>   Enum
>>   Name(lto_linker_output) Type(enum lto_linker_output) UnknownError(unknown linker output %qs)
>>
>> @@ -66,6 +69,65 @@ fwpa=
>>   LTO Driver RejectNegative Joined Var(flag_wpa)
>>   Whole program analysis (WPA) mode with number of parallel jobs specified.
>>
>> +
>> +[LTODump option records]
>> +
>> +
>>   fresolution=
>>   LTO Joined
>>   The resolution file.
> OK to push the attached
> "Better separate 'LTO' vs. 'LTODump' in 'gcc/lto/lang.opt'"?

OK.

jeff