[PATCH]Come up with -flive-patching master option.
Qing Zhao
qing.zhao@oracle.com
Fri Nov 16 16:05:00 GMT 2018
> On Nov 16, 2018, at 9:26 AM, Martin Liška <mliska@suse.cz> wrote:
>
> On 11/16/18 2:36 AM, Qing Zhao wrote:
>> Hi,
>>
>> this is the new version of the patch.
>>
>> I have bootstrapped it on both aarch64 and x86, no regression.
>>
>> please take a look.
>
> Thanks for the updated version of the patch.
> I have last small nits I see:
>
> - gcc/common.opt: when running --help=common, the line is too long
the following is the output for ./gcc —help=common:
-flive-patching Same as -flive-patching=. Use the latter option
instead.
-flive-patching=[inline-only-static|inline-clone] Control IPA optimizations
to provide a safe compilation for live-patching.
At the same time, provides multiple-level control
on the enabled IPA optimizations.
Not sure what’s you mean of “the line is too long”? could you please specify the above which line?
> - gcc/doc/invoke.texi - 2 spaces in between sentences + better gol
> - gcc/opts.c - do not mix spaces + tabs
I have used contrib/check_GNU_style.sh to check the patch, I did see one place that complains about 2 spaces in between sentences, fixed it.
but I didn’t see spaces + tabs mix issue with the script. could you please specify?
thanks.
Qing
More information about the Gcc-patches
mailing list