Re: [PATCH v5] Allocate constant size dynamic stack space in the prologue

2016-07-18 Thread Andreas Krebbel
> gcc/ChangeLog > > * cfgexpand.c (expand_stack_vars): Implement synamic stack space > allocation in the prologue. > * explow.c (get_dynamic_stack_base): New function to return an address > expression for the dynamic stack base. > (get_dynamic_stack_size): New

Re: [PATCH v5] Allocate constant size dynamic stack space in the prologue

2016-07-15 Thread Bernd Schmidt
On 07/15/2016 02:22 PM, Dominik Vogt wrote: Final version 6 with the stray comment removed (was a harmless oversight). Ok, let's put it in. Bernd

Re: [PATCH v5] Allocate constant size dynamic stack space in the prologue

2016-07-15 Thread Dominik Vogt
Final version 6 with the stray comment removed (was a harmless oversight). Initial description of the patch: https://gcc.gnu.org/ml/gcc-patches/2015-11/msg03052.html Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * cfgexpand.c (expand_stack_vars): Implement synamic

Re: [PATCH v5] Allocate constant size dynamic stack space in the prologue

2016-07-15 Thread Dominik Vogt
On Fri, Jul 15, 2016 at 01:51:51PM +0200, Bernd Schmidt wrote: > On 07/14/2016 11:10 AM, Dominik Vogt wrote: > > >- if (flag_stack_usage_info) > >-stack_usage_size += extra; > >+ /*!!!*/ > >+ if (flag_stack_usage_info && pstack_usage_size) > >+*pstack_usage_size += extra; > > What

Re: [PATCH v5] Allocate constant size dynamic stack space in the prologue

2016-07-15 Thread Bernd Schmidt
On 07/14/2016 11:10 AM, Dominik Vogt wrote: - if (flag_stack_usage_info) -stack_usage_size += extra; + /*!!!*/ + if (flag_stack_usage_info && pstack_usage_size) +*pstack_usage_size += extra; What does the comment mean? Whatever it is needs to be addressed and the comment removed.

Re: [PATCH v5] Allocate constant size dynamic stack space in the prologue

2016-07-14 Thread Dominik Vogt
On Wed, Jul 13, 2016 at 04:12:36PM -0600, Jeff Law wrote: > On 07/11/2016 05:44 AM, Dominik Vogt wrote: > >On Thu, Jul 07, 2016 at 12:57:16PM +0100, Dominik Vogt wrote: > >>On Wed, Jul 06, 2016 at 02:01:06PM +0200, Bernd Schmidt wrote: > >>>There's one thing I don't quite understand and which

Re: [PATCH v5] Allocate constant size dynamic stack space in the prologue

2016-07-13 Thread Jeff Law
On 07/11/2016 05:44 AM, Dominik Vogt wrote: On Thu, Jul 07, 2016 at 12:57:16PM +0100, Dominik Vogt wrote: On Wed, Jul 06, 2016 at 02:01:06PM +0200, Bernd Schmidt wrote: There's one thing I don't quite understand and which seems to have changed since v1: On 07/04/2016 02:19 PM, Dominik Vogt

Re: [PATCH v5] Allocate constant size dynamic stack space in the prologue

2016-07-11 Thread Dominik Vogt
On Thu, Jul 07, 2016 at 12:57:16PM +0100, Dominik Vogt wrote: > On Wed, Jul 06, 2016 at 02:01:06PM +0200, Bernd Schmidt wrote: > > There's one thing I don't quite understand and which seems to have > > changed since v1: > > > > On 07/04/2016 02:19 PM, Dominik Vogt wrote: > > >@@ -1099,8 +1101,10