Re: [CHKP] Never expand instrumentation thunks

2015-04-03 Thread Jan Hubicka
> 2015-04-03 0:13 GMT+03:00 Jan Hubicka : > >> On 04/02/2015 02:11 PM, Jan Hubicka wrote: > >> >>>2015-03-18 Ilya Enkovich > >> >>> > >> >>> * cgraphunit.c (cgraph_node::expand_thunk): Don't expand > >> >>> instrumentation thunks. > >> >>> > >> >>> > >> >>>diff --git a/gcc/cgraph

Re: [CHKP] Never expand instrumentation thunks

2015-04-03 Thread Ilya Enkovich
2015-04-03 0:13 GMT+03:00 Jan Hubicka : >> On 04/02/2015 02:11 PM, Jan Hubicka wrote: >> >>>2015-03-18 Ilya Enkovich >> >>> >> >>> * cgraphunit.c (cgraph_node::expand_thunk): Don't expand >> >>> instrumentation thunks. >> >>> >> >>> >> >>>diff --git a/gcc/cgraphunit.c b/gcc/cgrap

Re: [CHKP] Never expand instrumentation thunks

2015-04-02 Thread Jeff Law
On 04/02/2015 03:13 PM, Jan Hubicka wrote: I think it is really the transparent alias issue. The comment seems pretty clear about it. What is confusing is that instrumentation thunks are called thunks when they are really not - thunk is a small hunk of code, while instrumentation thunk is a t

Re: [CHKP] Never expand instrumentation thunks

2015-04-02 Thread Jan Hubicka
> On 04/02/2015 02:11 PM, Jan Hubicka wrote: > >>>2015-03-18 Ilya Enkovich > >>> > >>> * cgraphunit.c (cgraph_node::expand_thunk): Don't expand > >>> instrumentation thunks. > >>> > >>> > >>>diff --git a/gcc/cgraphunit.c b/gcc/cgraphunit.c > >>>index e640907..abc9cfe 100644 > >>>

Re: [CHKP] Never expand instrumentation thunks

2015-04-02 Thread Jeff Law
On 04/02/2015 02:11 PM, Jan Hubicka wrote: 2015-03-18 Ilya Enkovich * cgraphunit.c (cgraph_node::expand_thunk): Don't expand instrumentation thunks. diff --git a/gcc/cgraphunit.c b/gcc/cgraphunit.c index e640907..abc9cfe 100644 --- a/gcc/cgraphunit.c +++ b/gcc/cgraphunit.c

Re: [CHKP] Never expand instrumentation thunks

2015-04-02 Thread Jan Hubicka
> > 2015-03-18 Ilya Enkovich > > > > * cgraphunit.c (cgraph_node::expand_thunk): Don't expand > > instrumentation thunks. > > > > > > diff --git a/gcc/cgraphunit.c b/gcc/cgraphunit.c > > index e640907..abc9cfe 100644 > > --- a/gcc/cgraphunit.c > > +++ b/gcc/cgraphunit.c > > @@ -1

Re: [CHKP] Never expand instrumentation thunks

2015-04-02 Thread Jeff Law
On 04/02/2015 08:51 AM, Ilya Enkovich wrote: Ping. Patch doesn't affect not instrumented code. Thanks, Ilya 2015-03-18 15:19 GMT+03:00 Ilya Enkovich : Hi, This patch disables attempts to expand instrumentation thunks which appear when we create specialized function versions. Problem was fou

Re: [CHKP] Never expand instrumentation thunks

2015-04-02 Thread Ilya Enkovich
Ping. Patch doesn't affect not instrumented code. Thanks, Ilya 2015-03-18 15:19 GMT+03:00 Ilya Enkovich : > Hi, > > This patch disables attempts to expand instrumentation thunks which appear > when we create specialized function versions. Problem was found during > SPEC2006 insatrumentation wi

[CHKP] Never expand instrumentation thunks

2015-03-18 Thread Ilya Enkovich
Hi, This patch disables attempts to expand instrumentation thunks which appear when we create specialized function versions. Problem was found during SPEC2006 insatrumentation with '-Ofast -flto'. Unfortunately I couldn't make a small reproducer. Bootstrapped and tested on x86_64-unknown-lin