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

2016-07-22 Thread Dominik Vogt
On Thu, Jul 21, 2016 at 02:07:05PM -0600, Jeff Law wrote: > On 06/23/2016 03:57 AM, Dominik Vogt wrote: > >>and use that rather than rounding size up to an alignment boundary. > > > >Not exactly. Consider the unpatched code. At the beginning we > >have some amount of space to be allocated on the

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

2016-07-21 Thread Jeff Law
On 06/23/2016 03:57 AM, Dominik Vogt wrote: 0002-v3-ChangeLog gcc/ChangeLog * explow.c (round_push): Use know adjustment. (allocate_dynamic_stack_space): Pass known adjustment to round_push. gcc/testsuite/ChangeLog * gcc.dg/pr50938.c: New test. 0002-v3-Drop-excess-si

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

2016-06-23 Thread Dominik Vogt
On Wed, Jun 22, 2016 at 10:24:02PM -0600, Jeff Law wrote: > On 05/25/2016 07:32 AM, Dominik Vogt wrote: > >On Wed, May 25, 2016 at 02:30:54PM +0100, 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. > >>>

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

2016-06-22 Thread Jeff Law
On 05/25/2016 07:32 AM, Dominik Vogt wrote: On Wed, May 25, 2016 at 02:30:54PM +0100, 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: > > >

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

2016-06-20 Thread Bernd Schmidt
On 06/20/2016 02:19 PM, Dominik Vogt wrote: +/* PR/50938: Check that alloca () reserves the correct amount of stack space. + */ Same here really, even if it's only a test. In this case, the line gets too long with " */" appended. In that case we wrap before the last word. Bernd

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

2016-06-20 Thread Dominik Vogt
On Wed, Jun 08, 2016 at 01:21:09PM +0200, Bernd Schmidt wrote: > On 05/25/2016 03:32 PM, Dominik Vogt wrote: > > * explow.c (round_push): Use know adjustment. > > (allocate_dynamic_stack_space): Pass known adjustment to round_push. > >gcc/testsuite/ChangeLog > > > > I was thinking about wh

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

2016-06-08 Thread Bernd Schmidt
On 05/25/2016 03:32 PM, Dominik Vogt wrote: * explow.c (round_push): Use know adjustment. (allocate_dynamic_stack_space): Pass known adjustment to round_push. gcc/testsuite/ChangeLog I was thinking about whether it would be possible/desirable to eliminate the double add entire

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

2016-05-25 Thread Dominik Vogt
On Wed, May 25, 2016 at 02:32:41PM +0100, Dominik Vogt wrote: > On Wed, May 25, 2016 at 02:30:54PM +0100, 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

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

2016-05-25 Thread Dominik Vogt
On Wed, May 25, 2016 at 02:30:54PM +0100, 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: > >