Re: [CHKP] Support returned bounds in thunks expand

2015-04-07 Thread Ilya Enkovich
2015-04-07 23:33 GMT+03:00 Jan Hubicka : >> > > >> > > The problem with instrumented call is that instrumented function >> > > returns two values and call lhs gets only the first one. Thus we >> > > generate bndret call to get the second one to build own return with >> > > two values. >> > >> > I s

Re: [CHKP] Support returned bounds in thunks expand

2015-04-07 Thread Jan Hubicka
> > > > > > The problem with instrumented call is that instrumented function > > > returns two values and call lhs gets only the first one. Thus we > > > generate bndret call to get the second one to build own return with > > > two values. > > > > I see, patch is OK then (preferably merging as mu

Re: [CHKP] Support returned bounds in thunks expand

2015-04-07 Thread Ilya Enkovich
> > > > The problem with instrumented call is that instrumented function > > returns two values and call lhs gets only the first one. Thus we > > generate bndret call to get the second one to build own return with > > two values. > > I see, patch is OK then (preferably merging as much as possible

Re: [CHKP] Support returned bounds in thunks expand

2015-04-03 Thread Jan Hubicka
> > Yep, there is such code in place. It is done by > insert_bndret_call_after. You are right, I should use the same > interface for both these cases. That would be nice indeed. We should separate out and unify the code for porducing a wrapper call next stage1 (as Jeff mentions too). > > >> >

Re: [CHKP] Support returned bounds in thunks expand

2015-04-03 Thread Ilya Enkovich
2015-04-02 23:55 GMT+03:00 Jan Hubicka : >> Ping >> >> 2015-03-10 13:12 GMT+03:00 Ilya Enkovich : >> > Hi, >> > >> > Currentl we loose returned bounds when functions are merged. This patch >> > fixes it by adding returne bounds support for cgraph_node::expand_thunk. >> > Bootstrapped and tested

Re: [CHKP] Support returned bounds in thunks expand

2015-04-03 Thread Ilya Enkovich
2015-04-02 22:42 GMT+03:00 Jeff Law : > On 04/02/2015 08:49 AM, Ilya Enkovich wrote: >> >> Ping >> >> 2015-03-10 13:12 GMT+03:00 Ilya Enkovich : >>> >>> Hi, >>> >>> Currentl we loose returned bounds when functions are merged. This patch >>> fixes it by adding returne bounds support for cgraph_node

Re: [CHKP] Support returned bounds in thunks expand

2015-04-02 Thread Jan Hubicka
> Ping > > 2015-03-10 13:12 GMT+03:00 Ilya Enkovich : > > Hi, > > > > Currentl we loose returned bounds when functions are merged. This patch > > fixes it by adding returne bounds support for cgraph_node::expand_thunk. > > Bootstrapped and tested on x86_64-unknown-linux-gnu. OK for trunk? > >

Re: [CHKP] Support returned bounds in thunks expand

2015-04-02 Thread Jeff Law
On 04/02/2015 08:49 AM, Ilya Enkovich wrote: Ping 2015-03-10 13:12 GMT+03:00 Ilya Enkovich : Hi, Currentl we loose returned bounds when functions are merged. This patch fixes it by adding returne bounds support for cgraph_node::expand_thunk. Bootstrapped and tested on x86_64-unknown-linux-

Re: [CHKP] Support returned bounds in thunks expand

2015-04-02 Thread Ilya Enkovich
Ping 2015-03-10 13:12 GMT+03:00 Ilya Enkovich : > Hi, > > Currentl we loose returned bounds when functions are merged. This patch > fixes it by adding returne bounds support for cgraph_node::expand_thunk. > Bootstrapped and tested on x86_64-unknown-linux-gnu. OK for trunk? > > Thanks, > Ilya

[CHKP] Support returned bounds in thunks expand

2015-03-10 Thread Ilya Enkovich
Hi, Currentl we loose returned bounds when functions are merged. This patch fixes it by adding returne bounds support for cgraph_node::expand_thunk. Bootstrapped and tested on x86_64-unknown-linux-gnu. OK for trunk? Thanks, Ilya -- gcc/ 2015-03-06 Ilya Enkovich * cgraphunit.c (c