This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
PR37363: PR36090 and PR36182 all over again (was: Re: Call for testers, ppc64-linux)
- From: Hans-Peter Nilsson <hans-peter dot nilsson at axis dot com>
- To: gcc-patches at gcc dot gnu dot org
- Cc: hans-peter dot nilsson at axis dot com, janis187 at us dot ibm dot com
- Date: Fri, 5 Sep 2008 03:37:27 +0200
- Subject: PR37363: PR36090 and PR36182 all over again (was: Re: Call for testers, ppc64-linux)
> From: Janis Johnson <janis187@us.ibm.com>
> Date: Thu, 04 Sep 2008 17:20:47 -0700
> On Fri, 2008-09-05 at 02:03 +0200, Hans-Peter Nilsson wrote:
> > Could someone on a ppc64-linux system please regtest the patch
> > in PR37363, <http://gcc.gnu.org/bugzilla/attachment.cgi?id=16217>,
> > in-message below, on a ppc64-linux system with a trunk baseline
> > not older than svn revision 139763? If it helps, that patch is
> > already on the 4.3 branch, I just want to make sure it's not
> > regressing PR36090 on trunk. Many thanks in advance.
>
> I'll do it.
I got negative feedback on that patch (no, not regression
results :) on IRC from David Edelsohn and understandably you
held off your testing because of this, as for one the patch
affects the rs6000 backend.
I'll look a bit closer because I'm not sure I see the cause of
the objection. There was talk about non-canonical RTL generated
but I'd like to see it; the earlier discussion just indicates
insufficient predicates in the rs6000 backend but maybe I
misunderstood.
In the meantime, can I please ask Paolo Bonzini for advice on
how to proceed? (Involved in PR36090 as well the original code
and also PR36182 from which PR37363 was cloned.)
For CRIS (as well as other targets IIUC) the cause of PR37363 is
that there's code that wraps a MINUS of two symbol_ref's in a
CONST without checking that the two symbol_ref's make up a valid
address. After that, the CONST effectively acts as a barrier
for target hooks ("no need to look, we know that thing there is
a valid constant expression").
brgds, H-P