Re: [PATCH][1/n] Fix parts of PR19831

2011-09-08 Thread Richard Guenther
On Wed, 7 Sep 2011, Andrew Pinski wrote: > On Wed, Sep 7, 2011 at 7:31 AM, Richard Guenther wrote: > > In practice the patch will do something about alloca at most, > > unless, of course, you have a memleak that you don't use ;) > > I think we have "alloca (0);" being required still and aligning

Re: [PATCH][1/n] Fix parts of PR19831

2011-09-07 Thread Andrew Pinski
On Wed, Sep 7, 2011 at 7:31 AM, Richard Guenther wrote: > In practice the patch will do something about alloca at most, > unless, of course, you have a memleak that you don't use ;) I think we have "alloca (0);" being required still and aligning the stack. Thanks, Andrew Pinski

[PATCH][1/n] Fix parts of PR19831

2011-09-07 Thread Richard Guenther
This fixes the easy piece of PR19831 - removal of "dead" calls to allocation functions (thus, memleaks). It looks like this should be valid even for corner-cases like errno = 0; malloc (whatever); if (errno != 0) ...; as there is no standard defined value for whatever that is guaranteed t