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] Fix PR42805

From: Richard Guenther <rguenther at suse dot de>
To: Jakub Jelinek <jakub at redhat dot com>
Cc: gcc-patches at gcc dot gnu dot org, Martin Jambor <mjambor at suse dot de>, danglin at gcc dot gnu dot org
Date: Mon, 1 Mar 2010 13:39:19 +0100 (CET)
Subject: Re: [PATCH] Fix PR42805
References: <alpine.LNX.2.00.1003011314020.4650@zhemvz.fhfr.qr> <20100301122300.GE2817@tyan-ft48-01.lab.bos.redhat.com>

On Mon, 1 Mar 2010, Jakub Jelinek wrote:

> On Mon, Mar 01, 2010 at 01:16:17PM +0100, Richard Guenther wrote:
> > I'm going to bootstrap & test the following patch which addresses
> > the issue that SRA ends up introducing new accesses to DECLs with
> > DECL_HAS_VALUE_EXPR_P set.
> > 
> > Martin, does this look ok?  David - can you test this on hppa
> > (I verified the testcase no longer ICEs with a cross and the patch).
> 
> Isn't this a dup of PR41250 that should be fixed by the patch there?

Probably yes, but I'm not sure what is the most safe approach at this 
point (though Dave says the patch works for him).  Also the patch
probably doesn't preserve debug information - the newly built decl
is not linked anywhere.

Richard.

Follow-Ups:
- Re: [PATCH] Fix PR42805
  - From: Richard Guenther

References:
- [PATCH] Fix PR42805
  - From: Richard Guenther
- Re: [PATCH] Fix PR42805
  - From: Jakub Jelinek

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