This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Replace XALLOCAVEC with XCNEWVEC (PR c/79471).

From: Bernd Schmidt <bschmidt at redhat dot com>
To: Martin Liška <mliska at suse dot cz>, GCC Patches <gcc-patches at gcc dot gnu dot org>
Cc: dcb314 at hotmail dot com
Date: Mon, 13 Feb 2017 14:09:52 +0100
Subject: Re: [PATCH] Replace XALLOCAVEC with XCNEWVEC (PR c/79471).
Authentication-results: sourceware.org; auth=none
References: <a5e74aae-6df5-330e-1a67-39eab6c83793@suse.cz> <78d5254f-fd1f-9bf1-79a9-7b406a64e415@redhat.com> <de2ff76c-25aa-d4b3-72a7-ba4ba2576a91@suse.cz>

On 02/13/2017 02:06 PM, Martin Liška wrote:

On 02/13/2017 01:58 PM, Bernd Schmidt wrote:

On 02/13/2017 11:15 AM, Martin Liška wrote:

In order to not cause a stack overflow, lets use a vector allocated on heap instead of
the one created by XALLOCVEC.

Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.


Ok. I'm surprised this is marked as a regression, but it's simple enough that it probably ought to be fixed in any case.


Yep. I've just removed that. Is it also fine to both active branches after it finishes regression tests?


Sure.


Bernd

References:
- [PATCH] Replace XALLOCAVEC with XCNEWVEC (PR c/79471).
  - From: Martin Liška
- Re: [PATCH] Replace XALLOCAVEC with XCNEWVEC (PR c/79471).
  - From: Bernd Schmidt
- Re: [PATCH] Replace XALLOCAVEC with XCNEWVEC (PR c/79471).
  - From: Martin Liška

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]