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: [RS6000] Fix PR12817

From: Geoffrey Keating <geoffk at apple dot com>
To: Alan Modra <amodra at bigpond dot net dot au>
Cc: gcc-patches at gcc dot gnu dot org, Dale Johannesen <dalej at apple dot com>, Stan Shebs <shebs at apple dot com>, David Edelsohn <dje at watson dot ibm dot com>
Date: Fri, 26 Nov 2004 19:40:58 -0800
Subject: Re: [RS6000] Fix PR12817
References: <20041126101943.GA14779@bubble.modra.org> <200411262331.iAQNVwD25464@makai.watson.ibm.com> <20041127000000.GD14779@bubble.modra.org>

On 26/11/2004, at 4:00 PM, Alan Modra wrote:

On Fri, Nov 26, 2004 at 06:31:58PM -0500, David Edelsohn wrote:
Alan Modra writes:
	PR target/12817
	* config/rs6000/rs6000.c (rs6000_emit_prologue): Use r11 for vrsave.
Alan> Bootstrap and regression test powerpc-linux in progress. This ought to Alan> count as obvious, but I'll ask. OK to install? gcc-3.4 and gcc-3.3 Alan> too?
	This looks okay, but rs6000_emit_prologue is becoming quite
incomprehensible now.
Yes, I think we would do well to split the function into separate ones
for each major ABI.

I think that would just quadruple the amount of incomprehensible code.

Attachment: smime.p7s
Description: S/MIME cryptographic signature

Follow-Ups:
- Re: [RS6000] Fix PR12817
  - From: Alan Modra

References:
- [RS6000] Fix PR12817
  - From: Alan Modra
- Re: [RS6000] Fix PR12817
  - From: David Edelsohn
- Re: [RS6000] Fix PR12817
  - From: Alan Modra

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