Re: [PATCH][RFC] Come up with -flive-patching master option.

2018-11-15 Thread Martin Liška
On 11/14/18 6:54 PM, Qing Zhao wrote: > Hi, > > >> On Nov 14, 2018, at 9:03 AM, Martin Liška wrote: >> >>> Yes, you are right. I added this into my patch. >>> >>> I am attaching the new patch here. >> >> Hello. >> >> Please use >> git diff HEAD~ > /tmp/patch && ~/Programming/gcc/contrib/ch

Re: [PATCH][RFC] Come up with -flive-patching master option.

2018-11-14 Thread Miroslav Benes
On Tue, 13 Nov 2018, Qing Zhao wrote: > Hi, > > > On Nov 13, 2018, at 1:18 PM, Miroslav Benes wrote: > > > >> Attached is the patch for new -flive-patching=[inline-only-static | > >> inline-clone] master option. > >> > >> '-flive-patching=LEVEL' > >> Control GCC's optimizations to provide

Re: [PATCH][RFC] Come up with -flive-patching master option.

2018-11-14 Thread Martin Liška
On 11/13/18 10:16 PM, Qing Zhao wrote: > Hi, > >> On Nov 13, 2018, at 1:18 PM, Miroslav Benes wrote: >> >>> Attached is the patch for new -flive-patching=[inline-only-static | >>> inline-clone] master option. >>> >>> '-flive-patching=LEVEL' >>> Control GCC's optimizations to provide a safe c

Re: [PATCH][RFC] Come up with -flive-patching master option.

2018-11-13 Thread Qing Zhao
Hi, > On Nov 13, 2018, at 1:18 PM, Miroslav Benes wrote: > >> Attached is the patch for new -flive-patching=[inline-only-static | >> inline-clone] master option. >> >> '-flive-patching=LEVEL' >> Control GCC's optimizations to provide a safe compilation for >> live-patching. Provides m

Re: [PATCH][RFC] Come up with -flive-patching master option.

2018-11-13 Thread Miroslav Benes
On Tue, 13 Nov 2018, Qing Zhao wrote: > Hi, Hi, > Attached is the patch for new -flive-patching=[inline-only-static | > inline-clone] master option. > > '-flive-patching=LEVEL' > Control GCC's optimizations to provide a safe compilation for > live-patching. Provides multiple-level

[PATCH][RFC] Come up with -flive-patching master option.

2018-11-13 Thread Qing Zhao
Hi, Attached is the patch for new -flive-patching=[inline-only-static | inline-clone] master option. '-flive-patching=LEVEL' Control GCC's optimizations to provide a safe compilation for live-patching. Provides multiple-level control on how many of the optimizations are enabled

Re: [PATCH][RFC] Come up with -flive-patching master option.

2018-11-12 Thread Qing Zhao
> On Nov 12, 2018, at 2:53 AM, Martin Liška wrote: > >> >> Okay, I see. >> >> I am also working on a similar option as yours, but make the -flive-patching >> as two level control: >> >> +flive-patching >> +Common RejectNegative Alias(flive-patching=,inline-clone) >> + >> +flive-patching= >>

Re: [PATCH][RFC] Come up with -flive-patching master option.

2018-11-12 Thread Martin Liška
On 11/10/18 6:03 PM, Jan Hubicka wrote: >> On 11/9/18 6:43 PM, Qing Zhao wrote: >>> Hi, Martin, >>> >>> thanks a lot for the previous two new options for live-patching. >>> >>> >>> I have two more questions below: >> >> Hello. >> >>> >>> 1. do we still need new options to disable the following: >>

Re: [PATCH][RFC] Come up with -flive-patching master option.

2018-11-12 Thread Martin Liška
On 11/12/18 3:28 AM, Qing Zhao wrote: > Hi, > > >> On Nov 10, 2018, at 2:51 AM, Martin Liška wrote: >> >> On 11/9/18 6:43 PM, Qing Zhao wrote: >>> Hi, Martin, >>> >>> thanks a lot for the previous two new options for live-patching. >>> >>> >>> I have two more questions below: >> >> Hello. >> >>

Re: [PATCH][RFC] Come up with -flive-patching master option.

2018-11-11 Thread Qing Zhao
Hi, > On Nov 10, 2018, at 2:51 AM, Martin Liška wrote: > > On 11/9/18 6:43 PM, Qing Zhao wrote: >> Hi, Martin, >> >> thanks a lot for the previous two new options for live-patching. >> >> >> I have two more questions below: > > Hello. > >> >> 1. do we still need new options to disable th

Re: [PATCH][RFC] Come up with -flive-patching master option.

2018-11-10 Thread Martin Liška
On 11/9/18 6:43 PM, Qing Zhao wrote: > Hi, Martin, > > thanks a lot for the previous two new options for live-patching. > > > I have two more questions below: Hello. > > 1. do we still need new options to disable the following: >A. unreachable code/variable removal? I hope it's guarded

Re: [PATCH][RFC] Come up with -flive-patching master option.

2018-11-10 Thread Martin Liška
Hi. Sorry for attaching a wrong patch. Martin >From 7d3887b1b24901eca69614e601b6e8f36e5c86eb Mon Sep 17 00:00:00 2001 From: marxin Date: Fri, 9 Nov 2018 16:28:07 +0100 Subject: [PATCH] Come up with -flive-patching master option. gcc/ChangeLog: 2018-11-09 Martin Liska * common.opt: Add -fl

Re: [PATCH][RFC] Come up with -flive-patching master option.

2018-11-09 Thread Bernhard Reutner-Fischer
On 9 November 2018 16:33:22 CET, "Martin Liška" wrote: >Hi. > >After I added 2 new options, I would like to include a new master >option. >It's minimal version which only disables optimizations that we are >aware of >and can potentially cause problems for live-patching. I think you attached the w

Re: [PATCH][RFC] Come up with -flive-patching master option.

2018-11-09 Thread Qing Zhao
Hi, Martin, thanks a lot for the previous two new options for live-patching. I have two more questions below: 1. do we still need new options to disable the following: A. unreachable code/variable removal? B. Visibility changes with -flto and/or -fwhole-program? 2. for this new patch,

[PATCH][RFC] Come up with -flive-patching master option.

2018-11-09 Thread Martin Liška
Hi. After I added 2 new options, I would like to include a new master option. It's minimal version which only disables optimizations that we are aware of and can potentially cause problems for live-patching. Martin >From dd52cd0249fc30cf6d7bf01a8826323277817b78 Mon Sep 17 00:00:00 2001 From: marx