[testsuite] Fix gcc.dg/fstack-protector-strong.c on Solaris/x86

Rainer Orth ro@CeBiTec.Uni-Bielefeld.DE
Tue May 14 13:13:00 GMT 2013


Jeff Law <law@redhat.com> writes:

> On 05/14/2013 04:31 AM, Jakub Jelinek wrote:
>> On Tue, May 14, 2013 at 12:26:31PM +0200, Rainer Orth wrote:
>>> Tested with the appropriate runtest invocations on i386-pc-solaris2.11
>>> and x86_64-unknown-linux-gnu, installed on mainline.
>>
>> I'd say the test should just use __builtin_alloca instead.
> Yea, let's go with that.  OK with that fix.  I

Here's what I installed, tested as before:

2013-05-14  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>

	* gcc.dg/fstack-protector-strong.c: Don't include <stdlib.h>.
	(alloca): Remove declaration.
	(foo9): Replace alloca by __builtin_alloca.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: ba.patch
Type: text/x-patch
Size: 767 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20130514/14019c34/attachment.bin>
-------------- next part --------------

> Ranier, if you run into others, they're pre-approved to be fixed this way
> as well.

The one above is the only one that currently fails on Solaris.

	Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University


More information about the Gcc-patches mailing list