This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: Bad code generation on HPPA platform
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: "Steve Ellcey" <sje at cup dot hp dot com>
- Cc: bergner at vnet dot ibm dot com, law at redhat dot com, andrew_pinski at playstation dot sony dot com, dave at hiauly1 dot hia dot nrc dot ca, gcc at gcc dot gnu dot org
- Date: Wed, 7 May 2008 09:27:28 -0700
- Subject: Re: Bad code generation on HPPA platform
- References: <1210174214.7979.6.camel@otta> <200805071621.JAA05249@hpsje.cup.hp.com>
On Wed, May 7, 2008 at 9:21 AM, Steve Ellcey <sje@cup.hp.com> wrote:
> > From: Peter Bergner <bergner@vnet.ibm.com>
>
>
> >
> > On Wed, 2008-05-07 at 07:45 -0700, Steve Ellcey wrote:
> > > I have found that this problem does not occur on the ToT sources and
> > > that the problem went away with this patch:
> > >
> > > 2008-04-07 Peter Bergner <bergner@vnet.ibm.com>
> > >
> > > PR middle-end/PR28690
> > > * rtlanal.c: Update copyright years.
> > > (commutative_operand_precedence): Give SYMBOL_REF's the same precedence
> > > as REG_POINTER and MEM_POINTER operands.
> > > * emit-rtl.c (gen_reg_rtx_and_attrs): New function.
> > > (set_reg_attrs_from_value): Call mark_reg_pointer as appropriate.
> > > * rtl.h (gen_reg_rtx_and_attrs): Add prototype for new function.
> > > * gcse.c: Update copyright years.
> > > (pre_delete): Call gen_reg_rtx_and_attrs.
> > > (hoist_code): Likewise.
> > > (build_store_vectors): Likewise.
> > > (delete_store): Likewise.
> > > * loop-invariant.c (move_invariant_reg): Likewise.
> > > Update copyright years.
> > >
> > > I don't know if porting this patch to the 4.3 branch is an option or not
> > > but it might be the easiest way to fix this problem without having to
> > > revert Andrew's patch.
> >
> > Note that the rtlanal.c:commutative_operand_precedence() hunk was reverted
> > because it caused some problems on CRIS and was replaced by the following
> > safer change:
> >
> > http://gcc.gnu.org/ml/gcc-patches/2008-04/msg00693.html
> >
> > Peter
>
> I think that even with that hunk reverted this would fix the SPEC gcc
> problem. Unfortunately, I don't think it would fix the other problem I
> have found which is that SPECint2000/253.perlbmk also fails. I had
> hoped that the gcc fix would also fix perl but that doesn't seem to be
> the case. With ToT GCC sources the SPEC GCC test passes but the
> SPEC2000 and SPEC2006 perl tests appear to be going into infinite loops.
>
> This only seems to be happening on the 64 bit HPPA platform, not on the
> 32 bit one.
>
Hi Steve,
Is your perl test problem the same as
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33383
H.J.