Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-30 Thread Richard Sandiford
Trevor Saunders writes: > On Wed, Jul 29, 2015 at 09:11:21AM +0100, Richard Sandiford wrote: >> Trevor Saunders writes: >> > On Tue, Jul 28, 2015 at 09:24:17PM +0100, Richard Sandiford wrote: >> >> Trevor Saunders writes: >> >> > On Mon, Jul 27, 2015 at 09:05:08PM +0100, Richard Sandiford wrote:

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-30 Thread Richard Sandiford
David Edelsohn writes: > On Wed, Jul 29, 2015 at 11:16:40AM +0100, Richard Earnshaw wrote: >> I'm getting a bit worried about the potential performance impact from >> all these indirect function call hooks. This is a good example of when >> it's probably somewhat unnecessary. I doubt that the co

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-30 Thread David Edelsohn
On Wed, Jul 29, 2015 at 11:16:40AM +0100, Richard Earnshaw wrote: > I'm getting a bit worried about the potential performance impact from > all these indirect function call hooks. This is a good example of when > it's probably somewhat unnecessary. I doubt that the compiler could > function corre

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-29 Thread Trevor Saunders
On Wed, Jul 29, 2015 at 09:11:21AM +0100, Richard Sandiford wrote: > Trevor Saunders writes: > > On Tue, Jul 28, 2015 at 09:24:17PM +0100, Richard Sandiford wrote: > >> Trevor Saunders writes: > >> > On Mon, Jul 27, 2015 at 09:05:08PM +0100, Richard Sandiford wrote: > >> >> Alternatively we could

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-29 Thread Segher Boessenkool
On Wed, Jul 29, 2015 at 11:16:40AM +0100, Richard Earnshaw wrote: > I'm getting a bit worried about the potential performance impact from > all these indirect function call hooks. This is a good example of when > it's probably somewhat unnecessary. I doubt that the compiler could > function corre

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-29 Thread Richard Earnshaw
On 27/07/15 04:10, tbsaunde+...@tbsaunde.org wrote: > From: Trevor Saunders > > Hi, > > $subject. > > patches individually bootstrapped + regtested on x86_64-linux-gnu, and run > through config-list.mk with more patches removing usage of the macro. Ok? > > Trev > > Trevor Saunders (9): > r

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-29 Thread Richard Sandiford
Trevor Saunders writes: > On Tue, Jul 28, 2015 at 09:24:17PM +0100, Richard Sandiford wrote: >> Trevor Saunders writes: >> > On Mon, Jul 27, 2015 at 09:05:08PM +0100, Richard Sandiford wrote: >> >> Alternatively we could have a new target_globals structure that is >> >> initialised with the resul

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-28 Thread Trevor Saunders
On Tue, Jul 28, 2015 at 09:24:17PM +0100, Richard Sandiford wrote: > Trevor Saunders writes: > > On Mon, Jul 27, 2015 at 09:05:08PM +0100, Richard Sandiford wrote: > >> Alternatively we could have a new target_globals structure that is > >> initialised with the result of calling the hook. If we d

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-28 Thread Richard Sandiford
Trevor Saunders writes: > On Mon, Jul 27, 2015 at 09:05:08PM +0100, Richard Sandiford wrote: >> Alternatively we could have a new target_globals structure that is >> initialised with the result of calling the hook. If we do that though, >> it might make sense to consolidate the hooks rather than

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-27 Thread Trevor Saunders
On Mon, Jul 27, 2015 at 09:05:08PM +0100, Richard Sandiford wrote: > Jeff Law writes: > > On 07/27/2015 03:17 AM, Richard Biener wrote: > >> On Mon, Jul 27, 2015 at 5:10 AM, wrote: > >>> From: Trevor Saunders > >>> > >>> Hi, > >>> > >>> $subject. > >>> > >>> patches individually bootstrapped +

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-27 Thread Trevor Saunders
On Mon, Jul 27, 2015 at 10:14:41AM -0600, Jeff Law wrote: > On 07/27/2015 03:17 AM, Richard Biener wrote: > >On Mon, Jul 27, 2015 at 5:10 AM, wrote: > >>From: Trevor Saunders > >> > >>Hi, > >> > >>$subject. > >> > >>patches individually bootstrapped + regtested on x86_64-linux-gnu, and run > >>t

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-27 Thread Richard Sandiford
Jeff Law writes: > On 07/27/2015 03:17 AM, Richard Biener wrote: >> On Mon, Jul 27, 2015 at 5:10 AM, wrote: >>> From: Trevor Saunders >>> >>> Hi, >>> >>> $subject. >>> >>> patches individually bootstrapped + regtested on x86_64-linux-gnu, and run >>> through config-list.mk with more patches rem

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-27 Thread Jeff Law
On 07/27/2015 03:17 AM, Richard Biener wrote: On Mon, Jul 27, 2015 at 5:10 AM, wrote: From: Trevor Saunders Hi, $subject. patches individually bootstrapped + regtested on x86_64-linux-gnu, and run through config-list.mk with more patches removing usage of the macro. Ok? With POINTER_SIZ

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-27 Thread Richard Biener
On Mon, Jul 27, 2015 at 5:10 AM, wrote: > From: Trevor Saunders > > Hi, > > $subject. > > patches individually bootstrapped + regtested on x86_64-linux-gnu, and run > through config-list.mk with more patches removing usage of the macro. Ok? With POINTER_SIZE now being expensive (target hook) y

[PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-26 Thread tbsaunde+gcc
From: Trevor Saunders Hi, $subject. patches individually bootstrapped + regtested on x86_64-linux-gnu, and run through config-list.mk with more patches removing usage of the macro. Ok? Trev Trevor Saunders (9): remove POINTER_SIZE_UNITS macro add pointer_size target hook target.h: chan