Re: [PATCH 1/2][v3] Drop excess size used for run time allocated stack variables.

2016-07-04 Thread Andreas Krebbel
> gcc/ChangeLog0 > > * explow.c (allocate_dynamic_stack_space): Simplify knowing that > MUST_ALIGN was always true and extra_align ist always BITS_PER_UNIT. Applied. Thanks! -Andreas-

Re: [PATCH 1/2][v3] Drop excess size used for run time allocated stack variables.

2016-06-22 Thread Jeff Law
On 05/25/2016 07:30 AM, Dominik Vogt wrote: On Tue, May 03, 2016 at 03:17:53PM +0100, Dominik Vogt wrote: > Version two of the patch including a test case. > > On Mon, May 02, 2016 at 09:10:25AM -0600, Jeff Law wrote: > > On 04/29/2016 04:12 PM, Dominik Vogt wrote: > > >The attached patch remo

Re: [PATCH 1/2][v3] Drop excess size used for run time allocated stack variables.

2016-06-08 Thread Eric Botcazou
> There's some discussion about how STACK_POINT_OFFSET shouldn't cause us > to have to align, and postponing that optimization to gcc-4.7. Since > STACK_POINTER_OFFSET should be constant, it ought to be easy enough to > take it into account. See the "Minor cleanup to allocate_dynamic_stack_space"

Re: [PATCH 1/2][v3] Drop excess size used for run time allocated stack variables.

2016-06-08 Thread Bernd Schmidt
On 05/25/2016 03:30 PM, Dominik Vogt wrote: * explow.c (allocate_dynamic_stack_space): Simplify knowing that MUST_ALIGN was always true and extra_align ist always BITS_PER_UNIT. I tried to do some archaeology to find out how the code came to look the way it currently does. A re

Re: [PATCH 1/2][v3] Drop excess size used for run time allocated stack variables.

2016-05-25 Thread Dominik Vogt
On Tue, May 03, 2016 at 03:17:53PM +0100, Dominik Vogt wrote: > Version two of the patch including a test case. > > On Mon, May 02, 2016 at 09:10:25AM -0600, Jeff Law wrote: > > On 04/29/2016 04:12 PM, Dominik Vogt wrote: > > >The attached patch removes excess stack space allocation with > > >allo