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

On Wed, Aug 24, 2016 at 7:42 AM, Andreas Krebbel wrote: > On 08/24/2016 04:02 AM, David Edelsohn wrote: >> This patch broke bootstrap on AIX. Stage1 GCC is miscompiled. >> >> Please revert this patch. > > Done. Sorry for the breakage. Is the alignment assumption safe irrespective of ABI? Thank

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

On 08/24/2016 04:02 AM, David Edelsohn wrote: > This patch broke bootstrap on AIX. Stage1 GCC is miscompiled. > > Please revert this patch. Done. Sorry for the breakage. -Andreas-

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

This patch broke bootstrap on AIX. Stage1 GCC is miscompiled. Please revert this patch. - David

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

> gcc/ChangeLog > > * explow.c (get_dynamic_stack_size): Take known alignment of stack > pointer + STACK_DYNAMIC_OFFSET into account when calculating the size > needed. > Correct a typo in a comment. Applied. Thanks! -Andreas-

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

On 07/26/2016 09:53 AM, Dominik Vogt wrote: Finally a patch that works and is simple. Bootstrapped and regression tested on s390, s390x biarch and x86_64. The new patch exploits the known alignment of (stack pointer + STACK_DYNAMIC_OFFSET) as described earlier (see below). I think that is the

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

Finally a patch that works and is simple. Bootstrapped and regression tested on s390, s390x biarch and x86_64. The new patch exploits the known alignment of (stack pointer + STACK_DYNAMIC_OFFSET) as described earlier (see below). I think that is the right way to get rid of the extra allocation.