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

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

2015-07-30 Thread Richard Sandiford
David Edelsohn dje@gmail.com 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

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

2015-07-30 Thread Richard Sandiford
Trevor Saunders tbsau...@tbsaunde.org writes: On Wed, Jul 29, 2015 at 09:11:21AM +0100, Richard Sandiford wrote: Trevor Saunders tbsau...@tbsaunde.org writes: On Tue, Jul 28, 2015 at 09:24:17PM +0100, Richard Sandiford wrote: Trevor Saunders tbsau...@tbsaunde.org writes: On Mon, Jul 27,

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

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

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

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

2015-07-29 Thread Richard Sandiford
Trevor Saunders tbsau...@tbsaunde.org writes: On Tue, Jul 28, 2015 at 09:24:17PM +0100, Richard Sandiford wrote: Trevor Saunders tbsau...@tbsaunde.org 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

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 tbsaunde+...@tbsaunde.org 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

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 tbsau...@tbsaunde.org 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

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

2015-07-28 Thread Richard Sandiford
Trevor Saunders tbsau...@tbsaunde.org 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

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, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org Hi, $subject. patches individually bootstrapped + regtested on x86_64-linux-gnu, and run through config-list.mk with more patches

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

2015-07-27 Thread Richard Sandiford
Jeff Law l...@redhat.com writes: On 07/27/2015 03:17 AM, Richard Biener wrote: On Mon, Jul 27, 2015 at 5:10 AM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org Hi, $subject. patches individually bootstrapped + regtested on x86_64-linux-gnu, and run

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, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org Hi, $subject. patches individually bootstrapped + regtested on

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 l...@redhat.com writes: On 07/27/2015 03:17 AM, Richard Biener wrote: On Mon, Jul 27, 2015 at 5:10 AM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org Hi, $subject.

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, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org 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

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

2015-07-26 Thread tbsaunde+gcc
From: Trevor Saunders tbsaunde+...@tbsaunde.org 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