Re: [PATCH] C++ thunk section names

2014-09-02 Thread Sriraman Tallam
Ping. On Wed, Aug 6, 2014 at 2:42 PM, Sriraman Tallam wrote: > Hi, > > Just wondering if you got a chance to look at this? > > Sri > > On Tue, Jul 8, 2014 at 10:45 AM, Sriraman Tallam wrote: >> On Tue, Jul 8, 2014 at 10:38 AM, Sriraman Tallam wrote: >>> On Mon, Jul 7, 2014 at 11:48 AM, Jan Hu

Re: [PATCH] C++ thunk section names

2014-08-06 Thread Sriraman Tallam
Hi, Just wondering if you got a chance to look at this? Sri On Tue, Jul 8, 2014 at 10:45 AM, Sriraman Tallam wrote: > On Tue, Jul 8, 2014 at 10:38 AM, Sriraman Tallam wrote: >> On Mon, Jul 7, 2014 at 11:48 AM, Jan Hubicka wrote: >>> Hello, >>> I apologize for taking so long to get into this p

Re: [PATCH] C++ thunk section names

2014-07-08 Thread Sriraman Tallam
On Tue, Jul 8, 2014 at 10:38 AM, Sriraman Tallam wrote: > On Mon, Jul 7, 2014 at 11:48 AM, Jan Hubicka wrote: >> Hello, >> I apologize for taking so long to get into this patch. I ad busy time >> (wedding >> and teaching), should be back in regular schedule now. >> >>> Sri, can you provide exam

Re: [PATCH] C++ thunk section names

2014-07-08 Thread Sriraman Tallam
On Mon, Jul 7, 2014 at 11:48 AM, Jan Hubicka wrote: > Hello, > I apologize for taking so long to get into this patch. I ad busy time > (wedding > and teaching), should be back in regular schedule now. > >> Sri, can you provide examples to show why putting thunks into the same >> section as the t

Re: [PATCH] C++ thunk section names

2014-07-07 Thread Jan Hubicka
Hello, I apologize for taking so long to get into this patch. I ad busy time (wedding and teaching), should be back in regular schedule now. > Sri, can you provide examples to show why putting thunks into the same > section as the target function when function reorder is on can be bad > ? C++ AB

Re: [PATCH] C++ thunk section names

2014-07-07 Thread Xinliang David Li
Sri, can you provide examples to show why putting thunks into the same section as the target function when function reorder is on can be bad ? Thanks, David On Thu, Jun 26, 2014 at 10:29 AM, Sriraman Tallam wrote: > Hi Honza, > >Could you review this patch when you find time? > > Thanks > S

Re: [PATCH] C++ thunk section names

2014-06-26 Thread Sriraman Tallam
Hi Honza, Could you review this patch when you find time? Thanks Sri On Tue, Jun 17, 2014 at 10:42 AM, Sriraman Tallam wrote: > Ping. > > On Mon, Jun 9, 2014 at 3:54 PM, Sriraman Tallam wrote: >> Ping. >> >> On Mon, May 19, 2014 at 11:25 AM, Sriraman Tallam >> wrote: >>> Ping. >>> >>> On

Re: [PATCH] C++ thunk section names

2014-06-17 Thread Sriraman Tallam
Ping. On Mon, Jun 9, 2014 at 3:54 PM, Sriraman Tallam wrote: > Ping. > > On Mon, May 19, 2014 at 11:25 AM, Sriraman Tallam wrote: >> Ping. >> >> On Thu, Apr 17, 2014 at 10:41 AM, Sriraman Tallam >> wrote: >>> Ping. >>> >>> On Wed, Feb 5, 2014 at 4:31 PM, Sriraman Tallam wrote: Hi, >

Re: [PATCH] C++ thunk section names

2014-06-09 Thread Sriraman Tallam
Ping. On Mon, May 19, 2014 at 11:25 AM, Sriraman Tallam wrote: > Ping. > > On Thu, Apr 17, 2014 at 10:41 AM, Sriraman Tallam wrote: >> Ping. >> >> On Wed, Feb 5, 2014 at 4:31 PM, Sriraman Tallam wrote: >>> Hi, >>> >>> I would like this patch reviewed and considered for commit when >>> Stage 1

Re: [PATCH] C++ thunk section names

2014-05-19 Thread Sriraman Tallam
Ping. On Thu, Apr 17, 2014 at 10:41 AM, Sriraman Tallam wrote: > Ping. > > On Wed, Feb 5, 2014 at 4:31 PM, Sriraman Tallam wrote: >> Hi, >> >> I would like this patch reviewed and considered for commit when >> Stage 1 is active again. >> >> Patch Description: >> >> A C++ thunk's section name i

Re: [PATCH] C++ thunk section names

2014-04-17 Thread Sriraman Tallam
Ping. On Wed, Feb 5, 2014 at 4:31 PM, Sriraman Tallam wrote: > Hi, > > I would like this patch reviewed and considered for commit when > Stage 1 is active again. > > Patch Description: > > A C++ thunk's section name is set to be the same as the original function's > section name for which the t

[PATCH] C++ thunk section names

2014-02-05 Thread Sriraman Tallam
Hi, I would like this patch reviewed and considered for commit when Stage 1 is active again. Patch Description: A C++ thunk's section name is set to be the same as the original function's section name for which the thunk was created in order to place the two together. This is done in cp/metho