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: Peter Bergner <bergner at vnet dot ibm dot com>
- To: Steve Ellcey <sje at cup dot hp dot com>
- Cc: 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, 07 May 2008 10:30:14 -0500
- Subject: Re: Bad code generation on HPPA platform
- References: <200805071445.HAA04364@hpsje.cup.hp.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