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]

[PATCH] Re: Rewrite i386 string operation expansion

From: "Uros Bizjak" <ubizjak at gmail dot com>
To: "GCC Patches" <gcc-patches at gcc dot gnu dot org>
Cc: jh at suse dot cz
Date: Wed, 29 Nov 2006 15:04:15 +0100
Subject: [PATCH] Re: Rewrite i386 string operation expansion
References: <5787cf470611280249i2bc38423gb26d5d50b9ed3c3e@mail.gmail.com>

Hello!

Your patch:

2006-11-19 Jan Hubicka <jh@suse.cz>
        * gcc.target/i386/memcpy-1.c: Adjust size.
        * testsuite/gcc.dg/visibility-11.c: Likewise.
        ...
        (decide_alg): New function.
        ...
crashes for -minline-all-stringops for certain targets:

gcc -O2 -march=pentium4 -minline-all-stringops gcc.dg/i386/memcpy-1.c
memcpy-1.c: In function âtâ:
memcpy-1.c:16: internal compiler error: in decide_alg, at
config/i386/i386.c:13510
Please submit a full bug report, ...etc...
Also crashes for -march=pentium, -march=k6 and -march=nocona.

Attached patch fixes this failure. Bootstrapped on i686-pc-linux-gnu.

2006-11-29 Uros Bizjak <ubizjak@gmail.com>

	config/i386/i386.c (decide_alg): For TARGET_INLINE_ALL_STRINGOPS
	initialize alg to algs->size[0].alg.

OK for mainline?

Uros.

Attachment: i386-memcpy.diff
Description: Binary data

Follow-Ups:
- Re: [PATCH] Re: Rewrite i386 string operation expansion
  - From: Richard Guenther

References:
- Re: Rewrite i386 string operation expansion
  - From: Uros Bizjak

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